-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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 healthcheck to aws-iam-authenticator #8991
Add healthcheck to aws-iam-authenticator #8991
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rdrgmnzs 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 |
12413fc
to
09f418d
Compare
An alpha was just released with the CRD fixes: https://github.com/kubernetes-sigs/aws-iam-authenticator/releases/tag/v0.5.1-alpha.1 i'll try to do some testing soon |
@rifelpet sg, I've actually been running a internal build from master for about 3 weeks now with no issues. |
@rdrgmnzs 0.5.1 was just released, care to rebase this so we can get it merged? https://github.com/kubernetes-sigs/aws-iam-authenticator/releases/tag/v0.5.1 |
volumeMounts: | ||
- name: config | ||
mountPath: /etc/aws-iam-authenticator/ | ||
- name: state | ||
mountPath: /var/aws-iam-authenticator/ | ||
- name: output | ||
mountPath: /etc/kubernetes/aws-iam-authenticator/ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add this line back?
09f418d
to
733bed6
Compare
Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA. It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
8376f77
to
e20f3ba
Compare
Confirmed this path is valid.
and matches this commit from 2017 that was only recently added to authenticator: kubernetes-sigs/aws-iam-authenticator@47f0d8e#diff-09856fe9becddf0199651f451409356aR250-R252 /lgtm |
@rdrgmnzs now that I think about this more, I think we might have a similar issue as #9500 where the /healthz endpoint was only introduced in 0.5.0 but users may have overridden the image with a <0.5.0 version. When they upgrade to this version of kops, the livenessProbe will be added but will fail. How do you think we should handle that? |
This builds on PR #8423 by @rifelpet
A 5.0.1 version of aws-iam-authenticator has not been released as of the creation of this PR, however I had the need to create internal version with the health check and decided to create the PR as a reminder to merger it in the future.