Add support for warm up requests in google_app_engine_standard_app_version

[maroux]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment. If the issue is assigned to the "modular-magician" user, it is either in the process of being autogenerated, or is planned to be autogenerated soon. If the issue is assigned to a user, that user is claiming responsibility for the issue. If the issue is assigned to "hashibot", a community member has claimed the issue already.

Description

Unable to use google_app_engine_service_split_traffic because that requires warm up requests to be enabled and google_app_engine_standard_app_version doesn't provide any way to enable that.

New or Affected Resource(s)

• google_app_engine_service_split_traffic
• google_app_engine_standard_app_version

Potential Terraform Configuration

resource "google_app_engine_standard_app_version" "service" {
  service    = "service"
  runtime    = "go112"
  version_id = "v1"

  deployment {
    zip {
      source_url = "..."
    }
  }

  lifecycle {
    create_before_destroy = true
  }

  noop_on_destroy = true

  timeouts {
    create = "10m"
  }
}

resource "google_app_engine_service_split_traffic" "service" {
  service = google_app_engine_standard_app_version.service.service

  migrate_traffic = true

  split {
    shard_by = "IP"

    allocations = {
      (google_app_engine_standard_app_version.service.version_id) = 1.0
    }
  }
}

Fails with error:

Error: Error creating ServiceSplitTraffic: googleapi: Error 400: Invalid request.
Details:
[
  {
    "@type": "type.googleapis.com/google.rpc.BadRequest",
    "fieldViolations": [
      {
        "description": "Warmup requests must be enabled for all versions that will gain additional traffic as a result of the traffic migration.",
        "field": "service.split.allocations[v0-3-1]"
      }
    ]
  }
]

[danawillow]

This looks possible via https://www.terraform.io/docs/providers/google/r/app_engine_standard_app_version.html#inbound_services (as per https://cloud.google.com/appengine/docs/admin-api/reference/rest/v1/apps.services.versions#inboundservicetype).

Changing this to docs change.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!