Skip to content

Commit d45e7cc

Browse files
authored
Merge pull request kubernetes#4426 from haircommander/bump-4210-beta
KEP-4210: retarget to beta for 1.30
2 parents b63e9ae + 706d265 commit d45e7cc

File tree

3 files changed

+13
-8
lines changed

3 files changed

+13
-8
lines changed
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
11
kep-number: 4210
22
alpha:
33
approver: "@johnbelamaric"
4+
beta:
5+
approver: "@johnbelamaric"

keps/sig-node/4210-max-image-gc-age/README.md

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -208,12 +208,14 @@ Additional tests will be added to this file to cover the garbage collection e2e.
208208

209209
- Configuration field added to the Kubelet (disabled by default)
210210
- Feature supported by Kubelet Image Manager
211-
- Unit tests and e2e tests added
211+
- Unit tests
212212
- Add a metric `kubelet_image_garbage_collected_total` which tracks the number of images the kubelet is GC'ing through any mechanism.
213213

214214
#### Beta
215215

216-
- Gather feedback from users
216+
- Add e2e tests
217+
- Document `kubelet_image_garbage_collected_total` (a step missed in alpha)
218+
- Add "reason" field to `kubelet_image_garbage_collected_total` to allow distinguishing between GC reasons (space based or time based).
217219

218220
#### GA
219221

@@ -276,8 +278,8 @@ removed, so no running workloads can be affected.
276278

277279
###### What specific metrics should inform a rollback?
278280

279-
- `kubelet_image_garbage_collected_total` metric drastically (100x) increasing, indicating thrashing of the GC manager and
280-
images being pulled.
281+
- `kubelet_image_garbage_collected_total` metric drastically (100x) increasing, with the "reason" field being "age",
282+
indicating thrashing of the GC manager and images being pulled.
281283

282284
###### Were upgrade and rollback tested? Was the upgrade->downgrade->upgrade path tested?
283285

@@ -292,7 +294,7 @@ No.
292294
###### How can an operator determine if the feature is in use by workloads?
293295

294296
- Verify the Kubelet Configuration with the Kubelet's configz endpoint
295-
- Monitor the `kubelet_image_garbage_collected_total`, and expect a slight increase.
297+
- Monitor the `kubelet_image_garbage_collected_total`, and expect some images are removed for reason "age"
296298

297299
###### How can someone using this feature know that it is working for their instance?
298300

@@ -302,7 +304,6 @@ No.
302304
###### What are the reasonable SLOs (Service Level Objectives) for the enhancement?
303305

304306
- The eventual default value should increase the average `kubelet_image_garbage_collected_total` by no more than 10x
305-
- TODO: On what clusters?
306307

307308
###### What are the SLIs (Service Level Indicators) an operator can use to determine the health of the service?
308309

@@ -370,6 +371,7 @@ No
370371

371372

372373
2023-09-18: KEP opened, targeted at Alpha
374+
2024-01-22: KEP updated to Beta
373375

374376
## Drawbacks
375377

keps/sig-node/4210-max-image-gc-age/kep.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,16 +14,17 @@ approvers:
1414

1515

1616
# The target maturity stage in the current dev cycle for this KEP.
17-
stage: alpha
17+
stage: beta
1818

1919
# The most recent milestone for which work toward delivery of this KEP has been
2020
# done. This can be the current (upcoming) milestone, if it is being actively
2121
# worked on.
22-
latest-milestone: "v1.29"
22+
latest-milestone: "v1.30"
2323

2424
# The milestone at which this feature was, or is targeted to be, at each stage.
2525
milestone:
2626
alpha: "v1.29"
27+
beta: "v1.30"
2728

2829
# The following PRR answers are required at alpha release
2930
# List the feature gate name and the components for which it must be enabled

0 commit comments

Comments
 (0)