Skip to content

Rename CAGENT_HIDE_TELEMETRY & CAGENT_EXP_DEBUG_LAYOUT. Still support old env vars#1926

Merged
dgageot merged 1 commit intodocker:mainfrom
gtardif:more_env_rename
Mar 5, 2026
Merged

Rename CAGENT_HIDE_TELEMETRY & CAGENT_EXP_DEBUG_LAYOUT. Still support old env vars#1926
dgageot merged 1 commit intodocker:mainfrom
gtardif:more_env_rename

Conversation

@gtardif
Copy link
Contributor

@gtardif gtardif commented Mar 4, 2026

No description provided.

@gtardif gtardif requested a review from a team as a code owner March 4, 2026 16:21
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: 🟢 APPROVE

This PR successfully renames environment variables while maintaining backward compatibility:

  • CAGENT_HIDE_TELEMETRY_BANNERDOCKER_AGENT_HIDE_TELEMETRY_BANNER
  • CAGENT_EXPERIMENTAL_DEBUG_LAYOUTDOCKER_AGENT_EXPERIMENTAL_DEBUG_LAYOUT

The implementation correctly uses logical OR operators to support both old and new variable names during the transition period. No functional bugs were found in the changed code.

Findings

No issues requiring changes. The backward compatibility logic is implemented correctly.

… old env vars

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@dgageot dgageot merged commit a5dcb43 into docker:main Mar 5, 2026
5 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