-
Notifications
You must be signed in to change notification settings - Fork 43
vendor: update vendoring #121
vendor: update vendoring #121
Conversation
I got pretty confused by the vendoring and the changes in kubernetes but it now compiles and seems to work. |
@iaguis why using rkt v1.25.0 instead of the last release? |
I chose that kinda arbitrarily because that's what k8s vendors. Although I see not that it's only us (rktlet) that use it. I'll change it to the last release and also do |
Travis fails because |
3950aae
to
5ec404c
Compare
It's green now :) |
It compiles fine for me. A couple of changes in the rkt git repository were missed:
After the changes mentioned above, LGTM. |
It now uses: * Kubernetes v1.7.5 * rkt v1.25.0 * go-systemd v15 I had to explicitly list some dependencies to pin them to a version * client-go v4.0.0 * cni v0.3.0 (what rkt uses) I also changed some depencency revisions to match what Kubernetes has. Finally, since I was getting compilation errors similar to the ones mentioned in kelseyhightower/kubernetes-initializer-tutorial#4, I set the version of k8s.io/apimachinery to like mentioned there.
It's now github.com/rkt/rkt
5ec404c
to
f6e0c4e
Compare
Thanks! I fixed those and also removed |
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
It now uses:
I had to explicitly list some dependencies to pin them to a version
I also changed some depencency revisions to match what Kubernetes has.
Finally, since I was getting compilation errors similar to the ones
mentioned in kelseyhightower/kubernetes-initializer-tutorial#4,
I set the version of k8s.io/apimachinery like mentioned there.