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
Oc 4.7 kubernetes 1.20.0 beta.2 #467
Oc 4.7 kubernetes 1.20.0 beta.2 #467
Conversation
2088028
to
188f5d1
Compare
/hold |
188f5d1
to
3b0947c
Compare
/hold cancel |
c5fb61c
to
04e2077
Compare
/retest |
Had to patch hack/update-vendor.sh to get rid of: $ ./hack/update-vendor.sh +++ [1119 14:23:06] logfile at /tmp/update-vendor.USbV/update-vendor.log +++ [1119 14:23:06] go.mod: update staging references +++ [1119 14:23:14] go.mod: propagate to staging modules +++ [1119 14:23:15] go.mod: sorting staging modules +++ [1119 14:23:32] go.mod: tidying tsort: /tmp/update-vendor.USbV/tidy_deps.txt: input contains a loop: tsort: k8s.io/api tsort: k8s.io/code-generator tsort: k8s.io/client-go tsort: k8s.io/apimachinery tsort: /tmp/update-vendor.USbV/tidy_deps.txt: input contains a loop: tsort: k8s.io/api tsort: k8s.io/code-generator tsort: k8s.io/client-go tsort: /tmp/update-vendor.USbV/tidy_deps.txt: input contains a loop: tsort: k8s.io/api tsort: k8s.io/code-generator tsort: /tmp/update-vendor.USbV/tidy_deps.txt: input contains a loop: tsort: k8s.io/code-generator tsort: k8s.io/client-go tsort: k8s.io/apimachinery tsort: /tmp/update-vendor.USbV/tidy_deps.txt: input contains a loop: tsort: k8s.io/code-generator tsort: k8s.io/client-go tsort: /tmp/update-vendor.USbV/tidy_deps.txt: input contains a loop: tsort: k8s.io/client-go tsort: k8s.io/apimachinery github.com/openshift/* repos are adding unnecessary deps to staging repos. Patch: diff --git a/hack/update-vendor.sh b/hack/update-vendor.sh index 35b1ef22c49..9afa63a2124 100755 --- a/hack/update-vendor.sh +++ b/hack/update-vendor.sh @@ -191,7 +191,7 @@ for repo in $(kube::util::list_staging_repos); do pushd "staging/src/k8s.io/${repo}" >/dev/null 2>&1 echo "=== propagating to ${repo}" >> "${LOG_FILE}" # copy root go.mod, changing module name - sed "s#module k8s.io/kubernetes#module k8s.io/${repo}#" < "${KUBE_ROOT}/go.mod" > "${KUBE_ROOT}/staging/src/k8s.io/${repo}/go.mod" + cat "${KUBE_ROOT}/go.mod" | grep -v "github.com/openshift/" | sed "s#module k8s.io/kubernetes#module k8s.io/${repo}#" > "${KUBE_ROOT}/staging/src/k8s.io/${repo}/go.mod" # remove `require` directives for staging components (will get re-added as needed by `go list`) kube::util::list_staging_repos | xargs -n 1 -I {} echo "-droprequire k8s.io/{}" | xargs -L 100 go mod edit # rewrite `replace` directives for staging components to point to peer directories
04e2077
to
4144759
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: ingvagabund, soltysh 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 |
99ac8bc
into
openshift:oc-4.7-kubernetes-1.20.0-beta.2
see 1.20 pick spreadsheet
Following https://github.com/openshift/enhancements/blob/master/enhancements/oc/upgrade_kubernetes.md
/assign soltysh