Skip to content

Commit c4e8680

Browse files
author
Maddie Clayton
authored
Merge pull request Azure#7531 from dalibormesaric/preview
Fix -Tags for New-AzureRmSqlDatabaseCopy Azure#6049
2 parents d18367d + 31e2168 commit c4e8680

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/ResourceManager/Sql/Commands.Sql/ChangeLog.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121

2222
## Version 4.11.5
2323
* Fixed issue where some backup cmdlets would not recognize the current azure subscription
24+
* Fixed issue where Tags were not being added correctly in New-AzureRmSqlDatabaseCopy
2425

2526
## Version 4.11.3
2627
* Fixed issue with default resource groups not being set.

src/ResourceManager/Sql/Commands.Sql/Replication/Services/AzureSqlDatabaseReplicationAdapter.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,9 +137,8 @@ internal AzureSqlDatabaseCopyModel CopyDatabaseWithNewSdk(string copyResourceGro
137137
model.ElasticPoolName);
138138

139139
// Create copy of the database
140-
var resp = ReplicationCommunicator.CreateCopy(copyResourceGroup, copyServerName, model.CopyDatabaseName, new Management.Sql.Models.Database
140+
var resp = ReplicationCommunicator.CreateCopy(copyResourceGroup, copyServerName, model.CopyDatabaseName, new Management.Sql.Models.Database(model.CopyLocation, tags: model.Tags)
141141
{
142-
Location = model.CopyLocation,
143142
CreateMode = Management.Sql.Models.CreateMode.Copy,
144143
SourceDatabaseId = string.Format(AzureReplicationLinkModel.SourceIdTemplate, _subscription.Id.ToString(),
145144
model.ResourceGroupName, model.ServerName, model.DatabaseName),

0 commit comments

Comments
 (0)