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 servicebinding.io/v1beta1 #227

Merged
merged 2 commits into from
Mar 3, 2022
Merged

Conversation

scothis
Copy link
Contributor

@scothis scothis commented Mar 3, 2022

The API is compatible with v1alpha3 and does not require a conversion
webhook. The controller and webhook still operate against v1alpha3, and
can be upgraded at some point in the future.

Signed-off-by: Scott Andrews andrewssc@vmware.com

The API is compatible with v1alpha3 and does not require a conversion
webhook. The controller and webhook still operate against v1alpha3, and
can be upgraded at some point in the future.

Signed-off-by: Scott Andrews <andrewssc@vmware.com>
@vmwclabot vmwclabot added the cla-not-required CLA not required label Mar 3, 2022
@scothis
Copy link
Contributor Author

scothis commented Mar 3, 2022

Keeping as a draft until the Service Binding spec formerly releases v1beta1.

@scothis scothis requested a review from rashedkvm March 3, 2022 03:03
Signed-off-by: Scott Andrews <andrewssc@vmware.com>
@scothis scothis marked this pull request as ready for review March 3, 2022 16:59
@@ -212,3 +212,192 @@ spec:
storage: true
subresources:
status: {}
- additionalPrinterColumns:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

from https://github.com/servicebinding/spec/blob/v1.0.0/servicebinding.io_servicebindings.yaml#L19-L207

Only change is that we're keeping v1alpha3 as the stored version.

@rashedkvm rashedkvm merged commit c55c320 into vmware-tanzu:main Mar 3, 2022
@scothis scothis deleted the v1beta1 branch March 3, 2022 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-not-required CLA not required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants