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

New Installation breaks tool #8

Closed
embano1 opened this issue Jul 12, 2018 · 0 comments
Closed

New Installation breaks tool #8

embano1 opened this issue Jul 12, 2018 · 0 comments

Comments

@embano1
Copy link
Contributor

embano1 commented Jul 12, 2018

Hi Ashley,

Thx for switching to goreleaser :)

I found some issues with the new installation procedure:

export BINARY=kubeconfig-cleanup_1.0.1_Linux_x86_64.tar.gz
curl -LO -o cleanup https://github.com/ashleyschuett/kubeconfig-cleanup/releases/download/v1.0.1/$BINARY && \
curl -LO https://raw.githubusercontent.com/ashleyschuett/kubeconfig-cleanup/v1.0.1/plugin.yaml && \
mkdir ~/.kube/plugins/kubeconfig-cleanup && mv cleanup plugin.yaml
  1. The BINARY is a tgz, meaning that it must be extracted first and not just moved (will error out with error: fork/exec ./cleanup: exec format error)
  2. The binary then has to be executable (chmod +x), otherwise error: fork/exec ./cleanup: permission denied
  3. curl -LO -o does not work because -o has no effect here (just remove -O to fix)
  4. mkdir should use option -p to not error out when plugins does not exist or when kubeconfig-cleanup already exists (e.g. upgrade)
  5. the mv needs a target folder, i.e. mv cleanup plugin.yaml ~/.kube/plugins/kubeconfig-cleanup

Will file PR :)

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

No branches or pull requests

1 participant