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

Add OpenIDConnectProvider resource #639

Merged
merged 1 commit into from May 25, 2021

Conversation

mcavoyk
Copy link
Contributor

@mcavoyk mcavoyk commented Apr 23, 2021

Description of your changes

Fixes #636

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Has controller tests and have e2e tested this resource by adding the oidcIssuer of an EKS cluster.

@mcavoyk mcavoyk force-pushed the oidc-provider branch 4 times, most recently from 1884b88 to a5a3382 Compare April 27, 2021 22:17
@mcavoyk mcavoyk marked this pull request as ready for review April 27, 2021 22:17
@mcavoyk mcavoyk force-pushed the oidc-provider branch 2 times, most recently from 0aab913 to b1f37f9 Compare April 28, 2021 16:17
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.

Apologies for late review. LGTM! Though we need an example YAML in examples/iam folder that's proved to work with this implementation.

@mcavoyk mcavoyk force-pushed the oidc-provider branch 2 times, most recently from 8f25755 to 9341bdc Compare May 24, 2021 22:52
@mcavoyk
Copy link
Contributor Author

mcavoyk commented May 24, 2021

Thanks @muvaf, I rebased and added an example.

@muvaf
Copy link
Member

muvaf commented May 25, 2021

@mcavoyk Sorry, it seems like one of the PRs I merged made a change on the same file. Could you resolve the conflict?

Signed-off-by: Kellin McAvoy <kmcavoy@ea.com>
@muvaf muvaf merged commit 959cc5b into crossplane-contrib:master May 25, 2021
tektondeploy pushed a commit to gtn3010/provider-aws that referenced this pull request Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add IAM OpenIDConnectProvider resource
2 participants