Skip to content

Commit

Permalink
fix(bbl-up): Use mikefarah/yq flags
Browse files Browse the repository at this point in the history
It appears that we used to use `kislyuk/yq` but #167 switched us to
`mikefarah/yq` unintentionally.

It appears that `--output-format yaml` will work for `mikefarah/yq` the
same way that `--yaml-output` worked for `kislyuk/yq`.
  • Loading branch information
ctlong committed Oct 5, 2023
1 parent ef58fcb commit eba7af2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions bbl-up/task
Original file line number Diff line number Diff line change
Expand Up @@ -116,10 +116,10 @@ function main() {
ns_record_advisory="For DNS delegation to work, please ensure an NS record for ${LB_DOMAIN} is created in its parent DNS zone with the following nameservers:"
if [[ "$BBL_IAAS" == "aws" ]]; then
echo "$ns_record_advisory"
bbl outputs | yq '.env_dns_zone_name_servers[0:4]' --yaml-output | cut -d' ' -f2
bbl outputs | yq '.env_dns_zone_name_servers[0:4]' --output-format yaml | cut -d' ' -f2
elif [[ "$BBL_IAAS" == "gcp" ]]; then
echo "$ns_record_advisory"
bbl outputs | yq '.system_domain_dns_servers' --yaml-output | cut -d' ' -f2
bbl outputs | yq '.system_domain_dns_servers' --output-format yaml | cut -d' ' -f2
fi

if [[ "${DELETE_TERRAFORM_PLUGINS}" == "true" ]]; then
Expand Down

0 comments on commit eba7af2

Please sign in to comment.