Skip to content
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

Can't run Allocation example with Go 1.16 #2024

Closed
markmandel opened this issue Mar 11, 2021 · 8 comments · Fixed by #2167
Closed

Can't run Allocation example with Go 1.16 #2024

markmandel opened this issue Mar 11, 2021 · 8 comments · Fixed by #2167
Labels
kind/bug These are bugs.
Milestone

Comments

@markmandel
Copy link
Member

What happened:

Attempting to run the allocation service sample with Go 1.16 results in the following error:

$ go run main.go --ip 35.232.217.29 --namespace default --key /home/markmandel/cloud-game-servers-examples/terraform-4-cluster/allocation/certs/gke_gcgs-beta-demo_us-central1-b_game-cluster-us-2/client.key --cert /home/markmandel/cloud-game-servers-examples/terraform-4-cluster/allocation/certs/gke_gcgs-beta-demo_us-central1-b_game-cluster-us-2/client.crt --cacert /home/markmandel/cloud-game-servers-examples/terraform-4-cluster/allocation/certs/gke_gcgs-beta-demo_us-central1-b_game-cluster-us-2/ca.crt --multicluster truepanic: rpc error: code = Unavailable desc = all SubConns are in TransientFailure, latest connection error: connection error: desc = "transport: authentication handshake failed: x509: cannot validate certificate for 35.232.217.29 because it doesn't contain any IP SANs"
goroutine 1 [running]:main.main()        /home/markmandel/agones/examples/allocator-client/main.go:76 +0x81cexit status 2
$ go version
go version go1.16 linux/amd64

What you expected to happen:

It to successfully allocate.

How to reproduce it (as minimally and precisely as possible):

Follow https://agones.dev/site/docs/advanced/allocator-service/ with openssl certificates, and then attempt to allocate.

Anything else we need to know?:

Not sure of workaround at this stage.

Environment:

  • Agones version: 1.12.0
  • Kubernetes version (use kubectl version): 1.17.0
  • Cloud provider or hardware configuration: GKE
  • Install method (yaml/helm): Helm
  • Troubleshooting guide log(s): Above
  • Others: N/A
@markmandel markmandel added the kind/bug These are bugs. label Mar 11, 2021
@markmandel
Copy link
Member Author

@pooneh-m something you should probably be aware of.

@markmandel
Copy link
Member Author

Workaround!

export GODEBUG=x509ignoreCN=0 before running the command.

@pooneh-m
Copy link
Contributor

pooneh-m commented Mar 11, 2021

The error says: "because it doesn't contain any IP SANs".
Have you installed Agones using helm config agones.allocator.http.loadBalancerIP?

@pooneh-m
Copy link
Contributor

It is mentioned in https://agones.dev/site/docs/advanced/allocator-service/#server-tls-certificate:

EXTERNAL_IP=$(kubectl get services agones-allocator -n agones-system -o jsonpath='{.status.loadBalancer.ingress[0].ip}')
helm upgrade --install --wait \
   --set agones.allocator.http.loadBalancerIP=${EXTERNAL_IP} 

@highlyunavailable
Copy link
Contributor

I ran into an issue like this too - I'm not sure if it's a Windows Docker Desktop issue or something else, but that loadbalancer IP is always null for me and I have to set EXTERNAL_IP to the string "localhost" instead.

@markmandel
Copy link
Member Author

It is mentioned in https://agones.dev/site/docs/advanced/allocator-service/#server-tls-certificate:

EXTERNAL_IP=$(kubectl get services agones-allocator -n agones-system -o jsonpath='{.status.loadBalancer.ingress[0].ip}')
helm upgrade --install --wait \
   --set agones.allocator.http.loadBalancerIP=${EXTERNAL_IP} 

Does that mean we should remove the other instructions, and make this the one true way?

@pooneh-m
Copy link
Contributor

We can break it into two sections of quick start and security considerations.

@roberthbailey roberthbailey added this to the 1.16.0 milestone Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug These are bugs.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants