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

Update xpkg spec with webhook configurations in providers #2996

Merged
merged 1 commit into from Mar 22, 2022

Conversation

hasheddan
Copy link
Member

Description of your changes

Updates the xpkg spec to include AdmissionWebhookConfiguration and
MutatingWebhookConfiguration as valid types in a Provider package.

Signed-off-by: hasheddan georgedanielmangum@gmail.com

Fixes #2975

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

n/a -- documentation only

Updates the xpkg spec to include AdmissionWebhookConfiguration and
MutatingWebhookConfiguration as valid types in a Provider package.

Signed-off-by: hasheddan <georgedanielmangum@gmail.com>
Copy link
Member

@muvaf muvaf left a comment

Choose a reason for hiding this comment

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

Thanks @hasheddan ! Do you want to version the spec at one point?

Copy link
Member Author

@hasheddan hasheddan left a comment

Choose a reason for hiding this comment

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

Do you want to version the spec at one point?

@muvaf I think that could be a good idea -- although since it is included in the documentation, the versioning is somewhat implied by the version of the docs you select. I think it could be helpful to have a Added in vX.Y.Z note for features we add in the future though 👍🏻

@github-actions
Copy link

Successfully created backport PR #2997 for release-1.7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update xpkg spec with the new webhook functionality
2 participants