-
Notifications
You must be signed in to change notification settings - Fork 170
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
Fix service script #169
Fix service script #169
Conversation
- Fixed init service script to allow proper 'start' and 'stop' actions consistently. I kept struggling that service start only worked after killing the distro because sometimes it didn't start/stop properly leaving processes running and without pid file - Added service 'status' action - Added additional service debug by setting environment variable DEBUG, ie: wsl.exe -d wsl-vpnkit DEBUG=1 service wsl-vpnkit stop - Changed order in startup welcome screen having stop followed by start, this makes it easier for users to just copy/paste from welcome screen to have service running - Added log check command to see only the latest service execution logs
- Added to README how to do start with status firt and how to enable DEBUG - Run tests twice with and without DEBUG enabled
It's still glitchy for me. Note: I have I installed your patch, rebooted and it worked.
I wonder why |
Yes it is still flacky. Although it helps in case start-stop-daemon is not working at all, it still doesn't solve the problem in Windows 10 where wsl-gvproxy.exe dies on windows side shortly after it starts. I'm keeping this open as alternative, but this needs more refinement. |
- stop and restart actions are performed consistently Improved build.sh - able to build inside vpn if using http_proxy
I've reviewed the service script and now it behaves consistently to on all actions without breaking wsl-gvproxy connection. |
Just tested this and I can confirm that it now stops and starts correctly. Closes #165 |
Allow build with podman Improve automated test timings
@sakai135 please review. |
./test.sh | ||
``` | ||
|
||
Optionally you may build with `podman` instead of `docker` (default) by overriding environment variable `DOCKER`: | ||
```sh | ||
DOCKER=podman ./build.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -8,7 +8,7 @@ RUN GOOS=windows go build -ldflags '-H=windowsgui' -o bin/wsl-gvproxy.exe ./cmd/ | |||
GOOS=linux CGO_ENABLED=0 go build -ldflags '-s -w' -o bin/wsl-vm ./cmd/vm && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the gvisor-tap-vsock
repo I maintain there is an additional comment to explain why -ldflags '-H=windowsgui'
was added; as it allows backgrounding of the process. This got lost in the import.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added as comment
Thanks @josesa-xx @F21 |
Rework service init script without using buggy
start-stop-daemon
Script has error checks enables so there is no need to set
ret
variableRedirect debug messages to stderr