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

Avoid -u parameter when using the OS crontab for WMAgent #1564

Merged

Conversation

todor-ivanov
Copy link
Contributor

@todor-ivanov todor-ivanov commented Nov 14, 2024

Fixes: dmwm/WMCore#12166

With the current change we no longer try to use the -u parameter when the OS crontab is used. This is vital for the virtual environment setups, because it breaks with:

must be privileged to use -u

Related PRs

dmwm/WMCore#12167

@todor-ivanov
Copy link
Contributor Author

I tested it for prod WMAgnts. @LinaresToine could you please test it also for T0 ?

@LinaresToine
Copy link

The changes work correctly during a deployment of the agent. Thank you for providing this solution @todor-ivanov

Copy link
Member

@mapellidario mapellidario left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks todor, looks good to me!

@todor-ivanov
Copy link
Contributor Author

todor-ivanov commented Nov 21, 2024

ok, While further testing the solution I figured out two more, minor but important differences between the docker and virtual env cronjobs at runtime. I resolved them in the following two commits. @mapellidario @LinaresToine please take another look, before we merge.

Copy link
Member

@mapellidario mapellidario left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Todor, looks good to me!

@todor-ivanov todor-ivanov merged commit 1469215 into dmwm:master Nov 25, 2024
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

Successfully merging this pull request may close these issues.

Setup of cron jobs fail during initialization of the agent
3 participants