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

Remove secondary traces (Merge when deploying new cloud) #283

Merged

Conversation

mira-miracoli
Copy link
Contributor

@mira-miracoli mira-miracoli commented Mar 20, 2024

This PR is removing all htcondor-secondary traces.
Everything now uses the condor 23 version and manager on build.galaxyproject.eu.

  • renames machines
  • removes the htcondor-secondary tag and makes it the default (it was there to keep the old condor config that we don't need anymore)
  • removes the script (htcondor-mirgation.py) that creates a new resources.yml with secondary tags and images according to the set percentage
  • update schema.yml and remove htcondor-secondary tag

What it does not is changing the images to Rocky 9. This should be done before we roll out the new cloud.

@mira-miracoli mira-miracoli changed the title Remove secondary traces Remove secondary traces (Merge when rolling out new cloud) Mar 20, 2024
@mira-miracoli mira-miracoli changed the title Remove secondary traces (Merge when rolling out new cloud) Remove secondary traces (Merge when deploying new cloud) Mar 20, 2024
@mira-miracoli mira-miracoli marked this pull request as draft March 21, 2024 08:18
@mira-miracoli mira-miracoli marked this pull request as ready for review March 27, 2024 13:48
Copy link
Member

@sanjaysrikakulam sanjaysrikakulam left a comment

Choose a reason for hiding this comment

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

Cleanup looks good. Thank you!

@sanjaysrikakulam sanjaysrikakulam merged commit e79c428 into usegalaxy-eu:main Mar 28, 2024
2 checks passed
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.

2 participants