-
Notifications
You must be signed in to change notification settings - Fork 820
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
Update the default gRPC and HTTP ports for the sdkserver for the 1.2.0 release #1180
Update the default gRPC and HTTP ports for the sdkserver for the 1.2.0 release #1180
Conversation
This needs to have a big release note with 1.2. In the 1.1 release we warned folks that this was coming and made it possible to configure the port (which makes it easy to keep using the old port as long as you are using the 1.1 SDK) but we need to mention it again. I've labeled this with |
Build Succeeded 👏 Build Id: 76cf05f9-819d-47ca-bd47-aed784fb18b1 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.
Node.js parts OK!
Overall this looks good - do we want to add some documentation on the GameServer reference (I can't think of a better place) that explicitly shows how to setup the ports for the old version of the SDK? |
Good catch. I should update the comments in https://github.com/googleforgames/agones/blob/master/examples/gameserver.yaml and https://agones.dev/site/docs/reference/gameserver/. Both of those show examples of how to set the port, and we can point to them from the release notes. |
2fbecec
to
7e25f28
Compare
I've updated those two files. |
Build Succeeded 👏 Build Id: c35cf9f5-3c4b-4e6e-86af-61a0fa5dda0b 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:
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: markmandel, roberthbailey, steven-supersolid The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
New changes are detected. LGTM label has been removed. |
Build Failed 😱 Build Id: ca910839-7451-43e5-b9e4-2a070b96cbd5 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
88c6895
to
63e5580
Compare
|
Looks like the helm documentation has been rearranged (maybe with the release of helm 3?). |
Just submitted #1187 👍 |
It looks like the old docs are available at https://v2.helm.sh/docs/helm/#helm-upgrade. I've added a commit to point to that page for now (since we are still using helm 2). |
Your commit is the same as my new commit. :) |
We can merge yours and I'll rebase. |
Build Failed 😱 Build Id: 4f8694cc-3028-4f14-9b2b-303ef040e686 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
17e59c6
to
f6b7712
Compare
Rebased. Tests should pass now. |
Build Succeeded 👏 Build Id: ec8782f3-5115-4e32-a28f-e68de1f72a44 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:
|
Build Failed 😱 Build Id: 540948d0-fe73-43dc-a970-999f8dea479d To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
That's a new one? Hitting restart. |
Build Succeeded 👏 Build Id: 49f37b88-ca8f-46d6-a67f-b63d6c787bc7 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:
|
Build Succeeded 👏 Build Id: c41aa006-c8d0-4507-9210-98e5fba85fce 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:
|
Fixes #851.