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

Add Go Client example which could create GS #1213

Merged
merged 2 commits into from
Jan 9, 2020

Conversation

aLekSer
Copy link
Collaborator

@aLekSer aLekSer commented Dec 6, 2019

Add image and example job file which could be used as an example
provided by access-api document.

Closes #1212.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 70617dc8-b5f5-43d1-a5d6-c6f2ae8f1a83

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.2.0-012ddeb

site/content/en/docs/Guides/access-api.md Outdated Show resolved Hide resolved
go 1.13

require (
agones.dev/agones v1.1.0
Copy link
Member

Choose a reason for hiding this comment

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

Any thoughts on how we keep this up to date as we keep releasing Agones versions?

Copy link
Member

Choose a reason for hiding this comment

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

Should this now be 1.2.0 ?

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.

Coming back around to this - had a couple of small cleanup things.

Still not sure what to do with the hardcoded dependency on the previous versions of Agones as we keep doing releases. WDYT we should there? Just make sure we update it occasionally? (the agones.dev interface shouldn't change drastically from version to version?)

site/content/en/docs/Guides/access-api.md Outdated Show resolved Hide resolved
@@ -108,6 +117,18 @@ func main() {
fmt.Printf("New game servers' name is: %s", newGS.ObjectMeta.Name)
}
```
{{% feature publishVersion="1.2.0" %}}
Copy link
Member

Choose a reason for hiding this comment

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

Same here, not sure if we need the feature shortcode here. It's not a new feature, just a new example. WDYT?

Copy link
Collaborator Author

@aLekSer aLekSer Dec 20, 2019

Choose a reason for hiding this comment

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

I am using {{< release-branch >}} in lines below, so batch.yaml file would not be accessible from 1.2 release branch

go 1.13

require (
agones.dev/agones v1.1.0
Copy link
Member

Choose a reason for hiding this comment

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

Should this now be 1.2.0 ?

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: f34e4685-e401-48f5-b1db-b2fe42723643

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

@aLekSer
Copy link
Collaborator Author

aLekSer commented Dec 20, 2019

WDYT we should there? Just make sure we update it occasionally? (the agones.dev interface shouldn't change drastically from version to version?)

I think we can use latest there and also need to check that example image is in the registry.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: d6a53e24-f367-48e8-9c40-22c2fbdd1d80

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-f0c199a

@aLekSer
Copy link
Collaborator Author

aLekSer commented Dec 20, 2019

Could not find an option to use latest version in go.mod. Anyway we definitely need to update

	k8s.io/apimachinery v0.0.0-20191004074956-01f8b7d1121a // kubernetes-1.13.12

and other project dependencies, otherwise we could not build the example client as in #1212.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 71840cb0-1ca9-45ff-87fd-2cbfb7a6fe05

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-690e8ce

@aLekSer aLekSer marked this pull request as ready for review December 20, 2019 15:53
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 51253337-4edd-41f5-b95e-093eeae78898

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-9c2343a

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 6fbe59d2-1265-4eef-b94f-0e39af310237

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-7274121

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 2af23366-cff0-401a-8984-f8871a1cb39b

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-bb737eb

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: f087f0ae-b5c4-44b2-bfa9-d1a201b93d88

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-74a795c

@markmandel
Copy link
Member

Looks good! Happy to commit, but would like to add one more small thing: Add this to the list of examples that are checked to make sure they exist in the registry when we do releases.

Need two things to add it:
Add an entry here of example-image-test.crd-client: https://github.com/googleforgames/agones/blob/master/build/includes/examples.mk#L29

Add a gcr-check in your Makefile for the example, you can copy the example here:
https://github.com/googleforgames/agones/blob/master/examples/simple-udp/Makefile#L45

Also, let me know if you would like me to build this and push it up to gcr.io for you.

@markmandel
Copy link
Member

Gentle bump on this - would be good to have this in next week's RC release.

@aLekSer aLekSer force-pushed the example/client-set branch from 74a795c to 85fb57c Compare January 8, 2020 12:16
@aLekSer
Copy link
Collaborator Author

aLekSer commented Jan 8, 2020

Also, let me know if you would like me to build this and push it up to gcr.io for you.

Yes, please, now we need to have this image in a registry.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 76d11d54-c59d-4141-8f55-e240401e2dd3

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

@aLekSer
Copy link
Collaborator Author

aLekSer commented Jan 8, 2020

Build failure:

Already have image (with digest): gcr.io/cloud-builders/gcloud
Services to deploy:

descriptor: [/workspace/go/src/site/.app.yaml]
source: [/workspace/go/src/site]
target project: [agones-images]
target service: [preview]
target version: [85fb57c]
target url: [https://85fb57c-dot-preview-dot-agones-images.appspot.com]


(add --promote if you also want to make this service available from
[https://preview-dot-agones-images.appspot.com])

Do you want to continue (Y/n)?
Beginning deployment of service [preview]...
#============================================================#
#= Uploading 86 files to Google Cloud Storage =#
#============================================================#
File upload done.
Updating service [preview]...
...........................................................................................................................................................................................................................................failed.
ERROR: (gcloud.app.deploy) Error Response: [13] Cloud build 106451eb-f18d-4bbc-bb13-063ab23a3d5b status: INTERNAL_ERROR.
Check the build log for errors: https://console.cloud.google.com/gcr/builds/106451eb-f18d-4bbc-bb13-063ab23a3d5b?project=258182270954
Build error details not available.

@aLekSer aLekSer force-pushed the example/client-set branch from 85fb57c to ea66f7d Compare January 8, 2020 17:13
Add image and example job file which could be used as an example
provided by access-api document.
@aLekSer aLekSer force-pushed the example/client-set branch from ea66f7d to 44883ff Compare January 8, 2020 17:15
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 578455b0-e27e-46e8-8721-c471e947aeaf

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-44883ff

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 8eb53d02-229a-4ef7-8f79-3d75be71b3e2

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-ea66f7d

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.

👍

@google-oss-robot
Copy link

New changes are detected. LGTM label has been removed.

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aLekSer, markmandel

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aLekSer, markmandel

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 9bd1f84c-63d7-4218-bb8a-47321b5615a4

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/1213/head:pr_1213 && git checkout pr_1213
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.3.0-db4d3bf

@markmandel markmandel merged commit 2d0ec1d into googleforgames:master Jan 9, 2020
@markmandel markmandel added area/examples Examples. Usually found in the `examples` directory kind/feature New features for Agones labels Jan 9, 2020
@markmandel
Copy link
Member

and pushed to gcr!

@markmandel markmandel added this to the 1.3.0 milestone Jan 10, 2020
@aLekSer aLekSer deleted the example/client-set branch January 13, 2020 15:47
ilkercelikyilmaz pushed a commit to ilkercelikyilmaz/agones that referenced this pull request Oct 23, 2020
Add image and example job file which could be used as an example
provided by access-api document.

Co-authored-by: Mark Mandel <mark.mandel@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved area/examples Examples. Usually found in the `examples` directory kind/feature New features for Agones size/XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

devloping application to Access Agones via the Kubernetes API with compile error
4 participants