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

Configure Galaxy to be able to access the secondary HTCondor cluster #981

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

kysrpex
Copy link
Contributor

@kysrpex kysrpex commented Nov 7, 2023

Add a runner and TPV destinations for the secondary HTCondor cluster. Sending jobs to the secondary cluster is controlled via the condor-secondary TPV scheduling tag.

TPV destinations are automatically duplicated by Jinja. It creates extra destinations whose name is prefixed with secondary_, makes them inherit from the original destination, replaces the runner, and makes them require the condor-secondary scheduling tag.

job_conf.yml.j2 was also modified so that concurrency limits affecting condor destinations also get cloned.

This commit also lets the Galaxy user invoke the appropriate condor_* commands using systemd-run.

 Add a runner and TPV destinations for the secondary HTCondor cluster. Sending jobs to the secondary cluster is controlled via the `condor-secondary` TPV scheduling tag.

TPV destinations are automatically duplicated by Jinja. It creates extra destinations whose name is prefixed with `secondary_`, makes them inherit from the original destination, replaces the runner, and makes them require the `condor-secondary` scheduling tag.

`job_conf.yml.j2` was also modified so that concurrency limits affecting condor destinations also get cloned.

This commit also lets the Galaxy user invoke the appropriate `condor_*` commands using `systemd-run`.
@kysrpex
Copy link
Contributor Author

kysrpex commented Nov 7, 2023

Once this is merged I can start sending my own jobs there.

Just check if this is good enough in terms of it not disrupting your workflow.

Copy link
Contributor

@mira-miracoli mira-miracoli left a comment

Choose a reason for hiding this comment

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

I think I learned something new in jinja2, looks cool :)

@kysrpex kysrpex merged commit f34a332 into usegalaxy-eu:master Nov 8, 2023
4 checks passed
@kysrpex kysrpex deleted the htcondor_secondary_enable branch November 8, 2023 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants