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

Update coredns to 1.11.3 and metrics-server to 0.7.2 #10760

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

harsimranmaan
Copy link
Contributor

@harsimranmaan harsimranmaan commented Aug 27, 2024

Proposed Changes

Used https://github.com/coredns/corefile-migration to migrate the corefile. There are no changes for the default file from 1.10.1 to 1.11.3.

Notable plugin changes include the k8s_external with fallthrough option and rewrite with cname_target option.

These changes are not part of the default config that ships with k3s. Customers using these two plugins can start using the new options

Requires

Types of Changes

version bump

Verification

Check image version

Testing

Linked Issues

User-Facing Change


Further Comments

@harsimranmaan
Copy link
Contributor Author

/retest

@brandond
Copy link
Contributor

brandond commented Aug 29, 2024

  • You need to run make download && make generate, and then check in the changes to the embedded file.
  • You need to update the image tag in the airgap list text file

Also, don't use "fixes" or "resolves" before an issue in your comment, as that will auto-close the issue. We need it left open for QA to validate and close.

@brandond brandond changed the title Update coredns to 1.11.3 Update coredns to 1.11.3 and metrics-server to 0.7.2 Aug 29, 2024
brandond
brandond previously approved these changes Aug 29, 2024
dereknola
dereknola previously approved these changes Aug 29, 2024
Copy link

codecov bot commented Aug 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.67%. Comparing base (bd45aa5) to head (fb34e5c).
Report is 1 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (bd45aa5) and HEAD (fb34e5c). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (bd45aa5) HEAD (fb34e5c)
e2etests 7 6
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #10760      +/-   ##
==========================================
- Coverage   49.83%   43.67%   -6.17%     
==========================================
  Files         178      178              
  Lines       14753    14753              
==========================================
- Hits         7352     6443     -909     
- Misses       6062     7121    +1059     
+ Partials     1339     1189     -150     
Flag Coverage Δ
e2etests 36.15% <ø> (-10.02%) ⬇️
inttests 19.74% <ø> (ø)
unittests 13.59% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@brandond brandond dismissed stale reviews from dereknola and themself via 80b7550 August 29, 2024 21:04
brandond
brandond previously approved these changes Aug 29, 2024
dereknola
dereknola previously approved these changes Aug 29, 2024
Used https://github.com/coredns/corefile-migration to
migrate the corefile. There are no changes for the
default file from 1.10.1 to 1.11.3.

Notable plugin changes include the k8s_external with fallthrough option
and rewrite with cname_target option.

These changes are not part of the default config that ships
with k3s. Customers using these two plugins can start using the new options

Metrics does not have any new features other than build tooling updates.

Requires rancher/image-mirror#704

Signed-off-by: Harsimran Singh Maan <maan.harry@gmail.com>
@brandond
Copy link
Contributor

thanks for the PR!

@brandond brandond merged commit 0b4d249 into k3s-io:master Aug 29, 2024
28 checks passed
@harsimranmaan harsimranmaan deleted the coredns branch August 29, 2024 23:13
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.

3 participants