-
Notifications
You must be signed in to change notification settings - Fork 661
CI: see if wsl2 works #1826
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
CI: see if wsl2 works #1826
Conversation
AkihiroSuda
commented
Sep 21, 2023
- CI: test WSL2 #1781
98678cd
to
60e615c
Compare
"failed to run
https://github.com/lima-vm/lima/actions/runs/6264735883/job/17012085612?pr=1826 |
60e615c
to
5b454be
Compare
|
a5e9480
to
fc1ace2
Compare
https://github.com/lima-vm/lima/actions/runs/6265448839/job/17014322683?pr=1826 @pendo324 PTAL |
fc1ace2
to
3d0ec0f
Compare
@pendo324 @afbjorklund Do you repro the issue locally? Is it working on your local env? |
I think the error is related to using bash mode. Y not try a simple |
3d0ec0f
to
62cd771
Compare
a98985c
to
b261f21
Compare
Removed bash, still failing
https://github.com/lima-vm/lima/actions/runs/6340681012/job/17222647981?pr=1826 |
Added some debug prints, it tries to unmarshal
https://github.com/lima-vm/lima/actions/runs/6340933579/job/17223439234?pr=1826 @pendo324 Do you see this error locally? Is it working on your machine? |
Not using logrus here, will this be a problem here ?? lima/pkg/downloader/downloader.go Line 444 in 6938ae5
|
Yes, IIUC this was on purpose to avoid messing up the output, but probably we should use logrus, or at least switch the output to stderr. |
9a72494
to
7a83cd7
Compare
https://github.com/lima-vm/lima/actions/runs/6342128771/job/17227239927?pr=1826 |
2792574
to
b40483a
Compare
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
a57614c
to
5a8b908
Compare
The script was failing prior to this commit ``` time="2023-09-28T17:18:46Z" level=info msg="[hostagent] Driver stopped due to error: \"error running wslCommand that executes boot.sh: exit status 127, check /var/log/lima-init.log for more details (out=\\\"bash: line 1: $'\\\\\\\\r': command not found\\\\n\\\")\"" ``` Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
5a8b908
to
5c690de
Compare
Temporarily giving up: Still help wanted to carry this. |
Sorry @AkihiroSuda, I just saw this for this first time (apparently I had my notifications off? I'll try to fix that). The issue seems to be that you don't have the binaries needed to extract the rootfs archive. It works on my machine because I do have those in my path. Is the To install tar/gzip we can install Git for Windows and add it to the path. Something like this will do that: $env:Path = 'C:\Program Files\Git\usr\bin;' + $env:Path
Set-ItemProperty -Path 'Registry::HKEY_LOCAL_MACHINE\System\CurrentControlSet\Control\Session Manager\Environment' -Name PATH -Value $env:Path If you re-open this PR and change the rootfs in wsl2.yaml to this, it should work if tar/gzip are installed:
|
Thanks @pendo324 would you mind opening a PR? |