Skip to content

Commit

Permalink
fix: disable tracing in services testflight (#112)
Browse files Browse the repository at this point in the history
* fix: move tf var into services.sh script

* chore: remove quotes around boolean var
  • Loading branch information
krtk6160 authored Dec 14, 2022
1 parent d8d5c33 commit 6ea964a
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 5 deletions.
2 changes: 0 additions & 2 deletions ci/gcp.lib.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,8 +51,6 @@ KUBE_CA_CERT: #@ data.values.concourse_tf_kube_ca_cert
KUBE_TOKEN: #@ data.values.concourse_tf_kube_token
TF_VAR_gcp_project: galoy-infra-testflight
TF_VAR_enable_services: false
TF_VAR_enable_tracing: false

BRANCH: #@ data.values.git_branch
#@ end

Expand Down
2 changes: 1 addition & 1 deletion ci/tasks/gcp/services.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,4 +38,4 @@ for i in {1..60}; do
done
set -e

gcloud compute ssh --ssh-key-file=${CI_ROOT}/login.ssh ${bastion_name} --zone=${bastion_zone} -- "cd repo/examples/gcp; export GOOGLE_APPLICATION_CREDENTIALS=\$(pwd)/gcloud-creds.json; echo yes | make initial-services && echo yes | make services"
gcloud compute ssh --ssh-key-file=${CI_ROOT}/login.ssh ${bastion_name} --zone=${bastion_zone} -- "cd repo/examples/gcp; export GOOGLE_APPLICATION_CREDENTIALS=\$(pwd)/gcloud-creds.json; export TF_VAR_enable_tracing=false; echo yes | make initial-services && echo yes | make services"
2 changes: 1 addition & 1 deletion examples/gcp/services/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ variable "letsencrypt_issuer_email" {
}

variable "enable_tracing" {
default = "true"
default = true
}

data "google_client_config" "default" {
Expand Down
2 changes: 1 addition & 1 deletion modules/services/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ variable "letsencrypt_issuer_email" {}
variable "local_deploy" { default = false }
variable "small_footprint" { default = false }
variable "smoketest_cronjob" { default = false }
variable "enable_tracing" { default = "true" }
variable "enable_tracing" { default = true }

locals {
local_deploy = var.local_deploy
Expand Down

0 comments on commit 6ea964a

Please sign in to comment.