-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[SQL] az sql instance-pool
: Add instance pools command group
#11721
Merged
Changes from 10 commits
Commits
Show all changes
141 commits
Select commit
Hold shift + click to select a range
13c8465
Added CLI functions with tests
48e690b
Resolved comments, added vnet-name to create
3a3b6fd
Fixed no_Wait call in create command
83df496
Changed active/passive voice issues and added enum to license type
4685279
Deleted existing recording file
790d5af
Merge branch 'dev' into cli_for_ip
rohands 68e28be
Added test recording file
cc855e7
Merge branch 'cli_for_ip' of https://github.com/rohands/azure-cli int…
c09d4f7
upgrade azure-mgmt-version to 0.16.0
Juliehzl 7c68e19
preview enabled for flexibility
9c1c370
Added CLI functions with tests
65db7c5
Resolved comments, added vnet-name to create
c7f375c
Fixed no_Wait call in create command
6a406cf
Changed active/passive voice issues and added enum to license type
4a80b03
Deleted existing recording file
2056186
Added test recording file
01ce464
init colorama in az --version to fix issue on windows (#11609)
fengzhou-msft 65953fb
[Storage] Remove Preview for -enable-hierarchical-namespace (#11639)
Juliehzl aa26746
Adding new license type for az sql vm (#11570)
yareyes bd445ab
[Storage] Bump azure-mgmt-storage version to use api version 2019-06-…
Juliehzl 5776312
ACR: deprecate os parameter (#11699)
shahzzzam 04d059d
[App Service] On webapp up set httpsonly true (#11539)
ammanifold 88be5f3
Fix typo in `KeyVaultMgmtScenarioTest` (#11722)
bim-msft 3688593
[General][Minor]Update Code Onwers (#11705)
mmyyrroonn 0294081
[Compute][Feature]az vm create: Support default data source settings …
mmyyrroonn 22a4001
[RecoveryServices Backup]Added disk exclusion Feature for IAASVM (#11…
sambitratha b21ebc2
fix typo in authoring_commands.md (#11730)
dc0ef32
update codeowner for resource (#11739)
Juliehzl 575013f
Fix the test of test_managedapp (#11727)
zhoxing-ms 5cd1f31
Fix the test of test_resource_policyset (#11723)
zhoxing-ms f6450b4
add libc6-compat in dockerfile (#11742)
fengzhou-msft 3a0dcf3
Update Cosmos DB python package (#11551)
shurd ff5ea20
[AppConfig] Adding new command 'set-keyvault' to kv subgroup (#11571)
avanigupta ef997f3
[AppConfig] Support Import/Export of features in yaml files (#11637)
avanigupta 4afb97a
[App Service] Add az webapp up flag for HTML static sites (#11719)
ammanifold 70e3ee9
[Compute] Feature issue #11203 Add new reapply command action for az …
arrownj 08e8f50
Added new parameters for account blob service properties to manage de…
zhoxing-ms 384707c
[misc] Add 'az version' (#11680)
jiasli cd195b5
Removing ReceiveDisabled status from EventHub and ServiceBus CLI doc …
basilhariri 4ebc0e0
[RBAC] Return exit code 3 if app or sp doesn't exist (#11725)
jiasli 00e8a48
update azure-cli version to 2.0.79 (#11762)
33d1f1c
add codeonwer for sql (#11777)
Juliehzl 1780e56
fix: az monitor metric alert create. Fix test failure. (#11776)
mmyyrroonn 434b450
updating BYOS commands to mark them as preview (#11755)
btardif 34b5bd0
Fix: tox can use ADO default Python 2.7 (#11802)
21a528d
python3.8 migration for homebrew (#11803)
fengzhou-msft 2bb3f47
[Storage] Support Table and Queue Encryption Service for storage acco…
Juliehzl b6e84a7
update CODEOWNERS (#11814)
Juliehzl 8c19ded
[Webapp] Adding E2E tests for az webapp up (#11774)
panchagnula 0f66071
[Doc] Try new features before release (#11784)
qwordy 6f5c74a
Chore: move CredScansuppressions.json to codeownder guarded folder (#…
0e9029d
[Compute] disk update: Add --disk-encryption-set and --encryption-typ…
qwordy b40a37e
[AKS] Update example help text to reflect change of defaults to Stand…
seanmck 654493e
Upgrade to Azure CLI 2.0.80 (#11826)
8777cd9
Add table output to backup commands (#11764)
sambitratha fef4d3a
Fix typo in History.rst (#11835)
ankitkumarr 305bf6d
[IoT Central] Add new sku name to iotcentral (#11810)
poanchen 9aee89a
[Core] Expose example_provider hook to include AI examples (#10987)
mirdaki d1238c0
fix: enabling monitoring for OpenShift managed cluster (#11778)
ganga1980 8cc55f1
Update fedora install troubleshooting doc (#11827)
fengzhou-msft dbcddb3
Fix: 'az redis update' operation for RDB/AOF enabled redis caches (#1…
MECHANDR 93bb3c3
Chore: disable CredScan job temporary for ADO security concern (#11844)
bb38e1c
[KeyVault] Add example for container sas-definition (#11846)
bim-msft 823938c
refact: update global --tags help content (#11847)
cc100eb
[Rest] Make --method -m optional (#11843)
jiasli 3eb5247
[Core] Support filename and environment variable completion (#11817)
jiasli bebe8b4
Chore: add code owner for command module appconfig (#11860)
f0ee2b0
Update code owner (#11875)
jiasli 7aa766c
[AppService] Fix #10476: Support appservice in 2019-03-01-hybrid prof…
Juliehzl bfac05f
Bump argcomplete version (#11866)
jiasli 5e0672f
[AppConfig] Support import/export of keyvault ref from appservice (#1…
avanigupta c3f6073
fix test_get_clouds_concurrent on python 3.8 windows (#11864)
jiasli 51c85a6
Documented sql db restore time format (#11838)
jaredmoo f28c3d2
[ARM] Fix az group export command does not support --query and --outp…
zhoxing-ms bf15a70
acr: add notary version compatibility checks (#11792)
Isaac-Lee-msft 1bc435a
{Storage} Update help for pattern (#11782)
Juliehzl 7f2f54d
[ARM] Fix the exit code of az group deployment validate when the veri…
zhoxing-ms f89ab44
[Storage]: Add a new command group `az storage share-rm` to use the M…
jsntcy 1033ab5
delete cli_license_terms.md (#11902)
jiasli bbc5a01
[Network]az network dns record-set add/remove: show warning when a n…
mmyyrroonn 2716d99
[ACR]: Add a prompt for command "az acr delete" to avoid accidental o…
Wwwsylvia deb8cfe
[Storage] Fix errors for storage blob update (#11907)
Juliehzl b1cbfe7
[AppConfig] Support import/export of all labels between 2 config stor…
avanigupta 2fca734
{Storage} Refine help for storage data plane commands and log warning…
Juliehzl cb65e9f
[AppConfig] Validate key and feature names before setting and importi…
avanigupta c46779c
[IoT] Deprecated command group 'iot hub job' (#11794)
anusapan abe3550
[AppConfig] Expose sku and managed identity for GA (#11795)
shenmuxiaosen cba42c1
[Storage] Upgrade Azcopy for storage copy/remove and blob sync (#11874)
Juliehzl 8fff40d
remove pydocumentdb (#11918)
fengzhou-msft 2c5c6ce
[ACR] Add new commands `az acr taskrun show/list/delete` for show lis…
huanwu e88a711
[Azure Red Hat OpenShift] Add `monitor` subgroup to manage Log Analyt…
ca1e645
[CDN] Add support for Rules engine feature (#11854)
hytao 7e3db70
[Security] Add new commands `az atp show` and `az atp update` to view…
libronsh a87fd96
{Network} az network application-gateway create: set https when key_v…
sputnik13 e022aa8
[SQL] BREAKING CHANGE: az sql db create: Remove WideWorldImportersStd…
jaredmoo fd467fd
[RBAC] BREAKING CHANGE: Fix #11883: `az role assignment create`: empt…
arrownj ded0433
{Packaging} Generate Homebrew formula by updating existing one by def…
fengzhou-msft dd1fec7
[ARM] Fix az resource list cannot use tag when there is a default loc…
zhoxing-ms 50188cc
[ARM] Fix az resource tag crashed when the parameter --ids passed in …
zhoxing-ms f20761e
[Key Vault] Add a new command `az keyvault key download` for download…
bim-msft 3aa8f2e
[Core][Profile] Add `tenant` parameter to `get_raw_token` (#11798)
jiasli 9c007cf
Improve the error message of the conflict between tag and other filte…
zhoxing-ms 4b6cb76
[Deployment Manager] Add `az deploymentmanager rollout/service/servic…
deveshguha 4ba74ff
[SQL] Add new commands `sql db classification show/list/update/delete…
ranisha2 65d31f6
Improve intercluster isolation (#11848)
cristiklein 7afca98
{Document} Modify PR template and document (#11913)
fengzhou-msft b971dca
Service Fabric Application commands (#10666)
a-santamaria 1779468
[ACS] add support for the aks 1.16/1.17 in "aks browse" command. (#11…
andyliuliming 426e29b
[Policy] Update policyinsights to 0.4.0 package (#11820)
pilor d794c75
[SQL] BREAKING CHANGE: `az sql dw create`: Deprecate `--zone-redundan…
jaredmoo 9ba02c4
[BotService] Refactor tests, make bot create idempotent (#11935)
stevengum 832de7f
[SQL] az sql db audit-policy: Fix for empty audit actions and groups …
ranisha2 9cdfd3c
{CI} Add PR title check (#11982)
fengzhou-msft 9da3518
[ACS] aks create/update: add --load-balancer-outbound-ports and --loa…
gtracer d2b36f3
[functionapp]: add ability to create java apps on Linux (#11811)
ankitkumarr dc6a7d3
update azure-cli version to 2.0.81 (#12005)
11184ac
Re-run functionapp tests (#12006)
jiasli 1808216
{CI} Enable CredScan task of Microsoft Security Code Analysis (Previe…
fb13977
{Monitor} az monitor diagnostic-settings category list: fix bug(#12020)
mmyyrroonn c1dc7a1
[AMS] az ams is GA now (#12004)
giakas 3109764
{Compute} Fix typo in error message when attaching unmanaged disk (#1…
gaoruifeng f927a70
[KeyVault] az keyvault key create: add a new value `import` for param…
bim-msft b726d06
{Network} Clean up zone*.txt_export.txt files after tests (#12032)
jiasli bcc85ef
[ACS] aks create/update: add validation for --vnet-subnet-id argument…
Czhang0727 8cf30f0
[AppConfig] Revise help message to exclude unsupported key/label filt…
shenmuxiaosen 2884873
{Resource} Bump to azure-mgmt-resource~=8.0.1 (#12050)
jiasli 53965a7
[Compute] vm/vmss/availability-set update: add --ppg to allowing upda…
qwordy 077c9b6
[Compute] vmss create: add --data-disk-iops and --data-disk-mbps (#11…
qwordy d1aae73
upgrade azure-mgmt-version to 0.16.0
Juliehzl df0c460
preview enabled for flexibility
242ce28
Added SQL changes to the changeList
c226f5d
Merging with upstream/dev
7d084cd
Merge remote-tracking branch 'upstream/dev' into cli_for_ip
1f684d2
Added --no-wait and list command to the tests
2caed06
Fixing lint issues
df33780
Resolving PR comments
ba47cd7
Added transformer to create command
a807ec2
Merge remote-tracking branch 'upstream/dev' into cli_for_ip
a74dfec
Updated network by syncing with upstream/dev
7143623
Removed description from HISTORY
a140481
Adding new test for list
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -14,6 +14,7 @@ | |
ElasticPoolPerDatabaseSettings, | ||
ImportExtensionRequest, | ||
ExportRequest, | ||
InstancePool, | ||
ManagedDatabase, | ||
ManagedInstance, | ||
ManagedInstanceAdministrator, | ||
|
@@ -1020,6 +1021,69 @@ def _configure_security_policy_storage_params(arg_ctx): | |
c.argument('allow_data_loss', | ||
arg_type=allow_data_loss_param_type) | ||
|
||
############################################### | ||
# sql instance pool # | ||
############################################### | ||
|
||
with self.argument_context('sql instance-pool') as c: | ||
c.argument('instance_pool_name', | ||
options_list=['--name', '-n'], | ||
help="Instance Pool Name") | ||
rohands marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
c.argument( | ||
'tier', | ||
arg_type=tier_param_type, | ||
rohands marked this conversation as resolved.
Show resolved
Hide resolved
|
||
required=True, | ||
help='The edition component of the sku. Allowed value: GeneralPurpose.') | ||
|
||
c.argument('family', | ||
arg_type=family_param_type, | ||
required=True, | ||
help='The compute generation component of the sku. ' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The same as above. Please use get_enum_type() |
||
'Allowed value: Gen5') | ||
|
||
c.argument('license_type', | ||
arg_type=get_enum_type(DatabaseLicenseType), | ||
help='The license type to apply for this instance pool.') | ||
|
||
with self.argument_context('sql instance-pool create') as c: | ||
# Create args that will be used to build up the InstancePool object | ||
create_args_for_complex_type( | ||
rohands marked this conversation as resolved.
Show resolved
Hide resolved
|
||
c, 'parameters', InstancePool, [ | ||
'location', | ||
'license_type', | ||
rohands marked this conversation as resolved.
Show resolved
Hide resolved
|
||
'subnet_id', | ||
'vcores', | ||
'tags' | ||
]) | ||
|
||
c.argument('vcores', | ||
required=True, | ||
arg_type=capacity_param_type, | ||
help='Capacity of the instance pool in vcores.') | ||
|
||
c.argument( | ||
'subnet_id', | ||
rohands marked this conversation as resolved.
Show resolved
Hide resolved
rohands marked this conversation as resolved.
Show resolved
Hide resolved
|
||
options_list=['--subnet'], | ||
required=True, | ||
help='Name or ID of the subnet that allows access to an Instance Pool. ' | ||
'If subnet name is provided, --vnet-name must be provided.') | ||
|
||
# Create args that will be used to build up the Instance Pool's Sku object | ||
create_args_for_complex_type( | ||
c, 'sku', Sku, [ | ||
'family', | ||
'name', | ||
'tier', | ||
]) | ||
|
||
c.ignore('name') # Hide sku name | ||
|
||
c.extra('vnet_name', | ||
options_list=['--vnet-name'], | ||
help='The virtual network name', | ||
validator=validate_subnet) | ||
|
||
############################################### | ||
# sql server # | ||
############################################### | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This part should be removed and add it in description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need to change history.rst because we will generate it automatically.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh okay, where is the description section?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#11721 (comment) or title