Closed aklinkert closed 8 years ago
Ah - sorry... we're on a forked version of k8s while https://github.com/kubernetes/kubernetes/pull/32581 merges.
The submodules aren't actually needed; we copy them into vendor. We ship them so that there is some visibility into where vendor comes from, and so that you can easily update them if you need to.
I think that the go get should still have worked, and you should be able to proceed.
go get
clones submodules, but you actually don't have to. Indeed, it would be much faster not to! If you want to side-step the issue (particularly if you're running in CI), it can be easier and faster to mkdir -p ${GOPATH}/src/k8s.io; pushd ${GOPATH}/src/k8s.io; git clone https://github.com/kubernetes/kops.git; popd
I thought this would be a short-lived problem - if that turns out not to be the case I'll figure out what to do here...
Yep @justinsb , this works. At least as a short term solution. Thanks!
Nonetheless I guess we should leave this issue open unless the underlying issue is fixed, right?
cc @cbrand
The upstream PR merged, and so I updated the dependencies in #497.
Should now build cleanly again with go get
- thanks for reporting. Despite being fixed, I personally would leave the git pull
behaviour in place because it's a little more efficient, but that's up to you!
(Also, note that you have to use go get k8s.io/kops
otherwise the import names will be wrong)
Thanks for the info and fixing the issue! :)
Hey folks,
since this week we're unable to install kops any more, as well on Circle.CI as locally. This is the error trace:
Any ideas what's happening there? Thanks! 💪