Skip to content
This repository has been archived by the owner on Aug 13, 2024. It is now read-only.

After az capi delete show how to delete management cluster #148

Merged
merged 3 commits into from
Jun 29, 2022
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
show msg only if cluster name is the same
tal66 committed Jun 23, 2022
commit feaeaa3c7db60839e9de9894300418ccadf17901
5 changes: 3 additions & 2 deletions src/capi/azext_capi/custom.py
Original file line number Diff line number Diff line change
@@ -720,8 +720,9 @@ def delete_workload_cluster(cmd, capi_name, resource_group_name=None, yes=False)
if not yes and not prompt_y_n(msg, default="n"):
return
begin_msg = "Deleting workload cluster"
end_msg = "✓ Deleted workload cluster" \
+ f'\nNote: To also delete the management cluster, run "az capi management delete -n {capi_name}"'
end_msg = "✓ Deleted workload cluster"
if capi_name == kubectl_helpers.find_cluster_in_current_context():
end_msg += f'\nNote: To also delete the management cluster, run "az capi management delete -n {capi_name}"'
err_msg = "Couldn't delete workload cluster"
try_command_with_spinner(cmd, command, begin_msg, end_msg, err_msg)
if is_self_managed:
3 changes: 2 additions & 1 deletion src/capi/azext_capi/tests/latest/test_capi_scenario.py
Original file line number Diff line number Diff line change
@@ -98,9 +98,10 @@ def test_capi_show(self, mock_def):

self.assertEqual(mock.call_count, 3)

@patch('azext_capi.custom.kubectl_helpers.find_cluster_in_current_context', return_value="testcluster1")
@patch('azext_capi.custom.is_self_managed_cluster', return_value=False)
@patch('azext_capi.custom.exit_if_no_management_cluster')
def test_capi_delete(self, mock_def, mock_is_self_managed):
def test_capi_delete(self, mock_def, mock_is_self_managed, find_cluster_mock):
# Test (indirectly) that user is prompted for confirmation by default
with self.assertRaises(NoTTYException):
self.cmd('capi delete --name testcluster1')