search

cloudfoundry-incubator / fissile

Turn a BOSH release into container images / files
Apache License 2.0
67 stars 36 forks source link

Invoke pre-start scripts via explicit `bash` #486

Closed andreas-kupries closed 5 years ago

andreas-kupries commented 5 years ago

Ref https://trello.com/c/Ljs7a8lx/986-2-make-sure-authorize-internal-ca-pre-start-script-always-runs-first

PS to https://github.com/cloudfoundry-incubator/fissile/pull/485 Invoke the scripts with an explicit bash, like the old code did.

It looks as if at least one pre-start script has active options when called directly.

This PR is for testing if that is indeed the case. The "proper" change would be to fix the pre-start script, given that it is the only one (so far). This change here is a bit easier to create

:construction: :warning: :construction: Do not merge

cfdreddbot commented 5 years ago

:white_check_mark: Hey andreas-kupries! The commit authors and yourself have already signed the CLA.

andreas-kupries commented 5 years ago

Closing, confirmed issue, fix in SCF itself, keeping fissile as-is