Skip to content

Commit 91cc867

Browse files
committed
Cleanup terraform
1 parent c608b35 commit 91cc867

File tree

9 files changed

+2
-36
lines changed

9 files changed

+2
-36
lines changed

install/aws-terraform/main.tf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ module "gitpod" {
5656
# simply setting "{}" does not work as it does not override: https://github.com/helm/helm/issues/5407
5757
certificatesSecret:
5858
secretName: ""
59-
forceHTTPS: ${var.force_https}
6059
6160
installation:
6261
region: ${var.region}

install/aws-terraform/variables.tf

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,6 @@ variable "image_prefix" {
3535
description = "Image prefix for the registry in which the images for the components are hosted"
3636
}
3737

38-
variable "force_https" {
39-
type = bool
40-
default = false
41-
description = "Force the domain to use HTTPS"
42-
}
43-
4438
variable "certbot_enabled" {
4539
type = bool
4640
default = false

install/docker/gitpod-image/entrypoint.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,6 @@ fi
108108
case "$DOMAIN" in
109109
*ip.mygitpod.com)
110110
cat << EOF > /default_values/03_ip_mygitpod_com.yaml
111-
forceHTTPS: true
112111
components:
113112
imageBuilder:
114113
registry:

install/gcp-terraform/environment/installer/main.tf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ data "template_file" "values" {
6262
loadbalancer_ip = google_compute_address.gitpod.address
6363
certbot_enabled = var.certbot_enabled
6464
certbot_email = var.certificate_email
65-
force_https = var.force_https
6665
}
6766
}
6867
module "gitpod" {

install/gcp-terraform/environment/installer/templates/values.tpl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,4 @@ components:
1313
end: 11000
1414

1515
installPodSecurityPolicies: true
16-
forceHTTPS: ${force_https}
1716
authProviders: []

install/gcp-terraform/environment/installer/variables.tf

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,6 @@ variable "image_prefix" {
4141
default = "eu.gcr.io/gitpod-core-dev/build/"
4242
}
4343

44-
variable "force_https" {
45-
type = bool
46-
default = false
47-
}
48-
4944
variable "certbot_enabled" {
5045
type = bool
5146
default = false

install/gcp-terraform/modules/certmanager/templates/values.tpl

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
certificatesSecret:
22
secretName: ${secret_name}
3-
keyName: ${key_name}
4-
chainName: ${chain_name}
5-
fullChainName: ${full_chain_name}
63

74
components:
85
restarter:

install/installer/cmd/aws.go

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -73,10 +73,6 @@ var awsCmd = &cobra.Command{
7373
}
7474
} else if !strings.Contains(domain, "ip.mygitpod.com") {
7575
err = terraform.PersistVariable(tfvarsfn,
76-
terraform.PersistVariableOpts{
77-
Name: "force_https",
78-
Sources: []terraform.VariableValueSource{func(name string, spec terraform.VariableSpec) (value string, ok bool) { return "true", true }},
79-
},
8076
terraform.PersistVariableOpts{
8177
Name: "certbot_enabled",
8278
Sources: []terraform.VariableValueSource{func(name string, spec terraform.VariableSpec) (value string, ok bool) { return "true", true }},
@@ -134,11 +130,7 @@ var awsCmd = &cobra.Command{
134130
},
135131
},
136132
ForceOverwrite: true,
137-
}, terraform.PersistVariableOpts{Name: "force_https", ForceOverwrite: true, Sources: []terraform.VariableValueSource{
138-
func(name string, spec terraform.VariableSpec) (value string, ok bool) {
139-
return "true", true
140-
},
141-
}})
133+
})
142134
if err != nil {
143135
ui.Fatalf("cannot update the \"domain\" terraform variables - please re-run this installer.\n\t%q", err)
144136
}

install/installer/cmd/gcp.go

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -69,10 +69,6 @@ var gcpCmd = &cobra.Command{
6969
}
7070
} else if !strings.Contains(domain, "ip.mygitpod.com") {
7171
err = terraform.PersistVariable(tfvarsfn,
72-
terraform.PersistVariableOpts{
73-
Name: "force_https",
74-
Sources: []terraform.VariableValueSource{func(name string, spec terraform.VariableSpec) (value string, ok bool) { return "true", true }},
75-
},
7672
terraform.PersistVariableOpts{
7773
Name: "certbot_enabled",
7874
Sources: []terraform.VariableValueSource{func(name string, spec terraform.VariableSpec) (value string, ok bool) { return "true", true }},
@@ -121,11 +117,7 @@ var gcpCmd = &cobra.Command{
121117
},
122118
},
123119
ForceOverwrite: true,
124-
}, terraform.PersistVariableOpts{Name: "force_https", ForceOverwrite: true, Sources: []terraform.VariableValueSource{
125-
func(name string, spec terraform.VariableSpec) (value string, ok bool) {
126-
return "true", true
127-
},
128-
}})
120+
})
129121
if err != nil {
130122
ui.Fatalf("cannot update the \"domain\" terraform variables - please re-run this installer:\n\t%q", err)
131123
}

0 commit comments

Comments
 (0)