Skip to content

Conversation

@parnniti
Copy link
Contributor

Add image_maximum_gc_age and image_minimum_gc_age fields to KubernetesSettingsV1 struct to allow configuration of garbage collection age thresholds for container images.

Issue #, if available:
Depending on this issue bottlerocket-os/bottlerocket#4544

Description of changes:
I want to update kubelet fields of image_maximum_gc_age and image_minimum_gc_age following this docs https://kubernetes.io/docs/reference/config-api/kubelet-config.v1beta1/

@ytsssun
Copy link
Contributor

ytsssun commented Jun 13, 2025

Thanks for cutting us this PR. To proceed with the change, you would also need to bump the model version in Cargo.toml.

You can refer to this PR for the relevant change - https://github.com/bottlerocket-os/bottlerocket-settings-sdk/pull/80/files

@ytsssun
Copy link
Contributor

ytsssun commented Jun 13, 2025

Can you fix the lint warning?

You would also need to open up a PR in core-kit like this - bottlerocket-os/bottlerocket-core-kit#541

If your time allows, feel free to open one for that. If not we can work on that, but the work would need to be prioritized.

@koooosh koooosh force-pushed the feat/kubernetes-add-kubelet-image-gc-age branch 2 times, most recently from d8406bf to 83290a8 Compare October 8, 2025 23:52
@koooosh koooosh force-pushed the feat/kubernetes-add-kubelet-image-gc-age branch from 83290a8 to c2eeb44 Compare October 8, 2025 23:56
@koooosh
Copy link
Contributor

koooosh commented Oct 8, 2025

^ Force push edits commit msg to appease linter. Will merge this before my PR here: #104

@koooosh koooosh mentioned this pull request Oct 9, 2025
@koooosh koooosh merged commit 9306966 into bottlerocket-os:develop Oct 10, 2025
2 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.

4 participants