Skip to content

Commit

Permalink
Upgraded demo to use TF 0.12
Browse files Browse the repository at this point in the history
  • Loading branch information
wripley committed Jul 15, 2019
1 parent b4e2e44 commit 2b7d550
Show file tree
Hide file tree
Showing 5 changed files with 21 additions and 15 deletions.
2 changes: 1 addition & 1 deletion Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ metadata:
spec:
containers:
- name: ${containerName}
image: gcr.io/pso-helmsman-cicd/jenkins-k8s-node:${env.CONTAINER_VERSION}
image: gcr.io/pso-helmsman-cicd/jenkins-k8s-node:jenkins-cicd_images_builder-185
command: ['cat']
tty: true
volumeMounts:
Expand Down
18 changes: 10 additions & 8 deletions terraform/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,11 @@ See the License for the specific language governing permissions and
limitations under the License.
*/


// Provides access to available Google Container Engine versions in a zone for a given project.
// https://www.terraform.io/docs/providers/google/d/google_container_engine_versions.html
data "google_container_engine_versions" "on-prem" {
zone = "${var.zone}"
project = "${var.project}"
zone = var.zone
project = var.project
}

// https://www.terraform.io/docs/providers/google/d/google_container_cluster.html
Expand All @@ -28,9 +27,9 @@ data "google_container_engine_versions" "on-prem" {
// Create the GKE Cluster
resource "google_container_cluster" "primary" {
name = "tracing-demo-space"
zone = "${var.zone}"
zone = var.zone
initial_node_count = 1
min_master_version = "${data.google_container_engine_versions.on-prem.latest_master_version}"
min_master_version = data.google_container_engine_versions.on-prem.latest_master_version

// Scopes were a pre-IAM method of giving instances API access
// They are still around we need to give our cluster nodes
Expand All @@ -46,6 +45,8 @@ resource "google_container_cluster" "primary" {
]
}

// Here we use gcloud to gather authentication information about our new cluster and write that
// information to kubectls config file
// Here we use gcloud to gather authentication information about our new cluster and write that
// information to kubectls config file
provisioner "local-exec" {
Expand All @@ -62,13 +63,14 @@ resource "google_pubsub_topic" "tracing-demo-topic" {
// You need a subscription to pull messages from a topic
resource "google_pubsub_subscription" "tracing-demo-subscription" {
name = "tracing-demo-cli"
topic = "${google_pubsub_topic.tracing-demo-topic.name}"
topic = google_pubsub_topic.tracing-demo-topic.name
}

output "cluster_name" {
value = "${google_container_cluster.primary.name}"
value = google_container_cluster.primary.name
}

output "primary_zone" {
value = "${google_container_cluster.primary.zone}"
value = google_container_cluster.primary.zone
}

6 changes: 3 additions & 3 deletions terraform/provider.tf
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,9 @@ See the License for the specific language governing permissions and
limitations under the License.
*/


// Identifies allowable version range for Terraform Google Provider
provider "google" {
project = "${var.project}"
version = "~> 1.13"
project = var.project
version = "~> 2.5"
}

6 changes: 3 additions & 3 deletions terraform/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -14,13 +14,13 @@ See the License for the specific language governing permissions and
limitations under the License.
*/


variable "zone" {
description = "The zone in which to create the Kubernetes cluster. Must match the region"
type = "string"
type = string
}

variable "project" {
description = "the project for this network"
type = "string"
type = string
}

4 changes: 4 additions & 0 deletions terraform/versions.tf
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@

terraform {
required_version = ">= 0.12"
}

0 comments on commit 2b7d550

Please sign in to comment.