Skip to content

Commit 7fbfbbb

Browse files
author
kovirivong
committed
remove some useless comments
1 parent d1b2536 commit 7fbfbbb

1 file changed

Lines changed: 0 additions & 9 deletions

File tree

src/aks-preview/azext_aks_preview/tests/latest/test_aks_commands.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -17735,7 +17735,6 @@ def test_aks_jwtauthenticator_cmds(self, resource_group, resource_group_location
1773517735
'updated_jwt_config_file': _get_test_data_file('jwtauthenticator_update.json'),
1773617736
})
1773717737

17738-
# Create AKS cluster
1773917738
create_cmd = (
1774017739
"aks create --resource-group={resource_group} --name={name} --location={location} "
1774117740
"--vm-set-type=VirtualMachines --node-count 1 "
@@ -17745,9 +17744,6 @@ def test_aks_jwtauthenticator_cmds(self, resource_group, resource_group_location
1774517744
self.check('provisioningState', 'Succeeded'),
1774617745
])
1774717746

17748-
# Add JWT authenticator
17749-
# Since feature is not registered in subscription and requests from polling don't use custom headers,
17750-
# we have to use --no-wait and then use a custom polling function to wait until the JWT authenticator is ready.
1775117747
add_jwt_cmd = (
1775217748
"aks jwtauthenticator add --resource-group={resource_group} --cluster-name={name} "
1775317749
"--name={jwt_auth_name} --config-file={jwt_config_file} "
@@ -17758,7 +17754,6 @@ def test_aks_jwtauthenticator_cmds(self, resource_group, resource_group_location
1775817754
self.check('name', '{jwt_auth_name}'),
1775917755
])
1776017756

17761-
# Show JWT authenticator
1776217757
show_jwt_cmd = (
1776317758
"aks jwtauthenticator show --resource-group={resource_group} --cluster-name={name} "
1776417759
"--name={jwt_auth_name} "
@@ -17775,15 +17770,13 @@ def test_aks_jwtauthenticator_cmds(self, resource_group, resource_group_location
1777517770

1777617771
time.sleep(60) # wait before updating to avoid "Another operation is in progress" error
1777717772

17778-
# Update JWT authenticator with same config file (should succeed)
1777917773
update_jwt_cmd = (
1778017774
"aks jwtauthenticator update --resource-group={resource_group} --cluster-name={name} "
1778117775
"--name={jwt_auth_name} --config-file={updated_jwt_config_file} "
1778217776
"--aks-custom-headers AKSHTTPCustomFeatures=Microsoft.ContainerService/JWTAuthenticatorPreview "
1778317777
)
1778417778
self.cmd(update_jwt_cmd)
1778517779

17786-
# List JWT authenticators
1778717780
list_jwt_cmd = (
1778817781
"aks jwtauthenticator list --resource-group={resource_group} --cluster-name={name} "
1778917782
"--aks-custom-headers AKSHTTPCustomFeatures=Microsoft.ContainerService/JWTAuthenticatorPreview "
@@ -17794,7 +17787,6 @@ def test_aks_jwtauthenticator_cmds(self, resource_group, resource_group_location
1779417787
assert jwt_list[0]['properties']['provisioningState'] == 'Succeeded'
1779517788
assert jwt_list[0]['properties']['issuer']['url'] == 'https://accounts.google.com'
1779617789

17797-
# Delete JWT authenticator
1779817790
delete_jwt_cmd = (
1779917791
"aks jwtauthenticator delete --resource-group={resource_group} --cluster-name={name} "
1780017792
"--name={jwt_auth_name} --yes "
@@ -17804,7 +17796,6 @@ def test_aks_jwtauthenticator_cmds(self, resource_group, resource_group_location
1780417796
self.is_empty(),
1780517797
])
1780617798

17807-
# Verify JWT authenticator is deleted
1780817799
list_after_delete_cmd = (
1780917800
"aks jwtauthenticator list --resource-group={resource_group} --cluster-name={name} "
1781017801
"--aks-custom-headers AKSHTTPCustomFeatures=Microsoft.ContainerService/JWTAuthenticatorPreview "

0 commit comments

Comments
 (0)