microsoft / microsoft/vscode-azureresourcegroups
The "revealResource" command does not work for Container Apps
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 78
- Forks
- 55
- Avg merge
- 16h 49m
- Merged PRs (30d)
- 139
Description
Repro steps
See https://github.com/Azure/azure-dev/issues/2043
More information
In the Azure resource tree hierarchy, the subscription nodes (with id like /subscriptions/s1234 have children that are "groupings" (with id like /subscriptions/s1234/groupings/Azure Cosmos DB). The problem is "grouping" is not a valid part of an Azure resource ID, which typically have a resource group as the second designator, for example /subscriptions/s1234/resourceGroups/rg-foo/providers/Microsoft.App/containerApps/mygreatapp.
As a result, when the findItemById function starts processing children of the subscription tree item, none of the "groupings" is considered an ancestor of an Azure resource that is part of a resource group, so the tree item corresponding to the passed resource ID is not found and nothing is revealed.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the Container Apps reproduction in issue 2043 and read findItemById in src/tree/ResourceTreeDataProviderBase.ts. Trace how subscription children with grouping IDs are compared with the passed Azure resource ID. Done means revealResource can locate and reveal a Container App under its resource group, with the relevant behavior covered by the existing tests if available.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- azure, typescript
- Domain
- cloud, devtools
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100