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

Add ope-course-quota resourceQuota to cluster-scope components #341

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

dystewart
Copy link
Contributor

Closes: nerc-project/operations#377

Adding this resourceQuota as a component at the cluster scope and adding it to the rhods-notebooks namespace in the prod cluster. The quota values themselves were determined based on our resourceQuota in the namespace where we did scale testing, with a bump to accomodate all the expected students

Closes: nerc-project/operations#377

Adding this resourceQuota as a component at the cluster scope and adding it to
the rhods-notebooks namespace in the prod cluster. The quota values themselves were
determined based on our resourceQuota in the namespace where we did scale testing,
with a bump to accomodate all the expected students
Copy link
Member

@computate computate left a comment

Choose a reason for hiding this comment

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

Clever to add a resource quota to prevent large allocations.

@dystewart
Copy link
Contributor Author

Most of our resourceQuotas are created via ColdFront so this is a bit of a one-off but I think it makes the most sense to add this as a cluster-scope component

Copy link
Contributor

@IsaiahStapleton IsaiahStapleton left a comment

Choose a reason for hiding this comment

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

Looks good

@hpdempsey
Copy link

Looks good. Also something @schwesig should take into account for the class monitoring and alerting events, so we know when we are "approaching" a quota problem.

@schwesig
Copy link
Contributor

Looks good. Also something @schwesig should take into account for the class monitoring and alerting events, so we know when we are "approaching" a quota problem.

ack

Copy link
Contributor

@schwesig schwesig left a comment

Choose a reason for hiding this comment

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

Follows discussion in slack, makes sense👍

@dystewart dystewart merged commit 8641f62 into OCP-on-NERC:main Jan 15, 2024
2 checks passed
@dystewart dystewart deleted the ope-quota branch April 4, 2024 14:36
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.

Create resourceQuota in rhods-notebooks namespace (Prod cluster)
5 participants