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.

Sign up for GitHub

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

Jump to bottom

Traceback with Cylc 8 workflows #4315

Closed
oliver-sanders opened this issue Jul 21, 2021 · 2 comments · Fixed by #4320
Closed

Traceback with Cylc 8 workflows #4315

oliver-sanders opened this issue Jul 21, 2021 · 2 comments · Fixed by #4320
Assignees
@MetRonnie
Labels
bug Something is wrong :(
Milestone
cylc-7.8.9 and 7.9.4

Comments

@oliver-sanders
Copy link
Member

Reverse of #4309

Add logic to Cylc 7 to more gracefully handle Cylc 8 workflows.

Example traceback:

int() argument must be a string or a number, not 'NoneType'

Also ensure that the Cylc 7 logic for removing the contact files of crashed suites (included in cylc scan etc) does not interfere with Cylc 8 workflows.
@oliver-sanders oliver-sanders added the bug Something is wrong :( label Jul 21, 2021
@oliver-sanders oliver-sanders added this to the cylc-7.8.x milestone Jul 21, 2021
@MetRonnie MetRonnie self-assigned this Jul 22, 2021
@MetRonnie MetRonnie mentioned this issue Jul 22, 2021
Merged
6 tasks
@oliver-sanders
Copy link
Member Author

See also the related Rose issue: metomi/rose#2474

@oliver-sanders
Copy link
Member Author

Closed by #4320

@MetRonnie MetRonnie linked a pull request Aug 13, 2021 that will close this issue
Better handling of Cylc 8 workflows #4320
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MetRonnie MetRonnie

Labels
bug Something is wrong :(
Projects
None yet
Milestone
cylc-7.8.9 and 7.9.4
Development

Successfully merging a pull request may close this issue.

Better handling of Cylc 8 workflows MetRonnie/cylc-flow
3 participants
@hjoliver @oliver-sanders @MetRonnie