Skip to content
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

Disk resource does not support "Premium_ZRS" and few other sku's #4057

Closed
ganeshg2491 opened this issue Jun 3, 2024 · 4 comments · Fixed by #4171
Closed

Disk resource does not support "Premium_ZRS" and few other sku's #4057

ganeshg2491 opened this issue Jun 3, 2024 · 4 comments · Fixed by #4171
Assignees
Labels
new-resouce-version New version of a resource ASO already supports
Milestone

Comments

@ganeshg2491
Copy link

Describe the current behavior
Currently Disk resource is generated from " /compute/resource-manager/Microsoft.Compute/DiskRP/stable/2020-09-30/disk.json"
and it does not seem to be supported all the mention ZRS sku's in
https://learn.microsoft.com/en-us/azure/templates/microsoft.compute/disks?pivots=deployment-language-arm-template#disksku-1

Describe the improvement
Disk resource needs to be regenerated from the latest arm api version "2023-10-02".

Additional context
Add any other context about the suggested improvement.

@theunrepentantgeek theunrepentantgeek added new-resouce-version New version of a resource ASO already supports and removed needs-triage 🔍 labels Jun 3, 2024
@theunrepentantgeek theunrepentantgeek added this to the v2.9.0 milestone Jun 3, 2024
@theunrepentantgeek
Copy link
Member

We'll look into adding support for api-version 2023-10-02 - this will sit alongside the existing version, allowing users to update on their schedule, not ours.

@ganeshg2491
Copy link
Author

@theunrepentantgeek : Thanks. As a part of this change, can we also support including "Microsoft.Compute/diskAccesses" and "Microsoft.Compute/diskAccesses/privateEndpointConnections" resource which is require for Disk private access.

@matthchr matthchr self-assigned this Jul 15, 2024
matthchr added a commit to matthchr/azure-service-operator that referenced this issue Jul 31, 2024
matthchr added a commit to matthchr/azure-service-operator that referenced this issue Jul 31, 2024
matthchr added a commit to matthchr/azure-service-operator that referenced this issue Jul 31, 2024
@matthchr matthchr mentioned this issue Jul 31, 2024
3 tasks
@matthchr
Copy link
Member

The above PR resolves this, with one exception.

We added Microsoft.Compute/diskAccesses, but did not add Microsoft.Compute/diskAccesses/privateEndpointConnections. This is because the privateEndpointConnections resource is only used to approve privateEndpoints.

Our current thinking with this scenario is either:

  1. You're autoapproved, in which case you don't need Microsoft.Compute/diskAccesses/privateEndpointConnections.
  2. You're not autoapproved, in which case the connection likely needs to be reviewed by a person (not managed through IAC), in which case using something like portal or azcli for the approval probably makes more sense than managing it through ASO.

Let us know if you agree/disagree with that sentiment. Other than that though, all your asks are actioned by the linked PR.

@ganeshg2491
Copy link
Author

@matthchr : Yeah, I think that's fine. Good to go ahead with this PR.

matthchr added a commit to matthchr/azure-service-operator that referenced this issue Aug 1, 2024
github-merge-queue bot pushed a commit that referenced this issue Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-resouce-version New version of a resource ASO already supports
Projects
Status: Recently Completed
Development

Successfully merging a pull request may close this issue.

3 participants