cloudfoundry-incubator / kubo-release

Kubernetes BOSH release
https://www.cloudfoundry.org/container-runtime/
Apache License 2.0
161 stars 76 forks source link

Add special case for Azure Availability Sets #355

Closed larham closed 5 years ago

larham commented 5 years ago

Add special case for Azure Availability Sets

As a workaround, we add a conditional to sense the Azure-only, artificial zone value (the string "Availability Sets"), and omit its usage as a value for the output of environmental variable 'bosh.zone'.

What this PR does / why we need it: Adapt to OpsMan 2.5+; fix breakage on Azure

How can this PR be verified? Local test; integration pipeline under way

Is there any change in kubo-deployment? No

Is there any change in kubo-ci? No

Does this affect upgrade, or is there any migration required? No

Which issue(s) this PR fixes: https://pivotal-spike.atlassian.net/browse/PKS-349

Release note:

Fixes issue that prevented deployment with Azure Availability Sets
cfdreddbot commented 5 years ago

:x: Hey larham!

All pull request submitters and commit authors must have a Contributor License Agreement (CLA). Click here for details on the CLA process.

The following github user @larham is not covered by a CLA.

After the CLA process is complete, this pull request will need to be closed & reopened. DreddBot will then validate the CLA(s).