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

Handle create nodegroup and withOIDC for clusters without aws-node SA #2778

Merged
merged 3 commits into from
Oct 28, 2020

Conversation

michaelbeaumont
Copy link
Contributor

Description

See #2740

Checklist

  • Added tests that cover your change (if possible)
  • Added/modified documentation as required (such as the README.md, or the userdocs directory)
  • Manually tested
  • Added labels for change area (e.g. area/nodegroup), target version (e.g. version/0.12.0) and kind (e.g. kind/improvement)
  • Make sure the title of the PR is a good description that can go into the release notes

@michaelbeaumont michaelbeaumont changed the base branch from release-0.30 to release-0.31 October 28, 2020 12:47
@michaelbeaumont michaelbeaumont force-pushed the withoidc_old branch 4 times, most recently from 72ca680 to 51f1ec8 Compare October 28, 2020 16:45
@michaelbeaumont michaelbeaumont marked this pull request as ready for review October 28, 2020 16:46
@aclevername aclevername self-requested a review October 28, 2020 16:46
Copy link
Contributor

@aclevername aclevername left a comment

Choose a reason for hiding this comment

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

LGTM

@michaelbeaumont michaelbeaumont merged commit 3db6e0d into release-0.31 Oct 28, 2020
@michaelbeaumont michaelbeaumont deleted the withoidc_old branch October 28, 2020 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants