Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Handle case sensitivity in resourceID format #269

Closed
aramase opened this issue Jun 28, 2019 · 1 comment
Closed

Handle case sensitivity in resourceID format #269

aramase opened this issue Jun 28, 2019 · 1 comment
Assignees
@aramase
Labels
bug Something isn't working
Milestone
v1.5

Comments

@aramase
Copy link
Member

aramase commented Jun 28, 2019

Describe the bug
When identity is manually assigned to node using az cli, the resourceID uses resourceGroups in the format /subscriptions/[REDACTED]/resourceGroups/devcluster/providers/Microsoft.ManagedIdentity/userAssignedIdentities/keyvault-identity . When a new identity is created the resourceID returned however has resourcegroups /subscriptions/[REDACTED]/resourcegroups/devcluster/providers/Microsoft.ManagedIdentity/userAssignedIdentities/keyvault-identity. We need to normalize the resourceID format before performing a check to see if they are equal.

Steps To Reproduce

Expected behavior

AAD Pod Identity version

Kubernetes version

Additional context
@aramase aramase added the bug Something isn't working label Jun 28, 2019
@aramase aramase added this to the v1.5 milestone Jun 28, 2019
@aramase aramase mentioned this issue Jul 1, 2019
Merged
@kkmsft
Copy link
Contributor

kkmsft commented Jul 2, 2019

Closing it via #271

@kkmsft kkmsft closed this as completed Jul 2, 2019
@aramase aramase self-assigned this Jul 16, 2019
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@aramase aramase

Labels
bug Something isn't working
Projects
None yet
Milestone
v1.5
Development

No branches or pull requests
2 participants
@aramase @kkmsft