Skip to content

Commit 6f8dcbf

Browse files
committed
Merge pull request Azure#163 from hyonholee/crp2
Change parameter name 'ReferenceUri' to 'SourceVaultId' and Remove 'config' from some cmdlets name.
2 parents fae8ece + e9d54c5 commit 6f8dcbf

File tree

6 files changed

+19
-19
lines changed

6 files changed

+19
-19
lines changed

src/ResourceManager/Compute/Commands.Compute.Test/ScenarioTests/VirtualMachineProfileTests.ps1

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -87,19 +87,19 @@ function Test-VirtualMachineProfile
8787
$referenceUri = "/subscriptions/" + $subid + "/resourceGroups/RgTest1/providers/Microsoft.KeyVault/vaults/TestVault123";
8888
$certStore = "My";
8989
$certUrl = "https://testvault123.vault.azure.net/secrets/Test1/514ceb769c984379a7e0230bdd703272";
90-
$p = Add-AzureVMSecretConfig -VM $p -ReferenceUri $referenceUri -CertificateStore $certStore -CertificateUrl $certUrl;
90+
$p = Add-AzureVMSecret -VM $p -SourceVaultId $referenceUri -CertificateStore $certStore -CertificateUrl $certUrl;
9191

9292
$referenceUri2 = "/subscriptions/" + $subid + "/resourceGroups/RgTest1/providers/Microsoft.KeyVault/vaults/TestVault456";
93-
$p = Add-AzureVMSecretConfig -VM $p -ReferenceUri $referenceUri2 -CertificateStore $certStore -CertificateUrl $certUrl;
93+
$p = Add-AzureVMSecret -VM $p -SourceVaultId $referenceUri2 -CertificateStore $certStore -CertificateUrl $certUrl;
9494

9595
$certStore2 = "My2";
9696
$certUrl2 = "https://testvault123.vault.azure.net/secrets/Test1/514ceb769c984379a7e0230bddaaaaaa";
97-
$p = Add-AzureVMSecretConfig -VM $p -ReferenceUri $referenceUri -CertificateStore $certStore2 -CertificateUrl $certUrl2;
97+
$p = Add-AzureVMSecret -VM $p -SourceVaultId $referenceUri -CertificateStore $certStore2 -CertificateUrl $certUrl2;
9898

9999
$aucSetting = "AutoLogon";
100100
$aucContent = "<UserAccounts><AdministratorPassword><Value>" + $password + "</Value><PlainText>true</PlainText></AdministratorPassword></UserAccounts>";
101-
$p = Add-AzureVMAdditionalUnattendContentConfig -VM $p -Content $aucContent -SettingName $aucSetting;
102-
$p = Add-AzureVMAdditionalUnattendContentConfig -VM $p -Content $aucContent -SettingName $aucSetting;
101+
$p = Add-AzureVMAdditionalUnattendContent -VM $p -Content $aucContent -SettingName $aucSetting;
102+
$p = Add-AzureVMAdditionalUnattendContent -VM $p -Content $aucContent -SettingName $aucSetting;
103103

104104
Assert-AreEqual $p.OSProfile.AdminUsername $user;
105105
Assert-AreEqual $p.OSProfile.ComputerName $computerName;
@@ -145,9 +145,9 @@ function Test-VirtualMachineProfile
145145

146146
$sshPath = "/home/pstestuser/.ssh/authorized_keys";
147147
$sshPublicKey = "MIIDszCCApugAwIBAgIJALBV9YJCF/tAMA0GCSqGSIb3DQEBBQUAMEUxCzAJBgNV";
148-
$p = Add-AzureVMSshPublicKeyConfig -VM $p -KeyData $sshPublicKey -Path $sshPath;
149-
$p = Add-AzureVMSshPublicKeyConfig -VM $p -KeyData $sshPublicKey -Path $sshPath;
150-
$p = Add-AzureVMSecretConfig -VM $p -ReferenceUri $referenceUri -CertificateStore $certStore -CertificateUrl $certUrl;
148+
$p = Add-AzureVMSshPublicKey -VM $p -KeyData $sshPublicKey -Path $sshPath;
149+
$p = Add-AzureVMSshPublicKey -VM $p -KeyData $sshPublicKey -Path $sshPath;
150+
$p = Add-AzureVMSecret -VM $p -SourceVaultId $referenceUri -CertificateStore $certStore -CertificateUrl $certUrl;
151151

152152
Assert-AreEqual $p.OSProfile.AdminUsername $user;
153153
Assert-AreEqual $p.OSProfile.ComputerName $computerName;

src/ResourceManager/Compute/Commands.Compute/Commands.Compute.csproj

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -199,9 +199,9 @@
199199
<Compile Include="VirtualMachineSizes\VirtualMachineSizeBaseCmdlet.cs" />
200200
<Compile Include="VirtualMachine\Action\SaveAzureVMImageCommand.cs" />
201201
<Compile Include="VirtualMachine\Action\SetAzureVMCommand.cs" />
202-
<Compile Include="VirtualMachine\Config\AddAzureVMAdditionalUnattendContentConfigCommand.cs" />
203-
<Compile Include="VirtualMachine\Config\AddAzureVMSshPublicKeyConfigCommand.cs" />
204-
<Compile Include="VirtualMachine\Config\AddAzureVMSecretConfigCommand.cs" />
202+
<Compile Include="VirtualMachine\Config\AddAzureVMAdditionalUnattendContentCommand.cs" />
203+
<Compile Include="VirtualMachine\Config\AddAzureVMSshPublicKeyCommand.cs" />
204+
<Compile Include="VirtualMachine\Config\AddAzureVMSecretCommand.cs" />
205205
<Compile Include="VirtualMachine\Config\RemoveAzureVMNetworkInterfaceCommand.cs" />
206206
<Compile Include="VirtualMachine\Config\RemoveAzureVMDataDiskCommand.cs" />
207207
<Compile Include="VirtualMachine\Config\SetAzureVMSourceImage.cs" />

src/ResourceManager/Compute/Commands.Compute/Common/ConstantStringTypes.cs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -89,10 +89,9 @@ public static class ProfileNouns
8989

9090
public const string VirtualMachineUsage = "AzureVMUsage";
9191

92+
public const string SshPublicKey = "AzureVMSshPublicKey";
93+
public const string AdditionalUnattendContent = "AzureVMAdditionalUnattendContent";
94+
public const string VaultSecretGroup = "AzureVMSecret";
9295
public const string RemoteDesktopFile = "AzureRemoteDesktopFile";
93-
public const string SshPublicKey = "AzureVMSshPublicKeyConfig";
94-
public const string AdditionalUnattendContent = "AzureVMAdditionalUnattendContentConfig";
95-
public const string VaultCertificate = "AzureVMVaultCertificateConfig";
96-
public const string VaultSecretGroup = "AzureVMSecretConfig";
9796
}
9897
}
Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,13 @@ public class NewAzureVaultSecretGroupCommand : AzurePSCmdlet
4242
[ValidateNotNullOrEmpty]
4343
public PSVirtualMachine VM { get; set; }
4444

45+
[Alias("Id")]
4546
[Parameter(
4647
Position = 1,
4748
ValueFromPipelineByPropertyName = true,
48-
HelpMessage = "the relative URL")]
49+
HelpMessage = "The ID for Source Vault")]
4950
[ValidateNotNullOrEmpty]
50-
public string ReferenceUri { get; set; }
51+
public string SourceVaultId { get; set; }
5152

5253
[Parameter(
5354
Position = 2,
@@ -82,7 +83,7 @@ public override void ExecuteCmdlet()
8283
if (i == this.VM.OSProfile.Secrets.Count)
8384
{
8485
var sourceVault = new SourceVaultReference {
85-
ReferenceUri = this.ReferenceUri
86+
ReferenceUri = this.SourceVaultId
8687
};
8788

8889
var vaultCertificates = new List<VaultCertificate>{
@@ -103,7 +104,7 @@ public override void ExecuteCmdlet()
103104
break;
104105
}
105106

106-
if (this.VM.OSProfile.Secrets[i].SourceVault != null && this.VM.OSProfile.Secrets[i].SourceVault.ReferenceUri.Equals(this.ReferenceUri))
107+
if (this.VM.OSProfile.Secrets[i].SourceVault != null && this.VM.OSProfile.Secrets[i].SourceVault.ReferenceUri.Equals(this.SourceVaultId))
107108
{
108109
if (this.VM.OSProfile.Secrets[i].VaultCertificates == null)
109110
{

0 commit comments

Comments
 (0)