Skip to content

Commit 26d21d0

Browse files
authored
Fixing bug in az fleet namespace get-credentials command (#9383)
* updates * updates
1 parent 2d7f0d8 commit 26d21d0

File tree

5 files changed

+2807
-1508
lines changed

5 files changed

+2807
-1508
lines changed

src/fleet/HISTORY.rst

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,4 +164,8 @@ Release History
164164

165165
1.8.1
166166
++++++
167-
* Ensure role assignment is created for private fleets before fleet creation.
167+
* Ensure role assignment is created for private fleets before fleet creation.
168+
169+
1.8.2
170+
++++++
171+
* Fix fleet namespace get-credentials command with fleet member parameter.

src/fleet/azext_fleet/custom.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -237,8 +237,8 @@ def get_credentials(cmd,
237237
fleet_member = fleet_members_client.get(resource_group_name, name, member_name)
238238

239239
parsed_id = parse_resource_id(fleet_member.cluster_resource_id)
240-
if (parsed_id.get('resource_type') != 'managedClusters' or
241-
parsed_id.get('namespace') != 'Microsoft.ContainerService'):
240+
if (parsed_id.get('resource_type').lower() != 'managedclusters' or
241+
parsed_id.get('namespace').lower() != 'microsoft.containerservice'):
242242
raise CLIError(f"Fleet member '{member_name}' is not associated with an AKS managed cluster. "
243243
f"Currently, only AKS managed clusters are supported for this command.")
244244

0 commit comments

Comments
 (0)