Skip to content

Conversation

@naved001
Copy link
Contributor

@naved001 naved001 commented Sep 2, 2025

Some containers in the RHOAI application run with 64Mi of memory. The original reason we set these minimum is to prevent people from running containers without any cpu or memory request.

Some containers in the RHOAI application run with 64Mi of memory.
The original reason we set these minimum is to prevent people from running
containers without any cpu or memory request.
@joachimweyl
Copy link

@DanNiESh do you have any reasons this should not be included in the Sep 9th maintenance?

@DanNiESh
Copy link

DanNiESh commented Sep 2, 2025

@DanNiESh do you have any reasons this should not be included in the Sep 9th maintenance?

Yes. Without this PR, students will get errors when trying to creating notebooks. We need to quickly make them access the edu cluster without having issues when creating notebooks.

@DanNiESh
Copy link

DanNiESh commented Sep 2, 2025

Can we get this merged today? It's better to let students get access to their notebooks ASAP

@joachimweyl
Copy link

@knikolla my understanding is that this requires a maintenance to merge. Is that correct?

@DanNiESh
Copy link

DanNiESh commented Sep 3, 2025

@DanNiESh do you have any reasons this should not be included in the Sep 9th maintenance?

Yes. Without this PR, students will get errors when trying to creating notebooks. We need to quickly make them access the edu cluster without having issues when creating notebooks.

@knikolla my understanding is that this requires a maintenance to merge. Is that correct?

If that's the case, we might need to manually set those 18 student namespaces limitrange.

@DanNiESh
Copy link

DanNiESh commented Sep 3, 2025

I've manually edit the limit range for student namespaces, so no rush to merge this PR now.

@jtriley jtriley requested a review from knikolla September 3, 2025 17:22
@knikolla knikolla merged commit 6a88cbb into nerc-project:main Sep 8, 2025
4 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.

6 participants