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

[customizations/eks]: Update credential API version #6289

Merged
merged 1 commit into from
Jul 28, 2021

Conversation

micahhausler
Copy link
Member

Kubernetes has deprecated v1alpha1, v1beta1 has been available since Kubernetes
v1.11 (kubernetes/kubernetes#64482), and EKS currently supports Kubernetes
versions v1.16 through v1.21. This is a breaking change for clients running
versions v1.10 and older, which haven't been supported by EKS since September
2019.

Signed-off-by: Micah Hausler mhausler@amazon.com

Issue #, if available:
N/A

Description of changes:

Update Kubernetes client authentication API version.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@micahhausler micahhausler force-pushed the eks-get-token-update branch from a8b50d8 to 2378426 Compare July 21, 2021 20:24
Copy link
Contributor

@stealthycoin stealthycoin left a comment

Choose a reason for hiding this comment

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

This looks good to me. Can you add a change log by running:

./scripts/new-change -t enhancement -c eks -d "Short description here."

then adding/comitting the file it writes.

Kubernetes has deprecated v1alpha1, v1beta1 has been available since Kubernetes
v1.11 (kubernetes/kubernetes#64482), and EKS currently supports Kubernetes
versions v1.16 through v1.21. This is a breaking change for clients running
versions v1.10 and older, which haven't been supported by EKS since September
2019.

Signed-off-by: Micah Hausler <mhausler@amazon.com>
@micahhausler micahhausler force-pushed the eks-get-token-update branch from 2378426 to 3569d49 Compare July 28, 2021 16:14
@codecov-commenter
Copy link

Codecov Report

Merging #6289 (2378426) into develop (7734cee) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 2378426 differs from pull request most recent head 3569d49. Consider uploading reports for the commit 3569d49 to get more accurate results
Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #6289   +/-   ##
========================================
  Coverage    92.81%   92.81%           
========================================
  Files          204      204           
  Lines        16201    16201           
========================================
  Hits         15037    15037           
  Misses        1164     1164           
Impacted Files Coverage Δ
awscli/customizations/eks/get_token.py 100.00% <ø> (ø)
awscli/customizations/eks/update_kubeconfig.py 97.93% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7734cee...3569d49. Read the comment docs.

@stealthycoin stealthycoin merged commit 1b1fa0e into aws:develop Jul 28, 2021
@micahhausler micahhausler deleted the eks-get-token-update branch July 28, 2021 21:35
@stevehipwell
Copy link

When will these changes be included in v2 (sorry I'm unfamiliar with the workflow here)?

@ndegory
Copy link

ndegory commented Jul 30, 2021

@stevehipwell , it's in 2.2.24.

@stevehipwell
Copy link

I think there might have been a lack of communication around this change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customization Issues related to CLI customizations (located in /awscli/customizations) eks-get-token
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants