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

merge cli dev #6

Closed
wants to merge 834 commits into from
Closed

merge cli dev #6

wants to merge 834 commits into from

Conversation

DaeunYim
Copy link
Owner

Description

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.

evelyn-ys and others added 30 commits May 26, 2021 17:06
Co-authored-by: Ryan Kelly <rykelly@microsoft.com>
…ion-id (#18209)

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

* param

* application_id.lower comparison

* fix delete

* address comment
…021-03-01' (#18233)

* Upgrade api-version for compute

* Fix CI

* rerecord aks
…ature 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
…e-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
* 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
* recover for MHSM

* fix

* tests

* linter
Co-authored-by: Ryan Kelly <rykelly@microsoft.com>
…and 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
* 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
* reorder private link providers

* comment miss private link providers

* support private link provider for Microsoft.Sql/servers

* update recordings
…ack (#18301)

* Fix unsupported parameters in old api-version for vmss

* Run releated test in old profile

* Fix CI style
* [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 a197e3c, reversing
changes made to 10b1d7d.

* 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 <iamyfy@163.com>

* Update preparers.py

* Resolve conflicts

* Revert "Resolve conflicts"

This reverts commit 034b326.

* Resolve conflicts

* Update test_backup_wl_sql_policy.yaml

* Delete test_acs_create_kubernetes.yaml

Co-authored-by: Feiyue Yu <iamyfy@163.com>
* private link resource

* private endpoint connection

* tests

* network module tests

* linter

* merge network changes
…r 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
* fix virtual network preparer

* remove outdated checks
…ributeError: '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
…ailability 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
adewaleo and others added 29 commits August 4, 2021 11:27
…port 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 <oluwatosin.adewale@microsoft.com>
…rror: checking non-existent app name (#19126)

* added condition to check for non-existent app name

* updated functionapp tests recordings

Co-authored-by: parthagg <parthagg@umich.edu>
* codeowncli

* addcliteammember
Co-authored-by: jiasli <4003950+jiasli@users.noreply.github.com>
…ion for hybrid profile (#18443)

* Changing hybrid authorization api-version

* Adding test for acs and acr hybrid profiles

* adding azurestack check-health changes

* Acs hybrid test changes

* Modifying Hybrid test for acr

* Hybrid test acs

* Adding Aks hybrid test

* merge acr test changes

* Hybrid profile test  update

* Drop mock for acs hybrid test

* Drop mock for acr hybrid test

Co-authored-by: Bhuvaneswari Santharam <bhsantha@microsoft.com>
* Fix the enum name change in track2 Python SDK

* add AVAILABLE

* fix the available status

* pass all the tests

* fix style issue

* update the comment

* CapabilityGroup.SUPPORTED_ELASTIC_POOL_EDITIONS

* update comment
…leases (#19072)

* Add support for kubelogin darwin/arm64 releases

* Fix styling issues
* Fix resource group deletion for corss tenant tests

* Delete the group delete command
…eference to gateway load balancer fronted ID (#19162)

* fix unable to delete the reference to gateway load balancer fronted ID

* update test
…te`: Remove preview flag (#19177)

* WebApp: Remove preview flag for storage-account commands

* Removed Preview tag for List too.
* update test

* replace env var with file hook

* fix style

* catch test failure

* clarify functions

* replace CLIError with specific error types

* fix style

* update hook
…ame `--migration-id` to `--migration-name` (#19149)

* Changed migration_id to migration-name

* Changed File name to conform with naming conventions

Co-authored-by: Chanderprabh Jain <chajain@microsoft.com>
…s when the functionapp name does not reference an existing functionapp (#19127)

* added check for non-existent app

* fixed style

* changed error type

* returning identity instead of operation

* updated test recordings

* renamed variable
* create ssh key in advance

* update case without ssh-key-value

* update recording
Co-authored-by: jiasli <4003950+jiasli@users.noreply.github.com>
@DaeunYim DaeunYim closed this Aug 13, 2021
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.