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

compat mode ignored #319

Closed
oliver-sanders opened this issue May 7, 2024 · 3 comments · Fixed by #322 or cylc/cylc-flow#6097
Closed

compat mode ignored #319

oliver-sanders opened this issue May 7, 2024 · 3 comments · Fixed by #322 or cylc/cylc-flow#6097
Assignees
Labels
bug Something isn't working
Milestone

Comments

@oliver-sanders
Copy link
Member

oliver-sanders commented May 7, 2024

A bug which can be replicated by:

  • Running a Cylc 7 workflow under Cylc 8 compat mode.
  • Which contains an fcm_make task.
  • And a :finish trigger.

This should do it:

# suite.rc
[scheduling]
  [[graph]]
    R1 = fcm_make:finish => fin

Due to the way the workflow config is loaded from the log/config/flow-processed.cylc file, the Cylc 7 compat mode check is not being performed correctly causing the config to think it is not in compat mode.

This check is performed automatically by WorkflowConfig.__init__ so may require an upstream change to cylc-flow, hence 1.4.0.

Introduced by #302

@oliver-sanders oliver-sanders added the bug Something isn't working label May 7, 2024
@oliver-sanders oliver-sanders added this to the 1.4.0 milestone May 7, 2024
@wxtim wxtim self-assigned this May 9, 2024
@wxtim
Copy link
Member

wxtim commented May 9, 2024

I cannot replicate the bug with your example.

@oliver-sanders
Copy link
Member Author

oliver-sanders commented May 9, 2024

The error will be raised when the fcm_make task runs. It will need to be a "real" fcm_make task, not just a task called fcm_make, and it will need a sufficiently valid configuration that it gets far enough to trigger the loading of the workflow config. I don't have an example for this, you could try to butcher a rose-stem workflow to get one.

@wxtim
Copy link
Member

wxtim commented May 10, 2024

I have now replicated this error using

/metomi/rose/t/rose-task-run/26-app-fcm-make-new-mode-with-cont

with graph modified to

            hello-make:finish & hello-make => fin

Which should be ok in compat mode.

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
2 participants