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

Revisit the CYLC_VERSION use prior to setup.py #3081

Closed
kinow opened this issue Apr 6, 2019 · 3 comments
Closed

Revisit the CYLC_VERSION use prior to setup.py #3081

kinow opened this issue Apr 6, 2019 · 3 comments
Labels
bug? Not sure if this is a bug or not doc Documentation superseded

Comments

@kinow
Copy link
Member

kinow commented Apr 6, 2019

Prior to setuptools and setup.py, we used CYLC_VERSION to allow users to switch between versions of Cylc (as far as I understand it). After setuptools, it will be harder to maintain this feature.

But users might be able to have the same feature through virtual environments anaconda, etc.

This issue is a left over of #2989, to revisit its use, and document what must be done with it now.

Quoting @hjoliver comment from another issue:

ATM users can run different suites at different versions if necessary (e.g. long-running suite that's not upgraded yet, and a new suite written for new cylc) and every suite job invokes the right cylc version thanks to $CYLC_VERSION in the job environment, with cylc versions installed in (e.g.) in /opt/cylc/[VERSION].

@kinow kinow added bug? Not sure if this is a bug or not doc Documentation labels Apr 6, 2019
@kinow kinow added this to the cylc-8.0a1 milestone Apr 6, 2019
@matthewrmshin
Copy link
Contributor

Yes. The variable is used in remote commands such as when submitting jobs on remote hosts. The variable allows the correct version of Cylc to be chosen with login profiles, e.g. by loading the correct module or Python environment. If not Cylc version, we'll need a similar mechanism for a remote job host to recognise that it needs to load a matching Python environment before doing anything.

@kinow
Copy link
Member Author

kinow commented Apr 6, 2019

Hummmm, so looks like virtualenv/conda/etc are not a drop-in replacement for Cylc version 😟

@hjoliver
Copy link
Member

I'm absorbing this into #3150

@kinow kinow removed this from the cylc-8.0a1 milestone May 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug? Not sure if this is a bug or not doc Documentation superseded
Projects
None yet
Development

No branches or pull requests

3 participants