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

Remove the custom kubectl commands from quickstarts #556

Merged
merged 1 commit into from
Feb 11, 2019

Conversation

hpandeycodeit
Copy link
Contributor

Issue #521

Replace custom kubectl commands
kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports

with kubectl get gs

```

For a fleet of 2 replicas, you should see 2 Game Servers with a Status of Ready:
```
NAME STATUS IP PORT
NAME STATUS IP PORT
simple-udp-s2snf-765bc Ready 35.231.204.26 [map[name:default port:7260]]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm 99% sure the output will change too - i.e. the port only has the first value displayed, rather than the map - so we should adjust those too 👍

But this is awesome!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@markmandel yes you are right. I have updated the output. Thanks for pointing it out :)

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 26bfa174-87ed-4de6-a9ca-01434aade8ae

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website will exist for the next 30 days:

To install this version:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-298e64a

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 28d281fb-c652-4351-8655-5ffdaf44191a

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website will exist for the next 30 days:

To install this version:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-653d70e

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: d3641fa7-6159-465a-a5e5-5ce5a48b9180

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 908b5b0f-7c67-4051-ae54-8ec3db74004b

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website will exist for the next 30 days:

To install this version:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-107091d

Copy link
Member

@markmandel markmandel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I found some more outputs that need updating (inline)

Also some outputs on the webhook fleet autoscaler:
https://107091d-dot-preview-dot-agones-images.appspot.com/site/docs/getting-started/create-webhook-fleetautoscaler/

Thanks for doing all of these! 🍰

@@ -134,7 +134,7 @@ You might also be interested to see the `Events` section, which outlines when va
Let's retrieve the IP address and the allocated port of your Game Server :

```
kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports
kubectl get gs
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -339,7 +339,7 @@ allocated a `GameServer` out of the fleet, and you can now connect your players
A handy trick for checking to see how many `GameServers` you have `Allocated` vs `Ready`, run the following:

```
kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports
kubectl get gs
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -150,7 +150,7 @@ Last Scale Time has been updated, and a scaling event has been logged.
Double-check the actual number of game server instances and status by running

```
kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports
kubectl get gs
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@markmandel I have changed in above files. Let me know if you still see any old output in there. Thanks!

@markmandel markmandel added kind/documentation Documentation for Agones kind/cleanup Refactoring code, fixing up documentation, etc labels Feb 5, 2019
@markmandel markmandel added this to the 0.8.0 milestone Feb 5, 2019
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: a1fc1480-32d7-4cc9-8237-5f9ae1962b0b

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website will exist for the next 30 days:

To install this version:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-633b668

@markmandel
Copy link
Member

Found a few more:
https://633b668-dot-preview-dot-agones-images.appspot.com/site/docs/getting-started/create-gameserver/
Just under kubectl get gameservers

https://633b668-dot-preview-dot-agones-images.appspot.com/site/docs/getting-started/create-fleet/
Under kubectl get fleet and You can also see the GameServers that have been created by the Fleet by running kubectl get gameservers, the GameServer will be prefixed by simple-udp.

Also under the section Scale up the Fleet

Also This will get you a list of all the current GameSevers and their Status.State., has the wrong output (nodeName is not there usually)

https://633b668-dot-preview-dot-agones-images.appspot.com/site/docs/getting-started/create-webhook-fleetautoscaler/
This will get you a list of all the current GameSevers and their Status > State. - old format

Double-check the actual number of game server instances and status by running: - too much in the output.

You can run the test site with make site-server if you want to check manually before submitting the PR and looking for review.

@hpandeycodeit
Copy link
Contributor Author

@markmandel corrected some more. I have checked by running make site-server as well.
However for webhook-fleetautoscaler was getting few errors on my minikube so there might still be some discrepency.

Maybe you can double check?

Thanks

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 55e65c1c-089d-426a-938c-ab4a4bc85879

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website will exist for the next 30 days:

To install this version:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-2604a63

Copy link
Member

@markmandel markmandel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just realised I should use my own tools 😊

NAME               STATE     ADDRESS           PORT      NODE       AGE
simple-udp-g7hnv   Ready     192.168.122.220   7493      minikube   5m

I thought "NODE" was only -o wide - I was wrong. We can make this 'minikube' everywhere though.

Also found STATUS instead of state in places.

Sorry to keep bouncing this back to you 😞

@@ -339,18 +339,18 @@ allocated a `GameServer` out of the fleet, and you can now connect your players
A handy trick for checking to see how many `GameServers` you have `Allocated` vs `Ready`, run the following:

```
kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports
kubectl get gs
```

This will get you a list of all the current `GameSevers` and their `Status > State`.

```
NAME STATUS IP PORT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NAME STATUS IP PORT
NAME STATE IP PORT

@@ -424,15 +424,15 @@ kubectl create -f https://raw.githubusercontent.com/GoogleCloudPlatform/agones/{

We should now have four `Ready` `GameServers` and one `Allocated`.

We can check this by running `kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports`.
We can check this by running `kubectl get gs`.

```
NAME STATUS IP PORT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NAME STATUS IP PORT
NAME STATE IP PORT

@@ -150,16 +150,16 @@ Last Scale Time has been updated, and a scaling event has been logged.
Double-check the actual number of game server instances and status by running

```
kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports
kubectl get gs
```

This will get you a list of all the current `GameSevers` and their `Status > State`.

```
NAME STATUS IP PORT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NAME STATUS IP PORT
NAME STATE IP PORT

@@ -223,15 +223,15 @@ might stay a bit in 'Unhealthy' state (and its pod in 'Terminating' until it get
Double-check the actual number of game server instances and status by running

```
kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports
kubectl get gs
```

This will get you a list of all the current `GameSevers` and their `Status > State`.

```
NAME STATUS IP PORT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NAME STATUS IP PORT
NAME STATE IP PORT

@@ -241,17 +241,17 @@ applied live, without interruptions of service.

Run `kubectl edit fleetautoscaler simple-udp-autoscaler` and set the `bufferSize` field to `5`.

Let's look at the list of game servers again. Run `watch kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports`
Let's look at the list of game servers again. Run `watch kubectl get gs`
until you can see that are 5 ready server instances:

```
NAME STATUS IP PORT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NAME STATUS IP PORT
NAME STATE IP PORT

@@ -134,14 +134,14 @@ You might also be interested to see the `Events` section, which outlines when va
Let's retrieve the IP address and the allocated port of your Game Server :

```
kubectl get gs -o=custom-columns=NAME:.metadata.name,STATUS:.status.state,IP:.status.address,PORT:.status.ports
kubectl get gs
```

This should ouput your Game Server IP address and ports, eg:

```
NAME STATUS IP PORT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NAME STATUS IP PORT
NAME STATE IP PORT

@@ -205,10 +205,10 @@ This will get you a list of all the current `GameSevers` and their `Status > Sta

```
NAME STATUS IP PORT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NAME STATUS IP PORT
NAME STATE IP PORT

NAME STATUS IP PORT
simple-udp-s2snf-765bc Ready 35.231.204.26 [map[name:default port:7260]]
simple-udp-s2snf-vf6l8 Ready 35.196.162.169 [map[name:default port:7591]]
NAME STATUS IP PORT
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
NAME STATUS IP PORT
NAME STATE IP PORT

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@markmandel hopefully this time it will be done :)

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 4abdbf6f-7c6e-43b4-943a-cc19e5bb6dfd

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website will exist for the next 30 days:

To install this version:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-f1780fe

@markmandel
Copy link
Member

More stuff
IP => ADDRESS

Also, can see where the IP/Address column doesn't line up with the content (needs an extra space in the column)

@markmandel
Copy link
Member

Gentle bump on this - it's so close - would be awesome to have it for the 0.8.0 release! 👍

@hpandeycodeit
Copy link
Contributor Author

@markmandel I missed the last notification. :(
Let me know if there is still anything pending, I will work on that too :)

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 997d562c-9fd9-4183-9622-d510ac9b8be3

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:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-9793c85

@markmandel
Copy link
Member

This looks perfect 👍 🎆

I just need you to rebase this down to a single commit, and I will approve it, and push it through! 👌
https://github.com/GoogleCloudPlatform/agones/blob/master/CONTRIBUTING.md#submitting-code-via-pull-requests

Updated the output

Updated the output in multiple files

corrected the outputs

Added NODE and AGE in the output

Minor changes to the output
@hpandeycodeit
Copy link
Contributor Author

@markmandel done :)

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: a3edff60-7d1f-4647-8cb3-49c3f45d133a

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:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-f7fa52f

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 1821a88b-a325-4ca8-ab75-badd24a1cead

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:

  • git fetch https://github.com/GoogleCloudPlatform/agones.git pull/556/head:pr_556 && git checkout pr_556
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=0.8.0-fae7c07

Copy link
Member

@markmandel markmandel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@markmandel markmandel merged commit 5fa7cba into googleforgames:master Feb 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cleanup Refactoring code, fixing up documentation, etc kind/documentation Documentation for Agones
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants