Skip to content

Conversation

anjani-bhat
Copy link
Contributor

@anjani-bhat anjani-bhat commented Aug 14, 2025

Ticket: DEVPROD-19870
This PR is for cutting instance sizes to use high memory variants. This should result in ~ 15k savings in a year.

Proof of Work

Here are the successful patches for the variants which were downsized:
e2e_om60_kind_ubi
patch
This results in the variant being 3-4% slower (considering total duration not wall time) on an average.
The longest task went from
47m 58s to 48m 53s, so the wall clock time change would be ~2%. The cost savings would be ~30-32%.

e2e_om70_kind_ubi
patch
This results in the variant being 3-4% slower (considering total duration not wall time) on an average.
The longest task went from
48m 12s to 47m 8s, so the wall clock time shouldn't change by much. The cost savings would be ~32%.

e2e_om80_kind_ubi
patch
This results in the variant being 3-4% slower (considering total duration not wall time) on an average.
The longest task went from
45m 2s to 45m 51s, so the wall clock time should change be 1-2%. The cost savings would be ~30-32%.

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you added changelog file?

@anjani-bhat anjani-bhat changed the title Devprod 19870 part2 DEVPROD-19870: Use high memory instances to cut instance size where possible Aug 14, 2025
@anjani-bhat anjani-bhat marked this pull request as ready for review August 21, 2025 18:03
@anjani-bhat anjani-bhat requested a review from a team as a code owner August 21, 2025 18:03
@anjani-bhat
Copy link
Contributor Author

@anandsyncs: I don't have write access to this repo, so could you add the skip-changelog label for this PR? Also let me know if anything else needs to be done for this PR. Thank you !

Copy link
Collaborator

@nammn nammn left a comment

Choose a reason for hiding this comment

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

Thanks!
Have you've checked the same for e2e_static_om.*_kind_ubi variants?

And have you've considered verifying via multiple patches?

@nammn nammn added the skip-changelog Use this label in Pull Request to not require new changelog entry file label Aug 22, 2025
@anjani-bhat anjani-bhat changed the title DEVPROD-19870: Use high memory instances to cut instance size where possible DEVPROD-19870: Downsize instances where possible Aug 22, 2025
@anjani-bhat
Copy link
Contributor Author

Have you've checked the same for e2e_static_om.*_kind_ubi variants?

Thanks for calling this out. Downsized e2e_static_om60_kind_ubi as well. The other e2e_static_om.*_kind_ubi variants are already on the smaller variants.

And have you've considered verifying via multiple patches?

Yes, here are the patches:

Copy link
Collaborator

@nammn nammn left a comment

Choose a reason for hiding this comment

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

thanks a lot!
But why not also update

e2e_static_om70_kind_ubi and e2e_static_om80_kind_ubi?

Anyway, please update them as well if patches show the same effect

@anjani-bhat
Copy link
Contributor Author

anjani-bhat commented Aug 25, 2025

e2e_static_om70_kind_ubi and e2e_static_om80_kind_ubi?

They are already updated
https://github.com/mongodb/mongodb-kubernetes/blob/master/.evergreen.yml#L1362
https://github.com/mongodb/mongodb-kubernetes/blob/master/.evergreen.yml#L1385

@anandsyncs
Copy link
Contributor

evergreen retry

@anandsyncs
Copy link
Contributor

@anjani-bhat You probably have collaborator access, can you please raise this PR directly from this repo?

@anjani-bhat
Copy link
Contributor Author

@anjani-bhat You probably have collaborator access, can you please raise this PR directly from this repo?

@anandsyncs I don't have permissions to push to mongodb/mongodb-kubernetes. I can reach out to @mircea-cosbuc since he was able to merge the last PR.

@mircea-cosbuc mircea-cosbuc merged commit b90c986 into mongodb:master Aug 26, 2025
32 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Use this label in Pull Request to not require new changelog entry file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants