Skip to content

[WIP] Copilot Request#33589

Closed
Copilot wants to merge 1 commit into
devfrom
copilot/fix-6844498-51040886-3ba576f2-e46c-461d-a6a2-04fa920978ad
Closed

[WIP] Copilot Request#33589
Copilot wants to merge 1 commit into
devfrom
copilot/fix-6844498-51040886-3ba576f2-e46c-461d-a6a2-04fa920978ad

Conversation

Copilot AI commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>az aks namespace update --annotations replaces existing annotations instead of updating incrementally</issue_title>
<issue_description>### Describe the bug

The az aks namespace update --annotations command replaces all existing annotations on a managed namespace with only the values provided via the --annotations parameter.
This behavior prevents users from adding or updating individual annotations without first retrieving and re-specifying the entire existing annotation set.

Related command

az aks namespace update --annotations

Errors

N/A

Issue script & Debug output

Variables

RG=my-resource-group
CLUSTER=my-cluster-name
NS=my-namespace

Step 1: Set initial annotations

az aks namespace update -g $RG --cluster-name $CLUSTER -n $NS --annotations a=1 b=2 --debug

Step 2: Verify annotations

az aks namespace show -g $RG --cluster-name $CLUSTER -n $NS --debug

Step 3: Add a new annotation (expected: merge, actual: replace)

az aks namespace update -g $RG --cluster-name $CLUSTER -n $NS --annotations c=3 --debug

Step 4: Verify annotations again

az aks namespace show -g $RG --cluster-name $CLUSTER -n $NS --debug

Annotations after Step 4:

"annotations": {
"c": "3"
},

Expected behavior

Expected Annotations after Step 4:

"annotations": {
"a": "1",
"b": "2",
"c": "3"
},

Environment Summary

azure-cli 2.87.0

core 2.87.0
telemetry 1.1.0

Extensions:
resource-graph 2.1.1

Dependencies:
msal 1.36.0
azure-mgmt-resource 24.0.0

Python location '/opt/homebrew/Cellar/azure-cli/2.87.0/libexec/bin/python'
Config directory '/Users/sunehabose/.azure'
Extensions directory '/Users/sunehabose/.azure/cliextensions'

Python (Darwin) 3.13.13 (main, Apr 7 2026, 18:19:01) [Clang 21.0.0 (clang-2100.0.123.102)]

Additional context

No response</issue_description>

Comments on the Issue (you are @copilot in this section)

@yonzhan Thank you for opening this issue, we will look into it. @azclibot Thanks for the detailed report! I'll investigate the `az aks namespace update --annotations` behavior.

Summary of the issue:

  • Command: az aks namespace update --annotations
  • Problem: The --annotations flag performs a full replacement of all existing annotations instead of a merge/incremental update
  • Expected: Adding c=3 should preserve existing annotations a=1 and b=2, resulting in all three being set
  • CLI version: 2.87.0

I'll look into the implementation to fix the annotations update logic to merge new annotations with existing ones rather than replacing them entirely.


Posted by agent-assist (autonomous bug-fix pipeline).</comment_new>

@azure-client-tools-bot-prd

azure-client-tools-bot-prd Bot commented Jun 20, 2026

Copy link
Copy Markdown
️✔️AzureCLI-FullTest
️✔️acr
️✔️latest
️✔️3.12
️✔️3.14
️✔️acs
️✔️latest
️✔️3.12
️✔️3.14
️✔️advisor
️✔️latest
️✔️3.12
️✔️3.14
️✔️ams
️✔️latest
️✔️3.12
️✔️3.14
️✔️apim
️✔️latest
️✔️3.12
️✔️3.14
️✔️appconfig
️✔️latest
️✔️3.12
️✔️3.14
️✔️appservice
️✔️latest
️✔️3.12
️✔️3.14
️✔️aro
️✔️latest
️✔️3.12
️✔️3.14
️✔️backup
️✔️latest
️✔️3.12
️✔️3.14
️✔️batch
️✔️latest
️✔️3.12
️✔️3.14
️✔️batchai
️✔️latest
️✔️3.12
️✔️3.14
️✔️billing
️✔️latest
️✔️3.12
️✔️3.14
️✔️botservice
️✔️latest
️✔️3.12
️✔️3.14
️✔️cdn
️✔️latest
️✔️3.12
️✔️3.14
️✔️cloud
️✔️latest
️✔️3.12
️✔️3.14
️✔️cognitiveservices
️✔️latest
️✔️3.12
️✔️3.14
️✔️compute_recommender
️✔️latest
️✔️3.12
️✔️3.14
️✔️computefleet
️✔️latest
️✔️3.12
️✔️3.14
️✔️config
️✔️latest
️✔️3.12
️✔️3.14
️✔️configure
️✔️latest
️✔️3.12
️✔️3.14
️✔️consumption
️✔️latest
️✔️3.12
️✔️3.14
️✔️container
️✔️latest
️✔️3.12
️✔️3.14
️✔️containerapp
️✔️latest
️✔️3.12
️✔️3.14
️✔️core
️✔️latest
️✔️3.12
️✔️3.14
️✔️cosmosdb
️✔️latest
️✔️3.12
️✔️3.14
️✔️databoxedge
️✔️latest
️✔️3.12
️✔️3.14
️✔️dls
️✔️latest
️✔️3.12
️✔️3.14
️✔️dms
️✔️latest
️✔️3.12
️✔️3.14
️✔️eventgrid
️✔️latest
️✔️3.12
️✔️3.14
️✔️eventhubs
️✔️latest
️✔️3.12
️✔️3.14
️✔️feedback
️✔️latest
️✔️3.12
️✔️3.14
️✔️find
️✔️latest
️✔️3.12
️✔️3.14
️✔️hdinsight
️✔️latest
️✔️3.12
️✔️3.14
️✔️identity
️✔️latest
️✔️3.12
️✔️3.14
️✔️iot
️✔️latest
️✔️3.12
️✔️3.14
️✔️keyvault
️✔️latest
️✔️3.12
️✔️3.14
️✔️lab
️✔️latest
️✔️3.12
️✔️3.14
️✔️managedservices
️✔️latest
️✔️3.12
️✔️3.14
️✔️maps
️✔️latest
️✔️3.12
️✔️3.14
️✔️marketplaceordering
️✔️latest
️✔️3.12
️✔️3.14
️✔️monitor
️✔️latest
️✔️3.12
️✔️3.14
️✔️mysql
️✔️latest
️✔️3.12
️✔️3.14
️✔️netappfiles
️✔️latest
️✔️3.12
️✔️3.14
️✔️network
️✔️latest
️✔️3.12
️✔️3.14
️✔️policyinsights
️✔️latest
️✔️3.12
️✔️3.14
️✔️postgresql
️✔️latest
️✔️3.12
️✔️3.14
️✔️privatedns
️✔️latest
️✔️3.12
️✔️3.14
️✔️profile
️✔️latest
️✔️3.12
️✔️3.14
️✔️rdbms
️✔️latest
️✔️3.12
️✔️3.14
️✔️redis
️✔️latest
️✔️3.12
️✔️3.14
️✔️relay
️✔️latest
️✔️3.12
️✔️3.14
️✔️resource
️✔️latest
️✔️3.12
️✔️3.14
️✔️role
️✔️latest
️✔️3.12
️✔️3.14
️✔️search
️✔️latest
️✔️3.12
️✔️3.14
️✔️security
️✔️latest
️✔️3.12
️✔️3.14
️✔️servicebus
️✔️latest
️✔️3.12
️✔️3.14
️✔️serviceconnector
️✔️latest
️✔️3.12
️✔️3.14
️✔️servicefabric
️✔️latest
️✔️3.12
️✔️3.14
️✔️signalr
️✔️latest
️✔️3.12
️✔️3.14
️✔️sql
️✔️latest
️✔️3.12
️✔️3.14
️✔️sqlvm
️✔️latest
️✔️3.12
️✔️3.14
️✔️storage
️✔️latest
️✔️3.12
️✔️3.14
️✔️synapse
️✔️latest
️✔️3.12
️✔️3.14
️✔️telemetry
️✔️latest
️✔️3.12
️✔️3.14
️✔️util
️✔️latest
️✔️3.12
️✔️3.14
️✔️vm
️✔️latest
️✔️3.12
️✔️3.14

@azure-client-tools-bot-prd

azure-client-tools-bot-prd Bot commented Jun 20, 2026

Copy link
Copy Markdown
️✔️AzureCLI-BreakingChangeTest
️✔️Non Breaking Changes

@azclibot

Copy link
Copy Markdown
Collaborator

Live test results — azdev test --live --series

FAIL (exit )

Target: `` ()
PR head ref: copilot/fix-6844498-51040886-3ba576f2-e46c-461d-a6a2-04fa920978ad
PR head sha: `787084bef0512188e6599f51284ae11c4bb08c28`
PR base ref: `dev`
New test files in PR:

Workflow run: https://github.com/Azure/issue-sentinel/actions/runs/27866436623

Last 80 lines of azdev output
(no log captured)

Posted by agent-assist live-test workflow.

@azclibot azclibot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Agent Review — PR #33589

Live Test (issue-sentinel)

Failed — The azdev test --live step did not produce an exit code (EXIT_CODE was empty), indicating the test run did not execute properly.

CI Checks (Azure DevOps + GitHub Actions)

Check Result
Azure.azure-cli (build) ❌ Failed — 4 errors (ADO build 323581)
Azure.azure-cli (Credential Scanner) ❌ Failed — 4 errors (same ADO build)
copilot agent job ❌ Failed — HTTP 500 Internal Server Error from Copilot runtime when fetching job details (infrastructure error, not a code issue)
All other checks (46/49) ✅ Passed

Summary

The Copilot coding agent job failed due to a transient infrastructure error (Failed to fetch job details: HTTP 500) — no code changes were made to this PR (0 files changed). The build failures and credential scanner failures may be pre-existing pipeline issues unrelated to this PR.

@copilot please retry — the agent session failed before making any changes to address the issue (az aks namespace update --annotations replaces instead of merging annotations).


Posted by agent-assist (autonomous bug-fix pipeline).

@azclibot azclibot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🔴 CI and Live-Test Failures — Action Required

@copilot The PR has failures that need to be addressed before it can be merged.

CI Check Results (3 failures)

Check Result
Azure.azure-cli (Build #20260620.1) ❌ Failed — 4 errors
Azure.azure-cli (Credential Scanner) ❌ Failed — 4 errors
copilot ❌ Failed

Build / Credential Scanner errors: The Azure DevOps pipeline reported 4 errors. Please review the build log for details — common causes include linting errors, import issues, or credentials/secrets accidentally included in changed files.

Live-Test Result

Workflow Result
issue-sentinel live-test ❌ Failed

Please investigate and push a fix for the failing checks above.


Posted by agent-assist (autonomous bug-fix pipeline).

@yonzhan

yonzhan commented Jun 21, 2026

Copy link
Copy Markdown
Collaborator

Copilot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

act-observability-squad AKS az aks/acs/openshift Auto-Assign Auto assign by bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

az aks namespace update --annotations replaces existing annotations instead of updating incrementally

5 participants