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

Cut AWS testing costs by 75% #5162

Closed
unito-bot opened this issue Nov 15, 2024 · 7 comments
Closed

Cut AWS testing costs by 75% #5162

unito-bot opened this issue Nov 15, 2024 · 7 comments
Assignees

Comments

@unito-bot
Copy link

┆Issue is synchronized with this Jira Story
┆Issue Number: TOIL-1673

@unito-bot
Copy link
Author

➤ Adam Novak commented:

We should reduce the number of CI runs or the fraction of CI runs that hit AWS for running clusters. We need to look at the AWS billing reports to make sure really we are spending all this on compute for the tests that use clusters, like we think we are.

To reduce CI runs we could not bump every PR when any other PR merges, or we could not run the same tests both before and after a merge, or we could not require a passing back-merged commit in the branch to merge.

@unito-bot
Copy link
Author

➤ Adam Novak commented:

We could try and restrict dependabot: it has run tests 7 times to test an htcondor update that isn’t even under test.

@unito-bot
Copy link
Author

➤ Adam Novak commented:

Maybe dependabot can be told not to update particular packages (google, htcondor, dev stuff like flake8, cluster-integration packages) that we know aren’t under CI.

@unito-bot
Copy link
Author

➤ Adam Novak commented:

We could also make dependabot auto-merge bumps to dependencies that are actually tested instead of waiting for us to take it and making more test runs while we ignore it.

@adamnovak adamnovak mentioned this issue Nov 20, 2024
19 tasks
@unito-bot
Copy link
Author

➤ Adam Novak commented:

To actually decide if this worked we might need to compare a before month and an after month, and control for time somehow.

@unito-bot
Copy link
Author

➤ Adam Novak commented:

We think this probably has been successful.

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

No branches or pull requests

2 participants