Skip to content

adding computelimit commands#938

Merged
necusjz merged 3 commits intoAzure:mainfrom
parthshah2305:computelimit-cmds
Mar 4, 2026
Merged

adding computelimit commands#938
necusjz merged 3 commits intoAzure:mainfrom
parthshah2305:computelimit-cmds

Conversation

@parthshah2305
Copy link
Copy Markdown
Member

@parthshah2305 parthshah2305 commented Feb 5, 2026

Quota Service has a new RP Microsoft.ComputeLimit.
We have published the API specs here - ComputeLimit REST API as well as shipped the ComputeLimit SDK back in December.

This PR is to add the new commands for the same for CLI

Comment thread Commands/computelimit/guest-subscription/_add.md
Comment thread Commands/computelimit/shared-limit/_add.md

- Add a guest subscription
```bash
computelimit guest-subscription add --location eastus --guest-subscription-id 11111111-1111-1111-1111-111111111111
Copy link
Copy Markdown

@danielli90 danielli90 Feb 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"computelimit" should be replaced with "compute-limit" ((kebab-case)). Right?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually both are accepted. This files shows all the commands that are there, https://github.com/parthshah2305/aaz/blob/6995fad7672a090a2be998932eb2a2930156b25f/Commands/readme.md

Image

Do we have preference for kebab-case? I aligned it with the resource provider name ComputeLimit for simplicity.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed -

Image

Comment thread Commands/readme.md Outdated
@parthshah2305 parthshah2305 marked this pull request as ready for review February 20, 2026 16:30
@necusjz necusjz merged commit 04b806f into Azure:main Mar 4, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants