Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ACR] hotfix based on #18786 and #18850 #18853

Merged
merged 2 commits into from
Jul 14, 2021
Merged

Conversation

00Kai0
Copy link
Contributor

@00Kai0 00Kai0 commented Jul 14, 2021

Description

Hotfix:

  • Fix some parts of az acr build, az acr connected-registry, az acr pack, az acr run, az acr scope-map didn't migrate to track2 SDK

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


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

Copy link
Member

@northtyphoon northtyphoon left a comment

Choose a reason for hiding this comment

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

LGTM

@00Kai0 00Kai0 mentioned this pull request Jul 14, 2021
3 tasks
@00Kai0 00Kai0 changed the title [ACR] hotfix based on #18787 and #18850 [ACR] hotfix based on #18786 and #18850 Jul 14, 2021
@jsntcy jsntcy merged commit a15f723 into Azure:release Jul 14, 2021
kairu-ms added a commit that referenced this pull request Aug 5, 2021
* {Doc} Add managed identity command guideline (#17473)

* [WebApp] Add to `az webapp deploy` param help text (#17743)

* Add to `az webapp deploy` param help text

String changes for the `az webapp deploy` command's parameter help text.

* Update _params.py

* Update _params.py

* Style guide fixes

* Update src/azure-cli/azure/cli/command_modules/appservice/_params.py

Co-authored-by: Feiyue Yu <[email protected]>

* Update _params.py

Co-authored-by: Feiyue Yu <[email protected]>

* {Doc} Update track_2_migration_roadmap.md (#18206)

* [RDBMS] Add commands for Github actions  (#17949)

* basic functionality for github action deploy

* Modify deploy setup and run command

* add logger warnings

* mysql command added for deploy and run

* testing mysql and postgres

* Add github action file

* testing pg

* add erroring out vnet enabled server

* add help text

* fix help text

* fix help text

* style issue fixed

* trigger CI

* Update _flexible_server_util.py

* changed action file to template

* fix style

* style issue fixed

* {Pylint} Fix `self-assigning-variable` (#18119)

* [KeyVault] Fix #11871: AKV10032: Invalid issuer error for operations in nondefault tenant/subscription (#18162)

* keyvault data plane cross tenant

* all data plane

* cli_ctx for client factory

* other modules

* remove unused import

* omit extra `None`

* keyvault code owners (#18234)

* Fix what-if formatter (#18244)

* Fix for user-assigned identity ARM issue on remove (#18205)

Co-authored-by: Ryan Kelly <[email protected]>

* [Keyvault] `az keyvault set-policy/delete-policy`: Support --application-id (#18209)

* `az keyvault set-policy/delete-policy`: Support --application-id

* param

* application_id.lower comparison

* fix delete

* address comment

* [Compute] Upgrade api-version for VM and VMSS from '2020-12-01' to '2021-03-01' (#18233)

* Upgrade api-version for compute

* Fix CI

* rerecord aks

* [rdbms] az postgres flexible-server migration: Add customer facing feature to migrate postgres db servers from Sterling to Meru platform (#18161)

* Support for "az postgres flexible-server migration" commands

* Fix a linter error caused by copy-paste.

* Fix more linter errors

* Use custom_show_command instead of custom_command for 'show' to fix a linter finding.

* 1. Remove the --subscription-id parameter. Use the CLI command param --subscription instead

* Rename body to properties, which makes more sense in this context.

* Fix isses reported by "azdev style rdbms"

* Fix linter issues

* Fix issues reported by azdev style check

* Mark the migration command group as experimental

* Cannot have is_preview and is_experimental at the same time. So removed is_preview.

* 1. Added a confirmation for migration delete, 2. Deleted unused code, 3. Return instead of just the migration_id from update

* Replace parameters db1, db2, ...db8 with an db-names parameter which takes a space-separated list of values

* [Storage] `az storage blob copy start`: Support --tier and --rehydrate-priority (#18170)

* upgrade api version

* draft command design

* test pass

* test pass

* refine help

* checkout

* checkout

* add linter exclusion

* fix style

* fix error type

* fix error type

* fix style

* {monitor} live test fixing (#18189)

* fix test_monitor_action_group_basic_scenario tests

* fix test_metrics_alert_metric_name_with_special_characters

* fix test_monitor_create_log_profile

* skip test_monitor_clone_storage_metric_alerts_across_subs_scenario

* fix issue

* [KeyVault] `az keyvault recover`: Support MHSM (#18150)

* recover for MHSM

* fix

* tests

* linter

* Fix for user-assigned identity ARM issue on remove (#18283)

Co-authored-by: Ryan Kelly <[email protected]>

* [AppService] Hotfix: Fix # 18266 - webapp config appsettings set command causing all values to default to "false" in version 2.24.0 (#18277)

* #18266 - webapp config appsettings set command causing all values to default to "false" in version 2.24.0

* Added tests for validating settings through json with slot

* [Storage] GA release storage file share NFS and SMB multichannel (#18232)

* {Packaging} Bump `urllib3` and `requests` (#18213)

* {Pylint} Fix `unnecessary-comprehension` (#18013)

* {Compute} Adopt track2 SDK, azure-mgmt-batch==15.0.0 (#18113)

* Update test link

* Update requirements

* Fix batch_data_plane

* Fix 2 tests

* Fix their style and linter

* Fix _help

* Fix batch style and linter

* Upload test yaml

* Fix batch test in network

* [Network] private link add `Microsoft.Sql/servers` provider (#18268)

* reorder private link providers

* comment miss private link providers

* support private link provider for Microsoft.Sql/servers

* update recordings

* {CI} Unpin azdev (#18300)

* [VM] Hotfix: Fix the bad request issue when creating VMSS in Azure Stack (#18301)

* Fix unsupported parameters in old api-version for vmss

* Run releated test in old profile

* Fix CI style

* [HDInsight] Migrate to track2 python sdk 7.0.0 (#18237)

* [Backup] Migrate backup to track2 SDK (#17831)

* [Backup] Migrate backup to track2 SDK

* [Backup] Migrate backup to track2 SDK

* [Backup] Migrate backup to track2 SDK

afs_test passed

* workload_test

* workload_sql

* update

* workload_hana

* update afs_test

* update

* Revert "Merge branch 'dev' into backup"

This reverts commit a197e3ceb9f9f85e078c6a0c5707d2543b834c17, reversing
changes made to 10b1d7d53cb3e11868b196d57cd729fd83df4b74.

* Modified style

* update

* fix conflicts

* update

* update

* update

* update

There was a problem with the virtual environment, changed it back

* Apply suggestions from code review

Co-authored-by: Feiyue Yu <[email protected]>

* Update preparers.py

* Resolve conflicts

* Revert "Resolve conflicts"

This reverts commit 034b32671abd3c8c45fe6e5abc895d864e288e06.

* Resolve conflicts

* Update test_backup_wl_sql_policy.yaml

* Delete test_acs_create_kubernetes.yaml

Co-authored-by: Feiyue Yu <[email protected]>

* Hotfix for 2.24.1 (#18307)

* {Keyvault} Support managed HSM private link operations (#18273)

* private link resource

* private endpoint connection

* tests

* network module tests

* linter

* merge network changes

* [Compute] `az vm create`: Support delete option for NICs and Disks for VMs in Azure CLI (#18238)

* support delete option for new nic

* test pass for nix

* test pass for vm nic count

* test pass

* rebase dev

* suppress linter

* fix linter

* fix comment

* [EventGrid] Migrate track2 SDK (#18210)

* {AKS} Fix failed live test cases (#18310)

* fix virtual network preparer

* remove outdated checks

* [Synapse]update to  AZ Synapse mgmt 2.0.0 (#18195)

* [Container] Hotfix: Fix #18276: `az container create` fails with `AttributeError: 'ResourcesOperations' object has no attribute 'create_or_update'` (#18326)

* Fix the method signature for resource track2 client

* upgrade version

* Update src/azure-cli/HISTORY.rst

* {Packaging} Bump urllib3 from 1.26.4 to 1.26.5 (#18324)

* fix network tests due to synapse bum (#18340)

* [RDBMS] Private DNS zone parameter added for restore command, high availability validator (#18218)

* add private dns zone for restore and ha validator, template for github actions

* change auto registration and fix autogenerated name

* test  fixed

* add param explanation

* Fix typo

* Add functionalities

* update functionalities

* fix setup py

* delete automatic action run

* Update running condition

* {Role} `az ad sp create-for-rbac`: Stop setting `identifierUris` on app (#18312)

* {container} Fix style in CI (#18397)

* [signalr] az signalr identity: Add managed identity related command, az signalr cors update: Add update for CORS (#18309)

* update

* Update

* Add help

* Add tests

* Fix conflict

* Override signalr_name

* [NetAppFiles] Update azure-mgmt-netapp version to 3.0.0 (#18303)

* added missing files

* Backup tests updated and new recordings for them

* Add missing recordings

* Fix for check cli check

* More fixes

* More fixes

* update netapp sdk to version 3.0.0 and add new features from it. Updated tests and gathered new recordings

* update requirements

* Updated according to new sdk 2021-02-01. Tests updated and new recordings gathered

* update help text (#18365)

* [Storage] BREAKING CAHNGE: az storage account create: Remove StorageFileDataSmbShareOwner option for --default-share-permission (#18396)

* {EventGrid}: Update warning message (#18388)

* Fix warning message

* Fix style warnings

* [Synapse]spark configuration conversion, which cause the failure (#18328)

* Forward storage blob list delimiter parameter to client.walk_blobs (#18394)

* {container} Bugfix #15856,#18251,#18275: az container exec - decode received bytes as utf-8 string (#18384)

* decode binary string as utf-8

* updates suggested code style changes

* {Keyvault} `az keyvault update-hsm`: --secondary-location not supported (#18427)

* update hsm not support secondary locations

* update hsm

* fix typo

* bump up azure-mgmt-applicationinsights to version 1.0.0 (#18400)

* {AppService} Add custom location sdk (#18362)

* [App Service]:  Fixing az webapp deployment source config-local-git to set SiteConfig (#18364)

* Updating enable_local_git SiteConfig Settings

* Recording failing test

* Filter apps with kind null when returning list of apps

* Rerecording tests

* [Core] Fix #18360 and #18262: write to stderr issue (#18414)

* fix

* refine wait

* fix style

* [ACR] az acr connected-registry: Minor bug fixes (#18288)

* bug fix

* pylint fix

* fix style

Co-authored-by: Zunli Hu <[email protected]>

* [Compute] Direct Sharing support for Azure Shared Image Gallery (#18398)

* init

* 'test'

* update test

* update test

* test recording

* style

* remove generated test

* add command to get shared galleries

* fix test

* wait command

* Update src/azure-cli/azure/cli/command_modules/vm/generated/_help.py

Co-authored-by: Feiyue Yu <[email protected]>

* delete sig share update

* change the command format

* lint

* remove is_experimental

* update command design

* update command name and enum value

* add wait command

* linter

* Update src/azure-cli/azure/cli/command_modules/vm/manual/_help.py

Co-authored-by: Feiyue Yu <[email protected]>

* Update src/azure-cli/azure/cli/command_modules/vm/manual/_help.py

Co-authored-by: Feiyue Yu <[email protected]>

* Update src/azure-cli/azure/cli/command_modules/vm/generated/_params.py

Co-authored-by: Feiyue Yu <[email protected]>

* help info

* regenerate command interface but manual not work

* test pass

* fix test

* resolve comments

* fix style

* Update src/azure-cli/azure/cli/command_modules/vm/manual/_help.py

Co-authored-by: xfz11 <[email protected]>

Co-authored-by: xiaofanzhou <[email protected]>
Co-authored-by: xfz11 <[email protected]>
Co-authored-by: Feiyue Yu <[email protected]>

* replace fixed location in test case (#18429)

* [RDBMS] Change server default location (issue reported) (#18157)

* Change rdbms server default location

* style fixed

* fixed tests yaml file

* Fix the problem of tagging resources of public ip (#18254)

* [Compute] Support user_data for VM and VM Scale Sets (#18432)

* Add UserData for VM and VMSS

* Fix CI

* [ARM] Add 'az policy assignment non-compliance-message' (#18158)

* Non compliance message support

* Persist nulls when removing non compliance messages

* Fix variable naming

* Fix parameters for policy assignment update

* Fix style and linter errors

* Fix not scope validation

* Allow MG level notscopes

* Revert subscription

* Revert "[Compute] Direct Sharing support for Azure Shared Image Gallery (#18398)" (#18449)

This reverts commit 7f7276d712bda4618a489856aed7b70a99433c5d.

* {Release} Upgrade to Azure CLI 2.25.0 (#18452)

* {Misc} Update service mapping for az term (#18329)

* aro: add cidr values for pod/service to cli params (#18457)

* {CODEOWNERS} Update CODEOWNERS (#18455)

* Update CODEOWNERS

* Update .github/CODEOWNERS

Co-authored-by: Yishi Wang <[email protected]>

* Update .github/CODEOWNERS

Co-authored-by: Yishi Wang <[email protected]>

Co-authored-by: Yishi Wang <[email protected]>

* {Reference TOC} fixed Azure service names (#18459)

* {Cognitive Services} Adopt track2 SDK, azure-mgmt-cognitiveservices==12.0.0 (#18354)

* Update test link

* update requirements

* Fix tests

* Update cognitiviceservices==12.0.0

* Fix identity

* Fix tests

* Fix tests(delete unnecessary import)

* Fix tests(sku)

* Upload test_recordings

* [AKS] Migrate ACS module to track 2 SDK (#18117)

* small fix (#18505)

* {Packaging} Remove full Release History from project description (#18485)

* {Packaging} Bump PyJWT to 2.1.0 (#18506)

* {Packaging} Be more strict on `requests` (#18518)

* {Misc.} Fix typos in HISTORY.rst (#18504)

* {Pylint} Fix `use-a-generator` (#18523)

* {Core} `az extension`: Retry `shutil.rmtree` (#18061)

* Remove preview out of security module (#18529)

Co-authored-by: Ahmad Abas <[email protected]>

* {AppService} Make `appservice` tests serial (#18534)

* {Cognitive Services} Update CLI to latest SDK, add soft-delete support. (#18464)

* Update CLI to latest SDK, add soft-delete support.

* fix comments

* Fix style

* re-record tests

* fix style issue.

* fix comments

* remove unused import

* Update codeowners (#18537)

* [Azurestack][ACR][AKS] add azurestack hybrid profile for AKS (#18118)

* Adding ACR and AKS for azurestack hybrid profile

* Change managed cluster SDK profile for acs

* Container service resource type to managed cluster sdk

* Removing hardcoded api-version to import models

* Removing versioned models import

* Removing versioned models import

* Fixing style and models

* Changing Azure stack API version for container service

* Adding operation-group to container service

* Changing container service test_validator

* Changing container service test_validator

* Modifying operation_group to managed_clusters

* Adding managed_clusters operation group to aks run command

* Container service test code changes

* Revert "Container service test code changes"

This reverts commit c551f3b8fd0bfbe3570bb767070353d2011a9f6b.

* Adding mock cmd to acs test

* Fix resource tyope in acs test

* Fix resource tyope in acs test

* Removing resource type in mock cmd

* Changing acs get_models

* Adding operation group to get_models

* Adding magicmock to acs test

* Ading magicmock to acs test

* Modifying mock cmd

* Modifying acs mock cmd

* Adding operation_group to acs mock

* Adding mod=models for acs mock cmd

* removing redundant operation_group properties

* Adding operation_group to acs loadbalancer

Co-authored-by: Bhuvaneswari Santharam <[email protected]>

* Fix #17850 - prevent dublicate rules for service endpoints (#18024)

* [Compute] Direct sharing support (#18503)

* init

* 'test'

* update test

* update test

* test recording

* style

* remove generated test

* add command to get shared galleries

* fix test

* wait command

* Update src/azure-cli/azure/cli/command_modules/vm/generated/_help.py

Co-authored-by: Feiyue Yu <[email protected]>

* delete sig share update

* change the command format

* lint

* remove is_experimental

* update command design

* update command name and enum value

* add wait command

* linter

* Update src/azure-cli/azure/cli/command_modules/vm/manual/_help.py

Co-authored-by: Feiyue Yu <[email protected]>

* Update src/azure-cli/azure/cli/command_modules/vm/manual/_help.py

Co-authored-by: Feiyue Yu <[email protected]>

* Update src/azure-cli/azure/cli/command_modules/vm/generated/_params.py

Co-authored-by: Feiyue Yu <[email protected]>

* help info

* regenerate command interface but manual not work

* test pass

* fix test

* resolve comments

* fix style

* Update src/azure-cli/azure/cli/command_modules/vm/manual/_help.py

Co-authored-by: xfz11 <[email protected]>

* test pass

* revert

* revert

* delete

* fix typo

* refine help

* Apply suggestions from code review

Co-authored-by: Xing Zhou <[email protected]>

* add permission to create

Co-authored-by: xiaofanzhou <[email protected]>
Co-authored-by: xfz11 <[email protected]>
Co-authored-by: Feiyue Yu <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>

* [HDInsight]Support new azure monitor (#18519)

* {Docs} Remove stale reference in README to closed issue about extensions (#12771)

* Remove breaking change incoming notice and support new azure monitor

* fix cred scan ci error

* Replace CLIError with RequiredArgumentMissingError

Co-authored-by: Daniel Miller <[email protected]>
Co-authored-by: Azure CLI Team <[email protected]>
Co-authored-by: Zhenyu Zhou <[email protected]>

* [Resource] Third party S2S Consent/Permission Enumeration (#18433)

* feature permission list

* fix

* fix

* fix

* fix

* fix

* Apply suggestions from code review

* fix test and help info

* fix

Co-authored-by: Yu Chen <[email protected]>

* [AKS] Upgrade api-version to 2021-05-01 for ACS module (#18593)

* update api to 0501

* update api version

* update recording location

* {Advisor} Migrate Advisor module to track 2 SDK (#18525)

* ARO: Fail if resource doesn't exist on delete (#18546)

* [SQL] Bump sdk version (#18373)

* Bump sdk version

* Switch to LongTermRetentionPolicies

* Fix sql db threat-policy

* some test fixes

* Fix threat detection

* re-record test_sql_server_security_mgmt

* Fix tde tests

* add more passing tests

* fix ltr test

* Fix elastic pool

* fix sql mi aad tests

* Failover group

* fix firewall rule

* Fix for tde byok

* push test file

* Skip MI tests

* skip server trust groups

* Add dns alias management

* fix backupstorageredundancy

* backupstorage related fixes

* Push test files

* Fix test_sql_db_sensitivity_classifications, skip instance pools

* Push sens classification re-record

* retry sensitivity test

* Fix instance group failover management

* restore db test passing

* record restore deleted

* skip restore deleted and classification

* management endpoint

* remove skip tag

* rerecord import export

* modify validators to separate db and MI

* change back MI to expect LRS/ZRS/GRS

* Add CMK changes

* Fix linter and style issues

* remove current_bsr from create params

* Fix failure due to --ids

* more fixes, tests and linter

* More linter issues

* External admin changes

* Style and Linter fixes

* Fix show command

* Fix more style issues

* Add help for external admin

* linter fix

* Add examples for CMK with UMI

* more linter fixes

Co-authored-by: xaliciayang <[email protected]>
Co-authored-by: Vinit Dinesh Parekh <[email protected]>

* [APIM] Added version parameter for apim api import (#18604)

* added version parameter for api import

* added test api version

* updated test

* addressed ci issue

* remove preview tag (#18607)

* Fix for server create (#18640)

* [AppConfig] Stop overwriting KeyVault reference content type during import (#18602)

* Stop overwriting KVR content type during import

* Style fix

* [BACKUP] Workload container registration fix, SDK upgraded to 0.12.0, Fixed and Re-ran tests  (#18592)

* workload container registeration fix, sdk upgrade to 0.12.0, test fixed and re-ran

* deleted local files

* Linter and pyproj fix

* commented record_only tag for test_afs_commands

Co-authored-by: Akshay Neema <[email protected]>

* fix CI tests (#18643)

* [APIM] Fixed apim upgrade bug when specifying protocols  (#18605)

* fixed issue with protocol related command

* added test for protocol

* addressed ci issues

* [Billing] Migrate billing to track2 SDK (#18608)

* billing 1.0.0->6.0.0

* Update custom.py

* parameter

* [ARM] `az provider register`: Make parameter `--accept-term` become not required (#18509)

* [RDBMS] Add autogrow parameter for MySQL and add database name to output json when created (#18441)

* Add autogrow feature and postgres db name show when created

* autogrow parameter for update added

* add validators file

* Add different subnet to restore server

* private dns zone validator

* server name validator and database delete warninng message deleted

* restore network added

* pg restore dns zone add when only microsoft dns zone

* mysql ha validation added

* style fix

* [Storage] GA support secured SMB (#18638)

* [CosmosDB] Adding point-in-time restore commands to the stable branch. (#18568)

* Porting Retrieve BackupInfo and Tests from preview branch.

* Porting PITR Changes for Restore Commands.

* Adding validation changes, test fixes and use updated python sdk.

* Updating test run files.

* Fixing style/linting/test errors

* Fixing linter and style errors

* Fix failing restore test.

* Updating linter_exclusions

* Fixing some formatting

* Fixing linter warning.

* Adress code feedback.

* Fix #18500. Add service tag list check. (#18527)

* [Storage] `az storage account create`: Support `--enable-nfs-v3` to set NFS 3.0 protocol (#16766)

* design

* add test

* redesign

* test pass

* refine

* fix

* refine help

* {CI} Fix test (#18680)

* {CDN} add missing help (#18641)

* add miss help

* Update src/azure-cli/azure/cli/command_modules/cdn/_help.py

Co-authored-by: Jiashuo Li <[email protected]>

Co-authored-by: Jiashuo Li <[email protected]>

* [APIM] Fixed apim create when --enable-managed-identity is set to true (#18554)

* fixed --enable-managed-identity for apim create

* fixed ci error and updated test

* {APIM} Fixed import error (#18553)

* fixed import error

* addressed linter format

* addressed style check

* address PR ci issue

* addressed PR comments

Co-authored-by: kai ru <[email protected]>

Co-authored-by: kai ru <[email protected]>

* aks: add 'enable-ultra-ssd' parameter (#18649)

add empty line

* [BACKUP] Added Archive Support for Azure CLI (#18535)

* Code done!

* Modified code1

* Archive tests added.

* help_for_move, linter and az file changed

* New line added in az and test_backup_commands

* ran backup archive

* Commented test_backup_archive for debugging

* Changed recordings

* Adding archive recording back

* Added archive recording, and couple others

* ran all four failed tests locally

* akshay_review_changes

* Corrected a error in help-text of move command

* Corrected a small error in test_workload_commands.

* More changes after review.

* Additional changes

* Changed help text of move in help.py

* Made changes for use-secondary-region parameter

* Changed recordings

* Re-ran the 3 failing tests

* Further modifications to those 3 recordings

* Tweaked archive code for use_secondary_region

* Changes suggested by Xing Zhou

* Additional change after review

* Update src/azure-cli/azure/cli/command_modules/backup/_help.py

Co-authored-by: Xing Zhou <[email protected]>

* Update src/azure-cli/azure/cli/command_modules/backup/custom_afs.py

Co-authored-by: Xing Zhou <[email protected]>

* Update src/azure-cli/azure/cli/command_modules/backup/custom_afs.py

Co-authored-by: Xing Zhou <[email protected]>

* Final changes suggested by Xing Zhou

* Minor change

Co-authored-by: Shreyansh Joshi <[email protected]>
Co-authored-by: Akshay Neema <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>

* [SQL] `az sql db ledger-digest-uploads`:  Support SQL Ledger (#18672)

* ledger create db + ledger digest uploads

* adding tests, updating help message

* adding db tests

* adding help message for parameter

* fixing comments on pr

* [Container Registry] Migrate to track2 SDK (#18611)

* [AKS] Support use custom kubelet identity (#18615)

* Add custom kubelet identity implementation

* Add recording file

* Fix lint

* Fix lint 2

* Apply code review

* Fix errors

* Apply code review

* Fix lint

* Fix lint 2

* [CosmosDB] Adding support for selecting Cosmos DB analytical storage schema type (#18636)

* adding support for selecting cosmos db analytical storage schema type

* bumping cosomsdb mgmt package version

* fixing some unit tests

* fixing build errors

* adding shorter parameter name for --analytical-storage-schema-type

Co-authored-by: Tom Swedlund <[email protected]>

* [NetAppFiles] Update azure-mgmt-netapp version to 4.0.0 (#18666)

* Update netapp python sdk and add new features

* fix help

* fix restore status command name

* Fix for UMI (#18693)

* [Compute] `az vmss create`: Support NetworkApiVersion for Vmss with OrchestraionMode == Flexible (#18132)

* draft command design

* test pass

* refine code

* new change

* simple case pass

* existing test pass

* test pass

* test pass

* fix test

* refine error type

* support generate ssh key

* test pass

* fix style

* add namespace.disable_overprovisionTrue

* fix

* [Storage] Support container soft delete  (#18508)

* support restore and list

* add example

* [network] `az network routeserver create`: Add `--public-ip-address` argument (#18663)

* add public_ip_address parameter for 'az network routeserver create'

* change the default ip configuration name to align with powershell

* update recordings

* adding fuming as the owner of acs module. (#18665)

* [SQL] 'az sql db str-policy set/show': Add Set and Show ShortTermRetentionPolicy (#14919)

* {Docs} Remove stale reference in README to closed issue about extensions (#12771)

* draft sql db short term retention policy set and show cmdlets

* add defination of get_sql_database_short_term_retention_policies_operations

* renamed functions; removed deleted_time since STRPolicy doesn't support on dropped SQL DB; add parameter help code; shorter parameter length.

* fix name

* fix code style

* remove unused parameter; use variable name from sdk: diff_backup_interval_in_hours on client; use show_command for 'show' cmdlets.

* switch to use show_custom_command rather than show_command since hit error 'azure.mgmt.sql.operations#BackupShortTermRetentionPoliciesOperations.get_short_term_retention' is invalid.

* should be custom_show_command

* add SqlServerDbShortTermRetentionScenarioTest

* add 'create db' to SqlServerDbShortTermRetentionScenarioTest

* fix create db of SqlServerDbShortTermRetentionScenarioTest

* change another way to create db for SqlServerDbShortTermRetentionScenarioTest

* update SqlServerDbShortTermRetentionScenarioTest test location to southeastasia

* upgrade azure-mgmt-sql to 0.21.0

* Revert "upgrade azure-mgmt-sql to 0.21.0"

This reverts commit 9a0e5e688e17e192ae36fcc748edf6579ca2f20c.

* remove unrelated comments

* edit test file, scenario test passed

* add test record

* fix cli style, reupload record

* style fix

* edit test file + remove old scenario test file

* upload succeed scenario test record

* change custom.py changes back

* update python sdk version to 0.29

* update azure-mgmt-sql version in setup.py file

* fix ServerPublicNetworkAccess to ServerNetworkAccessFlag according to new python sdk changes

* re-record failed scenario tests

* upload more scenario test record

* push more records

* upload more scenario tests

* add more scenario test record

* upload more scenario test record

* remove incident added pyproj and sln files

* upload scenario test records

* solve comment

* fix code style

Co-authored-by: Daniel Miller <[email protected]>
Co-authored-by: Azure CLI Team <[email protected]>
Co-authored-by: Lillian Liu <[email protected]>

* [App Service] Add support for Linux Consumption and improve how content share name is generated. (#18675)

* Added support for Powershell on Linux Consumption. Improved how content share name is generated.

* Removed unused test variable.

* [App Service] BREAKING CHANGE: `az functionapp create`: Remove support for EOL Node 8 and 10 (#18676)

* Removed support for EOL Node 8 and 10

* Fixed tests that relied on Node 8 or 10.

* Fixed more tests that relied on Node 8

* [App Service]: Fix an issue where adding VNET integration & Hybrid connections on a slot is not working (#18582)

* Updating the tests & fix for vnet_integration add with slots

* Updating add Hybrid connection for Slots & adding test cases for the scenario

* Updating test verification

* [AppService] BREAKING CHANGE: az webapp deployment source config: Remove vsts-cd-manager (#18203)

* Remove vsts-cd-manager

* Rerun tests

* Remove vsts-cd-manager

* Rerun tests

* [AppService] az appservice domain create: Fix get correct domain agreements (#18622)

* Fix get correct domain agreements

* Add test for domain agreement keys

* Fixes

* [AppService] az webapp deployment github-actions add/remove: new commands (#18261)

* Github actions

* Add --login-with-github flag to specify if you want to use interactive method to get github personal access token

* Linter

* Fixes

* don't pin pynacl package

* Fix linter

* Unpin PyGithub

* Fixes

* [AKS] az aks get-credentials: Add a check for KUBECONFIG environmental variable (#18704)

* Add a check for KUBECONFIG environmental variable

* Refactor to account for -f/--file being used.

* {SQL} Fix SQL related record file to pass CI (#18713)

* [AppConfiguration] Add support for ```disable_local_auth``` (#18619)

* [Compute] Make dependent resources of VM/VMSS support edge zone (#18708)

* [Compute] Update from CoreOS to Flatcar (#18644)

* [Compute] Add the hint to suggest users use the standard public IP when creating VM (#18662)

* {Release} Upgrade to Azure CLI 2.26.0 (#18716)

* [CDN] az cdn endpoint rule: Add OriginGroupOverride action support (#18711)

* [CDN] az cdn endpoint rule: Add OriginGroupOverride action support

* Fix typo

Co-authored-by: Bo Zhang <[email protected]>

* Migrate to track 2 SDK (#18740)

* [DMS] Track1 to track2 and MySQL removal (#18709)

* [ARM]: Fix ARM What-If formatter (#18721)

* Fix What-if formatter

added missing parameter in method call

* Update test_property_modify

Co-authored-by: Shenglong Li <[email protected]>

* [Compute] Fix issue #18633: az sig image-version create does not support mixing disks, snapshots, and vhd (#18741)

* fix issue

* test pass

* [ARM] `az bicep uninstall`: Add new command to uninstall bicep (#18744)

* Add az bicep uninstall and fix --stdout

* Add help for bicep uninstall

* [Packaging] Add licenses to all Python packages (#18749)

* [ACR] az acr import: Support no-wait  (#18572)

* add no-wait feature to acr import

* add no-wait feature to acr import

* revise no-wait feature for acr import

* add recording files

* style changes

* add no-wait warning and made style changes

* style fix

* method name changes to follow track 2 sdk changes

* style changes

* style changes

Co-authored-by: Caroline Barker <[email protected]>

* [Compute] Upgrade package version to fix securityProfile issue in `az vmss update` (#18788)

* upgrade version

* test pass:

* add test

* {IOT} Iot central track 2 migration (#18717)

* track 2 changes

* Rerecord yaml files that service team doesn't have permission

* record of test_certificate and test_iotcental

* Add yaml file for test_dps_lifecycle

Co-authored-by: zhoxing-ms <[email protected]>

* fix custom kubelet identity (#18795)

* add tags for create command (#18783)

* [ARM] `az provider register`: Add deprecate info for `--accept-term` (#18739)

* [ACR] Fix some bugs on uncovered cmds (#18786)

* {BatchAI} Migrate BatchAI module to track 2 SDK, azure-mgmt-batchai==7.0.0b1 (#18681)

* [Keyvault] BREAKING CHANGE: Fix #13752: az keyvault create not idempotent. Creating existing keyvault will fail. (#18520)

* check before create vault

* part of keyvault tests

* fix keyvault&storage&network tests

* other module tests

* pylint

* hybrid profile tests

* fix CI

* sql conflict

* fix test

* linter

* {Maps} Support managed maps creator operations (#18450)

* track 2

* uodate

* update

* update

* update

* update

* Compatible with old commands

* update

* creator

* Update _params.py

* Update _params.py

* Linter

* Update _help.py

* linter

* Update _params.py

* update

If property is None, they should not be in maps_account_update_parameters.
Modify the error of help.

* Update custom.py

* Update custom.py

* {Monitor} log analysis bump up SDK (#18374)

* bump up azure-mgmt-loganalysis sdk to version 11.0.0

* [Storage] `az storage blob copy start`: Fix issue in copy from different account (#18730)

* fix sas issue

* test pass

* fix style

* [Storage] Fix issues (#18811)

* fix

* fix jwt

* Fix #18352 (#18816)

* [ACR] hotfix based on #18786 and #18850 (#18853)

* {Release} HotFix: Release 2.26.1 (#18856)

* Hotfix for 2.26.1

* Add PR number

* add missing help to fix linter (#18859)

* Update help description for pool-change (#18835)

Co-authored-by: Audunn Baldvinsson <[email protected]>

* Fix style issue in conflict resolution

* [Storage] `az storage fs generate-sas`:  Support generate sas token for file system in ADLS Gen2 account (#18768)

* test pass

* fix linter and add test

* update and add dtest

* [Storage] az storage account blob-service-properties: Support last access tracking policy (#18731)

* [BACKUP] Added support for Customer Managed Keys(CMK) for Recovery Services Vault (#18733)

* Merged with commit added support for CMK feature

* Merged with commit Changes made

* Merged with commit More changes

* Changes with new version

* Linter file changed

* Changed test file

* encryption test changed

* Help Text Changed

* Changes in command

* help_text changed again

* modified param names based on review comments

* Encryption examples added

Co-authored-by: Vidhi Katyal <[email protected]>
Co-authored-by: Himanshu Agarwal <[email protected]>

* {Redis} Migrate to Track 2 Mgmt SDK (#18793)

* Fix style issue

* [Synapse]Workspace key update failure (#18719)

* 'workspacekeyupdatefailure'

* 'reformat'

* 'edithelp'

* 'format'

* Fix vm get boot log (#18830)

* [AKS] az aks create: Add edge zone support (#18852)

- Add the extended location parameter to create AKS in the edge zones

- Add a test case for the cluster creation in edge zone

Co-authored-by: Rhea Parekh <[email protected]>

* Fix help for lock (#18890)

* [SQL] `az sql server outbound-firewall-rule create`: Azure CLI Commands for Outbound Firewall Rules (#18671)

* AnotherCommit

* New Updates

* Working Test

* Updated version

* Updated Tests

* Linting updates

* Cleanup

* review comments

* Apply suggestions from code review

Co-authored-by: Yishi Wang <[email protected]>

* {APIM} Onboard to rest.az code autogen (#18724)

* prepare folder and file for autogen onboarding

* fixed style issues

* fixed style

* disabled unimportant pylint alerts

* updated test files

* [Synapse]Reimport notebook Failure (#18718)

* 'reimportnotebookerror'

* 'refine'

* 'paranamechange'

* 'addchanges'

* 'addexplanation'

* 'addimportprocess'

* 'addremovingredundantpara'

* 'addhelpandmissingpara'

* 'format1'

* 'format2'

* 'format3'

* 'format4'

* 'format5'

* {Keyvault} Fix keyvault packaging tests (#18922)

* Fix keyvault packaging tests

* Apply suggestions from code review

* address comment

* linter

* delete cert_secret file

* [App Config] BREAKING CHANGE: `appconfig kv set`: Set value to be empty string if not specified  (#18907)

* set value to be empty string

* revert change for default json type value

* [Container] `az container exec`: Fix and improve terminal experience (#18909)

* improve the azure container exec experience

* more convergence in the linux/windows paths, handle arrows and ctrlc in windows

* fix pylint errors

Co-authored-by: Novin Changizi <[email protected]>

* support ssl-certificate-name argument (#18861)

* {Docs} Update track2 migration roadmap (#18431)

* [Storage] Support Version level Worm (VLW) (#18540)

* test pass

* test pass

* [App Service] Add support for vnet route all setting (#18460)

* vnet route all arg + help fix

* quotes in help text

* vnet-route-all default behavior

* Add new and rerun tests. Clarify help text

* rerun tests

* Remove disable-route-all

* updated tests

* Fix linter issue

* [SQL]  BREAKING CHANGE: Update Microsoft.Sql to track2 SDK (#18637)

* Update sql track2

* Update commands.py

update begin_delete

* update setter_name to pass sdk validation

* replace_all_delete

* pass az self-check

* update recording tests

* fix all begin_xyz issue

* run test to unblock some issues

* pass SqlServerMSSupportScenarioTest

* change region to pass tests

* pass LTR test

* add recorded tests for MI

* update db_rename parameters

* pass db management tests

* address aad tests

* pass sql server aad

* revert azure-mgmt-netapp version change

* record test after rebase

* add three passed tests

* modify short term policy

* complete another three tests

* pass tde test

* pass two more tests

* Update custom.py

* pass two more tests

* pass classification test

* pass long and short backup

* update test file to match new config

* pass server and db security tests

* record test_sql_db_sensitivity_classifications again

* update MI tests

* pass MI tests

* pass tests

* modify instance_pool_test

* update aad_admin and aad_only_auth

* update sdk 3.0.1

* record the mssuport and db security tests

* pass instance pool and virtual cluster tests

* update hide uuid

* hack Microsoft.Management UUID

* pass instance failover

* fix lint/style and pass MI tests

* rebase

* pass outbound command test

* change show command and make log replay as live_only

* Delete test_sql_midb_logreplay_mgmt.yaml

* address the comments

* modify get_long_term_retention

* remove the --security-alert-policy-name

* pass two MI AAD tests with specific setup

* add comment and remove record only

* [ARM] `az deployment group/sub/mg/tenant create`: Add `--what-if` and `--proceed-if-no-change` parameters (#18924)

* Add --what-if and --proceed-if-no-change params

* Fix linter failures

* [ACR] `az acr build`: Make .dockerignore include directories with `!`  (#18821)

* solve path issue of included directories in dockerignore

* remove debugging command

* style changes

Co-authored-by: Caroline Barker <[email protected]>

* [ARM] Bump api-version from 2020-10-01 to 2021-04-01 (#18923)

* {EventGrid} - Fix delivery-identity, delivery-endpoint-type and delivery-identity-endpoint (#18989)

* [App Service] `az appservice ase create`: Command changes for ASE v3 GA (#18748)

* ga changes - draft

* exclude missing v3 apis

* remove debug line

* Add zone support

* Add test for zone redundancy

* Update help

* Hybrid connections improve error/help msg (#18843)

* {Eventgrid} Fix eventgrid test api version (#19003)

* [BACKUP] `az backup restore restore-disks`: Add MSI usage in IaaS VM Restore (#18961)

* merged latest dev

* vanguardm9 restore disks changes

* restore azure wl log point in time bug fix

* changed sdk version to 0.13.0

* ran tests on 2021-04 sdk

* ran tests on 2021-04 sdk

* sdk upgrade, ran tests, pylint fixes

* help text and linter fixes

* null check in vault identity

* changed mi related parameter name and help text

* removed linter exclusion

* changed the version

* removed unnecessary files

Co-authored-by: Akshay Neema <[email protected]>

* [ARM] `az ts create`: Make parameter `--template-file` support bicep file (#18888)

* Add support for bicep files in TS

* Add tests

* re-record test

* Changed new test to a live test

* Changed new test to a live test #2

* [Redis] Add `--redis-version` param to `az redis create` and `az redis update` (#18996)

* initial changes to support redis 6 create/update

* Updating test records

* Revert history.rst changes

* Add private link test for service bus (#18999)

* [Network] `az network application-gateway waf-policy custom-rule match-condition add`: Add examples (#18957)

* add example for 'az network application-gateway waf-policy custom-rule match-condition add'

* fix example in 'network application-gateway ssl-cert show'

* [network] `az network express-route port link update`: Add `--macsec-sci-state` argument. (#18814)

* support sci state for express route link

* add tests for network express route port

* fix style issue

* fix issue in 'az monitor alert create' (#18995)

* Remove trailing period from URL (#19019)

* {Telemetry} Import unittest.mock in telemetry test (#18977)

* {AKS} Refactor acs module (#18975)

* * add test for `aks browse`
* add preserve_location option for AKSCustomResourceGroupPreparer
* modify aks browse

* fix tag bug

* extract internal func for aks_browse

* add test for install-cli

* update test for browse

* fix @live_only tag

* fix test case & update recording file

* update recording

* fix style issue

* Add example for creating site extension to web app (#18935)

* [Network] private link add Microsoft.Web/hostingEnvironments provider (#19025)

* Add private link test for web ase

* fix

* network gateway loadbalance support cross tenant (#18792)

* [Network] private link add Microsoft.StorageSync/storageSyncServices provider (#19000)

* Add private link test for storagesync

* Skip storagesync test in record test

* {ACR} Migrate to Track 2 (#19006)

* trackV2 fix

* trackv2 fix

* pylint fix

* [Keyvault] Fix #6372: table output for secrets isn't correct (#18308)

* table transformer

* import

* move build_table_output to core

* [Databoxedge] Migrate databoxedge to track2 SDK (#18678)

* track2

* update

* update

* update

* style

* Linter

* manual

* test

* old test

* Update _help.py

* order

* bandwidth

* [Network] private link add Microsoft.Media/mediaservices provider  (#18997)

* Add private link test for media service

* fix

* [Packaging] Add SOCKS Proxy Support (#18931)

* {Core} Add `-NoProfile` when calling `powershell.exe` (#18770)

* [RDBMS] PostgreSQL, MySQL migration to GA API (#18921)

* migrated to PG GA API

* mysql migration initial commit

* mysql migration and vnet subnet

* fix params and mysql ga migration

* add tests for GA changes

* linter issue resolve

* linter issue fix2

* linter fix 3

* style linter issue fix

* test recordings

* test fix to network and resource

* test fiex 2

* add validator update test

* trigger CI

* Disable geo restore

* [App Config] BREAKING CHANGE: `az appconfig feature`: Support namespacing for feature flags and change output fields (#18990)

* Support prefix on feature flags

* Edit help messages

* List by key filter

* Raise more specific errors

* [Network] private link add Microsoft.Batch/batchAccounts provider (#18970)

* Add private link test for batch account

* fi

* delete old test

* Remove non-numerical chars from kube minor version (#18727)

* [Compute] `az vm list-skus`: Fix the issue that it can't query the SKU which with partially zones available (#18939)

* Fix vm list available skus

* Fix vm list skus

* Fix CI issue

* Fix CI style issue

* Fix Style issue

* [App Service] `az webapp config access-restriction remove`: Fix the broken caused by AttributeError (#18986)

* Fix #18947

* style fix

* [IoT] `az iot hub create\update`: File upload and certificate updates (#18966)

* Fileupload code and parameter updates

* Changed certificate uploads to always be base64-encoded.

* renamed fileupload-lock-duration arg to fileupload-notification-lock-duration

* More specific argument errors

* File upload test additions and minor logic updates

* PR Feedback and some parameter/logic updates

* Argument error class update

* Test timing and recording updates

Co-authored-by: Ryan Kelly <[email protected]>

* [ARM] `az ts`: Fix the issue that export template specs with no linked templates failed (#18928)

* Removed preview tag from template specs, fixed iteration bug

* Specified error in packing engine

* Fixed styling error

Co-authored-by: Dania Etienne <[email protected]>

* [ACR] `az acr connected-registry install info/renew-credentials`: Add new parameter `--parent-protocol` to specify parent protocol (#18954)

* Add specifying parent protocol to 'acr connected-registry install' to fetch complete connection string

* add one more blank line

* improve style

* change to enum type

* style fix

* Update src/azure-cli/azure/cli/command_modules/acr/_help.py

* Update src/azure-cli/azure/cli/command_modules/acr/_help.py

Co-authored-by: Xing Zhou <[email protected]>

* [Compute] `az vm auto-shutdown`: Fix the issue that `--webhook` is required when `--email` is passed in (#18958)

* Fix the issue that --webhook is required when parameter --email is passed in

* resolve conflicts

* [Compute] `az vm create`: Support creating VM from shared gallery image (#19037)

* Support shared gallery image for vm creation

* Fix help

* Fix help

* Fix CI issue

* Fix style issue

* Fix CI issue

* [Storage] `az storage copy` add new option `--cap-mbps` (#18344)

* [storage] Add new option `--cap-mbps`

* Update azcopy.py

* Update test_storage_azcopy_scenarios.py

* Update src/azure-cli/azure/cli/command_modules/storage/_params_azure_stack.py

Co-authored-by: Zunli Hu <[email protected]>

Co-authored-by: Zunli Hu <[email protected]>

* [Compute] az vm extension set: Add note to use Azure Key Vault VM extension instead (#19045)

* Add note to az add secret

* Update _help.py

* Update _help.py

* Update src/azure-cli/azure/cli/command_modules/vm/_help.py

Co-authored-by: Xing Zhou <[email protected]>

* {Core} Log `_get_attr` exceptions (#19030)

* [App Service]: Fix #17424: `az appservice plan show`: Provide correct exit status (#18994)

* fix #17424 - az appservice plan show does not provide correct exit status

* Tests are re-recorded with latest changes

* StyleError

* Disable pylint check temporarily

* Update src/azure-cli/azure/cli/command_modules/appservice/custom.py

* Fix pylint check

* Updated review comments

Co-authored-by: Xing Zhou <[email protected]>

* {Release} Upgrade to Azure CLI 2.27.0 (#19052)

* Increase TestYumPackage's timeoutInMinutes (#19058)

* Improve nodepool help text for OS Disk options (#19073)

* {AKS} Refactor acs module: add test cases for sub-command `create` (#19026)

* add test for autoscaler

* add test for loadbalancer

* add test for node public ip

* add test for network cidr

* update create default test

* add test for create attach acr

* add AKSCustomRoleBasedServicePrincipalPreparer

* update test ingress appgw

* add test case for outbound type udr

* add test for node_osdisk_diskencryptionset

* add test for enable-encryption-at-host, enable-azure-rbac, disable-rbac and dns-service-ip

* add test for workspace-resource-id

* add recordings

* update recordings

* [SQL] az sql mi update: Add --subnet and --vnet-name parameters to support the cross subnet update SLO (#18886)

* Update sql track2

* Update commands.py

update begin_delete

* update setter_name to pass sdk validation

* replace_all_delete

* pass az self-check

* update recording tests

* fix all begin_xyz issue

* run test to unblock some issues

* pass SqlServerMSSupportScenarioTest

* change region to pass tests

* pass LTR test

* add recorded tests for MI

* update db_rename parameters

* pass db management tests

* address aad tests

* pass sql server aad

* revert azure-mgmt-netapp version change

* record test after rebase

* add three passed tests

* modify short term policy

* complete another three tests

* pass tde test

* pass two more tests

* Update custom.py

* pass two more tests

* pass classification test

* pass long and short backup

* update test file to match new config

* pass server and db security tests

* record test_sql_db_sensitivity_classifications again

* update MI tests

* pass MI tests

* pass tests

* modify instance_pool_test

* Iteration 1:
- Adding support for cross subnet update SLO

* Iteration 2:
- Removing the change from History.rst file

* Iteration 3:
- Fixing merge errors

* Iteration 4:
- Removing merge errors

* Iteration 5:
- Removing merge errors

* Iteration 6:
- Fixed merge errors

* Iteration 7:
- Fix merge errors

* Iteration 7:
- Fixed CLI style issue - removed two trailing spaces
- Fixed CLI linter issue - fixed the cross-subnet update slo command in _help.py

Co-authored-by: Ji Wang <[email protected]>
Co-authored-by: Nemanja <[email protected]>

* [ACR] `acr create/update`: Add new parameter `--allow-exports` to support export (#19065)

* Update azure-mgmt-containerregistry from 8.0.0 to 8.1.0. Add support for export disable.

* Add acr export policy tests

style fix

* re-record acr dependent tests

* style: disable too many locals

* fix style issue

Co-authored-by: Tosin Adewale <[email protected]>

* Fix linter failures (#19116)

* {Packaging} Drop `mock` library (#19024)

* {AppService} Fix #17947: az functionapp update fails with Attribute error: checking non-existent app name (#19126)

* added condition to check for non-existent app name

* updated functionapp tests recordings

Co-authored-by: parthagg <[email protected]>

* removing preview stats for --enable-analytical-storage (#19113)

Co-authored-by: Feng Zhou <[email protected]>
Co-authored-by: Jason Freeberg <[email protected]>
Co-authored-by: Feiyue Yu <[email protected]>
Co-authored-by: Daeun Yim <[email protected]>
Co-authored-by: Jiashuo Li <[email protected]>
Co-authored-by: Yishi Wang <[email protected]>
Co-authored-by: Shenglong Li <[email protected]>
Co-authored-by: Ryan K <[email protected]>
Co-authored-by: Ryan Kelly <[email protected]>
Co-authored-by: Xing Zhou <[email protected]>
Co-authored-by: Rajesh1Ganesan <[email protected]>
Co-authored-by: Zunli Hu <[email protected]>
Co-authored-by: Kota Sudhakar Reddy <[email protected]>
Co-authored-by: Jiefeng Chen <[email protected]>
Co-authored-by: aim-for-better <[email protected]>
Co-authored-by: songlu <[email protected]>
Co-authored-by: Feiyue Yu <[email protected]>
Co-authored-by: Kerwin(Kaihui) Sun <[email protected]>
Co-authored-by: FumingZhang <[email protected]>
Co-authored-by: zesluo <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Chenyang Liu <[email protected]>
Co-authored-by: hildurhodd <[email protected]>
Co-authored-by: Calvin <[email protected]>
Co-authored-by: Vidya Kukke <[email protected]>
Co-authored-by: Alexandre Arpin <[email protected]>
Co-authored-by: Jan Schmidle <[email protected]>
Co-authored-by: Sisira Panchagnula <[email protected]>
Co-authored-by: Rodrigo Mendoza <[email protected]>
Co-authored-by: xiaofanzhou <[email protected]>
Co-authored-by: xfz11 <[email protected]>
Co-authored-by: Chris Eggert <[email protected]>
Co-authored-by: Azure CLI Bot <[email protected]>
Co-authored-by: Yu Chen <[email protected]>
Co-authored-by: Troy Connor <[email protected]>
Co-authored-by: Delora Bradish <[email protected]>
Co-authored-by: Ahmad Abbas <[email protected]>
Co-authored-by: Ahmad Abas <[email protected]>
Co-authored-by: Yang Yuan <[email protected]>
Co-authored-by: Bhuvaneswari Santharam <[email protected]>
Co-authored-by: Bhuvaneswari Santharam <[email protected]>
Co-authored-by: Mads Damgård <[email protected]>
Co-authored-by: Daniel Miller <[email protected]>
Co-authored-by: Azure CLI Team <[email protected]>
Co-authored-by: Zhenyu Zhou <[email protected]>
Co-authored-by: Ben Vesel <[email protected]>
Co-authored-by: strehan1993 <[email protected]>
Co-authored-by: xaliciayang <[email protected]>
Co-authored-by: Vinit Dinesh Parekh <[email protected]>
Co-authored-by: Alan Feng <[email protected]>
Co-authored-by: Avani Gupta <[email protected]>
Co-authored-by: Akshay Neema <[email protected]>
Co-authored-by: Akshay Neema <[email protected]>
Co-authored-by: Amit Singh <[email protected]>
Co-authored-by: Andy Zhang <[email protected]>
Co-authored-by: Shreyansh Joshi <[email protected]>
Co-authored-by: Shreyansh Joshi <[email protected]>
Co-authored-by: rewongmicrosoft <[email protected]>
Co-authored-by: Tongyao Si <[email protected]>
Co-authored-by: tomswedlund <[email protected]>
Co-authored-by: Tom Swedlund <[email protected]>
Co-authored-by: viparek <[email protected]>
Co-authored-by: Liming Liu <[email protected]>
Co-authored-by: lululilliancoding <[email protected]>
Co-authored-by: Lillian Liu <[email protected]>
Co-authored-by: Graham Zuber <[email protected]>
Co-authored-by: Ismayil <[email protected]>
Co-authored-by: Shuai Wang <[email protected]>
Co-authored-by: t-bzhan <[email protected]>
Co-authored-by: Bo Zhang <[email protected]>
Co-authored-by: Robert Gao <[email protected]>
Co-authored-by: Artyom Pavlichenko <[email protected]>
Co-authored-by: Mikolaj Mackowiak <[email protected]>
Co-authored-by: Major Hayden <[email protected]>
Co-authored-by: Caroline Barker <[email protected]>
Co-authored-by: Caroline Barker <[email protected]>
Co-authored-by: abalogun-dev <[email protected]>
Co-authored-by: Auðunn <[email protected]>
Co-authored-by: Audunn Baldvinsson <[email protected]>
Co-authored-by: Vidhi-2510 <[email protected]>
Co-authored-by: Vidhi Katyal <[email protected]>
Co-authored-by: Himanshu Agarwal <[email protected]>
Co-authored-by: Praveen kumar <[email protected]>
Co-authored-by: Rhea Parekh <[email protected]>
Co-authored-by: Rhea Parekh <[email protected]>
Co-authored-by: Vineet Mahadik <[email protected]>
Co-authored-by: Novin <[email protected]>
Co-authored-by: Novin Changizi <[email protected]>
Co-authored-by: Ji Wang <[email protected]>
Co-authored-by: Gokul Premraj <[email protected]>
Co-authored-by: MECHANDR <[email protected]>
Co-authored-by: John Downs <[email protected]>
Co-authored-by: Graham Hayes <[email protected]>
Co-authored-by: Dania Etienne <[email protected]>
Co-authored-by: Dania Etienne <[email protected]>
Co-authored-by: Yuan Xi <[email protected]>
Co-authored-by: Micah McKittrick <[email protected]>
Co-authored-by: v-nestan <[email protected]>
Co-authored-by: Nemanja <[email protected]>
Co-authored-by: Tosin Adewale <[email protected]>
Co-authored-by: Tosin Adewale <[email protected]>
Co-authored-by: Parth Aggarwal <[email protected]>
Co-authored-by: parthagg <[email protected]>
Co-authored-by: toswedlu <[email protected]>
@@ -224,7 +223,7 @@ def acr_connected_registry_delete(cmd,
try:
connected_registry = acr_connected_registry_show(
cmd, client, connected_registry_name, registry_name, resource_group_name)
result = client.delete(resource_group_name, registry_name, connected_registry_name)
result = client.begin_delete(resource_group_name, registry_name, connected_registry_name)
Copy link
Member

@jiasli jiasli Oct 22, 2024

Choose a reason for hiding this comment

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

The result is not polled.

Then at L237, acr_token_delete makes another client.begin_delete call. This causes Message: The subscription '00000000-0000-0000-0000-000000000000' could not be found. error: https://dev.azure.com/azclitools/public/_build/results?buildId=199090&view=logs&jobId=76ded334-3544-5630-8508-787a6cf96d29&j=76ded334-3544-5630-8508-787a6cf96d29&t=c62dd615-c6c6-5fdb-a846-07cb3711ef74

2024-10-22T02:44:50.2908872Z _________ AcrConnectedRegistryCommandsTests.test_acr_connectedregistry _________
2024-10-22T02:44:50.2909322Z [gw0] linux -- Python 3.11.8 /opt/az/bin/python3
2024-10-22T02:44:50.2909646Z self = <azure.cli.testsdk.base.ExecutionResult object at 0x7f32918d1d10>
2024-10-22T02:44:50.2910002Z cli_ctx = <azure.cli.core.mock.DummyCli object at 0x7f32929aa910>
2024-10-22T02:44:50.2910500Z command = 'acr connected-registry delete -n grandchild -r clireg000002 --cleanup -y'
2024-10-22T02:44:50.2910805Z expect_failure = False
2024-10-22T02:44:50.2910906Z 
2024-10-22T02:44:50.2911181Z     def _in_process_execute(self, cli_ctx, command, expect_failure=False):
2024-10-22T02:44:50.2911473Z         from io import StringIO
2024-10-22T02:44:50.2911788Z         from vcr.errors import CannotOverwriteExistingCassetteException
2024-10-22T02:44:50.2913231Z     
2024-10-22T02:44:50.2913589Z         if command.startswith('az '):
2024-10-22T02:44:50.2913841Z             command = command[3:]
2024-10-22T02:44:50.2914058Z     
2024-10-22T02:44:50.2914267Z         stdout_buf = StringIO()
2024-10-22T02:44:50.2914504Z         logging_buf = StringIO()
2024-10-22T02:44:50.2914724Z         try:
2024-10-22T02:44:50.2915015Z             # issue: stderr cannot be redirect in this form, as a result some failure information
2024-10-22T02:44:50.2915318Z             # is lost when command fails.
2024-10-22T02:44:50.2915626Z >           self.exit_code = cli_ctx.invoke(shlex.split(command), out_file=stdout_buf) or 0
2024-10-22T02:44:50.2915784Z 
2024-10-22T02:44:50.2916158Z /opt/az/lib/python3.11/site-packages/azure/cli/testsdk/base.py:302: 
2024-10-22T02:44:50.2916475Z _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
2024-10-22T02:44:50.2917154Z /opt/az/lib/python3.11/site-packages/knack/cli.py:245: in invoke
2024-10-22T02:44:50.2917462Z     exit_code = self.exception_handler(ex)
2024-10-22T02:44:50.2917915Z /opt/az/lib/python3.11/site-packages/azure/cli/core/__init__.py:129: in exception_handler
2024-10-22T02:44:50.2918231Z     return handle_exception(ex)
2024-10-22T02:44:50.2918510Z _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
2024-10-22T02:44:50.2918642Z 
2024-10-22T02:44:50.2919333Z ex = ResourceNotFoundError("(SubscriptionNotFound) The subscription '00000000-0000-0000-0000-000000000000' could not be found.\nCode: SubscriptionNotFound\nMessage: The subscription '00000000-0000-0000-0000-000000000000' could not be found.")
2024-10-22T02:44:50.2919782Z args = (), kwargs = {}
2024-10-22T02:44:50.2919881Z 
2024-10-22T02:44:50.2920277Z     def _handle_main_exception(ex, *args, **kwargs):  # pylint: disable=unused-argument
2024-10-22T02:44:50.2920652Z         if isinstance(ex, CannotOverwriteExistingCassetteException):
2024-10-22T02:44:50.2920998Z             # This exception usually caused by a no match HTTP request. This is a product error
2024-10-22T02:44:50.2921322Z             # that is caused by change of SDK invocation.
2024-10-22T02:44:50.2921569Z             raise ex
2024-10-22T02:44:50.2921770Z     
2024-10-22T02:44:50.2921997Z >       raise CliExecutionError(ex)
2024-10-22T02:44:50.2922374Z E       azure.cli.testsdk.exceptions.CliExecutionError: The CLI throws exception ResourceNotFoundError during execution and fails the command.
2024-10-22T02:44:50.2922582Z 
2024-10-22T02:44:50.2922998Z /opt/az/lib/python3.11/site-packages/azure/cli/testsdk/patches.py:35: CliExecutionError
2024-10-22T02:44:50.2923169Z 
2024-10-22T02:44:50.2923431Z During handling of the above exception, another exception occurred:
2024-10-22T02:44:50.2923580Z 
2024-10-22T02:44:50.2923928Z self = <command_modules.acr.tests.latest.test_acr_connectedregistry_commands.AcrConnectedRegistryCommandsTests testMethod=test_acr_connectedregistry>
2024-10-22T02:44:50.2924399Z resource_group = 'clitest.rg000001'
2024-10-22T02:44:50.2924512Z 
2024-10-22T02:44:50.2924733Z     @ResourceGroupPreparer()
2024-10-22T02:44:50.2925019Z     def test_acr_connectedregistry(self, resource_group):
2024-10-22T02:44:50.2925327Z         # Agentpool prerequisites for connected registry testing
2024-10-22T02:44:50.2925687Z         crName = 'connectedregistry'
2024-10-22T02:44:50.2926010Z         rootName = 'rootregistry'
2024-10-22T02:44:50.2926322Z         childName = 'child'
2024-10-22T02:44:50.2926637Z         grandchildName = 'grandchild'
2024-10-22T02:44:50.2926947Z         repo1 = 'repo1'
2024-10-22T02:44:50.2927232Z         repo2 = 'repo2'
2024-10-22T02:44:50.2927510Z         repo3 = 'repo3'
2024-10-22T02:44:50.2927750Z         self.kwargs.update({
2024-10-22T02:44:50.2928127Z             'registry_name': self.create_random_name('clireg', 20),
2024-10-22T02:44:50.2928472Z             'cr_name': crName,
2024-10-22T02:44:50.2928790Z             'root_name': rootName,
2024-10-22T02:44:50.2929102Z             'child_name': childName,
2024-10-22T02:44:50.2929442Z             'grandchild_name': grandchildName,
2024-10-22T02:44:50.2929766Z             'rg_loc': 'eastus',
2024-10-22T02:44:50.2930066Z             'sku': 'Premium',
2024-10-22T02:44:50.2930380Z             'syncToken': 'syncToken',
2024-10-22T02:44:50.2930710Z             'clientToken': 'clientToken',
2024-10-22T02:44:50.2931064Z             'clientToken2': 'clientToken2',
2024-10-22T02:44:50.2931386Z             'scopeMap': 'scopeMap1',
2024-10-22T02:44:50.2931687Z             'repo_1': repo1,
2024-10-22T02:44:50.2931984Z             'repo_2': repo2,
2024-10-22T02:44:50.2932274Z             'repo_3': repo3,
2024-10-22T02:44:50.2932601Z             'syncSchedule': '0 0/10 * * *',
2024-10-22T02:44:50.2932949Z             'defaultSyncSchedule': '* * * * *',
2024-10-22T02:44:50.2933269Z             'syncWindow': 'PT4H',
2024-10-22T02:44:50.2933758Z             'notificationStr': 'hello-world:tag:push',
2024-10-22T02:44:50.2934227Z             'notificationStr2': '*:*'
2024-10-22T02:44:50.2934452Z         })
2024-10-22T02:44:50.2934689Z         # Create Registr and enable data endpoint
2024-10-22T02:44:50.2935109Z         self.cmd('acr create -n {registry_name} -g {rg} -l {rg_loc} --sku {sku}',
2024-10-22T02:44:50.2935529Z                  checks=[self.check('name', '{registry_name}'),
2024-10-22T02:44:50.2935911Z                          self.check('location', '{rg_loc}'),
2024-10-22T02:44:50.2936301Z                          self.check('adminUserEnabled', False),
2024-10-22T02:44:50.2936666Z                          self.check('sku.name', '{sku}'),
2024-10-22T02:44:50.2937034Z                          self.check('sku.tier', '{sku}'),
2024-10-22T02:44:50.2937431Z                          self.check('provisioningState', 'Succeeded')])
2024-10-22T02:44:50.2937855Z         self.cmd('acr update -n {registry_name} --data-endpoint-enabled true',
2024-10-22T02:44:50.2938277Z                  checks=self.check('dataEndpointEnabled', True))
2024-10-22T02:44:50.2938534Z     
2024-10-22T02:44:50.2938766Z         # Create a default connected registry.
2024-10-22T02:44:50.2939289Z         self.cmd('acr connected-registry create -n {cr_name} -r {registry_name} -m ReadWrite --repository {repo_1} {repo_2} {repo_3}',
2024-10-22T02:44:50.2939753Z                  checks=[self.check('name', '{cr_name}'),
2024-10-22T02:44:50.2940118Z                          self.check('mode', 'ReadWrite'),
2024-10-22T02:44:50.2940509Z                          self.check('logging.logLevel', 'Information'),
2024-10-22T02:44:50.2940915Z                          self.check('provisioningState', 'Succeeded'),
2024-10-22T02:44:50.2941297Z                          self.check('resourceGroup', '{rg}')])
2024-10-22T02:44:50.2941530Z     
2024-10-22T02:44:50.2941916Z         # Create a custom connected-registry with a previously created token.
2024-10-22T02:44:50.2942609Z         self.cmd('acr token create -r {registry_name} -n {syncToken} --repository {repo_1} content/read metadata/read --gateway {root_name} config/read config/write message/read message/write --no-passwords')
2024-10-22T02:44:50.2943264Z         self.cmd('acr token create -r {registry_name} -n {clientToken} --repository {repo_1} content/read --no-passwords')
2024-10-22T02:44:50.2944016Z         self.cmd('acr connected-registry create -n {root_name} -r {registry_name} --sync-token {syncToken} --log-level Warning -s "{syncSchedule}" -w PT4H --client-tokens {clientToken} --notifications {notificationStr}',
2024-10-22T02:44:50.2944554Z                  checks=[self.check('name', '{root_name}'),
2024-10-22T02:44:50.2944925Z                          self.check('mode', 'ReadOnly'),
2024-10-22T02:44:50.2945306Z                          self.check('provisioningState', 'Succeeded'),
2024-10-22T02:44:50.2945705Z                          self.check('logging.logLevel', 'Warning'),
2024-10-22T02:44:50.2946142Z                          self.check('parent.syncProperties.schedule', '{syncSchedule}'),
2024-10-22T02:44:50.2946633Z                          self.check('parent.syncProperties.syncWindow', '4:00:00'),
2024-10-22T02:44:50.2947053Z                          self.check('resourceGroup', '{rg}'),
2024-10-22T02:44:50.2947475Z                          self.check('notificationsList[0]', '{notificationStr}')])
2024-10-22T02:44:50.2947733Z     
2024-10-22T02:44:50.2947960Z         # Create Child connected registry
2024-10-22T02:44:50.2948472Z         self.cmd('acr connected-registry create -n {child_name} -p {root_name} -r {registry_name} --repository {repo_2} -m ReadOnly',
2024-10-22T02:44:50.2948926Z                  checks=[self.check('name', '{child_name}'),
2024-10-22T02:44:50.2949297Z                          self.check('mode', 'ReadOnly'),
2024-10-22T02:44:50.2949692Z                          self.check('provisioningState', 'Succeeded'),
2024-10-22T02:44:50.2950111Z                          self.check("ends_with(parent.id, '{root_name}')", True),
2024-10-22T02:44:50.2950712Z                          self.check('resourceGroup', '{rg}')])
2024-10-22T02:44:50.2950960Z     
2024-10-22T02:44:50.2951197Z         # Create Grandchild connected registry
2024-10-22T02:44:50.2951717Z         self.cmd('acr connected-registry create -n {grandchild_name} -p {child_name} -r {registry_name} --repository {repo_2} -m ReadOnly',
2024-10-22T02:44:50.2952511Z                  checks=[self.check('name', '{grandchild_name}'),
2024-10-22T02:44:50.2952950Z                          self.check('mode', 'ReadOnly'),
2024-10-22T02:44:50.2953374Z                          self.check('provisioningState', 'Succeeded'),
2024-10-22T02:44:50.2953854Z                          self.check("ends_with(parent.id, '{child_name}')", True),
2024-10-22T02:44:50.2954289Z                          self.check('resourceGroup', '{rg}')])
2024-10-22T02:44:50.2954550Z     
2024-10-22T02:44:50.2954801Z         # List connected registries
2024-10-22T02:44:50.2955219Z         self.cmd('acr connected-registry list -r {registry_name}',
2024-10-22T02:44:50.2955636Z                  checks=[self.check('[0].name', '{cr_name}'),
2024-10-22T02:44:50.2956045Z                          self.check('[1].name', '{root_name}')])
2024-10-22T02:44:50.2956302Z     
2024-10-22T02:44:50.2956530Z         # List client tokens
2024-10-22T02:44:50.2956987Z         self.cmd('acr connected-registry list-client-tokens -n {root_name} -r {registry_name}',
2024-10-22T02:44:50.2957444Z                  checks=[self.check('[0].name', '{clientToken}')])
2024-10-22T02:44:50.2957677Z     
2024-10-22T02:44:50.2957891Z         # Update the connected registry
2024-10-22T02:44:50.2958343Z         self.cmd('acr token create -r {registry_name} -n {clientToken2} --repository {repo_2} metadata/read --no-passwords')
2024-10-22T02:44:50.2959187Z         self.cmd('acr connected-registry update -n {root_name} -r {registry_name} --log-level Information -s "{defaultSyncSchedule}" --remove-client-tokens {clientToken} --add-client-tokens {clientToken2} --add-notifications {notificationStr2} --remove-notifications {notificationStr}',
2024-10-22T02:44:50.2959811Z                  checks=[self.check('name', '{root_name}'),
2024-10-22T02:44:50.2960205Z                          self.check('logging.logLevel', 'Information'),
2024-10-22T02:44:50.2960648Z                          self.check('parent.syncProperties.schedule', '{defaultSyncSchedule}'),
2024-10-22T02:44:50.2961117Z                          self.check('parent.syncProperties.syncWindow', 'None'),
2024-10-22T02:44:50.2961540Z                          self.check('resourceGroup', '{rg}'),
2024-10-22T02:44:50.2961997Z                          self.check('notificationsList[0]', '{notificationStr2}')])
2024-10-22T02:44:50.2962298Z     
2024-10-22T02:44:50.2962546Z         # List client tokens
2024-10-22T02:44:50.2963008Z         self.cmd('acr connected-registry list-client-tokens -n {root_name} -r {registry_name}',
2024-10-22T02:44:50.2963476Z                  checks=[self.check('[0].name', '{clientToken2}')])
2024-10-22T02:44:50.2963741Z     
2024-10-22T02:44:50.2963982Z         # Show connected registry properties
2024-10-22T02:44:50.2964436Z         self.cmd('acr connected-registry show -n {cr_name} -r {registry_name}',
2024-10-22T02:44:50.2964904Z                  checks=[self.check('name', '{cr_name}')])
2024-10-22T02:44:50.2965194Z     
2024-10-22T02:44:50.2965491Z         # Get connection string a generate new password.
2024-10-22T02:44:50.2966148Z         self.cmd('acr connected-registry get-settings -n {root_name} -r {registry_name} --parent-protocol https --generate-password 2 -y')
2024-10-22T02:44:50.2966745Z         self.cmd('acr token show -n {syncToken} -r {registry_name}', checks=[
2024-10-22T02:44:50.2967184Z             self.check('credentials.passwords[0].name', 'password2')])
2024-10-22T02:44:50.2967437Z     
2024-10-22T02:44:50.2967693Z         # Update and check connected registry repo permissions
2024-10-22T02:44:50.2968200Z         self.cmd('acr connected-registry permissions update -n {grandchild_name} -r {registry_name} --add {repo_1} {repo_3}')
2024-10-22T02:44:50.2968763Z     
2024-10-22T02:44:50.2969218Z         scope_map = self.cmd('acr connected-registry permissions show -n {root_name} -r {registry_name}').get_output_in_json()
2024-10-22T02:44:50.2969693Z         self.assertListEqual(sorted(scope_map['actions']),
2024-10-22T02:44:50.2970161Z                              ['gateway/'+ childName +'/config/read', 'gateway/'+ childName +'/config/write',
2024-10-22T02:44:50.2970675Z                               'gateway/'+ childName +'/message/read', 'gateway/'+ childName +'/message/write',
2024-10-22T02:44:50.2971199Z                               'gateway/'+ grandchildName +'/config/read', 'gateway/'+ grandchildName +'/config/write',
2024-10-22T02:44:50.2971773Z                               'gateway/'+ grandchildName +'/message/read', 'gateway/'+ grandchildName +'/message/write',
2024-10-22T02:44:50.2972329Z                               'gateway/'+ rootName +'/config/read', 'gateway/'+ rootName +'/config/write',
2024-10-22T02:44:50.2972868Z                               'gateway/'+ rootName +'/message/read', 'gateway/'+ rootName +'/message/write',
2024-10-22T02:44:50.2973421Z                               'repositories/'+ repo1 +'/content/read', 'repositories/'+ repo1 +'/metadata/read',
2024-10-22T02:44:50.2973957Z                               'repositories/'+ repo2 +'/content/read', 'repositories/'+ repo2 +'/metadata/read',
2024-10-22T02:44:50.2974504Z                               'repositories/'+ repo3 +'/content/read', 'repositories/'+ repo3 +'/metadata/read'])
2024-10-22T02:44:50.2974812Z     
2024-10-22T02:44:50.2975303Z         self.cmd('acr connected-registry permissions update -n {root_name} -r {registry_name} --remove {repo_1} {repo_2}')
2024-10-22T02:44:50.2975935Z         self.cmd('acr connected-registry permissions update -n {child_name} -r {registry_name} --remove {repo_3} --add {repo_1}')
2024-10-22T02:44:50.2976286Z     
2024-10-22T02:44:50.2976799Z         scope_map = self.cmd('acr connected-registry permissions show -n {grandchild_name} -r {registry_name}').get_output_in_json()
2024-10-22T02:44:50.2977321Z         self.assertListEqual(sorted(scope_map['actions']),
2024-10-22T02:44:50.2977867Z                              ['gateway/'+ grandchildName +'/config/read', 'gateway/'+ grandchildName +'/config/write',
2024-10-22T02:44:50.2978454Z                               'gateway/'+ grandchildName +'/message/read', 'gateway/'+ grandchildName +'/message/write'])
2024-10-22T02:44:50.2979148Z     
2024-10-22T02:44:50.2979626Z         # Delete connected registry grand child
2024-10-22T02:44:50.2980171Z >       self.cmd('acr connected-registry delete -n {grandchild_name} -r {registry_name} --cleanup -y')
2024-10-22T02:44:50.2980371Z 
2024-10-22T02:44:50.2981129Z /opt/az/lib/python3.11/site-packages/azure/cli/command_modules/acr/tests/latest/test_acr_connectedregistry_commands.py:146: 
2024-10-22T02:44:50.2981589Z _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
2024-10-22T02:44:50.2982090Z /opt/az/lib/python3.11/site-packages/azure/cli/testsdk/base.py:176: in cmd
2024-10-22T02:44:50.2982507Z     return execute(self.cli_ctx, command, expect_failure=expect_failure).assert_with_checks(checks)
2024-10-22T02:44:50.2983037Z /opt/az/lib/python3.11/site-packages/azure/cli/testsdk/base.py:251: in __init__
2024-10-22T02:44:50.2983428Z     self._in_process_execute(cli_ctx, command, expect_failure=expect_failure)
2024-10-22T02:44:50.2983928Z /opt/az/lib/python3.11/site-packages/azure/cli/testsdk/base.py:314: in _in_process_execute
2024-10-22T02:44:50.2984249Z     raise ex.exception
2024-10-22T02:44:50.2984641Z /opt/az/lib/python3.11/site-packages/knack/cli.py:233: in invoke
2024-10-22T02:44:50.2984960Z     cmd_result = self.invocation.execute(args)
2024-10-22T02:44:50.2985418Z /opt/az/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py:666: in execute
2024-10-22T02:44:50.2985714Z     raise ex
2024-10-22T02:44:50.2986126Z /opt/az/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py:733: in _run_jobs_serially
2024-10-22T02:44:50.2986731Z     results.append(self._run_job(expanded_arg, cmd_copy))
2024-10-22T02:44:50.2987185Z /opt/az/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py:703: in _run_job
2024-10-22T02:44:50.2987481Z     result = cmd_copy(params)
2024-10-22T02:44:50.2987885Z /opt/az/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py:336: in __call__
2024-10-22T02:44:50.2988188Z     return self.handler(*args, **kwargs)
2024-10-22T02:44:50.2988613Z /opt/az/lib/python3.11/site-packages/azure/cli/core/commands/command_operation.py:121: in handler
2024-10-22T02:44:50.2988916Z     return op(**command_args)
2024-10-22T02:44:50.2989387Z /opt/az/lib/python3.11/site-packages/azure/cli/command_modules/acr/connected_registry.py:268: in acr_connected_registry_delete
2024-10-22T02:44:50.2989791Z     acr_scope_map_delete(cmd, scope_map_client, registry_name, sync_scope_map_name, yes, resource_group_name)
2024-10-22T02:44:50.2990306Z /opt/az/lib/python3.11/site-packages/azure/cli/command_modules/acr/scope_map.py:73: in acr_scope_map_delete
2024-10-22T02:44:50.2990738Z     return client.begin_delete(resource_group_name, registry_name, scope_map_name)
2024-10-22T02:44:50.2991260Z /opt/az/lib/python3.11/site-packages/azure/core/tracing/decorator.py:94: in wrapper_use_tracer
2024-10-22T02:44:50.2991599Z     return func(*args, **kwargs)
2024-10-22T02:44:50.2992411Z /opt/az/lib/python3.11/site-packages/azure/mgmt/containerregistry/v2020_11_01_preview/operations/_scope_maps_operations.py:635: in begin_delete
2024-10-22T02:44:50.2992840Z     raw_result = self._delete_initial(  # type: ignore
2024-10-22T02:44:50.2993441Z /opt/az/lib/python3.11/site-packages/azure/mgmt/containerregistry/v2020_11_01_preview/operations/_scope_maps_operations.py:589: in _delete_initial
2024-10-22T02:44:50.2993950Z     map_error(status_code=response.status_code, response=response, error_map=error_map)
2024-10-22T02:44:50.2994311Z _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
2024-10-22T02:44:50.2994464Z 
2024-10-22T02:44:50.2994681Z status_code = 404
2024-10-22T02:44:50.2995161Z response = <RequestsTransportResponse: 404 Not Found, Content-Type: application/json; charset=utf-8>
2024-10-22T02:44:50.2996034Z error_map = {304: <class 'azure.core.exceptions.ResourceNotModifiedError'>, 401: <class 'azure.core.exceptions.ClientAuthenticatio..., 404: <class 'azure.core.exceptions.ResourceNotFoundError'>, 409: <class 'azure.core.exceptions.ResourceExistsError'>}
2024-10-22T02:44:50.2996387Z 
2024-10-22T02:44:50.2996614Z     def map_error(
2024-10-22T02:44:50.2996949Z         status_code: int, response: _HttpResponseCommonAPI, error_map: Mapping[int, Type[HttpResponseError]]
2024-10-22T02:44:50.2997327Z     ) -> None:
2024-10-22T02:44:50.2997561Z         if not error_map:
2024-10-22T02:44:50.2997797Z             return
2024-10-22T02:44:50.2998067Z         error_type = error_map.get(status_code)
2024-10-22T02:44:50.2998339Z         if not error_type:
2024-10-22T02:44:50.2998581Z             return
2024-10-22T02:44:50.2998845Z         error = error_type(response=response)
2024-10-22T02:44:50.2999120Z >       raise error
2024-10-22T02:44:50.2999695Z E       azure.core.exceptions.ResourceNotFoundError: (SubscriptionNotFound) The subscription '00000000-0000-0000-0000-000000000000' could not be found.
2024-10-22T02:44:50.3000079Z E       Code: SubscriptionNotFound
2024-10-22T02:44:50.3000542Z E       Message: The subscription '00000000-0000-0000-0000-000000000000' could not be found.
2024-10-22T02:44:50.3000716Z 
2024-10-22T02:44:50.3001163Z /opt/az/lib/python3.11/site-packages/azure/core/exceptions.py:161: ResourceNotFoundError

This is similar to #17185.

Copy link
Member

Choose a reason for hiding this comment

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

It even causes a test error:

https://dev.azure.com/azclitools/public/_build/results?buildId=199090&view=logs&jobId=76ded334-3544-5630-8508-787a6cf96d29&j=76ded334-3544-5630-8508-787a6cf96d29&t=c62dd615-c6c6-5fdb-a846-07cb3711ef74

2024-10-22T02:44:50.2811890Z ==================================== ERRORS ====================================
2024-10-22T02:44:50.2812330Z _ ERROR at teardown of AcrConnectedRegistryCommandsTests.test_acr_connectedregistry _
2024-10-22T02:44:50.2813027Z [gw0] linux -- Python 3.11.8 /opt/az/bin/python3
2024-10-22T02:44:50.2813530Z /opt/az/lib/python3.11/site-packages/azure/cli/testsdk/base.py:146: in tearDown
2024-10-22T02:44:50.2813893Z     super(ScenarioTest, self).tearDown()
2024-10-22T02:44:50.2814223Z _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
2024-10-22T02:44:50.2814367Z 
2024-10-22T02:44:50.2814770Z self = <command_modules.acr.tests.latest.test_acr_connectedregistry_commands.AcrConnectedRegistryCommandsTests testMethod=test_acr_connectedregistry>
2024-10-22T02:44:50.2815030Z 
2024-10-22T02:44:50.2815272Z     def tearDown(self):
2024-10-22T02:44:50.2815553Z         os.environ = self.original_env
2024-10-22T02:44:50.2815839Z         # Autorest.Python 2.x
2024-10-22T02:44:50.2816215Z         assert not [t for t in threading.enumerate() if t.name.startswith("AzureOperationPoller")], \
2024-10-22T02:44:50.2817186Z             "You need to call 'result' or 'wait' on all AzureOperationPoller you have created"
2024-10-22T02:44:50.2817550Z         # Autorest.Python 3.x
2024-10-22T02:44:50.2817887Z >       assert not [t for t in threading.enumerate() if t.name.startswith("LROPoller")], \
2024-10-22T02:44:50.2818411Z             "You need to call 'result' or 'wait' on all LROPoller you have created"
2024-10-22T02:44:50.2818942Z E       AssertionError: You need to call 'result' or 'wait' on all LROPoller you have created
2024-10-22T02:44:50.2819126Z 
2024-10-22T02:44:50.2819611Z /opt/az/lib/python3.11/site-packages/azure/cli/testsdk/scenario_tests/base.py:160: AssertionError

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.

5 participants