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

system instance "dry run" option #6469

Open
chu11 opened this issue Dec 3, 2024 · 1 comment
Open

system instance "dry run" option #6469

chu11 opened this issue Dec 3, 2024 · 1 comment

Comments

@chu11
Copy link
Member

chu11 commented Dec 3, 2024

before loading (or re-loading) a new set of configs for a system instance, it would be nice to be able to 'dry run' the configs to ensure they parse correctly, seem to make sense, etc.

My initial thought was to support some "dry run" config option that could effectively run sleep 0 as the initial process, thus exiting immediately if everything worked. However, this technique probably wouldn't work for testing flux config reload.

not sure if a dry-run option could be parsed during flux config reload, and the broker would know "do not save this config, just report errors" or something.

just initial thoughts, haven't thought much more than this

@garlick
Copy link
Member

garlick commented Dec 3, 2024

Hmm, good thought. The bummer is that config is parsed separately by different components. If any part of a flux config reload fails, the caller will get an error but the "good parts" of the config would have been accepted. IOW a config update does not atomically roll back to the previous config on failure. Maybe it should.

I guess for a dry run there would need to be a flag passed with the config update to tell each component not to apply the change, just to check it. Not a bad idea!

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

No branches or pull requests

2 participants