Skip to content

Commit

Permalink
Remove App Engine backend from GCLB (#2257)
Browse files Browse the repository at this point in the history
This just removes App Engine from the load balancer. I'll manually apply
this on prod after it's merged.

I still need to understand what happens to Datastore if we delete App
Engine entirely.
  • Loading branch information
michaelkedar authored May 30, 2024
1 parent bd0df63 commit 381f459
Showing 1 changed file with 0 additions and 33 deletions.
33 changes: 0 additions & 33 deletions deployment/terraform/modules/osv/website.tf
Original file line number Diff line number Diff line change
Expand Up @@ -94,31 +94,6 @@ module "gclb" {
url_map = google_compute_url_map.website.id

backends = {
appengine = {
groups = [
{
group = google_compute_region_network_endpoint_group.appengine_neg.id
}
]
protocol = "HTTPS"
enable_cdn = true
cdn_policy = {
cache_key_policy = {
include_host = true
include_protocol = true
include_query_string = true
}
signed_url_cache_max_age_sec = 0
}

iap_config = {
enable = false
}
log_config = {
enable = false
}
}

cloudrun = {
groups = [
{
Expand Down Expand Up @@ -157,14 +132,6 @@ resource "google_compute_region_network_endpoint_group" "website_neg" {
}
}

resource "google_compute_region_network_endpoint_group" "appengine_neg" {
project = var.project_id
name = "appengine-neg"
network_endpoint_type = "SERVERLESS"
region = google_app_engine_application.app.location_id
app_engine {}
}

resource "google_compute_url_map" "website" {
project = var.project_id
name = "website-url-map"
Expand Down

0 comments on commit 381f459

Please sign in to comment.