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

added first support for user managed auth #4826

Merged
merged 1 commit into from
Jun 3, 2022

Conversation

dlaloue-vmware
Copy link
Collaborator

this change is an incremental update to add support for user-managed auth.

(final support is pending spec review and finalization)

@netlify
Copy link

netlify bot commented Jun 2, 2022

Deploy Preview for kubeapps-dev ready!

Name Link
🔨 Latest commit 3969880
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/6298f6ab36d34f0009838102
😎 Deploy Preview https://deploy-preview-4826--kubeapps-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Contributor

@antgamdia antgamdia left a comment

Choose a reason for hiding this comment

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

Great, thanks!

@antgamdia
Copy link
Contributor

Let me merge the PR to start using it in the UI

@antgamdia antgamdia merged commit b58d4fc into vmware-tanzu:main Jun 3, 2022
@vmwclabot
Copy link

@dlaloue-vmware, VMware has rejected your signed contributor license agreement. The change has already been merged, but it will be backed out by the project maintainers if the agreement is not resigned in a timely manner. Click here to resign the agreement.

@absoludity
Copy link
Contributor

@dlaloue-vmware is a member of vmware-tanzu org. Removing cla-rejected label.

@vmwclabot
Copy link

@dlaloue-vmware, your company's legal contact has approved your signed contributor license agreement. It will also be reviewed by VMware, but the merge can proceed.

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.

4 participants