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

Document agones-allocator and multi-cluster allocation #1155

Merged
merged 1 commit into from
Oct 29, 2019

Conversation

pooneh-m
Copy link
Contributor

No description provided.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: d1cbb1ab-1d64-4c64-9f9f-3cedb67df210

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

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 8afe0aca-f362-4105-9660-1f888dbe76cb

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

@roberthbailey
Copy link
Member

Because the builds are failing I can't see a nice version of the markdown rendered in docsy.

This change seems unrelated to the build failures (as does #1119 which also failed last time I rebased) so I think something is going on with the CI system right now....

@roberthbailey
Copy link
Member

I found this in the build logs:

Error: Error building site: logged 2 error(s)
make[3]: *** [site-static] Error 255
docker run --rm -v /workspace/build//.config/gcloud:/root/.config/gcloud -v ~/.kube/:/root/.kube -v ~/.helm:/root/.helm -v /workspace:/go/src/agones.dev/agones -v /workspace/build//.gomod:/go/pkg/mod -v /workspace/build//.gocache:/root/.cache/go-build --workdir=/go/src/agones.dev/agones/site  agones-build:418360257b rm -r ./public
includes/website.mk:35: recipe for target 'site-static' failed

If you run make site-static locally does it succeed?

@pooneh-m
Copy link
Contributor Author

pooneh-m commented Oct 28, 2019

There was a reference to an image and the reference could not be resolved. I removed the reference as it is not needed. I'll add later, when I figured out the issue.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 2f4d9507-9d98-4368-8e94-16caa566dee1

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

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: f5d5f767-df5e-4dce-a510-03573e2338c2

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/1155/head:pr_1155 && git checkout pr_1155
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.1.0-12da9f2

@roberthbailey
Copy link
Member

There was a reference to an image and the reference could not be resolved.

If you add a property data-proofer-ignore to the image it should be possible to leave it in and have the tests pass.

@pooneh-m
Copy link
Contributor Author

There was a reference to an image and the reference could not be resolved.

If you add a property data-proofer-ignore to the image it should be possible to leave it in and have the tests pass.

Thanks. I'll leave it out for now. I hope to get the documentation reviewed by release to merge it in in-time. @roberthbailey will you have time for the review?

site/content/en/docs/Advanced/allocator-service.md Outdated Show resolved Hide resolved
title: "Allocator Service"
date: 2019-10-25T05:45:05Z
description: >
Agones provides mTLS based allocator service that is accessible from outside the cluster using a load balancer. The service is deployed and scales independent to Agones controller.
Copy link
Member

Choose a reason for hiding this comment

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

Is the service always deployed? And an ELB always created by default using helm / yaml? Are there flags to turn it on / off?

Copy link
Contributor Author

@pooneh-m pooneh-m Oct 28, 2019

Choose a reason for hiding this comment

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

It is by default deployed. Because the service is of type LoadBalancer, the cloud provider or GKE on-prem provides the LB.

site/content/en/docs/Advanced/allocator-service.md Outdated Show resolved Hide resolved
site/content/en/docs/Advanced/allocator-service.md Outdated Show resolved Hide resolved
site/content/en/docs/Advanced/multi-cluster-allocation.md Outdated Show resolved Hide resolved
site/content/en/docs/Advanced/multi-cluster-allocation.md Outdated Show resolved Hide resolved
site/content/en/docs/Advanced/multi-cluster-allocation.md Outdated Show resolved Hide resolved
site/content/en/docs/Advanced/multi-cluster-allocation.md Outdated Show resolved Hide resolved
@pooneh-m
Copy link
Contributor Author

Thanks @roberthbailey for your detailed review. Your feedback is very helpful.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: c3666f02-6e80-42b8-8bc8-249a626eef66

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/1155/head:pr_1155 && git checkout pr_1155
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.1.0-e309c83

Copy link
Member

@roberthbailey roberthbailey left a comment

Choose a reason for hiding this comment

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

Are there flags to turn it on / off?

We've missed the release train for 1.1, but I think that we should make it possible to disable this in the helm configuration (since it's an alpha feature). Unless we plan to promote it to beta before 1.2, I think doing that before 1.2 would be really nice.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 8c3753f4-ba60-4c30-bc08-6c80fdb2b5b2

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/1155/head:pr_1155 && git checkout pr_1155
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.1.0-4e728fd

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 1664ed0f-e7d0-403b-97b9-50d04caa9568

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

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: a8c1db00-476e-4f68-9db5-cbb305fe386e

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/1155/head:pr_1155 && git checkout pr_1155
  • helm install install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.1.0-900b1bf

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pooneh-m, roberthbailey

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:
  • OWNERS [pooneh-m,roberthbailey]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@roberthbailey roberthbailey merged commit 25e4705 into googleforgames:master Oct 29, 2019
@markmandel markmandel added this to the 1.1.0 milestone Oct 29, 2019
@markmandel markmandel added kind/documentation Documentation for Agones kind/feature New features for Agones labels Oct 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved kind/documentation Documentation for Agones kind/feature New features for Agones lgtm size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants