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
Kubernetes v0.19.0 #167
Kubernetes v0.19.0 #167
Conversation
/hold |
/assign @adambkaplan /cc @gabemontero, @coreydaley |
the org gets rather precise when it comes to the exact commit message @otaviof for these gomod/vendor changes take a look at the commit message from either my samples or @coreydaley 's OCM PRs and try to follow that pattern |
/test e2e-aws |
fdecc26
to
e9fcb0b
Compare
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.
A nit on the commit history - when dependency bumps force changes in our code, we always split the PR into two commits:
- The
bump(*)
commit with the changes to go.mod, go.sum, and the vendored dependencies - A follow-up commit with the changes needed to get the application to compile/function.
This makes reviews simpler to follow. Please split the changes not related to vendoring on go.mod updates into a separate commit.
) | ||
|
||
replace ( | ||
github.com/containerd/containerd => github.com/containerd/containerd v1.3.2 | ||
github.com/docker/docker => github.com/openshift/moby-moby v1.4.2-0.20190308215630-da810a85109d | ||
github.com/docker/libnetwork => github.com/docker/libnetwork v0.8.0-dev.2.0.20171107005402-dcf79f8c7fcb | ||
github.com/godbus/dbus => github.com/godbus/dbus v0.0.0-20181101234600-2ff6f7ffd60f | ||
// testing changes | ||
github.com/opencontainers/runc => github.com/opencontainers/runc v1.0.0-rc90 | ||
github.com/opencontainers/runtime-spec => github.com/opencontainers/runtime-spec v1.0.1 |
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.
Is this replace necessary?
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.
I'm removing this replace block, and running go mod tidy
. As you can see it skewed runc
and runtime-spec
a bit. Should we keep with a replace statement instead?
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.
I end up adding the replace statements again, due the following errors:
$ make build
hack/build.sh
# github.com/opencontainers/runtime-tools/generate
vendor/github.com/opencontainers/runtime-tools/generate/config.go:184:13: g.Config.VM undefined (type *specs.Spec has no field or method VM)
vendor/github.com/opencontainers/runtime-tools/generate/config.go:185:11: g.Config.VM undefined (type *specs.Spec has no field or method VM)
vendor/github.com/opencontainers/runtime-tools/generate/config.go:185:18: undefined: specs.VM
vendor/github.com/opencontainers/runtime-tools/generate/config.go:191:14: g.Config.VM undefined (type *specs.Spec has no field or method VM)
vendor/github.com/opencontainers/runtime-tools/generate/config.go:192:11: g.Config.VM undefined (type *specs.Spec has no field or method VM)
vendor/github.com/opencontainers/runtime-tools/generate/config.go:192:28: undefined: specs.VMHypervisor
vendor/github.com/opencontainers/runtime-tools/generate/config.go:198:14: g.Config.VM undefined (type *specs.Spec has no field or method VM)
vendor/github.com/opencontainers/runtime-tools/generate/config.go:199:11: g.Config.VM undefined (type *specs.Spec has no field or method VM)
vendor/github.com/opencontainers/runtime-tools/generate/config.go:199:24: undefined: specs.VMKernel
vendor/github.com/opencontainers/runtime-tools/generate/config.go:205:14: g.Config.VM undefined (type *specs.Spec has no field or method VM)
vendor/github.com/opencontainers/runtime-tools/generate/config.go:205:14: too many errors
make: *** [Makefile:11: build] Error 2
2a3502a
to
e36cfae
Compare
/retest |
4 similar comments
/retest |
/retest |
/retest |
/retest |
/approve |
/hold cancel |
/test e2e-aws |
/test e2e-aws |
1 similar comment
/test e2e-aws |
/retest |
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adambkaplan, otaviof The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Please review the full test history for this PR and help us cut down flakes. |
6 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
6 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
Upgrading Go modules and replacement statements to use
v0.19.0-rc.3
.