Skip to content

google_compute_subnetwork (beta), updating purpose forces replacement #6481

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
edwardmedia opened this issue May 28, 2020 · 2 comments
Closed
Assignees
Labels
bug forward/review In review; remove label to forward service/compute-vpc

Comments

@edwardmedia
Copy link
Contributor

edwardmedia commented May 28, 2020

Affected Resource(s)

  • google_compute_subnetwork (beta)

Expected Behavior

Updating purpose should like role do partial replacement

Actual Behavior

Changing purpose will force the subnetwork recreation.

Steps to Reproduce

Important Factoids

It appears the PATCH is supported for field purpose
( Note role was introduced at the same time and if d.HasChange("role") was added in the code )

https://cloud.google.com/compute/docs/reference/rest/beta/subnetworks/patch

@edwardmedia edwardmedia changed the title google_compute_subnetwork (beta) purpose change forces replacement google_compute_subnetwork (beta), purpose change forces replacement May 28, 2020
@edwardmedia edwardmedia changed the title google_compute_subnetwork (beta), purpose change forces replacement google_compute_subnetwork (beta), updating purpose forces replacement May 28, 2020
@danawillow
Copy link
Contributor

@edwardmedia I don't think the field can be updated. I tried it in API explorer and got the response:

{
  "error": {
    "code": 400,
    "message": "Invalid value for field 'resource': '{  \"fingerprint\": \"KN4X1G0qzB0=\",  \"purpose\": \"INTERNAL_HTTPS_LOAD_BALANCER\"}'. The following field(s) specified in the request cannot be modified: [purpose]",
    "errors": [
      {
        "message": "Invalid value for field 'resource': '{  \"fingerprint\": \"KN4X1G0qzB0=\",  \"purpose\": \"INTERNAL_HTTPS_LOAD_BALANCER\"}'. The following field(s) specified in the request cannot be modified: [purpose]",
        "domain": "global",
        "reason": "invalid"
      }
    ]
  }
}

Closing this since I think there's nothing to do here.

@ghost
Copy link

ghost commented Jul 3, 2020

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!

@ghost ghost locked and limited conversation to collaborators Jul 3, 2020
@github-actions github-actions bot added forward/review In review; remove label to forward service/compute-vpc labels Jan 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug forward/review In review; remove label to forward service/compute-vpc
Projects
None yet
Development

No branches or pull requests

2 participants