Make sure we use python from /usr/local environment in crontab proxy renewal#1588
Open
khurtado wants to merge 1 commit intodmwm:masterfrom
Open
Make sure we use python from /usr/local environment in crontab proxy renewal#1588khurtado wants to merge 1 commit intodmwm:masterfrom
khurtado wants to merge 1 commit intodmwm:masterfrom
Conversation
todor-ivanov
reviewed
Apr 2, 2025
|
|
||
| ### The main manage script for WMAgent | ||
| WMCoreVersion=$(python -c "from WMCore import __version__ as WMCoreVersion; print(WMCoreVersion)") | ||
| WMCoreVersion=$(/usr/local/bin/python -c "from WMCore import __version__ as WMCoreVersion; print(WMCoreVersion)") |
Contributor
There was a problem hiding this comment.
@khurtado this may provide a solution for some containers, but will break many things in the virtual environment and for T0.
Could we avoid using hardcoded && absolute paths in our setups. we have a very well defined environment now and we must always start from the root of the WMAgent deployment, which is referred through: the $WMA_DEPLOY_DIR variable.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is related to the MM comment below:
https://mattermost.web.cern.ch/cms-o-and-c/pl/t816wz1xpfyzjmdzzmio1nx99h
When proxies are renewed via crontab,
/usr/bin/pythonis used, but we deploy our python in/usr/local/bin.Solutions are to either enforce
/usr/local/binas part of PATH in the environment that crontab sees while executing this, or simply enforcing/usr/local/bin/python, which is the python version we use. This solution does the latter, as it is the simplest one.If the future python 3.12 will be installed in
/usr/local/binas well, this should be compatible.