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
bump go version to build kubernetes #2955
Conversation
Signed-off-by: Peter Hunt <pehunt@redhat.com>
another day, another set of CI flakes 😃 |
As it's not included in go installation anymore, and I don't think it's needed anyway Signed-off-by: Peter Hunt <pehunt@redhat.com>
Codecov Report
@@ Coverage Diff @@
## master #2955 +/- ##
=======================================
Coverage 52.45% 52.45%
=======================================
Files 88 88
Lines 7145 7145
=======================================
Hits 3748 3748
Misses 3015 3015
Partials 382 382 |
cri-tools is running into golangci/golangci-lint#659 because it uses v1.17.1. Opening a PR now... |
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
critest* and integration* need kubernetes-sigs/cri-tools#552 |
that includes fixes for a compile error with golangci-lint, and also builds with go 1.13. Signed-off-by: Peter Hunt <pehunt@redhat.com>
I also added a temp cri-tools bump until the 1.13 changes are released. this should get tests to pass. |
d1a64a7
to
4e85ef7
Compare
Signed-off-by: Peter Hunt <pehunt@redhat.com>
4e85ef7
to
8a45293
Compare
finally |
/retest |
2 similar comments
/retest |
/retest |
/retest |
1 similar comment
/retest |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: haircommander, mrunalp, saschagrunert 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 |
1 similar comment
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: haircommander, mrunalp, saschagrunert 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 |
/lgtm |
Signed-off-by: Peter Hunt pehunt@redhat.com