Skip to content

[3.0] toolkit golang version to go1.24.12#704

Merged
cheeyanglee merged 7 commits intoopen-edge-platform:3.0-devfrom
cheeyanglee:3.0-dev-toolkit-go1.24.12
Jan 30, 2026
Merged

[3.0] toolkit golang version to go1.24.12#704
cheeyanglee merged 7 commits intoopen-edge-platform:3.0-devfrom
cheeyanglee:3.0-dev-toolkit-go1.24.12

Conversation

@cheeyanglee
Copy link
Copy Markdown
Contributor

Merge Checklist

All boxes should be checked before merging the PR

  • The changes in the PR have been built and tested
  • [] cgmanifest file has been updated if required
  • Ready to merge

Description

The goal for this PR is to fix for CVE listed under #593 .

As ubuntu 22.04 does not officially support for golang 1.24 or newer version through apt package, we will take the golang binaries from upstream go.dev site .

disable vet as this would failed the test for known minor issue "non-constant format string in call"

This required changes in build host too.

Any Newly Introduced Dependencies

How Has This Been Tested?

dependabot bot and others added 4 commits January 27, 2026 10:32
Bumps the go_modules group with 1 update in the /toolkit/tools directory: [golang.org/x/crypto](https://github.com/golang/crypto).

Updates `golang.org/x/crypto` from 0.36.0 to 0.45.0
- [Commits](golang/crypto@v0.36.0...v0.45.0)
as the toolkit were expected to run with golang 1.23, some of the
component and dependencies does not met the 1.24 vet requirement hence
skip this.

these are currently reported error

pkgworker/pkgworker.go:427:22: non-constant format string in call to fmt.Errorf
imagegen/installutils/installutils.go:2079:21: non-constant format string in call to (*github.com/sirupsen/logrus.Logger).Warnf
imagegen/installutils/installutils.go:3142:20: non-constant format string in call to (*github.com/sirupsen/logrus.Logger).Warnf
graphpkgfetcher/graphpkgfetcher.go:315:79: non-constant format string in call to (*github.com/sirupsen/logrus.Logger).Debugf
scheduler/schedulerutils/preparerequest.go:96:21: non-constant format string in call to (*github.com/sirupsen/logrus.Logger).Warnf
scheduler/schedulerutils/preparerequest.go:189:21: non-constant format string in call to (*github.com/sirupsen/logrus.Logger).Warnf
internal/packagerepo/repocloner/rpmrepocloner/rpmrepocloner.go:920:21: non-constant format string in call to fmt.Errorf
internal/packagerepo/repomanager/rpmrepomanager/rpmrepomanager.go:114:78: non-constant format string in call to fmt.Errorf

Signed-off-by: Lee Chee Yang <chee.yang.lee@intel.com>
Signed-off-by: Lee Chee Yang <chee.yang.lee@intel.com>
Signed-off-by: Lee Chee Yang <chee.yang.lee@intel.com>
@cheeyanglee cheeyanglee requested a review from a team as a code owner January 29, 2026 02:04
liulis-sg
liulis-sg previously approved these changes Jan 29, 2026
Copy link
Copy Markdown
Contributor

@liulis-sg liulis-sg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@andy-vm andy-vm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@andy-vm andy-vm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to fix the CI errors

Signed-off-by: Lee Chee Yang <chee.yang.lee@intel.com>
@cheeyanglee cheeyanglee force-pushed the 3.0-dev-toolkit-go1.24.12 branch 2 times, most recently from d1b8690 to b587927 Compare January 29, 2026 10:28
Signed-off-by: Lee Chee Yang <chee.yang.lee@intel.com>
@cheeyanglee cheeyanglee force-pushed the 3.0-dev-toolkit-go1.24.12 branch from b587927 to 763201a Compare January 29, 2026 10:49
@cheeyanglee cheeyanglee merged commit e730576 into open-edge-platform:3.0-dev Jan 30, 2026
17 of 21 checks passed
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