Skip to content

Commit f6ba777

Browse files
committed
Fixes #107738048
1 parent 336ce8b commit f6ba777

File tree

4 files changed

+23
-7
lines changed

4 files changed

+23
-7
lines changed

src/ResourceManager/Common/Commands.ResourceManager.Common/Properties/Resources.Designer.cs

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/ResourceManager/Common/Commands.ResourceManager.Common/Properties/Resources.resx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,10 +167,10 @@ Select Y to enable data collection [Y/N]:</value>
167167
<value>Succeeded to register resource provider '{0}'</value>
168168
</data>
169169
<data name="SubscriptionIdNotFound" xml:space="preserve">
170-
<value>The provided account {0} does not have access to subscription ID "{1}". Please try logging in with different credentials not a different subscription ID.</value>
170+
<value>The provided account {0} does not have access to subscription ID "{1}". Please try logging in with different credentials or a different subscription ID.</value>
171171
</data>
172172
<data name="SubscriptionNameNotFound" xml:space="preserve">
173-
<value>The provided account {0} does not have access to subscription name "{1}". Please try logging in with different credentials not a different subscription name.</value>
173+
<value>The provided account {0} does not have access to subscription name "{1}". Please try logging in with different credentials or a different subscription name.</value>
174174
</data>
175175
<data name="TenantNotFound" xml:space="preserve">
176176
<value>Tenant '{0}' was not found. Please verify that your account has access to this tenant.</value>

src/ResourceManager/Profile/Commands.Profile/Account/AddAzureRmAccount.cs

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,46 +38,62 @@ public class AddAzureRMAccountCommand : AzureRMCmdlet , IModuleAssemblyInitializ
3838
private const string ServicePrincipalParameterSet = "ServicePrincipal";
3939
private const string ServicePrincipalCertificateParameterSet = "ServicePrincipalCertificate";
4040
private const string AccessTokenParameterSet = "AccessToken";
41+
private const string SubscriptionNameParameterSet = "SubscriptionName";
42+
private const string SubscriptionIdParameterSet = "SubscriptionId";
4143

4244
[Parameter(Mandatory = false, HelpMessage = "Environment containing the account to log into")]
4345
[ValidateNotNullOrEmpty]
4446
public AzureEnvironment Environment { get; set; }
4547

4648
[Parameter(ParameterSetName = UserParameterSet, Mandatory = false, HelpMessage = "Optional credential")]
4749
[Parameter(ParameterSetName = ServicePrincipalParameterSet, Mandatory = true, HelpMessage = "Credential")]
50+
[Parameter(ParameterSetName = SubscriptionIdParameterSet, Mandatory = false, HelpMessage = "Optional credential")]
51+
[Parameter(ParameterSetName = SubscriptionNameParameterSet, Mandatory = false, HelpMessage = "Optional credential")]
4852
public PSCredential Credential { get; set; }
4953

5054
[Parameter(ParameterSetName = ServicePrincipalCertificateParameterSet, Mandatory = true, HelpMessage = "Certificate Hash (Thumbprint)")]
55+
[Parameter(ParameterSetName = SubscriptionIdParameterSet, Mandatory = false, HelpMessage = "Certificate Hash (Thumbprint)")]
56+
[Parameter(ParameterSetName = SubscriptionNameParameterSet, Mandatory = false, HelpMessage = "Certificate Hash (Thumbprint)")]
5157
public string CertificateThumbprint { get; set; }
5258

5359
[Parameter(ParameterSetName = ServicePrincipalCertificateParameterSet, Mandatory = true, HelpMessage = "SPN")]
60+
[Parameter(ParameterSetName = SubscriptionIdParameterSet, Mandatory = false, HelpMessage = "SPN")]
61+
[Parameter(ParameterSetName = SubscriptionNameParameterSet, Mandatory = false, HelpMessage = "SPN")]
5462
public string ApplicationId { get; set; }
5563

5664
[Parameter(ParameterSetName = ServicePrincipalParameterSet, Mandatory = true)]
5765
[Parameter(ParameterSetName = ServicePrincipalCertificateParameterSet, Mandatory = true)]
66+
[Parameter(ParameterSetName = SubscriptionIdParameterSet, Mandatory = false)]
67+
[Parameter(ParameterSetName = SubscriptionNameParameterSet, Mandatory = false)]
5868
public SwitchParameter ServicePrincipal { get; set; }
5969

6070
[Parameter(ParameterSetName = UserParameterSet, Mandatory = false, HelpMessage = "Optional tenant name or ID")]
6171
[Parameter(ParameterSetName = ServicePrincipalParameterSet, Mandatory = true, HelpMessage = "TenantId name or ID")]
6272
[Parameter(ParameterSetName = AccessTokenParameterSet, Mandatory = false, HelpMessage = "TenantId name or ID")]
6373
[Parameter(ParameterSetName = ServicePrincipalCertificateParameterSet, Mandatory = true, HelpMessage = "TenantId name or ID")]
74+
[Parameter(ParameterSetName = SubscriptionIdParameterSet, Mandatory = false, HelpMessage = "TenantId name or ID")]
75+
[Parameter(ParameterSetName = SubscriptionNameParameterSet, Mandatory = false, HelpMessage = "TenantId name or ID")]
6476
[Alias("Domain")]
6577
[ValidateNotNullOrEmpty]
6678
public string TenantId { get; set; }
6779

6880
[Parameter(ParameterSetName = AccessTokenParameterSet, Mandatory = true, HelpMessage = "AccessToken")]
81+
[Parameter(ParameterSetName = SubscriptionIdParameterSet, Mandatory = false, HelpMessage = "AccessToken")]
82+
[Parameter(ParameterSetName = SubscriptionNameParameterSet, Mandatory = false, HelpMessage = "AccessToken")]
6983
[ValidateNotNullOrEmpty]
7084
public string AccessToken { get; set; }
7185

7286
[Parameter(ParameterSetName = AccessTokenParameterSet, Mandatory = true, HelpMessage = "Account Id for access token")]
87+
[Parameter(ParameterSetName = SubscriptionIdParameterSet, Mandatory = false, HelpMessage = "Account Id for access token")]
88+
[Parameter(ParameterSetName = SubscriptionNameParameterSet, Mandatory = false, HelpMessage = "Account Id for access token")]
7389
[ValidateNotNullOrEmpty]
7490
public string AccountId { get; set; }
7591

76-
[Parameter(Mandatory = false, HelpMessage = "Subscription Id")]
92+
[Parameter(ParameterSetName = SubscriptionIdParameterSet, Mandatory = false, HelpMessage = "Subscription", ValueFromPipelineByPropertyName = true)]
7793
[ValidateNotNullOrEmpty]
7894
public string SubscriptionId { get; set; }
7995

80-
[Parameter(Mandatory = false, HelpMessage = "Subscription name")]
96+
[Parameter(ParameterSetName = SubscriptionNameParameterSet, Mandatory = false, HelpMessage = "Subscription Name", ValueFromPipelineByPropertyName = true)]
8197
[ValidateNotNullOrEmpty]
8298
public string SubscriptionName { get; set; }
8399

src/ResourceManager/Profile/Commands.Profile/Models/RMProfileClient.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ public AzureRMProfile Login(
9595
}
9696
else if (subscriptionName != null)
9797
{
98-
throw new PSInvalidOperationException(String.Format(Properties.Resources.SubscriptionNameNotFound, account.Id, subscriptionId));
98+
throw new PSInvalidOperationException(String.Format(Properties.Resources.SubscriptionNameNotFound, account.Id, subscriptionName));
9999
}
100100

101101
_profile.Context = new AzureContext(account, environment, newTenant);

0 commit comments

Comments
 (0)