Skip to content

Dockerfile with docker-agent binary, keeping cagent only as compatible symlink#1992

Merged
dgageot merged 2 commits intodocker:mainfrom
gtardif:docker-agent_image
Mar 9, 2026
Merged

Dockerfile with docker-agent binary, keeping cagent only as compatible symlink#1992
dgageot merged 2 commits intodocker:mainfrom
gtardif:docker-agent_image

Conversation

@gtardif
Copy link
Contributor

@gtardif gtardif commented Mar 9, 2026

Changes on cross will require changes on the release action

…e symlink

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@gtardif gtardif force-pushed the docker-agent_image branch from 1ac4d9a to 6c0e083 Compare March 9, 2026 08:52
dgageot
dgageot previously approved these changes Mar 9, 2026
Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@gtardif gtardif marked this pull request as ready for review March 9, 2026 09:13
@gtardif gtardif requested a review from a team as a code owner March 9, 2026 09:13
Copy link

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Review Summary

Assessment: 🟡 NEEDS ATTENTION

Reviewed the Dockerfile changes for the binary rename from cagent to docker-agent. Found 1 issue requiring attention:

MEDIUM severity: The ldflags in the build command still reference the old module path github.com/docker/cagent/pkg/version. If the Go module has been renamed to match the binary, this will cause version information to be set incorrectly or the build may fail.

Statistics

  • Files changed: 1
  • Issues found: 1 (0 critical, 1 medium, 0 low)
  • Issues verified: CONFIRMED

@dgageot dgageot merged commit 75e58e1 into docker:main Mar 9, 2026
8 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.

2 participants