Skip to content
This repository was archived by the owner on Feb 2, 2022. It is now read-only.

Commit e7bc863

Browse files
stishkinstas
andauthored
Fix deployment failure caused by updated az-cli (#135)
* adjust deployment warning handling to newer az cli * Fix deployment failure caused by updated az-cli - AZ CLI prints "WARNING:" in front of the warning message now to std-out Co-authored-by: stas <statis@microsoft.com>
1 parent 52e7ba4 commit e7bc863

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

cli/raft_sdk/raft_deploy.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -278,13 +278,15 @@ def init_app_insights(self):
278278
app_insights_installed = True
279279

280280
if not app_insights_installed:
281-
p = ("The installed extension"
282-
" 'application-insights' is in preview.")
281+
p = [("The installed extension"
282+
" 'application-insights' is in preview."),
283+
("WARNING: The installed extension"
284+
" 'application-insights' is in preview.")]
283285
try:
284286
print('Installing application insights extension')
285287
az('extension add --name application-insights')
286288
except RaftAzCliException as ex:
287-
if ex.error_message.strip() == p:
289+
if ex.error_message.strip() in p:
288290
pass
289291
else:
290292
raise ex
@@ -929,7 +931,7 @@ def update_deployment_context(self):
929931
json.dump(defaults, d, indent=4)
930932

931933
def test_az_version(self):
932-
supported_versions = ['2.15.0', '2.16.0', '2.17.1']
934+
supported_versions = ['2.19.0', '2.19.1']
933935
# az sometimes reports the version with an asterisk.
934936
# Perhaps when a new version of the CLI is available.
935937
tested_az_cli_versions = (

0 commit comments

Comments
 (0)