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

"arkade install" fails on Win (Git Bash) due to helm download URL guess? #92

Closed
mkol5222 opened this issue Apr 13, 2020 · 5 comments · Fixed by #93
Closed

"arkade install" fails on Win (Git Bash) due to helm download URL guess? #92

mkol5222 opened this issue Apr 13, 2020 · 5 comments · Fixed by #93

Comments

@mkol5222
Copy link

Hello, I am using arkade on Win10 in Git Bash installed via "choco install git"

WHAT FAILS
arkade install fails because trying to get helm from
https://get.helm.sh/helm-v3.1.1-mingw64_nt-10.0-18362-amd64.tar.gz

REPLICATION

$ uname -a
MINGW64_NT-10.0-18362 username-7480 3.0.7-338.x86_64 2019-11-21 23:07 UTC x86_64 Msys

$ arkade install mongodb
Using kubeconfig: C:\Users\username/.kube/config
Node architecture: "amd64"
Client: "x86_64", "MINGW64_NT-10.0-18362"
2020/04/13 10:16:15 User dir established as: C:\Users\username/.arkade/
https://get.helm.sh/helm-v3.1.1-mingw64_nt-10.0-18362-amd64.tar.gz
2020/04/13 10:16:16 error extracting tarball into C:\Users\username/.arkade/bin/helm3 after 0 files, 0 dirs, 0s: requires gzip-compressed body: gzip: invalid header
Error: unable to add repo exec: "C:\\Users\\username/.arkade/bin/helm3/helm": file does not exist

Am I using wrong distro of Git Bash or startegy to guess helm download URL is not complete? (uname -a abobe)

Thank you

@mkol5222
Copy link
Author

manual fix was:

  • download helm from
curl -OL https://get.helm.sh/helm-v3.1.2-windows-amd64.tar.gz
  • extract and copy helm.exe
tar xzvf helm-v3.1.2-windows-amd64.tar.gz
mv windows-amd64/helm.exe ~/.arkade/bin/helm3/
  • testing some arkade install then worked for me
arkade install mongodb

@alexellis
Copy link
Owner

Hi "mkol5222"

Thanks for logging this issue. We should be able to resolve the auto-download of helm on Windows. I'll ping you when there's something to test.

Alex

alexellis added a commit that referenced this issue Apr 13, 2020
Fixes: #92

Tested with a unit test, with similar logic to the get.sh
script.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
alexellis added a commit that referenced this issue Apr 13, 2020
Fixes: #92

Tested with a unit test, with similar logic to the get.sh
script.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@alexellis
Copy link
Owner

PR #93 fixes this issue, you'll be able to test the new binary release when it's published via https://github.com/alexellis/arkade/releases/tag/0.2.6

To test simply arkade upgrade, then rm -rf ~/.arkade before running arkade install again with an app that uses helm3.

@mkol5222
Copy link
Author

Many thanks for amazingly fast fix delivery. I was starting our new lab and it got latest arkade release that was able to fetch helm and all was working fine and smooth even on Win10 inside Git Bash.

@alexellis
Copy link
Owner

Tell me about what you're using arkade for? We have a Slack community with a general and contributors channel inside OpenFaaS slack https://slack.openfaas.io

aidun added a commit to aidun/arkade that referenced this issue Jun 16, 2020
- Helm3 only
- No arm support

Signed-off-by: Markus Hartmann <markush1986@gmail.com>

Updated comment

Signed-off-by: kadern0 <kaderno@gmail.com>

Fix download of Helm on Git Bash

Fixes: alexellis#92

Tested with a unit test, with similar logic to the get.sh
script.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Fix issue with Istio init timing

Fixes: 94

A user reported an error about a missing Istio CRD and it
appears that one must wait for at least three bespoke init
jobs to complete before attempting to do the installation.
With KinD, on my home Internet connection this took several
minutes to run.

Technique sourced from Flagger / @stefanprodan.

In the future, when arkade uses istioctl, hopefully all this
bespoke code won't be required for an unattended installation.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Fixes alexellis#82: Add Jenkins App

- Uses Jenkins Helm Chart
- Helm 3 only
- no ARM support
- Only default settings

Signed-off-by: Markus Hartmann <markush1986@gmail.com>

Update Jenkins messages

* updated the jenkins message to wrap to 80 chars for
readability
* extracted variable for non-Intel architecture warning to
reduce duplicated code
* rollout status command added since Jenkins with KinD took
around 6 minutes to come up and looked like it might have
failed to work

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Inline travis build environment

Prevents the travis build using multiple stages when only
one is required.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Fixes alexellis#98: Add stable repo for grafana

Signed-off-by: Markus Hartmann <markush1986@gmail.com>

Add max-inflight flag to openfaas

Specify max tasks to execute in parallel per queue worker

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Fix issue with max_inflight flag mismatch

Fixes: alexellis#106

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Update name and location of nginx-ingress chart

Signed-off-by: Manuel Alejandro de Brito Fontes <aledbf@gmail.com>

Trim whitespace from license read from file

A user reported an error when they left whitespace inside a
license JWT file, so this forces a trim of any new lines or
spaces to help prevent errors.

Tested with a PRO license.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Update as per lgtm.com

Analysis showed a couple of unused variables

https://lgtm.com/projects/g/alexellis/arkade/?mode=list

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Update README.md with the renaming of nginx-ingress

Signed-off-by: Manuel Alejandro de Brito Fontes <aledbf@gmail.com>

Add note for running Nginx on K8s 1.18

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Add new t-shirt

Add sponsored app offer via OpenFaaS Ltd

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Add subscription link

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Enable more verbose errors in helm download

Print an error if unable to create folder for helm.
Print an error if getting non 200 code from server for
download.

Testing: manual install with local binary of openfaas app after
removing .arkade folder.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Allow namespace for inlets-operator with helm3

With helm3 the namespace isn't an issue - so allow the user
to give an alternative namespace.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Update README.md

Refactor into pkg using an options input and add Loki App

Signed-off-by: Alistair Hey <alistair@heyal.co.uk>

Update .gitignore

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Remove k3sup reference

It's been long enough for people to have switched over from
k3sup app install, the note should not be required any longer.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Update headings

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Add "get" command

The get command downloads tools from GitHub release pages
and from custom URLs using a go template to define the
URL and filename according to OS and architecture.

Tested with unit tests and with the two initial apps: faas-cli
and kubectl.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>

Document "arkade get"

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
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 a pull request may close this issue.

2 participants