-
Notifications
You must be signed in to change notification settings - Fork 90
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
Fix the definition of scope #207
base: main
Are you sure you want to change the base?
Conversation
Scope is an internal CosmosDB thing, not the ARM resource ID
@iouri-s : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit d818c11: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
It can be both. The Azure CLI and Azure PowerShell implementations allow you to do a shorthand relative scope but it’s converted into the full resource id + relative scope behind the scenes. You can see this in Bicep or the source code for each tool. |
I'll double check the script, but it should work as-is. |
This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions. |
Scope is an internal CosmosDB thing, not the ARM resource ID