diff --git a/microsoft/testsuites/vm_extensions/azureperformancediagnostics.py b/microsoft/testsuites/vm_extensions/azureperformancediagnostics.py index 1525a7a105..bd4fa3e588 100644 --- a/microsoft/testsuites/vm_extensions/azureperformancediagnostics.py +++ b/microsoft/testsuites/vm_extensions/azureperformancediagnostics.py @@ -26,9 +26,9 @@ from lisa.sut_orchestrator import AZURE from lisa.sut_orchestrator.azure.common import ( AzureNodeSchema, + add_system_assign_identity, check_or_create_storage_account, get_node_context, - add_system_assign_identity, list_blobs, ) from lisa.sut_orchestrator.azure.features import AzureExtension @@ -60,7 +60,7 @@ def before_case(self, log: Logger, **kwargs: Any) -> None: storage account key, which we cannot use currently. Will change it back once the extension works with MSI. """, - priority=5, + priority=1, requirement=simple_requirement( supported_features=[AzureExtension], ), @@ -92,8 +92,12 @@ def verify_azure_performance_diagnostics( ) # Assign system identity to VM - # This MSI will be pre-configured with the necessary role assigments to Storage Account from Subscription level - # Permissions required: Storage Account Contributor, Storage Blob Data Contributor, Storage Blob Data Contributor + # This MSI will be pre-configured with the necessary + # role assigments to Storage Account from Subscription level + # Permissions required: + # Storage Account Contributor + # Storage Blob Data Contributor + # Storage Blob Data Contributor add_system_assign_identity( platform=platform, resource_group_name=node_context.resource_group_name, @@ -119,7 +123,7 @@ def verify_azure_performance_diagnostics( protected_settings = { "storageAccountName": storage_account_name, - "authenticationType": "SystemManagedIdentity" + "authenticationType": "SystemManagedIdentity", } extension_result = extension.create_or_update( @@ -174,7 +178,8 @@ def _is_supported_linux_distro(self, node: Node) -> bool: Suse: [12, 15], SLES: [12, 15], AlmaLinux: [8], - CBLMariner: [2, 3], #AzureLinux uses CBLMariner class + # AzureLinux uses CBLMariner class + CBLMariner: [2, 3], } for distro in supported_major_versions: