Azure / pcs-cli

A CLI for deploying Azure IoT PCS
MIT License
33 stars 37 forks source link

fix failing local deployment on run time due to environment variables #449

Closed sakimsft closed 5 years ago

sakimsft commented 5 years ago

Description and Motivation

Local deployment from CLI failed on run-time because of wrong environment variables. There were two issues.

  1. executing multiple setx commands joined by new line failed.
  2. PCS_DIAGNOSTICS_ENDPOINT_URL did not have value so that it was syntax error.

Fix:

  1. adding default value for diagnostics endpoint environment variable
  2. execute setx command individually and if command fails, catch an error and print error message to console.

Change type

Checklist:


This change is Reviewable