-
Notifications
You must be signed in to change notification settings - Fork 205
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
boltdb/bolt is not maintained. Use bbolt instead. #202
Conversation
Signed-off-by: John Howard <[email protected]>
Signed-off-by: John Howard <[email protected]>
Replaces #200. Ping @thaJeztah. Added a second commit to bump golang to latest 1.8 release (although entirely not supported, but slightly less unsupported than 1.7.1....) |
1f22601
to
c6b4d57
Compare
Added third commit to bump zookeeper - CI fails as those versions are no longer present. |
Signed-off-by: John Howard <[email protected]>
c6b4d57
to
da0a707
Compare
Yay, green CI :) |
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.
LGTM
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.
LGTM
"github.com/docker/libkv" | ||
"github.com/docker/libkv/store" | ||
bolt "go.etcd.io/bbolt" |
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.
nit: Was the alias needed? I think it worked without
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.
Ah, the package name in the new repo is bbolt
, not bolt
. I'm following the import recommendation at https://github.com/etcd-io/bbolt#installing
Signed-off-by: John Howard [email protected]
Part fix for moby/libnetwork#1950
gitHub.com/boltdb/bolt is no longer being maintained. From https://github.com/boltdb/bolt:
In order to fix the libnetwork issue, I need to change consumers over to the new package, so that I can ultimately pull in this fix: etcd-io/bbolt#122.
This PR uses the import syntax recommended at https://github.com/etcd-io/bbolt#importing-bbolt
@mavenugo PTAL. cc @vieux @johnstep
@carlfischer1 - This will be one change of a few to get that libnetwork issue fixed and pulled into backport.