-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Switch off KUBE_BUILD_CONFORMANCE to diagnose a ci-cross issue #69817
Switch off KUBE_BUILD_CONFORMANCE to diagnose a ci-cross issue #69817
Conversation
Looking at job results, there seems to be some flakiness, let's see if the PR that added the conformance image is the one to blame. https://prow.k8s.io/?type=periodic&job=ci-kubernetes-cross-build https://k8s-testgrid.appspot.com/sig-release-misc#cross-build&graph-metrics=test-duration-minutes&width=20 Change-Id: Ic26431a0e262100dec0444279d94277482972b07
/assign @spiffxp |
/assign @cblecker |
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
thanks @dims, I'll start another alpha.1 build once this has merged.
/lgtm |
/approve Looks like this is a new option that was added in #69368, so I'm comfortable turning it off. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cblecker, dims 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 |
/test pull-kubernetes-integration |
It looks like the root cause is that both the hyperkube and conformance image Makefiles are trying to run |
ah wow! that's great bit of analysis @ixdy |
@ixdy we could use flock? (example https://www.msully.net/blog/2015/02/25/parallelizing-compiles-without-parallelizing-linking-using-make/ ) |
|
What this PR does / why we need it:
Looking at job results, there seems to be some flakiness, let's see if
the PR that added the conformance image is the one to blame.
https://prow.k8s.io/?type=periodic&job=ci-kubernetes-cross-build
https://k8s-testgrid.appspot.com/sig-release-misc#cross-build&graph-metrics=test-duration-minutes&width=20
Change-Id: Ic26431a0e262100dec0444279d94277482972b07
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Release note: