-
Notifications
You must be signed in to change notification settings - Fork 60
Plugin doesn't install on Windows OS #1
Comments
I think the issue is due to the fact that the installation is done with a shell script and windows tells that sh is not present. Do you mean the plugin installation works with WSL2, but doesn't without WSL2? |
Some related links - helm/helm-2to3#55 |
No it doesn't work with WSL2 as well. |
Okay. I don't have access to a Windows system currently, so I can't help currently. |
Works fine with git bash. Though this returns a different error. |
@sfxworks Thanks for reporting. I'll check how it works in https://github.com/chartmuseum/helm-push :) |
I think the culprit might be in this piece of code that I wrote |
@sfxworks Not sure what the above error means. Could you explain which version of helm you are using? I'm not sure if the above message is related to this plugin as I don't find that error message in the install script, and it has been working fine for others with helm, not sure what's causing this particular issue |
I checked if helm has this error message in it's master branch, but didn't find anything |
It will be nice to have it working on Windows @karuppiah7890. I still get the below error while installing the plugin. |
@rajurh Can you tell me if this plugin works for you - https://github.com/chartmuseum/helm-push $ helm plugin install https://github.com/chartmuseum/helm-push.git Also, can you try in something like git bash? or mingw / cygwin ? |
Works with A bit annoying that I have to use that to install it though, but it still shows up post-install in the windows terminal 👍
|
@sfxworks but the schema-gen plugin does not work for you now? Also, I understand it's annoying that you need another shell to just install this. I think the problem inherently lies in Helm's plugin installation mechanism - I can only provide one script to install. I can't provide different scripts (bash, sh, powershell etc) based on platform. It's a crazy thing. I need to dig in and see if there's any other solution to this |
I can confirm that the current version works on Windows when using Git Bash for the installation. Afterwards it works across all windows native shells (pwsh, cmd, git bash). An installation in WSL2/Bash (with installed dash, helm, git) fails with the following error message:
|
@alexrashed Thanks for the information! That's new. I'm not sure what's causing a file / executable not found issue Since I don't have access to windows machines, I'm not sure how to debug these issues. PRs are welcome though. Usually my approach for this issue has been to check what helm project recommends and what other cross platform plugins do. Currently I use an auto generated script to install the plugin. https://github.com/goreleaser/godownloader There seem to be issues with the script - it's based on |
Since godownloader isn't maintained anymore in quite some time, in my opinion there are two options:
EDIT: However, it might be reasonable to assume that an sh binary is in the path (even on Windows due to Git Bash). |
The releases already contain the binaries. I'm planning to update the readme to inform about the usage of the tool as a stand alone tool - install it standalone, instead of installing it as a helm plugin Any other ideas, I don't have time to work on it. PRs are welcome. I would like to see some test too, with GitHub actions, as I have no windows environment to manually test |
I know that the releases contain the binaries, but it does not contain the metadata for installing it as a Helm plugin. |
Ah. I didn't know about this! Thanks for this idea. I really am not a fan of the git based installation. I believe it tries to get the git repo with history included I think. But for this plugin, the repo files (golang files etc) is not needed at all, so, just the binary and plugin.yaml would do. But with tar ball, I think there might be issues for people using --version flag in "helm plugin install" command. But atleast tar ball can be an alternative. I can mention about how to install the latest version and other versions too, with tar ball and not git. But I would have to update the releases for it and also change the automated release process which supports both ways. I can't do this now though. If I start working on it later (idk when), I'll comment here. Till then it's open for PRs. Anyone working on the PR can check goreleaser config and work on it or use custom scripts too, so that a tar ball is created with binary and plugin.yaml for automated releases in the future. Maybe the same thing can be used to update the existing releases - I can help with that as you won't have access to edit releases |
I am trying on Windows laptop with WSL2 and without WSL2 , plugin fails to install with below error
Error: exec: "sh": executable file not found in %PATH%
The text was updated successfully, but these errors were encountered: