Skip to content

Conversation

misteriaud
Copy link
Contributor

@misteriaud misteriaud commented Jul 4, 2024

Buildimages update

This PR updates the current buildimages (v37150619-8250a4fc) to v38375519-53773733, here is the full changelog.

Golang update

This PR updates the current Golang version (1.22.4) to 1.22.5.

Describe how to test/QA your changes

Covered by the QA of #26488.

@misteriaud misteriaud requested review from a team as code owners July 4, 2024 11:41
@github-actions github-actions bot added the team/agent-devx-loops Deprecated, please use agent-devx instead label Jul 4, 2024
@@ -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),
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this change expected?

Copy link
Contributor Author

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 ?

@misteriaud misteriaud changed the title adding missing used_by_otel labels [ASCII-1936] Update Golang to 1.22.5 Jul 4, 2024
Copy link
Contributor

@amenasria amenasria left a 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

@pgimalac pgimalac added [deprecated] team/agent-shared-components Deprecated. Use team/agent-configuration or team/agent-runtimes labels instead. qa/done QA done before merge and regressions are covered by tests labels Jul 5, 2024
@misteriaud
Copy link
Contributor Author

/merge

@dd-devflow
Copy link

dd-devflow bot commented Jul 5, 2024

🚂 MergeQueue: pull request added to the queue

The median merge time in main is 24m.

Use /merge -c to cancel this operation!

@dd-mergequeue dd-mergequeue bot merged commit 3edb4c1 into main Jul 5, 2024
@dd-mergequeue dd-mergequeue bot deleted the buildimages/gobot/go-update-1.22.5 branch July 5, 2024 13:17
@github-actions github-actions bot added this to the 7.56.0 milestone Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[deprecated] team/agent-shared-components Deprecated. Use team/agent-configuration or team/agent-runtimes labels instead. qa/done QA done before merge and regressions are covered by tests team/agent-devx-loops Deprecated, please use agent-devx instead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants