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

conf: hierarchical configuration path #199

Closed
oliver-sanders opened this issue Apr 6, 2021 · 4 comments · Fixed by #301
Closed

conf: hierarchical configuration path #199

oliver-sanders opened this issue Apr 6, 2021 · 4 comments · Fixed by #301
Assignees
Labels

Comments

@oliver-sanders
Copy link
Member

Cylc Flow has a configuration file hierarchy which allows us to provide different configuration for different installed versions of Cylc E.G:

  • ~/.cylc/flow/global.cylc
  • ~/.cylc/flow/8/global.cylc
    *~/.cylc/flow/8.0/global.cylc

The UIS/Hub could probably do with the same treatment.

I would make more sense to use the Cylc Flow (meta) version for this hierarchy because:

  • It is already used for this purpose.
  • The UI Server version isn't user-facing (visible via cylc version --long).
  • The Hub version is obscure (implementation detail visible via pip list).

Pull requests welcome!

@oliver-sanders oliver-sanders added this to the cylc-uiserver 1.0.0 milestone Apr 6, 2021
@oliver-sanders oliver-sanders added the question Flag this as a question for the next Cylc project meeting. label Apr 27, 2021
@MetRonnie
Copy link
Member

Is this not already implemented? E.g. see #252

@oliver-sanders
Copy link
Member Author

No, this issue is about adding the version number to the configuration hierarchy.

@oliver-sanders
Copy link
Member Author

Should be easy, can copy the solution from cylc-flow. This will protect sites from future interface changes.

@oliver-sanders
Copy link
Member Author

Update: The meta package is no more.

Meeting: Change the config dir from hub to uiserver (#291) and add the uiserver to the hierarchy.

~/.cylc/uiserver/<uiserver-version>

@datamel datamel self-assigned this Jan 27, 2022
@oliver-sanders oliver-sanders removed the question Flag this as a question for the next Cylc project meeting. label Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants