-
Notifications
You must be signed in to change notification settings - Fork 65
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
Update UBC EOAS resource requests based on observed usage #3173
Conversation
Merging this PR will trigger the following deployment actions. Support and Staging deployments
Production deployments
|
memory: | ||
# Based on measurements of observed usage | ||
guarantee: 1G | ||
limit: 1G |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we increase the limit a touch for both courses? There are a few notebooks that they will use later in the term that might exceed 1G (just barely, but they will: specifically, for 350 it is the notebook https://github.com/geoscixyz/gpgLabs/blob/main/notebooks/dcip/DC_SurveyDataInversion.ipynb). For EOSC 350, 2G would be plenty and we shouldn't exceed that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lheagy based on comments from Henryk, these are now set to 4G as they requested that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After adjusting to two comments, this LGTM!
Ref https://2i2c.freshdesk.com/a/tickets/955 for measurement and reasoning
Co-authored-by: Erik Sundell <[email protected]>
d7b4079
to
13ba4f7
Compare
🎉🎉🎉🎉 Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/6333922509 |
Ref https://2i2c.freshdesk.com/a/tickets/955 for measurement and reasoning