Skip to content

Commit f50f1d5

Browse files
authored
[Service Fabric] Fix #5180: az sf cluster create: Change behavior to read cluster_name from parameters file if provided (#31816)
1 parent ed0af96 commit f50f1d5

1 file changed

Lines changed: 15 additions & 13 deletions

File tree

  • src/azure-cli/azure/cli/command_modules/servicefabric

src/azure-cli/azure/cli/command_modules/servicefabric/custom.py

Lines changed: 15 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@
7575
SERVICE_FABRIC_WINDOWS_NODE_EXT_NAME = "servicefabricnode"
7676
SERVICE_FABRIC_LINUX_NODE_EXT_NAME = "servicefabriclinuxnode"
7777

78+
CLUSTER_NAME_VALUE = "clusterName"
7879
SOURCE_VAULT_VALUE = "sourceVaultValue"
7980
CERTIFICATE_THUMBPRINT = "certificateThumbprint"
8081
CERTIFICATE_URL_VALUE = "certificateUrlValue"
@@ -145,7 +146,7 @@ def new_cluster(cmd,
145146
'when \'--secret-identifier\' is specified')
146147
if parameter_file or template_file:
147148
if parameter_file is None or template_file is None:
148-
raise CLIError('If using customize template to deploy,both \'--parameter-file\' and \'--template-file\' can not be None ' + '\n For example:\n az sf cluster create --resource-group myRg --location westus --certificate-subject-name test.com --parameter-file c:\\parameter.json --template-file c:\\template.json' +
149+
raise CLIError('If using customize template to deploy, neither \'--parameter-file\' and \'--template-file\' can be None ' + '\n For example:\n az sf cluster create --resource-group myRg --location westus --certificate-subject-name test.com --parameter-file c:\\parameter.json --template-file c:\\template.json' +
149150
'\n az sf cluster create --resource-group myRg --location westus --parameter-file c:\\parameter.json --template-file c:\\template.json --certificate_file c:\\test.pfx' + '\n az sf cluster create --resource-group myRg --location westus --certificate-subject-name test.com --parameter-file c:\\parameter.json --template-file c:\\template.json --certificate-output-folder c:\\certoutput')
150151
if cluster_size or vm_sku or vm_user_name:
151152
raise CLIError('\'cluster_size\',\'vm_sku\',\'vm_os\',\'vm_user_name\' can not be specified when using customize template deployment')
@@ -230,18 +231,19 @@ def new_cluster(cmd,
230231
os_type=vm_os,
231232
linux=linux)
232233
else:
233-
parameters, output_file = _set_parameters_for_customize_template(cmd,
234-
cli_ctx,
235-
resource_group_name,
236-
certificate_file,
237-
certificate_password,
238-
vault_name,
239-
vault_resource_group_name,
240-
certificate_output_folder,
241-
certificate_subject_name,
242-
secret_identifier,
243-
parameter_file)
244-
234+
parameters, output_file = _set_parameters_for_customize_template(cmd=cmd,
235+
cli_ctx=cli_ctx,
236+
resource_group_name=resource_group_name,
237+
certificate_file=certificate_file,
238+
certificate_password=certificate_password,
239+
vault_name=vault_name,
240+
vault_resource_group_name=vault_resource_group_name,
241+
certificate_output_folder=certificate_output_folder,
242+
certificate_subject_name=certificate_subject_name,
243+
secret_identifier=secret_identifier,
244+
parameter_file=parameter_file)
245+
246+
cluster_name = parameters[CLUSTER_NAME_VALUE]['value']
245247
vault_id = parameters[SOURCE_VAULT_VALUE]['value']
246248
certificate_uri = parameters[CERTIFICATE_URL_VALUE]['value']
247249
cert_thumbprint = parameters[CERTIFICATE_THUMBPRINT]['value']

0 commit comments

Comments
 (0)