Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support stemcell name property #74

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Support stemcell name property #74

wants to merge 1 commit into from

Conversation

abg
Copy link
Member

@abg abg commented Sep 25, 2024

Broadcom has started supporting FIPS enabled stemcells whose version matches the non-FIPS stemcells and without a name specified the FIPS may unintentionally be used when a deployment might not support FIPS.

To avoid this allow consumers to set a stemcell name property to point to the expected stemcell name. This allows BOSH to ensure the desired stemcell is always used and not unintentionally deploying with the wrong variant.

This commit updates the spec property of the broker job to allow defining the stemcell.name or stemcells[].name property for a data service product.

To fully support the name property this depends on changes in the ODB SDK in pivotal-cf/on-demand-services-sdk#200

Broadcom has started supporting FIPS enabled stemcells whose version
matches the non-FIPS stemcells and without a name specified the FIPS may
unintentionally be used when a deployment might not support FIPS.

To avoid this allow consumers to set a stemcell name property to point
to the expected stemcell name. This allows BOSH to ensure the desired
stemcell is always used and not unintentionally deploying with the wrong
variant.

This commit updates the spec property of the broker job to allow
defining the stemcell.name or stemcells[].name property for a data
service product.

To fully support the name property this depends on changes in the ODB
SDK in pivotal-cf/on-demand-services-sdk#200
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