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

Load all available k8s auth plugins instead of GCP only #784

Merged
merged 1 commit into from
Mar 2, 2021
Merged

Load all available k8s auth plugins instead of GCP only #784

merged 1 commit into from
Mar 2, 2021

Conversation

skymeyer
Copy link
Contributor

When using wash with kubernetes clusters a warning is returned:

WARN loading context freedom failed: no Auth Provider found for name "oidc"

This PR loads all available auth plugins instead of the one for gcp only (https://github.com/kubernetes/client-go/blob/master/plugin/pkg/client/auth/plugins.go), but will eat up some additional dependencies doing so.

@skymeyer skymeyer requested review from a team and ekinanp and removed request for a team February 25, 2021 04:56
@welcome
Copy link

welcome bot commented Feb 25, 2021

Hi! 💖 Thanks for your contribution 🎉🥳

We'll get it reviewed as soon as possible. If you'd like to chat with the developers, the #puppet-dev channel on Slack would probably be a great place to start. Make sure that your commit messages fit our contribution standards to make the code review go quickly.

If this is your first contribution to our repositories, then we'd love to send you a small welcome gift 🎁. Drop us a line at community@puppet.com and let us know where to send it! (/cc @puppetlabs/community)

@CLAassistant
Copy link

CLAassistant commented Feb 25, 2021

CLA assistant check
All committers have signed the CLA.

Signed-off-by: skymeyer <jelle.vink@gmail.com>
Copy link
Contributor

@MikaelSmith MikaelSmith left a comment

Choose a reason for hiding this comment

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

Thanks! This seems like a simple improvement.

@MikaelSmith MikaelSmith merged commit e1c3a95 into puppetlabs-toy-chest:main Mar 2, 2021
@welcome
Copy link

welcome bot commented Mar 2, 2021

Congrats on merging your first pull request in this repository! 🏅

@MikaelSmith
Copy link
Contributor

I'll try to get a release of it out later today.

@skymeyer skymeyer deleted the k8s-auth-plugins branch March 9, 2021 06:35
@skymeyer skymeyer restored the k8s-auth-plugins branch March 9, 2021 06:35
@skymeyer skymeyer deleted the k8s-auth-plugins branch March 9, 2021 06:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants