-
Notifications
You must be signed in to change notification settings - Fork 810
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
Add explicit local version of agones in go.mod #706
Conversation
Update go.mod which is used by ahmetb/gen-crd-api-reference-docs utility. Now go does not upload the file from agones.dev/agones repo instead uses local version of sources.
Build Succeeded 👏 Build Id: e1391042-57df-40b0-970e-8c551d43a1df The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
LGTM, but sanity check - @heartrobotninja you see any issues? |
Would it make sense to make the agones import path absolute instead of relative? |
The |
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! 🚒
Build Failed 😱 Build Id: 74741215-f4d0-4eaf-aec3-6181bb562284 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: 6f65b2ed-5cca-4357-8738-d8835919ea3c The following development artifacts have been built, and will exist for the next 30 days:
A preview of the website (the last 30 builds are retained): To install this version:
|
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.
So I'm going to block this.. since it changes go.mod... (and it feels like we should commit the change to go.mod?) and then if I try and run:
➜ build git:(master) ✗ make build-controller-binary
mkdir -p ~/.kube/
mkdir -p /home/markmandel/workspace/agones/src/agones.dev/agones/build//.gocache
mkdir -p /home/markmandel/workspace/agones/src/agones.dev/agones/build//.config/gcloud
mkdir -p ~/.helm
docker run --rm -e "CGO_ENABLED=0" -e "GO111MODULE=on" -w /go/src/agones.dev/agones -v /home/markmandel/workspace/agones/src/agones.dev/agones/build//.config/gcloud:/root/.config/gcloud -v ~/.kube/:/root/.kube -v ~/.helm:/root/.helm -v /home/markmandel/workspace/agones/src/agones.dev/agones:/go/src/agones.dev/agones -v /home/markmandel/workspace/agones/src/agones.dev/agones/build//.gocache:/root/.cache/go-build -v /home/markmandel/workspace/agones/src/agones.dev/agones/build//.gomod:/go/pkg/mod agones-build:4f3bdaf670 go build -mod=vendor \
-tags none -o /go/src/agones.dev/agones/cmd/controller/bin/controller \
-ldflags "-X agones.dev/agones/pkg.Version=0.10.0-97fdbf6" -installsuffix cgo agones.dev/agones/cmd/controller
go: errors parsing go.mod:
/go/src/agones.dev/agones/go.mod:59: invalid module version latest: module lookup disabled by -mod=vendor
Makefile:285: recipe for target 'build-controller-binary' failed
make: *** [build-controller-binary] Error 1
As you can see, it breaks things. So this is a little scary 😢
I need to investigate this, however it looks strange, cause I make change only in local directory:
|
Oh! I read this wrong. Sorry, entirely my fault. Merging! |
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.
I'm not smart. Sorry! Merging!
The new go mod versioning system could lead to some misunderstandings at first. |
Update go.mod which is used by ahmetb/gen-crd-api-reference-docs
utility. Now go does not upload the file from agones.dev/agones repo
instead uses local version of sources.
Closes #705.
For #682 will generate docs properly.