Disable go modules when compiling binaries used in tests#42391
Disable go modules when compiling binaries used in tests#42391thaJeztah merged 2 commits intomoby:masterfrom
Conversation
INFO: Running integration tests at 05/17/2021 12:54:50...
INFO: DOCKER_HOST at tcp://127.0.0.1:2357
INFO: Integration API tests being run from the host:
INFO: make.ps1 starting at 05/17/2021 12:54:50
powershell.exe : go: cannot find main module, but found vendor.conf in D:\gopath\src\github.com\docker\docker
At D:\gopath\src\github.com\docker\docker@tmp\durable-1ed00396\powershellWrapper.ps1:3 char:1
+ & powershell -NoProfile -NonInteractive -ExecutionPolicy Bypass -Comm ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo : NotSpecified: (go: cannot find...m\docker\docker:String) [], RemoteException
+ FullyQualifiedErrorId : NativeCommandError
to create a module there, run:
go mod init
INFO: make.ps1 ended at 05/17/2021 12:54:51
Signed-off-by: Sebastiaan van Stijn <[email protected]>
=== RUN TestServicePlugin
plugin_test.go:42: assertion failed: error is not nil: error building basic plugin bin: no required module provides package github.com/docker/docker/testutil/fixtures/plugin/basic: go.mod file not found in current directory or any parent directory; see 'go help modules'
: exit status 1
Signed-off-by: Sebastiaan van Stijn <[email protected]>
|
@cpuguy83 @AkihiroSuda ptal |
|
failure on amd64 is a known flaky test; #42357 kicking ci again |
|
This will only work until Go 1.17, right? 😞 |
Good question; haven't fully caught up on that; will Go 1.17 still support a |
(So in short, no, I do not think so -- for Go 1.17 we either have to "convert or die" so to speak.) |
|
So much for "no breaking changes'. Will they still support vendoring? (that's a bit odd as they're not modules either) |
|
Yes! From my testing, even an empty |
relates to #40353
These caused CI to fail when using Go 1.16, so disabling Go modules to prevent that