-
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
Helm support for Terraform #696
Helm support for Terraform #696
Conversation
60c1e02
to
b24921e
Compare
Build Succeeded 👏 Build Id: 8cfb6b56-8150-4760-b25e-878a75d107a1 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: 60b46c96-4272-413c-9b9b-3c7b5c77a44a To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
b24921e
to
17caae7
Compare
Build Failed 😱 Build Id: be0b185a-5d99-456f-a5da-e2542fa07a08 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
17caae7
to
afb7d73
Compare
Build Succeeded 👏 Build Id: aea04b64-9318-4a00-ba74-b0266304df8a 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:
|
1f26549
to
7ba51a8
Compare
Build Failed 😱 Build Id: b4414958-d502-4e66-8ca1-26309816a331 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Failed 😱 Build Id: 31b65a77-cef4-4b03-be9e-ca01cccde7e9 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
f590a34
to
1b05a96
Compare
Build Succeeded 👏 Build Id: 54f76f26-fad9-49f2-9c0a-c734d4ec8bbb 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: 65294f0c-6f24-4b12-9225-c007aa1e7fc5 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:
|
1b05a96
to
7bce733
Compare
Build Failed 😱 Build Id: 14c62b8d-7f66-43bc-9870-7245759383b1 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
c34fa3a
to
063cd3a
Compare
Build Failed 😱 Build Id: a94d7191-c5c4-453e-a218-83c363b5a92f To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: bd213ac5-ce6d-4778-adbb-79e452bfa1e1 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:
|
063cd3a
to
53b68eb
Compare
Build Succeeded 👏 Build Id: 0c02bbaa-d8f8-4506-ae60-2807ccf91da9 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:
|
53b68eb
to
33015ab
Compare
Build Succeeded 👏 Build Id: 3e44dde8-88ee-409a-8e5c-b48a5adc52f1 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:
|
33015ab
to
96c6d54
Compare
Build Succeeded 👏 Build Id: 12ed7db8-5bce-4ed1-9552-b7ba103e42e4 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:
|
7827d0c
to
59e99d0
Compare
Build Succeeded 👏 Build Id: 8acbb7ca-614b-4e4b-810d-a786cdab6962 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: 1afd6873-2a6b-4fef-b197-7ad51650ec6a 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:
|
5c26ffa
to
f936550
Compare
f936550
to
2b297cd
Compare
Build Failed 😱 Build Id: 3476afc9-ee28-454b-a03b-ad18858ac088 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Build Succeeded 👏 Build Id: 1933fa71-cb76-41f4-a59f-6491daf53987 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:
|
2b297cd
to
b48b480
Compare
Build Failed 😱 Build Id: 23f99e1d-e209-496e-88d6-c71eba2abcb2 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
b48b480
to
e9a0f4b
Compare
Build Failed 😱 Build Id: 0687462c-c675-441c-b179-20be7925555a To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
e9a0f4b
to
e1b69b9
Compare
Build Succeeded 👏 Build Id: 52820e77-523b-489e-8a66-6cb31e626302 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.
This is kinda awesome 😄 Just have some niggly things I'd like to double check on. But I'm pretty excited about this!
/cc @Kuqd who has way more terraform experience than I.
build/includes/google-cloud.mk
Outdated
gcloud-terraform-cluster: GCP_CLUSTER_LEGACYABAC ?= false | ||
gcloud-terraform-cluster: GCP_CLUSTER_NODEPOOL_INITIALNODECOUNT ?= 4 | ||
gcloud-terraform-cluster: GCP_CLUSTER_NODEPOOL_MACHINETYPE ?= n1-standard-4 | ||
gcloud-terraform-cluster: AGONES_VERSION ?= 0.9.0 |
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.
I can't think of a better param than this - I just see it going out of date as releases continue forward.
Can we default to '' which will install the latest chart, and then have an optional override?
default= "" | ||
} | ||
variable "agones_version" { | ||
default="0.9.0" |
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.
I'm just hesitant to set the version as default. I already have a long list of things to update on release - this just adds another one. Can we use the helm default of installing the latest, with the ability to override?
@@ -0,0 +1,65 @@ | |||
--- | |||
title: "Install Agones using Terraform" |
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.
Do we want to be clear this is GKE only somehow?
e1b69b9
to
cc6a9fb
Compare
Build Failed 😱 Build Id: e486fa45-d810-4324-8688-34f82725ad58 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
cc6a9fb
to
18584ba
Compare
Hello @markmandel ,
|
Build Failed 😱 Build Id: 5a303fa9-b7f4-4043-ae38-27b58973c4c1 To get permission to view the Cloud Build view, join the agones-discuss Google Group. |
Deploying agones using two options: 1. From repository release charts by specifying version. 2. Local version using Chart.yaml file with values.yaml.
18584ba
to
f6444b3
Compare
Build Succeeded 👏 Build Id: 8fbae0aa-8b7b-4996-b664-65dfbc03784d 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: bffbab42-681a-4f5b-bb89-eb4c9a34ffde 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:
|
Deploying agones using
agones.dev
chart. Version is specified inhelm.tf
config.Two options of installation:
make install
does.This Pull Request adds an ability to install Agones along with creating the cluster using Terraform.
Contains part of #670 Pull Request, so leave it as draft until rebased with new master.
For #657.