-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[ASCII-1936] Update Golang to 1.22.5 #27318
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
@@ -130,11 +130,11 @@ def dependency_path(self, agent_version): | |||
".", | |||
targets=["./pkg", "./cmd", "./comp"], | |||
), | |||
"cmd/agent/common/path": GoModule("cmd/agent/common/path", independent=True), | |||
"comp/api/api/def": GoModule("comp/api/api/def", independent=True), | |||
"cmd/agent/common/path": GoModule("cmd/agent/common/path", independent=True, used_by_otel=True), |
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.
Is this change expected?
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.
cmd/agent/common/path
is used in comp/core/config
module.
It is only used in comp/core/config/params_test.go
to get DefaultConfPath
.
Should we try to use another way to get DefaultConfPath
to avoid having it as a dependency ?
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 ! Don't forget to remove the _test_only
before merging
/merge |
🚂 MergeQueue: pull request added to the queue The median merge time in Use |
Buildimages update
This PR updates the current buildimages (
v37150619-8250a4fc
) tov38375519-53773733
, here is the full changelog.Golang update
This PR updates the current Golang version (
1.22.4
) to1.22.5
.Describe how to test/QA your changes
Covered by the QA of #26488.