Skip to content

Support multiple deployments on same cf org/space#868

Open
bonzofenix wants to merge 3 commits intomainfrom
multiple-mta-deployments
Open

Support multiple deployments on same cf org/space#868
bonzofenix wants to merge 3 commits intomainfrom
multiple-mta-deployments

Conversation

@bonzofenix
Copy link
Contributor

No description provided.

@@ -13,7 +13,7 @@ function deploy() {
bosh_login "${BBL_STATE_PATH}"
cf_login
cf_target "${autoscaler_org}" "${autoscaler_space}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [shellcheck] reported by reviewdog 🐶
autoscaler_org is referenced but not assigned. SC2154

@@ -13,7 +13,7 @@ function deploy() {
bosh_login "${BBL_STATE_PATH}"
cf_login
cf_target "${autoscaler_org}" "${autoscaler_space}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [shellcheck] reported by reviewdog 🐶
autoscaler_space is referenced but not assigned. SC2154

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [shellcheck] reported by reviewdog 🐶
Not following: ./common.sh: openBinaryFile: does not exist (No such file or directory) SC1091

source "${script_dir}/common.sh"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 [shellcheck] reported by reviewdog 🐶
Not following: ./vars.source.sh: openBinaryFile: does not exist (No such file or directory) SC1091

source "${script_dir}/vars.source.sh"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [shellcheck] reported by reviewdog 🐶
deployment_name is referenced but not assigned. SC2154

log "Deploying autoscaler apps for bosh deployment '${deployment_name}' "

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant