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
avoid builds on non-linux nodes #22885
avoid builds on non-linux nodes #22885
Conversation
2a551f7
to
36e9e56
Compare
unit test failure k8s flake (rebase related knee jerk assumption) the integration test failures are related to the changes I made...just pushed update |
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.
@gabemontero not sure why you have to reference the staging code - shouldn't the vendored API lib include the new label?
36e9e56
to
b098040
Compare
/assign @adambkaplan @soltysh - I didn't do a super detailed analysis, but some simple grepping saw more references to |
/approve |
upgrade failure was mco related
|
/test e2e-aws-upgrade |
we have passing tests @adambkaplan :-) ... bump / any more comments ? @soltysh is marked away on slack this AM but we could always create a follow up PR for my question to him at #22885 (comment) if this merges before he responds |
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, bparees, gabemontero 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 |
/skip pr |
/skip travis-ci/pr |
/skip continuous-integration/travis-ci/pr |
/retest Please review the full test history for this PR and help us cut down flakes. |
Looking at https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.14.md#deprecations it looks like we should be ok with just the new label, unless you plan to support that in 4.1 in which case both. |
thanks for the confirmation @soltysh And fyi this will not be backported to 4.1 |
https://jira.coreos.com/browse/DEVEXP-341