-
-
Notifications
You must be signed in to change notification settings - Fork 5.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
go 1.14 #10467
go 1.14 #10467
Conversation
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.
Please update the go.mod version to use latest go std lib.
Need to update Makefile, README.md, .drone.yml and docs with |
Failing due to error described in #10552 (and a few other places) |
Current template that need resolution:
|
Ok, now that template issues have been resolved, it seems that on ARM there are failures when |
#10597 got merged can you update/use goversion var ; |
Signed-off-by: jolheiser <[email protected]>
Resolved conflicts. 👍 |
looks like some kind of deadlock in queues on arm |
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.
Removing my blocking review since this part has been fixed.
Removing my blocking review since this part has been fixed.
Should we be concerned about this? How many of our vendored packages use system calls? |
$ grep '"syscall"' vendor -r | wc -l
161
$ grep '"golang.org/x/sys/unix"' vendor -r | wc -l
11 |
ping LG-TM |
Note: This also increases the minimum supported version of go to 1.12