Skip to content

[ACR] az acr connected-registry create/update: Add new parameters for enabling and setting interval for garbage collection within a connected registry#30956

Merged
zhoxing-ms merged 13 commits intoAzure:devfrom
trisavo-msft:trisavo/connected-registry-gc
Apr 23, 2025
Merged

Conversation

@trisavo-msft
Copy link
Member

@trisavo-msft trisavo-msft commented Mar 5, 2025

Related command

az acr connected-registry create, az acr connected-registry update

Description

  • Added new parameter "gc-enabled" which enables or disables garbage collection on a connected registry
  • Added new parameter "gc-schedule" which uses cron expression to set a collection interval.
  • Modified connected registry unit tests with the changes
  • Update default acr mgmt client and connected registry client to 2024-11-01-preview.
  • Update test recordings

Testing Guide

History Notes

[ACR] az acr connected-registry create/update: Add new parameter --gc-enabled which enables or disables garbage collection on a connected registry
[ACR] az acr connected-registry create/update: Added new parameter --gc-schedule which uses cron expression to set a collection interval.


This checklist is used to make sure that common guidelines for a pull request are followed.

@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Mar 5, 2025

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

@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Mar 5, 2025

⚠️AzureCLI-BreakingChangeTest
⚠️acr
rule cmd_name rule_message suggest_message
⚠️ 1006 - ParaAdd acr connected-registry create cmd acr connected-registry create added parameter garbage_collection_enabled
⚠️ 1006 - ParaAdd acr connected-registry create cmd acr connected-registry create added parameter garbage_collection_schedule
⚠️ 1006 - ParaAdd acr connected-registry update cmd acr connected-registry update added parameter garbage_collection_enabled
⚠️ 1006 - ParaAdd acr connected-registry update cmd acr connected-registry update added parameter garbage_collection_schedule

@yonzhan
Copy link
Collaborator

yonzhan commented Mar 5, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

@github-actions
Copy link

github-actions bot commented Mar 5, 2025

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

@trisavo-msft trisavo-msft changed the title update sdk [ACR] az acr create/update: Add new arguments for enabling and setting interval for garbage collection within a connected registry Mar 5, 2025
@trisavo-msft trisavo-msft changed the title [ACR] az acr create/update: Add new arguments for enabling and setting interval for garbage collection within a connected registry [ACR] az acr connected-registry create/update: Add new arguments for enabling and setting interval for garbage collection within a connected registry Mar 6, 2025
'child_name': childName,
'grandchild_name': grandchildName,
'rg_loc': 'eastus',
'rg_loc': 'westcentralus',
Copy link
Contributor

Choose a reason for hiding this comment

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

Question, why need make this change?

Copy link
Member Author

Choose a reason for hiding this comment

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

It was initially for testing to verify garbage collection since the RP release made it to westcentralus. I will change it back to eastus.

@trisavo-msft trisavo-msft marked this pull request as ready for review April 1, 2025 03:25
@trisavo-msft trisavo-msft requested a review from bebound as a code owner April 1, 2025 03:25
@zhoxing-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

zhoxing-ms
zhoxing-ms previously approved these changes Apr 1, 2025
@zhoxing-ms
Copy link
Contributor

@trisavo-msft Please address these CI issues

@zhoxing-ms
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@trisavo-msft trisavo-msft changed the title [ACR] az acr connected-registry create/update: Add new arguments for enabling and setting interval for garbage collection within a connected registry [ACR] ‘az acr connected-registry create/update’: Add new arguments for enabling and setting interval for garbage collection within a connected registry Apr 21, 2025
@zhoxing-ms zhoxing-ms changed the title [ACR] ‘az acr connected-registry create/update’: Add new arguments for enabling and setting interval for garbage collection within a connected registry [ACR] az acr connected-registry create/update: Add new arguments for enabling and setting interval for garbage collection within a connected registry Apr 21, 2025
@zhoxing-ms zhoxing-ms changed the title [ACR] az acr connected-registry create/update: Add new arguments for enabling and setting interval for garbage collection within a connected registry [ACR] az acr connected-registry create/update: Add new parameters for enabling and setting interval for garbage collection within a connected registry Apr 21, 2025
@zhoxing-ms
Copy link
Contributor

image
Could you add detailed History Notes for this PR, as these History Notes will be written into CLI release notes

@zhoxing-ms zhoxing-ms merged commit d90dc34 into Azure:dev Apr 23, 2025
55 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.

4 participants