pulumi / pulumi-google-native

Apache License 2.0
71 stars 18 forks source link

dataproc.v1.Cluster fails to update with bad request #254

Open ColeSiegelTR opened 2 years ago

ColeSiegelTR commented 2 years ago

When I try to increase the number of worker nodes from 2 to 3 it now registers as an update instead of a replace but the request fails with a 400 error. It seems like maybe updateMask noted here is not being set https://cloud.google.com/dataproc/docs/reference/rest/v1/projects.regions.clusters/patch

     Type                                                       Name                                                                         Plan       Info
 ~   ├─ google-native:dataproc/v1:Cluster                       dp_cluster                                                                   update     [diff: +config]

Resources:
    ~ 1 to update

Do you want to perform this update? yes
Updating:
     Type                                                       Name                                                                         Status                  Info
 ~   └─ google-native:dataproc/v1:Cluster                       dp_cluster                                                                   **updating failed**     [diff: +config

Diagnostics:
    error: update failed

  google-native:dataproc/v1:Cluster (dp_cluster):
    error: error sending request: googleapi: Error 400: Request must specify update_mask with at least one path.:
mikhailshilkov commented 2 years ago

Similar to #95 - I guess the fix would be similar

lblackstone commented 2 years ago

This should be fixed by https://github.com/pulumi/pulumi-google-native/pull/386

viveklak commented 2 years ago

As per https://github.com/pulumi/pulumi-google-native/pull/386#issuecomment-1087641534 the issue isn't fixed for some resources which require nested properties highlighted in the updateMask.