Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

nvmm is tested and works fine #914

Merged
merged 1 commit into from
Jun 20, 2022
Merged

nvmm is tested and works fine #914

merged 1 commit into from
Jun 20, 2022

Conversation

iamleot
Copy link
Contributor

@iamleot iamleot commented Jun 19, 2022

At least on NetBSD/amd64 -current(-ish) it works well.

Signed-off-by: Leonardo Taccari <leot@NetBSD.org>
@afbjorklund
Copy link
Member

afbjorklund commented Jun 19, 2022

But -cpu host failed, right ? Did -cpu max work, btw ?

@afbjorklund
Copy link
Member

afbjorklund commented Jun 19, 2022

@iamleot do the unit tests pass, on netbsd ?

go test ./...

@iamleot
Copy link
Contributor Author

iamleot commented Jun 19, 2022

But -cpu host failed, right ? Did -cpu max work, btw ?

Yes, -cpu max works.

@iamleot
Copy link
Contributor Author

iamleot commented Jun 19, 2022

@iamleot do the unit tests pass, on netbsd ?

go test ./...

If I try on a clean checked out source it fails with:

$ go test -v ./...
go: github.com/containerd/containerd@v1.6.6 requires
        github.com/AdaLogics/go-fuzz-headers@v0.0.0-20210715213245-6c3934b029d8: missing go.sum entry; to add it:
        go mod download github.com/AdaLogics/go-fuzz-headers

I guess I'm missing something obvious, right? :)

@iamleot
Copy link
Contributor Author

iamleot commented Jun 19, 2022

I have done a go mod download github.com/AdaLogics/go-fuzz-headers and after that:

$ go test ./...
# github.com/lima-vm/lima/pkg/cidata
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/cidata [setup failed]
# github.com/lima-vm/lima/pkg/downloader
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/downloader [setup failed]
# github.com/lima-vm/lima/pkg/guestagent/procnettcp
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/guestagent/procnettcp [setup failed]
# github.com/lima-vm/lima/pkg/hostagent/dns
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/hostagent/dns [setup failed]
# github.com/lima-vm/lima/pkg/limayaml
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/limayaml [setup failed]
# github.com/lima-vm/lima/pkg/networks
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/networks [setup failed]
# github.com/lima-vm/lima/pkg/osutil
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/osutil [setup failed]
# github.com/lima-vm/lima/pkg/qemu
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/qemu [setup failed]
# github.com/lima-vm/lima/pkg/sshutil
pkg/cidata/template_test.go:8:2: missing go.sum entry needed to verify package gotest.tools/v3/assert (imported by github.com/lima-vm/lima/pkg/cidata) is provided by exactly one module; to add:
        go get -t github.com/lima-vm/lima/pkg/cidata
FAIL    github.com/lima-vm/lima/pkg/sshutil [setup failed]
?       github.com/lima-vm/lima/cmd/limactl     [no test files]
?       github.com/lima-vm/lima/pkg/guestagent  [no test files]
?       github.com/lima-vm/lima/pkg/guestagent/api      [no test files]
?       github.com/lima-vm/lima/pkg/guestagent/api/client       [no test files]
?       github.com/lima-vm/lima/pkg/guestagent/api/server       [no test files]
ok      github.com/lima-vm/lima/pkg/guestagent/iptables (cached)
?       github.com/lima-vm/lima/pkg/hostagent   [no test files]
?       github.com/lima-vm/lima/pkg/hostagent/api       [no test files]
?       github.com/lima-vm/lima/pkg/hostagent/api/client        [no test files]
?       github.com/lima-vm/lima/pkg/hostagent/api/server        [no test files]
?       github.com/lima-vm/lima/pkg/hostagent/events    [no test files]
?       github.com/lima-vm/lima/pkg/httpclientutil      [no test files]
?       github.com/lima-vm/lima/pkg/httputil    [no test files]
?       github.com/lima-vm/lima/pkg/iso9660util [no test files]
?       github.com/lima-vm/lima/pkg/localpathutil       [no test files]
?       github.com/lima-vm/lima/pkg/lockutil    [no test files]
?       github.com/lima-vm/lima/pkg/logrusutil  [no test files]
?       github.com/lima-vm/lima/pkg/networks/reconcile  [no test files]
?       github.com/lima-vm/lima/pkg/qemu/const  [no test files]
?       github.com/lima-vm/lima/pkg/qemu/imgutil        [no test files]
?       github.com/lima-vm/lima/pkg/start       [no test files]
?       github.com/lima-vm/lima/pkg/store       [no test files]
?       github.com/lima-vm/lima/pkg/store/dirnames      [no test files]
?       github.com/lima-vm/lima/pkg/store/filenames     [no test files]
?       github.com/lima-vm/lima/pkg/templateutil        [no test files]
?       github.com/lima-vm/lima/pkg/usrlocalsharelima   [no test files]
?       github.com/lima-vm/lima/pkg/version     [no test files]
FAIL

@iamleot
Copy link
Contributor Author

iamleot commented Jun 19, 2022

After doing a go get -t github.com/lima-vm/lima/pkg/cidata:

$ go test ./...
?       github.com/lima-vm/lima/cmd/limactl     [no test files]
ok      github.com/lima-vm/lima/pkg/cidata      0.012s
ok      github.com/lima-vm/lima/pkg/downloader  0.751s
?       github.com/lima-vm/lima/pkg/guestagent  [no test files]
?       github.com/lima-vm/lima/pkg/guestagent/api      [no test files]
?       github.com/lima-vm/lima/pkg/guestagent/api/client       [no test files]
?       github.com/lima-vm/lima/pkg/guestagent/api/server       [no test files]
ok      github.com/lima-vm/lima/pkg/guestagent/iptables (cached)
ok      github.com/lima-vm/lima/pkg/guestagent/procnettcp       0.004s
?       github.com/lima-vm/lima/pkg/hostagent   [no test files]
?       github.com/lima-vm/lima/pkg/hostagent/api       [no test files]
?       github.com/lima-vm/lima/pkg/hostagent/api/client        [no test files]
?       github.com/lima-vm/lima/pkg/hostagent/api/server        [no test files]
ok      github.com/lima-vm/lima/pkg/hostagent/dns       0.006s
?       github.com/lima-vm/lima/pkg/hostagent/events    [no test files]
?       github.com/lima-vm/lima/pkg/httpclientutil      [no test files]
?       github.com/lima-vm/lima/pkg/httputil    [no test files]
?       github.com/lima-vm/lima/pkg/iso9660util [no test files]
ok      github.com/lima-vm/lima/pkg/limayaml    0.010s
?       github.com/lima-vm/lima/pkg/localpathutil       [no test files]
?       github.com/lima-vm/lima/pkg/lockutil    [no test files]
?       github.com/lima-vm/lima/pkg/logrusutil  [no test files]
--- FAIL: TestUser (0.00s)
    commands_test.go:72: assertion failed: wheel (user.Group string) != root (string)
FAIL
FAIL    github.com/lima-vm/lima/pkg/networks    0.008s
?       github.com/lima-vm/lima/pkg/networks/reconcile  [no test files]
ok      github.com/lima-vm/lima/pkg/osutil      0.003s
ok      github.com/lima-vm/lima/pkg/qemu        0.004s
?       github.com/lima-vm/lima/pkg/qemu/const  [no test files]
?       github.com/lima-vm/lima/pkg/qemu/imgutil        [no test files]
ok      github.com/lima-vm/lima/pkg/sshutil     0.006s
?       github.com/lima-vm/lima/pkg/start       [no test files]
?       github.com/lima-vm/lima/pkg/store       [no test files]
?       github.com/lima-vm/lima/pkg/store/dirnames      [no test files]
?       github.com/lima-vm/lima/pkg/store/filenames     [no test files]
?       github.com/lima-vm/lima/pkg/templateutil        [no test files]
?       github.com/lima-vm/lima/pkg/usrlocalsharelima   [no test files]
?       github.com/lima-vm/lima/pkg/version     [no test files]
FAIL

@afbjorklund
Copy link
Member

afbjorklund commented Jun 19, 2022

Thanks for checking, I guess the "wheel" is easy to fix (same as darwin)

        if runtime.GOOS == "darwin" {
                assert.Equal(t, user.Group, "wheel")
        } else {
                assert.Equal(t, user.Group, "root")
        }

Most likely, also FreeBSD.

@iamleot
Copy link
Contributor Author

iamleot commented Jun 19, 2022

Thanks for checking, I guess the "wheel" is easy to fix (same as darwin)

        if runtime.GOOS == "darwin" {
                assert.Equal(t, user.Group, "wheel")
        } else {
                assert.Equal(t, user.Group, "root")
        }

Most likely, also FreeBSD.

Yeah, I will provide a patch about that in a separate PR (and I can confirm that it fixes all the tests then).

Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@AkihiroSuda AkihiroSuda merged commit 30c5594 into lima-vm:master Jun 20, 2022
@AkihiroSuda AkihiroSuda added this to the vNext milestone Jul 7, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants