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

client-go: promote exec plugin support to beta #64482

Merged
merged 4 commits into from
Jun 4, 2018

Conversation

ericchiang
Copy link
Contributor

@ericchiang ericchiang commented May 29, 2018

/sig auth
/kind feature

Adds a "v1beta1" API group for client authentication. Because of a lack of usage with the input parameters, these have been dropped for the beta. Would like to re-introduce them as users feel they require them.

updates #61796

client-go: credential exec plugins have been promoted to beta

@k8s-ci-robot k8s-ci-robot added release-note Denotes a PR that will be considered when it comes time to generate release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. sig/auth Categorizes an issue or PR as relevant to SIG Auth. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature. labels May 29, 2018
@ericchiang ericchiang changed the title Exec plugin beta client-go: promote exec plugin support to beta May 29, 2018
@ericchiang ericchiang force-pushed the exec-plugin-beta branch 3 times, most recently from 10849c4 to d3b16da Compare May 29, 2018 23:40
@ericchiang
Copy link
Contributor Author

/test pull-kubernetes-kubemark-e2e-gce

@ericchiang
Copy link
Contributor Author

Going to let #61803 merge first.

@ericchiang ericchiang added this to the v1.11 milestone May 30, 2018
@ericchiang
Copy link
Contributor Author

/sig auth
/kind feature
/milestone v1.11
/priority important-longterm
/status approved-for-milestone

@k8s-ci-robot k8s-ci-robot added priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. status/approved-for-milestone labels May 30, 2018
@ericchiang
Copy link
Contributor Author

/test pull-kubernetes-bazel-build

@dims
Copy link
Member

dims commented Jun 1, 2018

/assign @thockin

@ericchiang ericchiang assigned liggitt and awly and unassigned thockin Jun 1, 2018
@ericchiang
Copy link
Contributor Author

cc @kubernetes/sig-auth-pr-reviews

@@ -420,6 +486,13 @@ func TestRefreshCreds(t *testing.T) {
t.Errorf("expected expiry %v got %v", test.wantExpiry, a.exp)
}

if test.wantInput == "" {
if got := strings.TrimSpace(stderr.String()); got != "" {
t.Errorf("expectd no input parameters, got %q", got)
Copy link
Contributor

Choose a reason for hiding this comment

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

s/expectd/expected/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks! updated


// Response defines metadata about a failed request, including HTTP status code and
// response headers.
type Response struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep. thanks for pointing that out

	./hack/update-bazel.sh
	./hack/update-codegen.sh
	./hack/update-staging-godeps.sh
@awly
Copy link
Contributor

awly commented Jun 1, 2018

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 1, 2018
@ericchiang
Copy link
Contributor Author

/assign @thockin

For final approval

@k8s-github-robot
Copy link

[MILESTONENOTIFIER] Milestone Pull Request: Up-to-date for process

@awly @ericchiang @liggitt @thockin

Pull Request Labels
  • sig/auth: Pull Request will be escalated to these SIGs if needed.
  • priority/important-longterm: Escalate to the pull request owners; move out of the milestone after 1 attempt.
  • kind/feature: New functionality.
Help

@thockin
Copy link
Member

thockin commented Jun 4, 2018

I have no context on this API, so it's hard for me to review. Given that it is mostly empty, I guess it's not a big deal yet :)

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: awly, ericchiang, thockin

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 4, 2018
@k8s-github-robot
Copy link

Automatic merge from submit-queue (batch tested with PRs 63453, 64592, 64482, 64618, 64661). If you want to cherry-pick this change to another branch, please follow the instructions here.

@k8s-github-robot k8s-github-robot merged commit 819d515 into kubernetes:master Jun 4, 2018
micahhausler added a commit to micahhausler/aws-cli that referenced this pull request Jul 21, 2021
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 added a commit to micahhausler/aws-iam-authenticator that referenced this pull request Jul 21, 2021
Kubernetes has deprecated v1alpha1, v1beta1 has been available since Kubernetes
v1.11 (kubernetes/kubernetes#64482). At the time of this commit,
community-supported versions are v1.19, v1.20, v1.21. This change will break
for Kubernetes clients using client-go or kubectl v1.10 and earlier, which
reached end-of-life community support on Feb 13, 2019. EKS ended support
for 1.10 in September of 2019.

Kubernetes 1.22 will introduce v1 of this API, but v1beta1 will likely be
supported for a period of 2 more years.

Signed-off-by: Micah Hausler <mhausler@amazon.com>
micahhausler added a commit to micahhausler/aws-cli that referenced this pull request Jul 28, 2021
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 added a commit to micahhausler/aws-cli that referenced this pull request Oct 13, 2021
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.

"aws eks get-token" now respects the KUBERNETES_EXEC_INFO environment
variable and conservatively falls back to v1alpha1, which is supported
by Kubernetes versions 1.10 through 1.22 (released upstream August 2021, to be
released by EKS in Q4 2021). It also now supports "v1beta1" and "v1".

"aws eks update-kubeconfig" now writes "v1beta1" in the kubeconfig which
will be supported by Kubernetes until 1.29 (aproximately December 2023).
At or around that date, we can change the default version written to
kubeconfigs to "v1"

Signed-off-by: Micah Hausler <mhausler@amazon.com>
justindho pushed a commit to micahhausler/aws-cli that referenced this pull request May 5, 2022
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.

"aws eks get-token" now respects the KUBERNETES_EXEC_INFO environment
variable and conservatively falls back to v1alpha1, which is supported
by Kubernetes versions 1.10 through 1.22 (released upstream August 2021, to be
released by EKS in Q4 2021). It also now supports "v1beta1" and "v1".

"aws eks update-kubeconfig" now writes "v1beta1" in the kubeconfig which
will be supported by Kubernetes until 1.29 (aproximately December 2023).
At or around that date, we can change the default version written to
kubeconfigs to "v1"

Signed-off-by: Micah Hausler <mhausler@amazon.com>
ConnorKirk pushed a commit to ConnorKirk/aws-cli that referenced this pull request Oct 28, 2022
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.

"aws eks get-token" now respects the KUBERNETES_EXEC_INFO environment
variable and conservatively falls back to v1alpha1, which is supported
by Kubernetes versions 1.10 through 1.22 (released upstream August 2021, to be
released by EKS in Q4 2021). It also now supports "v1beta1" and "v1".

"aws eks update-kubeconfig" now writes "v1beta1" in the kubeconfig which
will be supported by Kubernetes until 1.29 (aproximately December 2023).
At or around that date, we can change the default version written to
kubeconfigs to "v1"

Signed-off-by: Micah Hausler <mhausler@amazon.com>
godd9170 pushed a commit to godd9170/aws-cli that referenced this pull request Mar 7, 2023
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>
godd9170 pushed a commit to godd9170/aws-cli that referenced this pull request Mar 7, 2023
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.

"aws eks get-token" now respects the KUBERNETES_EXEC_INFO environment
variable and conservatively falls back to v1alpha1, which is supported
by Kubernetes versions 1.10 through 1.22 (released upstream August 2021, to be
released by EKS in Q4 2021). It also now supports "v1beta1" and "v1".

"aws eks update-kubeconfig" now writes "v1beta1" in the kubeconfig which
will be supported by Kubernetes until 1.29 (aproximately December 2023).
At or around that date, we can change the default version written to
kubeconfigs to "v1"

Signed-off-by: Micah Hausler <mhausler@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. release-note Denotes a PR that will be considered when it comes time to generate release notes. sig/auth Categorizes an issue or PR as relevant to SIG Auth. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants