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

Fix in-notebook config for culling kernels #518

Closed
1 task
consideRatio opened this issue Jul 14, 2021 · 4 comments · Fixed by #563
Closed
1 task

Fix in-notebook config for culling kernels #518

consideRatio opened this issue Jul 14, 2021 · 4 comments · Fixed by #563
Assignees

Comments

@consideRatio
Copy link
Contributor

consideRatio commented Jul 14, 2021

Description

I think the following config section is inactive. We should use the proper configuration so that this code gets loaded as-expected and kernels are culled properly.

https://github.com/2i2c-org/pilot-hubs/blob/1565a35c031e66fe083fd5c6056c986938496b11/hub-templates/basehub/values.yaml#L1-L15

Benefit

Currently we aren't properly configuring our notebook culling, and so this is largely causing many clusters to behave in unexpected ways. Moreover, the current state gives us confidence that we have configured it a certain way, even though this isn't actually true, which is particularly problematic.

Implementation

This kind of config can be replaced by hub.extraFiles. See the hub.extraFiles docs

Tasks to complete

@consideRatio consideRatio added the Task Actions that don't involve changing our code or docs. label Jul 14, 2021
@GeorgianaElena GeorgianaElena self-assigned this Jul 28, 2021
@damianavila
Copy link
Contributor

I don't see a trace of this config being referenced.

So that means we were not culling as we intended by that config stuff?

@consideRatio
Copy link
Contributor Author

Yepp, i think thats correct to say @damianavila!

@yuvipanda
Copy link
Member

Yep, we were only culling with the idle culler

@choldgraf choldgraf changed the title etcJupyter config not implemented? Use hub.extraFiles feature perhaps? Fix our culling logic Aug 6, 2021
@damianavila
Copy link
Contributor

Thanks for the confirmation, folks!

@choldgraf choldgraf added type: bug and removed Task Actions that don't involve changing our code or docs. labels Aug 6, 2021
@GeorgianaElena GeorgianaElena self-assigned this Aug 17, 2021
@yuvipanda yuvipanda changed the title Fix our culling logic Fix in-notebook config for culling kernels Aug 23, 2021
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 a pull request may close this issue.

5 participants