-
Notifications
You must be signed in to change notification settings - Fork 18
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
running workflows showing with the "?" symbol #485
Comments
This diff shows the status is getting into the data store correctly: diff --git a/cylc/flow/data_store_mgr.py b/cylc/flow/data_store_mgr.py
index a15341d8f..cf292ecfe 100644
--- a/cylc/flow/data_store_mgr.py
+++ b/cylc/flow/data_store_mgr.py
@@ -1855,6 +1855,7 @@ class DataStoreMgr:
# Set status & msg if changed.
status, status_msg = map(
str, get_workflow_status(self.schd))
+ LOG.warning(f'{status=}, {status_msg=}')
if w_data.status != status or w_data.status_msg != status_msg:
w_delta.status = status
w_delta.status_msg = status_msg Producing messages like this:
Which I think narrows it down to the UIS. |
It's already been added, but we received an extra added workflow delta... I suspect the UIS is constructing this.. |
Thing is.. that delta isn't consistently sent, so it makes me think it's from the UIS scan or something.. |
Finding it really hard to replicate the issue.. the content of the delta says two things:
Will carry on tomorrow.. might be fixed by just making sure those status fields aren't being set, but the WUI might break if the workflow has no status fields. |
The issue is more easily reproduced with higher scan rate (lower scan_interval), which is a clue in itself.. slowly narrowing in on it. |
I believe I've figured out what's going on here.. Two things really. At the UIS, occasionally the UIS connects to the Scheduler before the first start-up delta arrives from the scheduler;
gets published by the scheduler.. The UIS publishes the Fixing this (not sending this startup subscription if the initial burst has been sent) will be enough to resolve the issue. However the 2nd issue is the WUI's handling of the above delta, by my understanding if it encounters an
However, having no status would probably trip the scan panel up .. (and if the update was applied it wouldn't be an issue) Addressing either of these should resolve the issue.. |
Empty deltas are unnecessary but harmless. |
This might be an unrelated bug, but we see this when we are running more than 8 workflows. |
Release including fix coming out soon, hopefully today, will announce on Discourse. |
Can this be closed now? |
I don't think so, I've seen this recently (note the 8 workflow thing commented above had a different root cause). I think the underlying issue here is that sometimes the workflow status can be |
When starting a stopped workflow, it will sometimes show in the GUI with the "?" symbol rather than the "▶" icon for a running workflow.
From inspecting the UI data store, when the workflow is started, the status and statusMsg fields are initially provided correctly:
But are then quickly set to
""
in a subsequent delta:Then, this happens again leaving the workflow with status
""
causing the workflow-icon component to display the?
and preventing mutations from being run against the workflow.The text was updated successfully, but these errors were encountered: