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

install latest aws-iam-authenticator (0.5.9) from GitHub release #39

Merged
merged 1 commit into from
Oct 12, 2022

Conversation

rofreytag
Copy link
Contributor

Addresses #35 by downloading aws-iam-authenticator from GitHub.
Uses the GitHub API to check what the latest release actually is.

I moved the whole block down so I can use the previously installed jq to get the latest version from github API. AFAIK this should automatically download the latest release.

Let me know if this needs fine-tuning.

When building the container this should give you

$ aws-iam-authenticator version
{"Version":"0.5.9","Commit":"1209cfe28e95e32e719d0d69a323e6172a423333"}

@ozbillwang ozbillwang merged commit d1f7647 into alpine-docker:master Oct 12, 2022
@ozbillwang
Copy link
Collaborator

thanks for the PR.

ozbillwang pushed a commit that referenced this pull request Oct 12, 2022
ozbillwang pushed a commit that referenced this pull request Oct 12, 2022
@ozbillwang
Copy link
Collaborator

ozbillwang commented Oct 12, 2022

Images are ready.
image

docker run -ti --rm alpine/k8s:1.25.2 aws-iam-authenticator version
{"Version":"0.5.9","Commit":"1209cfe28e95e32e719d0d69a323e6172a423333"}

docker run -ti --rm alpine/k8s:1.22.15 aws-iam-authenticator version
{"Version":"0.5.9","Commit":"1209cfe28e95e32e719d0d69a323e6172a423333"}

@rofreytag rofreytag deleted the update-authenticator branch October 12, 2022 08:31
@rofreytag
Copy link
Contributor Author

Thank you for the quick turn-around!

@ozbillwang ozbillwang mentioned this pull request Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants