Skip to content
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

not enough arguments in call to graphdriver.GetDriver #1

Closed
vielmetti opened this issue Oct 14, 2016 · 5 comments
Closed

not enough arguments in call to graphdriver.GetDriver #1

vielmetti opened this issue Oct 14, 2016 · 5 comments

Comments

@vielmetti
Copy link

GOPATH=$HOME go test
# dsdbench
./layer_store_util.go:64: not enough arguments in call to graphdriver.GetDriver
FAIL    dsdbench [build failed]

environment:

# go version
go version go1.6.2 linux/arm64
@dmcgowan
Copy link
Owner

dmcgowan commented Oct 14, 2016

Are you building against docker/docker master? Can you try docker 1.12.1 first, I haven't pushed my update to fix this yet.

I was trying to avoid vendoring docker so I could test different versions, but then one of the function interfaces changed :(

@vielmetti
Copy link
Author

# cat /root/src/docker/VERSION
1.12.1

What should I set GOPATH to?

@dmcgowan
Copy link
Owner

checkout the version in $GOPATH/src/github.com/docker/docker to 1.12.1, also update GOPATH=$GOPATH/src/github.com/docker/docker/vendor:$GOPATH to use the vendor directory.

@vielmetti
Copy link
Author

I guessed at this patch:

diff --git a/layer_store_util.go b/layer_store_util.go
index 128f870..1bdc11b 100644
--- a/layer_store_util.go
+++ b/layer_store_util.go
@@ -61,7 +61,7 @@ func getLayerStore() (layer.Store, error) {
                driverOptions = strings.Split(options, " ")
        }

-       gd, err := graphdriver.GetDriver(driverName, td, driverOptions, nil, nil)
+       gd, err := graphdriver.GetDriver(driverName, td, driverOptions, nil, nil, nil)
        if err != nil {
                return nil, errors.Wrap(err, "failed to get graph driver")
        }

which got the tests to "work", though I don't have any confidence in them since I don't know what I added nil to.

The test run gist is here (again, I don't know if the patch is sensible).

https://gist.github.com/423b469d21da240608e4ff13bef047f0

@dmcgowan
Copy link
Owner

Updated to compile with docker master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants