search

cloudfoundry-community-attic / bosh-bootstrap

From zero to a running micro BOSH in one command line
MIT License
63 stars 46 forks source link

add support for ignore_server_availability_zone in openstack provider #317

Open metahertz opened 9 years ago

metahertz commented 9 years ago

ignore_server_availability_zone is a standard micro_bosh option and is referred to here:

https://lists.cloudfoundry.org/archives/list/cf-bosh@lists.cloudfoundry.org/message/UV6YQEV4QL4XW7BIBWO3KOZLM3KHMBW2/

Setting the option in settings.yml does not work as it is not surfaced through bosh-bootstrap to the eventual micro_bosh.yml, this PR addresses that.

tlawrence commented 8 years ago

Any idea why this never got pulled Matt? Looks like it has possibly broken a couple of other tests?

geofffranks commented 7 years ago

Is this still needed since the introduction of bosh-init? If so, can the tests be fixed?

tlawrence commented 7 years ago

pretty sure this is still required. Why would it not be?

geofffranks commented 7 years ago

I wasn't sure if you'd migrated to bosh-init or not, that's all.

Sent from my iPhone

On Dec 2, 2016, at 6:17 AM, Tim Lawrence notifications@github.com wrote:

pretty sure this is still required. Why would it not be?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

tlawrence commented 7 years ago

I haven't but I don't know what it does either! Presumably it still needs to talk to the infrastructure via a CPI though doesn't it?

On Fri, Dec 2, 2016 at 12:18 PM, Geoff Franks notifications@github.com wrote:

I wasn't sure if you'd migrated to bosh-init or not, that's all.

Sent from my iPhone

On Dec 2, 2016, at 6:17 AM, Tim Lawrence notifications@github.com wrote:

pretty sure this is still required. Why would it not be?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/cloudfoundry-community/bosh-bootstrap/pull/317#issuecomment-264442140, or mute the thread https://github.com/notifications/unsubscribe-auth/AFa6z4zFKJvu_mG1RV8beRGEKLxIpUhGks5rEAyQgaJpZM4F2KKo .

geofffranks commented 7 years ago

It allows you to deploy things via bosh using standard bosh manifests + bosh logic. All the special configuration + logic related to bosh-bootstrap and the bosh micro plugin go away, since any property supported in the CPI specs are automatically supported in your BOSH manifest