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

Fix job state with pre-submitted failure #4941

Merged
merged 2 commits into from
Jun 30, 2022

Conversation

dwsutherland
Copy link
Member

@dwsutherland dwsutherland commented Jun 29, 2022

These changes close #4905

Requirements check-list

  • I have read CONTRIBUTING.md and added my name as a Code Contributor.
  • Contains logically grouped changes (else tidy your branch by rebase).
  • Does not contain off-topic changes (use other PRs for other changes).
  • Applied any dependency changes to both setup.cfg and conda-environment.yml.
  • Already covered by existing tests.
  • Appropriate change log entry included.
  • No documentation update required.

@dwsutherland dwsutherland self-assigned this Jun 29, 2022
@dwsutherland dwsutherland added this to the cylc-8.0rc4 milestone Jun 29, 2022
Copy link
Member

@wxtim wxtim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Looks sane, but I'm not very familiar with the data store manage.
  • Works when tried out.

@hjoliver hjoliver merged commit ae2cf46 into cylc:master Jun 30, 2022
@hjoliver hjoliver modified the milestones: cylc-8.0rc4, cylc-8.0.0 Jul 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

data store: waiting jobs
3 participants