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

Upgrade: metrics server version bump from v0.5.0 to v0.5.2 #4867

Merged
merged 2 commits into from
Jan 18, 2022
Merged

Upgrade: metrics server version bump from v0.5.0 to v0.5.2 #4867

merged 2 commits into from
Jan 18, 2022

Conversation

uGiFarukh
Copy link
Contributor

@uGiFarukh uGiFarukh commented Jan 5, 2022

Proposed Changes

Types of Changes

Bugfix

Verification

v0.5.2

Linked Issues

User-Facing Change

Update metrics-server to 0.5.2

Further Comments

The upgraded version of metrics server is a good to have in the latest releases of k3s because it fixes one elevated permissions security issue along with some minor bug fixes.

@uGiFarukh uGiFarukh requested a review from a team as a code owner January 5, 2022 03:11
@brandond
Copy link
Member

brandond commented Jan 5, 2022

Thanks for the contribution! Don't forget about the image list at https://github.com/k3s-io/k3s/blob/master/scripts/airgap/image-list.txt

@brandond
Copy link
Member

This ended up needing more involved fixes to track the upstream manifest changes that come with updating the image. I've pushed to your branch, and will backport to older branches.

uGiFarukh and others added 2 commits January 15, 2022 01:33
Kubernetes Metrics Server version upgrade with the following changes:

Remove elevated permissions cap_net_bind_service from metrics-server

Fix bad architecture metadata

Fix reporting inconsistent number of nodes, caused by using invalid Kubelet node start time metric

Fix reporting partial container metrics for pod

Signed-off-by: Farukh Khan <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jan 15, 2022

Codecov Report

Merging #4867 (0c28f2b) into master (48ffed3) will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4867      +/-   ##
==========================================
- Coverage   10.52%   10.45%   -0.08%     
==========================================
  Files         142      142              
  Lines       10250    10323      +73     
==========================================
  Hits         1079     1079              
- Misses       8943     9016      +73     
  Partials      228      228              
Flag Coverage Δ
unittests 10.45% <ø> (-0.08%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/deploy/zz_generated_bindata.go 0.00% <ø> (ø)
pkg/etcd/etcd.go 13.40% <0.00%> (-1.08%) ⬇️
pkg/cli/cmds/server.go 0.00% <0.00%> (ø)
pkg/cli/server/server.go 0.00% <0.00%> (ø)
pkg/daemons/config/types.go 50.00% <0.00%> (ø)
pkg/cli/cmds/etcd_snapshot.go 0.00% <0.00%> (ø)
pkg/cli/etcdsnapshot/etcd_snapshot.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 48ffed3...0c28f2b. Read the comment docs.

@brandond brandond merged commit 27fe2c3 into k3s-io:master Jan 18, 2022
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