-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[release/1.2 backport] Fix CI due to Golang 1.10.6 / 1.11.3 regressions (workaround) #2881
[release/1.2 backport] Fix CI due to Golang 1.10.6 / 1.11.3 regressions (workaround) #2881
Conversation
Attempt to fix CI is failing due to a regression in Go 1.10.6 / 1.11.3 (see golang/go#29241) ``` package github.com/containernetworking/plugins/...: github.com/containernetworking/plugins/...: invalid import path: malformed import path "github.com/containernetworking/plugins/...": double dot ``` Signed-off-by: Sebastiaan van Stijn <github@gone.nl> (cherry picked from commit 52de371) Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
ping @crosbymichael @andrewhsu @estesp PTAL |
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
looks like something failed on the appveyor build:
even though the pr to master had all green |
Interesting. Can't see the full output from my phone, but can have a look when I'm at my computer |
Missed that you posted a link 😅 Looks like that's the failure ? Could be a flaky not sure. Don't suspect it's related to my changes |
Codecov Report
@@ Coverage Diff @@
## release/1.2 #2881 +/- ##
===============================================
- Coverage 47.41% 43.74% -3.68%
===============================================
Files 91 100 +9
Lines 8410 10728 +2318
===============================================
+ Hits 3988 4693 +705
- Misses 3699 5305 +1606
- Partials 723 730 +7
Continue to review full report at Codecov.
|
LGTM |
Backport of #2879 for the 1.2 release branch
Attempt to fix CI is failing due to a regression in Go 1.10.6 / 1.11.3 (see golang/go#29241)
Signed-off-by: Sebastiaan van Stijn github@gone.nl
(cherry picked from commit 52de371)
Signed-off-by: Sebastiaan van Stijn github@gone.nl