-
Notifications
You must be signed in to change notification settings - Fork 95
cylc review: issues inherited from rose bush origin #2833
Copy link
Copy link
Open
Labels
Milestone
Description
All GitHub issues concerning Rose Bush (to be closed in parallel under metomi/rose) plus those informally registered in the migration PR #2614, collated here to aid systemisation & discussion post-migration to Cylc as Cylc Review.
N.B. there is a new GitHub beta feature to transfer issues across repositories, but better to compile than transfer en-masse.
Can we agree on a (rough) prioritisation order, for anyone who has time to tackle any? We should also decide which 'feature request' issues should await the (later) development stages of a reformed Cylc Review as a component of the new Cylc GUI (#1873).
[Descriptions paraphrased given the overall, up-to-date context.]
Flaws or bugs:
- Rose Bush utility migration to cylc: incoming #2614 (comment) suite view broadcasts (states & events):
- show empty tables not just blank pages (for feedback) for suites with no broadcasts;
- raise appropriate HTTP status codes for non-existent relevant file/table & for lack of view permission.
- suite metadata (see also Rose Bush utility migration to cylc: incoming #2614 (comment) & Consolidation of suite metadata metomi/rose#2056 RE metadata division & duplication across
rose-suite.info&suite.rc, though not directly relevant) NB not marking these as duplicates as each concerns a separate aspect, as italicised:- Rose bush suite metadata metomi/rose#2147: dynamic metadata display (esp. for
rosie-suite.info); - rose bush: suites list: display more suite info metomi/rose#2144: display further (optional) suite info settings.
- Rose bush suite metadata metomi/rose#2147: dynamic metadata display (esp. for
- inappropriate icons e.g. for the waiting task state (see bullet point 1 of review: fix waiting tasks #2938 (review))
- long query strings with default (or multiple custom) task statuses: e.g:
Since no task status contains a
http://localhost:8080/taskjobs?user=sbarth&suite=consolidation-tutorial&no_fuzzy_time=0&cycles=&tasks=&task_status=runahead&task_status=held&task_status=queued&task_status=expired&task_status=ready&task_status=submit-failed&task_status=submit-retrying&task_status=submitted&task_status=retrying&task_status=running&task_status=failed&task_status=succeeded&job_status=&order=time_desc&per_page=,character, could we instead use a comma to delimit these query parameter values i.e.task_status=runahead,held,queued...instead?
New feature requests (extensions):
- rose-bush viewing multiple users at once metomi/rose#2189: view multiple users at once on suites listing page.
- bush: Display auto-refresh in pages metomi/rose#2153: display auto-refresh icon on main/all pages.
- rose bush: filter jobs by time run/updated metomi/rose#1308: filter jobs by time run or updated.
- rose bush: task list: group by cycles and/or tasks metomi/rose#1014: group by cycles &/or tasks in task list.
- rose bush: user preference metomi/rose#1013: store user preference.
Miscallenous issues:
- Rose Bush utility migration to cylc: incoming #2614 (comment) Codacy issues (~350!) on Cylc Review exclusive web (HTML, JS & CSS) files: sift through & either resolve or exclude from Codacy checks, as appropriate.
- More comprehensive utility documentation: currently there is only that from Document Cylc Review (minimally for next version) #2821.
Issues that are no longer issues (to not re-open in Cylc?):
- rose bush: suite analytics graphs metomi/rose#1310 & rose bush: plot suite graph metomi/rose#1009 suite analytics & graphing. Suite analytics will be implemented in the new Cylc GUI (#1873), but this is now being considered a separate domain to Bush/Review.
- rose bush: cycles/jobs list view: tar-gzip log directories and other log files metomi/rose#1030: given migration to Cylc, historical
rose suite-runlogs no longer appropriate to display?
Reactions are currently unavailable