Skip to content

Commit

Permalink
{CosmosDB} | Getting location from the Resource Group Name (#18449)
Browse files Browse the repository at this point in the history
* {CosmosDB} | Getting location from the Resource Group Name

Fixes #17978.
Though we say location parameter is optional we still expect it to be provided https://docs.microsoft.com/en-us/powershell/module/az.cosmosdb/new-azcosmosdbaccount?view=azps-8.0.0. This change will fix that issue.

* {CosmosDB} | adding missing semicolon

adding missing semicolon

* {CosmosDB} | Location is not required and can be null

Fixes #17978

* {CosmosDB} | Update change log

Though we say location parameter is optional we still expect it to be provided https://docs.microsoft.com/en-us/powershell/module/az.cosmosdb/new-azcosmosdbaccount?view=azps-8.0.0. This change will fix that issue.
  • Loading branch information
RakeshMohanMSFT authored Jun 17, 2022
1 parent a193ffa commit d40078c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
3 changes: 3 additions & 0 deletions src/CosmosDB/CosmosDB/ChangeLog.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,9 @@
## Upcoming Release
* Fixed bug related to Update-AzCosmosDBSqlContainer command on containers with Client Encryption Policy.

## Version 1.9.0
* Fixed the optional Location paramater of New-AzCosmosDBAccount cmdlet.

## Version 1.8.0
* Introduced support for creating containers with Client Encryption Policy. The current supported version of Client Encryption Policy is 1.

Expand Down
6 changes: 0 additions & 6 deletions src/CosmosDB/CosmosDB/CosmosDBAccount/NewAzCosmosDBAccount.cs
Original file line number Diff line number Diff line change
Expand Up @@ -118,12 +118,6 @@ public override void ExecuteCmdlet()
}
}

if (string.IsNullOrEmpty(writeLocation))
{
WriteWarning("Cannot create Account without a Write Location.");
return;
}

Dictionary<string, string> tags = new Dictionary<string, string>();
if (Tag != null)
{
Expand Down

0 comments on commit d40078c

Please sign in to comment.