Skip to content

Commit 39b34e2

Browse files
committed
Migrate vm diagnostics set
1 parent 2bc85ae commit 39b34e2

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

src/azure-cli/azure/cli/command_modules/vm/commands.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -318,7 +318,7 @@ def load_command_table(self, _):
318318
g.custom_command('enable', 'enable_boot_diagnostics')
319319
g.custom_command('get-boot-log', 'get_boot_log')
320320

321-
with self.command_group('vm diagnostics', compute_vm_sdk) as g:
321+
with self.command_group('vm diagnostics') as g:
322322
g.custom_command('set', 'set_diagnostics_extension')
323323
g.custom_command('get-default-config', 'show_default_diagnostics_configuration')
324324

src/azure-cli/azure/cli/command_modules/vm/custom.py

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -2276,30 +2276,30 @@ def get_boot_log(cmd, resource_group_name, vm_name):
22762276

22772277

22782278
# region VirtualMachines Diagnostics
2279-
def set_diagnostics_extension(
2280-
cmd, resource_group_name, vm_name, settings, protected_settings=None, version=None,
2281-
no_auto_upgrade=False):
2282-
client = _compute_client_factory(cmd.cli_ctx)
2283-
vm = client.virtual_machines.get(resource_group_name, vm_name, expand='instanceView')
2284-
# pylint: disable=no-member
2285-
is_linux_os = _is_linux_os(vm)
2279+
def set_diagnostics_extension(cmd, resource_group_name, vm_name, settings, protected_settings=None, version=None,
2280+
no_auto_upgrade=False):
2281+
from .aaz.latest.vm.extension import Delete as VmExtentionDelete
2282+
vm = get_instance_view(cmd, resource_group_name, vm_name)
2283+
is_linux_os = _is_linux_os_by_aaz(vm)
22862284
vm_extension_name = _LINUX_DIAG_EXT if is_linux_os else _WINDOWS_DIAG_EXT
22872285
if is_linux_os: # check incompatible version
2288-
exts = vm.instance_view.extensions or []
2286+
exts = vm.get('instanceView', {}).get('extensions', [])
22892287
major_ver = extension_mappings[_LINUX_DIAG_EXT]['version'].split('.', maxsplit=1)[0]
2290-
if next((e for e in exts if e.name == vm_extension_name and
2291-
not e.type_handler_version.startswith(major_ver + '.')), None):
2288+
if next((e for e in exts if e.get('name') == vm_extension_name and
2289+
not e.get('typeHandlerVersion', '').startswith(major_ver + '.')), None):
22922290
logger.warning('There is an incompatible version of diagnostics extension installed. '
22932291
'We will update it with a new version')
2294-
poller = client.virtual_machine_extensions.begin_delete(resource_group_name, vm_name, vm_extension_name)
2292+
poller = VmExtentionDelete(cmd.cli_ctx)(command_args={
2293+
'resource_group': resource_group_name,
2294+
'vm_extension_name': vm_extension_name,
2295+
'vm_name': vm_name
2296+
})
22952297
LongRunningOperation(cmd.cli_ctx)(poller)
22962298

22972299
return set_extension(cmd, resource_group_name, vm_name, vm_extension_name,
22982300
extension_mappings[vm_extension_name]['publisher'],
22992301
version or extension_mappings[vm_extension_name]['version'],
2300-
settings,
2301-
protected_settings,
2302-
no_auto_upgrade)
2302+
settings, protected_settings, no_auto_upgrade)
23032303

23042304

23052305
def show_default_diagnostics_configuration(is_windows_os=False):

0 commit comments

Comments
 (0)