-
Notifications
You must be signed in to change notification settings - Fork 6.5k
chore: upgrade Go from 1.24.1 to 1.24.3 #23026
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
Conversation
Updates Go version across all workflows, Dockerfile and go.mod to address an os.Root related security fix. Signed-off-by: Ville Vesilehto <[email protected]>
❌ Preview Environment deleted from BunnyshellAvailable commands (reply to this comment):
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #23026 +/- ##
==========================================
+ Coverage 60.01% 60.03% +0.02%
==========================================
Files 343 343
Lines 57846 57846
==========================================
+ Hits 34717 34730 +13
+ Misses 20361 20346 -15
- Partials 2768 2770 +2 ☔ View full report in Codecov by Sentry. |
blakepettersson
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Ville Vesilehto <[email protected]> Signed-off-by: Kanika Rana <[email protected]>
Signed-off-by: Ville Vesilehto <[email protected]> Signed-off-by: Oliver Gondža <[email protected]>
Signed-off-by: Ville Vesilehto <[email protected]> Signed-off-by: Lyheng <[email protected]>
Signed-off-by: Ville Vesilehto <[email protected]>
Signed-off-by: Ville Vesilehto <[email protected]>
|
/cherry-pick release-3.0 |
|
@thevilledev would you mind manually raising the cherry pick PRs for v2.14 and v3.0? |
|
@nitishfy sure thing, I'll do that later today! |
|
@blakepettersson and @nitishfy, just realised Go 1.24.4 will be released tomorrow. It includes one private (yet to be disclosed) CVE fix, so upgrading might be a high priority thing tomorrow. Shall I open those 1.24.4 upgrades for all affected branches then instead of 1.24.3? |
|
@thevilledev I'm totally fine with waiting for that release |
Signed-off-by: Ville Vesilehto <[email protected]> Signed-off-by: dsuhinin <[email protected]>
Signed-off-by: Ville Vesilehto <[email protected]> Signed-off-by: dsuhinin <[email protected]>
Description
Updates Go version across all workflows, Dockerfile and go.mod to address an
os.Rootrelated security fix.This functionality is being used by #22936 (merged) and #22956 (draft PR).
For older ArgoCD release branches it seems there's no need to cherry-pick, as they operate on older Go versions than 1.24 (where
os.Rootwas originally introduced).Checklist