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

Expose garbageCollection setting in kubernetes variable as a helm value #1173

Closed
ben-dov opened this issue Aug 15, 2024 · 2 comments · Fixed by #1175
Closed

Expose garbageCollection setting in kubernetes variable as a helm value #1173

ben-dov opened this issue Aug 15, 2024 · 2 comments · Fixed by #1175
Labels
enhancement New feature or request

Comments

@ben-dov
Copy link
Contributor

ben-dov commented Aug 15, 2024

Is your feature request related to a problem? Please describe

I find that if jenkins controller pod restarts when a worker pod is still Pending, then the resulting worker cannot connect to the controller.
It seems the worker is stuck in a loop where it tries to connect and must be terminated.

Describe the solution you'd like

The kubernetes plugin allows configuring a garbage collection that seems to delete these pods, but there is no way to configure it from the helm values when setting controller.JCasC.defaultConfig to true. In that case, attempting to use controller.JCasC.defaultConfig just creates another cloud.

I would like to to be able to configure the functionality without setting controller.JCasC.defaultConfig to false.

Describe alternatives you've considered

I can set controller.JCasC.defaultConfig to false, but it will be a non trivial refactor.

Additional context

the change it self should be pretty small, just adding the following lines to the the helm values

controller:
  garbageCollection:
    enabled: false
    namespaces: ""
    timeout: 300

And relevant condition in the _helpers.tpl that sets jenkins.casc.defaults

@ben-dov ben-dov added the enhancement New feature or request label Aug 15, 2024
@timja
Copy link
Member

timja commented Aug 15, 2024

A pull request would be welcome

@ben-dov
Copy link
Contributor Author

ben-dov commented Aug 18, 2024

@timja created 😃

@timja timja closed this as completed Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants