Skip to content

Commit

Permalink
Fix initial value type of allow-empty-targets
Browse files Browse the repository at this point in the history
**Why?**

ADF fails to build in the bootstrap pipeline if no `allow-empty-targets` value
is configured in the `adfconfig.yml` file.

Root cause, the default value is a boolean False. While the Parameter Store
service expects values to be of type string.

**What?**

This change set fixes it by converting the default to string when written to
Parameter Store.
  • Loading branch information
sbkok committed Jan 17, 2024
1 parent f1fec85 commit d20bc10
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ def prepare_deployment_account(sts, deployment_account_id, config):
'/adf/deployment-maps/allow-empty-target',
config.config.get('deployment-maps', {}).get(
'allow-empty-target',
ADF_DEFAULT_DEPLOYMENT_MAPS_ALLOW_EMPTY_TARGET,
str(ADF_DEFAULT_DEPLOYMENT_MAPS_ALLOW_EMPTY_TARGET),
)
)
deployment_account_parameter_store.put_parameter(
Expand Down

0 comments on commit d20bc10

Please sign in to comment.