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

cylc clean: continue if remote clean fails? #4935

Closed
hjoliver opened this issue Jun 27, 2022 · 5 comments
Closed

cylc clean: continue if remote clean fails? #4935

hjoliver opened this issue Jun 27, 2022 · 5 comments
Labels
bug Something is wrong :(

Comments

@hjoliver
Copy link
Member

hjoliver commented Jun 27, 2022

If cylc clean can't clean a remote job platform used (or attempted) by the target workflow, it should continue on and clean other job platforms, and locally, rather than abort immediately.

I'm reasonably sure this is a bug and not a feature(?)

E.g. install and run a workflow that uses a non-existent job platform, then try to clean it.

@hjoliver hjoliver added the bug Something is wrong :( label Jun 27, 2022
@hjoliver hjoliver added this to the cylc-8.0rc4 milestone Jun 27, 2022
@hjoliver
Copy link
Member Author

see also #4673

@oliver-sanders
Copy link
Member

oliver-sanders commented Jun 29, 2022

It should continue onto other platforms, yes, however, it should not clean locally because doing so would delete the database prohibiting cylc clean from being able to remove the remote directory at a later date e.g. if a login node goes down.

@oliver-sanders oliver-sanders added could be better Not exactly a bug, but not ideal. bug Something is wrong :( and removed bug Something is wrong :( could be better Not exactly a bug, but not ideal. labels Jun 29, 2022
@oliver-sanders
Copy link
Member

If you try to run a task on a non-existant platform, then, ideally, cylc clean should not attempt to clean that platform since it never installed on it in the first place. But cylc clean does not have access to the remote-init map, perhaps it should, it wouldn't hurt to store remote-init events in the DB?

@oliver-sanders
Copy link
Member

(taking a quick look at the broken platform side of this)

@oliver-sanders
Copy link
Member

This has now been fixed by other changes, see #4950 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong :(
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants