Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

addons/packages/pinniped: Update Pinniped cert-manager API version to v1 #2046

Merged
merged 1 commit into from
Sep 30, 2021
Merged

addons/packages/pinniped: Update Pinniped cert-manager API version to v1 #2046

merged 1 commit into from
Sep 30, 2021

Conversation

sabbey37
Copy link
Contributor

What this PR does / why we need it

The Pinniped package currently uses cert-manager.io/v1alpha2 cert-manager APIs. This version of cert-manager is deprecated and will be removed. This PR updates Pinniped cert-manager API version to v1.

Details for the Release Notes (PLEASE PROVIDE)

Update Pinniped cert-manager API version from v1alpha2 to v1

Which issue(s) this PR fixes

None

Describe testing done for PR

I built the Pinniped package revision from this PR into a core PackageRepository and deployed that into a 1.5 TKG environment. Then I configured OIDC on a management cluster, configured RBAC for the given user, successfully logged in and listed pods

Special notes for your reviewer

Feel free to reach out with any questions!

@sabbey37 sabbey37 requested a review from a team as a code owner September 29, 2021 20:59
@github-actions github-actions bot added the owner/packages Work executed by a package's maintainer label Sep 29, 2021
Copy link
Contributor

@ankeesler ankeesler left a comment

Choose a reason for hiding this comment

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

change lgtm

@sabbey37 sabbey37 merged commit 8223879 into vmware-tanzu:main Sep 30, 2021
@sabbey37 sabbey37 deleted the pinniped-cert-manager branch September 30, 2021 14:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required owner/packages Work executed by a package's maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants