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

Bump multus, cilium and flannel version #7091

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

manuelbuil
Copy link
Contributor

@manuelbuil manuelbuil commented Oct 22, 2024

Proposed Changes

Bump multus, cilium and flannel charts so that we use the latest version of cni-plugins.

Flannel version includes windows and linux

Types of Changes

Version bump

Verification

Deploy those CNIs and verify that things are still working:

  • Cilium
  • Multus (with whatever other CNI plugin)
  • Flannel in both windows and linux

Testing

Linked Issues

#7090

User-Facing Change


Further Comments

@manuelbuil manuelbuil requested a review from a team as a code owner October 22, 2024 11:26
@codecov-commenter
Copy link

codecov-commenter commented Oct 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 23.38%. Comparing base (b173ef5) to head (6cbb183).
Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #7091       +/-   ##
===========================================
+ Coverage    8.87%   23.38%   +14.51%     
===========================================
  Files          33       33               
  Lines        3438     3438               
===========================================
+ Hits          305      804      +499     
+ Misses       3110     2587      -523     
- Partials       23       47       +24     
Flag Coverage Δ
inttests 8.87% <ø> (ø)
unittests 16.62% <ø> (?)

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.

@manuelbuil
Copy link
Contributor Author

On hold, waiting for @brooksn to explain this PR: rancher/rke2-charts#548. Maybe he found problems in the new version?

@cwayne18
Copy link
Member

@manuelbuil is this to fix a specific CVE or is this just generally a "good hygiene" bump

@cwayne18
Copy link
Member

/trivy

Copy link
Contributor

❌ Trivy scan action failed, check logs ❌

@manuelbuil
Copy link
Contributor Author

@manuelbuil is this to fix a specific CVE or is this just generally a "good hygiene" bump

a "good hygiene" bump

@manuelbuil manuelbuil force-pushed the updatecni branch 2 times, most recently from 76aa9b0 to 063a121 Compare November 5, 2024 09:39
@manuelbuil manuelbuil changed the title Bump multus, cilium and flannel charts Bump multus, cilium and flannel version Nov 5, 2024
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.

6 participants