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

Adding Kubectl to Docker container #277

Open
wants to merge 3 commits into
base: v2.6
Choose a base branch
from
Open

Conversation

Arno500
Copy link

@Arno500 Arno500 commented May 15, 2020

This would actually be nice to be able to use Rancher CLI with kubectl commands in CI/CD pipelines and only provide a token. It's also not much to add to the container.

@gaby
Copy link

gaby commented Jul 21, 2021

Any plans to merge this?

Trackhe
Trackhe previously approved these changes May 23, 2022
@Trackhe
Copy link

Trackhe commented May 23, 2022

Please merge this.

@Arno500
Copy link
Author

Arno500 commented May 23, 2022

I updated the commit to the latest Alpine as in the main branch

@gaby
Copy link

gaby commented May 24, 2022

@Arno500 alpine comes with wget, that way you dont have to install curl.

Copy link

@Trackhe Trackhe left a comment

Choose a reason for hiding this comment

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

@Arno500 you can take this.

FROM alpine:3.13
RUN apk add --no-cache ca-certificates openssh-client
RUN wget -q https://storage.googleapis.com/kubernetes-release/release/$(wget -q --output-document -  https://storage.googleapis.com/kubernetes-release/release/stable.txt)/bin/linux/amd64/kubectl && chmod u+x kubectl && mv kubectl /bin/kubectl
COPY rancher /usr/bin/
WORKDIR /mnt
ENTRYPOINT ["rancher"]
CMD  ["--help"]

@Trackhe
Copy link

Trackhe commented May 26, 2022

@Arno500 the master branch is not the default branch and dosn`t build. Maybe you can try to pullrequest it to the v2.6 for faster merge.

@Arno500 Arno500 changed the base branch from master to v2.6 May 26, 2022 16:26
@Arno500
Copy link
Author

Arno500 commented May 26, 2022

@Trackhe, can you relaunch the checks? Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants