Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Last task in workflow doesn't update to succeeded in graph GUI #630

Open
jfrost-mo opened this issue Sep 24, 2024 · 8 comments
Open

Last task in workflow doesn't update to succeeded in graph GUI #630

jfrost-mo opened this issue Sep 24, 2024 · 8 comments
Labels
bug Something isn't working
Milestone

Comments

@jfrost-mo
Copy link

Description

After a workflow reaches its final cycle point, and the last task completes, the reported state in the graph view of the GUI still claims the task to be running, when it is not.

Reproducible Example

Run a workflow, opening it in cylc gui on the graph view. Once the workflow reaches its final cycle point, and the last task completes, observe the reported state of that task on the graph. It will be left in a running state, despite actually having succeeded.

Expected Behaviour

The task in the GUI should be marked as succeeded.

@jfrost-mo jfrost-mo added the bug Something isn't working label Sep 24, 2024
@Fraetor
Copy link

Fraetor commented Sep 24, 2024

An example. Note the workflow is stopped after having completed, but the task still claims to be running.

20240924_122816

@MetRonnie
Copy link
Member

Duplicate of cylc/cylc-ui#1756

@MetRonnie MetRonnie marked this as a duplicate of cylc/cylc-ui#1756 Sep 24, 2024
@MetRonnie MetRonnie closed this as not planned Won't fix, can't repro, duplicate, stale Sep 24, 2024
@MetRonnie MetRonnie added duplicate This issue or pull request already exists and removed bug Something isn't working labels Sep 24, 2024
@oliver-sanders
Copy link
Member

I think that's a different issue.

@jfrost-mo, if you switch back to the tree view, the task should still be visible there too, what state is it in?

@MetRonnie
Copy link
Member

I can reproduce with a simple workflow.

if you switch back to the tree view, the task should still be visible there too, what state is it in?

The workflow completed so the tree view is empty after shutdown

@oliver-sanders
Copy link
Member

Good point, would need an incomplete task to test the graph view against the tree view.

@MetRonnie MetRonnie added bug Something isn't working and removed duplicate This issue or pull request already exists labels Sep 24, 2024
@MetRonnie
Copy link
Member

Ok so not a duplicate because cylc/cylc-ui#1756 is just about clearing the graph view but the bug here is we are missing a delta update for the final state of the task before it is pruned

@MetRonnie MetRonnie reopened this Sep 24, 2024
@MetRonnie MetRonnie transferred this issue from cylc/cylc-ui Sep 24, 2024
@MetRonnie MetRonnie added this to the pending milestone Sep 24, 2024
@jfrost-mo
Copy link
Author

Clearing the graph would resolve this issue too, though I do personally find it quite disconcerting that the workflow finished state is just an empty window. Maybe some kind of graphic to indicate "Congratulations, your workflow finished." I see there is Issue cylc/cylc-ui#332 for this.

@oliver-sanders
Copy link
Member

That's the plan, we just haven't managed to get around to it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants