-
Notifications
You must be signed in to change notification settings - Fork 19
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
[Demo] try go mod #91
Conversation
@ChihChengLiang I am running go 1.11.1 and the repo is in my $GOPATH and I get this error occasionally when running from the CLI and no docker container: Steps to reproduce: Open a new terminal sesh and run
Open another terminal sesh and run:
You can see that we get an error, but if i run it again:
You can see that it worked. I am thinking that it has something to go with the code running on go1.11.1. I will trace through the code and investigate. If either @ChihChengLiang or @mhchia can confirm that it works on a previous version of Go...that might be a good lead. |
It works for me on the first try. I'm using go1.11.1. Note that I've updated my environment with
|
@NIC619 ahhh... interesting. Let me know if you have a fix and I will test it. @ChihChengLiang I'll try your suggestion as well |
@NIC619 your fix seemed to work. |
Closed in favor of #121 |
What was wrong?
This is an attempt to fix #85 by using the Go mod approach.
How was it fixed?
By introducing
go.mod
andgo.sum
, thego mod download
step in thedev.Dockerfile
can cache the dependencies download, so no need togo get
every time on codebase changes.But there's an issue. The containers spun from the newly built images fails randomly when executing
add_peer
commands in Ansible scripts. The error message is:To reproduce this PR, one needs to
$GOPATH
Cute Animal Picture