Terraform-provider-google: Allow specifying accelerators in cluster node_config

[kmontag]

Beta support for GPUs on Kubernetes clusters has been released: https://cloud.google.com/kubernetes-engine/docs/concepts/gpus

It looks like they can be added via REST API using the accelerators field in a node pool configuration, e.g. (when creating a cluster):

POST https://container.googleapis.com/v1/projects/foo/zones/us-west1-a/clusters
{
  "cluster": {
    "name": "gpu-test",
    "zone": "us-west1-a",
    "nodePools": [
      {
        "name": "default-pool",
        "initialNodeCount": 1,
        "config": {
          // ...
          "accelerators": [
            {
              "acceleratorCount": 1,
              "acceleratorType": "nvidia-tesla-k80"
            }
          ]
        },
        // ...
      }
    ],
    // ...
  }
}

Would be great if this could be supported in the node_config block for google_container_cluster / google_container_node_pool.

Thanks!

Terraform Version

Run terraform -v to show the version. If you are not running the latest version of Terraform, please upgrade because your issue may have already been fixed.

Terraform v0.11.3
+ provider.google v1.5.0

Affected Resource(s)

• google_container_cluster
• google_container_node_pool

[dhague]

I have the same requirement - currently hacking on the code for it, unless someone else has already started work on this?

[ndmckinley]

That's great! Tag me in the PR and I'll expedite the review.

[dhague]

Almost done with this - in case anyone is desperate for it, the code here works, I just need to tweak the tests to pass (so caveat emptor!). I'm on vacation for a long weekend, so I expect to finish this on Tuesday.

[thomasjungblut]

I can also pick this up if there's anything urgent, synced with Darren yesterday on what's missing.

[hashibot[bot]]

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 👉 [email protected]. Thanks!