From ae45bf384e9f1436525181ddf16b6275401eae3b Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Mon, 8 Dec 2025 15:48:03 +0800 Subject: [PATCH 01/11] test(container): disable failing test due to policy restriction - Added @Disabled annotation to testBeginCreateWithFileShareVolume - Included reason for disabling: internal policy denies local auth methods - Preserved existing @DoNotRecord annotation and skipInPlayback setting --- .../resourcemanager/containerinstance/ContainerGroupTest.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index 2ed7cce6084e..2822d3a91175 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -19,6 +19,7 @@ import com.azure.resourcemanager.resources.fluentcore.model.Accepted; import com.azure.resourcemanager.resources.fluentcore.utils.ResourceManagerUtils; import org.junit.jupiter.api.Assertions; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import java.time.Duration; @@ -212,6 +213,7 @@ public void testBeginCreate() { // test contains a data-plane call @DoNotRecord(skipInPlayback = true) @Test + @Disabled("This request was denied due to internal policy. Local authentication methods are not allowed.") public void testBeginCreateWithFileShareVolume() { String containerGroupName = generateRandomResourceName("container", 20); Region region = Region.US_WEST3; From ea2cae2bf5da88fbd1a39f45b2746658358d124c Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Tue, 23 Dec 2025 21:05:58 +0800 Subject: [PATCH 02/11] feat(test): enable file share volume testing for container groups - Add storage account creation and key retrieval for file share testing - Integrate Azure file share volume configuration with existing storage credentials - Include ShareServiceClientBuilder for file endpoint management - Remove @Disabled annotation to activate file share volume tests - Add storage manager initialization in test base class - Update container group volume definition to use existing Azure file share --- .../containerinstance/ContainerGroupTest.java | 37 +++++++++++++++++-- .../ContainerInstanceManagementTest.java | 3 ++ 2 files changed, 36 insertions(+), 4 deletions(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index 2822d3a91175..e43861b8becd 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -3,12 +3,15 @@ package com.azure.resourcemanager.containerinstance; +import com.azure.core.management.Region; import com.azure.core.test.annotation.DoNotRecord; +import com.azure.identity.AzureAuthorityHosts; +import com.azure.identity.ClientSecretCredential; +import com.azure.identity.DefaultAzureCredentialBuilder; import com.azure.resourcemanager.containerinstance.models.Container; import com.azure.resourcemanager.containerinstance.models.ContainerAttachResult; import com.azure.resourcemanager.containerinstance.models.ContainerExec; import com.azure.resourcemanager.containerinstance.models.ContainerGroup; -import com.azure.core.management.Region; import com.azure.resourcemanager.containerinstance.models.ContainerGroupRestartPolicy; import com.azure.resourcemanager.containerinstance.models.ContainerHttpGet; import com.azure.resourcemanager.containerinstance.models.ContainerProbe; @@ -18,8 +21,11 @@ import com.azure.resourcemanager.network.models.Network; import com.azure.resourcemanager.resources.fluentcore.model.Accepted; import com.azure.resourcemanager.resources.fluentcore.utils.ResourceManagerUtils; +import com.azure.resourcemanager.storage.models.StorageAccount; +import com.azure.resourcemanager.storage.models.StorageAccountKey; +import com.azure.storage.file.share.ShareServiceAsyncClient; +import com.azure.storage.file.share.ShareServiceClientBuilder; import org.junit.jupiter.api.Assertions; -import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import java.time.Duration; @@ -213,11 +219,28 @@ public void testBeginCreate() { // test contains a data-plane call @DoNotRecord(skipInPlayback = true) @Test - @Disabled("This request was denied due to internal policy. Local authentication methods are not allowed.") public void testBeginCreateWithFileShareVolume() { String containerGroupName = generateRandomResourceName("container", 20); Region region = Region.US_WEST3; + StorageAccount storageAccount = storageManager.storageAccounts() + .define(containerGroupName) + .withRegion(region) + .withNewResourceGroup(rgName) + .disableSharedKeyAccess() + .create(); + + StorageAccountKey key = storageAccount.getKeys().get(0); + + String shareName = "share1"; + String fileEndpoint = String.format("https://%s.file.core.windows.net", storageAccount.name()); + ShareServiceAsyncClient shareServiceAsyncClient = new ShareServiceClientBuilder() + .endpoint(fileEndpoint) + .credential(new DefaultAzureCredentialBuilder().build()) + .httpClient(containerInstanceManager.httpPipeline().getHttpClient()) + .buildAsyncClient(); + shareServiceAsyncClient.createShare(shareName).block(); + // create storage account (and virtual network), before create container group Accepted acceptedContainerGroup = containerInstanceManager.containerGroups() .define(containerGroupName) @@ -226,7 +249,13 @@ public void testBeginCreateWithFileShareVolume() { .withLinux() .withPublicImageRegistryOnly() // definition step only allow creating one file share volume - .withNewAzureFileShareVolume("vol1", "share1") + // https://learn.microsoft.com/azure/container-instances/container-instances-volume-azure-files +// .withEmptyDirectoryVolume("vol2") + .defineVolume("vol1") + .withExistingReadWriteAzureFileShare(shareName) + .withStorageAccountName(storageAccount.name()) + .withStorageAccountKey(key.value()) + .attach() .withContainerInstance("nginx", 80) .withNewVirtualNetwork("10.0.0.0/24") .beginCreate(); diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerInstanceManagementTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerInstanceManagementTest.java index 684736c28122..a0ab0705d4b1 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerInstanceManagementTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerInstanceManagementTest.java @@ -12,6 +12,7 @@ import com.azure.core.management.profile.AzureProfile; import com.azure.resourcemanager.resources.fluentcore.utils.HttpPipelineProvider; import com.azure.resourcemanager.resources.fluentcore.utils.ResourceManagerUtils; +import com.azure.resourcemanager.storage.StorageManager; import com.azure.resourcemanager.test.ResourceManagerTestProxyTestBase; import com.azure.resourcemanager.test.utils.TestDelayProvider; import com.azure.resourcemanager.test.utils.TestIdentifierProvider; @@ -22,6 +23,7 @@ public class ContainerInstanceManagementTest extends ResourceManagerTestProxyTestBase { protected ContainerInstanceManager containerInstanceManager; protected String rgName = ""; + protected StorageManager storageManager; @Override protected HttpPipeline buildHttpPipeline(TokenCredential credential, AzureProfile profile, @@ -37,6 +39,7 @@ protected void initializeClients(HttpPipeline httpPipeline, AzureProfile profile internalContext.setIdentifierFunction(name -> new TestIdentifierProvider(testResourceNamer)); ResourceManagerUtils.InternalRuntimeContext.setDelayProvider(new TestDelayProvider(!isPlaybackMode())); containerInstanceManager = buildManager(ContainerInstanceManager.class, httpPipeline, profile); + storageManager = buildManager(StorageManager.class, httpPipeline, profile); setInternalContext(internalContext, containerInstanceManager); } From 2927af5a1fa94605aecf0a7736d973aadd7f4fc7 Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Thu, 25 Dec 2025 11:50:51 +0800 Subject: [PATCH 03/11] fix(test): update ContainerGroupTest to clarify Azure Storage limitations with managed identity --- .../resourcemanager/containerinstance/ContainerGroupTest.java | 1 + 1 file changed, 1 insertion(+) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index e43861b8becd..7d9e5ce0d0e8 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -250,6 +250,7 @@ public void testBeginCreateWithFileShareVolume() { .withPublicImageRegistryOnly() // definition step only allow creating one file share volume // https://learn.microsoft.com/azure/container-instances/container-instances-volume-azure-files + // Azure Storage doesn't support SMB mounting of file share using managed identity // .withEmptyDirectoryVolume("vol2") .defineVolume("vol1") .withExistingReadWriteAzureFileShare(shareName) From fe8fc1cbe4a4b59f9a7190ca4afb74950b0b8d67 Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Mon, 5 Jan 2026 16:05:00 +0800 Subject: [PATCH 04/11] test(containerinstance): add tests for container group volume configurations - Added testBeginCreateWithEmptyDirectoryVolume method - Disabled testBeginCreateWithFileShareVolume due to internal policy restrictions - Updated volume configuration logic to use withEmptyDirectoryVolume instead of manual volume definition - Added proper annotations for test recording and policy compliance - Implemented container group creation with virtual network and volume mounting - Added assertions for volume count validation --- .../containerinstance/ContainerGroupTest.java | 36 ++++++++++++++----- 1 file changed, 27 insertions(+), 9 deletions(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index 7d9e5ce0d0e8..dbc967b941dd 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -26,6 +26,7 @@ import com.azure.storage.file.share.ShareServiceAsyncClient; import com.azure.storage.file.share.ShareServiceClientBuilder; import org.junit.jupiter.api.Assertions; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import java.time.Duration; @@ -217,12 +218,37 @@ public void testBeginCreate() { } // test contains a data-plane call + // https://learn.microsoft.com/azure/container-instances/container-instances-volume-azure-files + // Azure Storage doesn't support SMB mounting of file share using managed identity @DoNotRecord(skipInPlayback = true) @Test + @Disabled("This request was denied due to internal policy. Local authentication methods are not allowed.") public void testBeginCreateWithFileShareVolume() { String containerGroupName = generateRandomResourceName("container", 20); Region region = Region.US_WEST3; + // create storage account (and virtual network), before create container group + Accepted acceptedContainerGroup = containerInstanceManager.containerGroups() + .define(containerGroupName) + .withRegion(region) + .withNewResourceGroup(rgName) + .withLinux() + .withPublicImageRegistryOnly() + // definition step only allow creating one file share volume + .withNewAzureFileShareVolume("vol1", "share1") + .withContainerInstance("nginx", 80) + .withNewVirtualNetwork("10.0.0.0/24") + .beginCreate(); + ContainerGroup containerGroup = acceptedContainerGroup.getSyncPoller().getFinalResult(); + Assertions.assertEquals(1, containerGroup.volumes().size()); + } + + @DoNotRecord(skipInPlayback = true) + @Test + public void testBeginCreateWithEmptyDirectoryVolume() { + String containerGroupName = generateRandomResourceName("container", 20); + Region region = Region.US_WEST3; + StorageAccount storageAccount = storageManager.storageAccounts() .define(containerGroupName) .withRegion(region) @@ -248,15 +274,7 @@ public void testBeginCreateWithFileShareVolume() { .withNewResourceGroup(rgName) .withLinux() .withPublicImageRegistryOnly() - // definition step only allow creating one file share volume - // https://learn.microsoft.com/azure/container-instances/container-instances-volume-azure-files - // Azure Storage doesn't support SMB mounting of file share using managed identity -// .withEmptyDirectoryVolume("vol2") - .defineVolume("vol1") - .withExistingReadWriteAzureFileShare(shareName) - .withStorageAccountName(storageAccount.name()) - .withStorageAccountKey(key.value()) - .attach() + .withEmptyDirectoryVolume("vol2") .withContainerInstance("nginx", 80) .withNewVirtualNetwork("10.0.0.0/24") .beginCreate(); From a792d38c0ddf31f87fa0a89e12b126cf7bfcf69a Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Tue, 6 Jan 2026 14:54:04 +0800 Subject: [PATCH 05/11] fix(containerinstance): update test comment and remove outdated reference Removed outdated comment about Azure Storage SMB mounting with managed identity and updated the test documentation to reflect current behavior. The test now properly references the Azure documentation for container instances volume mounting. --- .../resourcemanager/containerinstance/ContainerGroupTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index dbc967b941dd..8a8a14a5cb3c 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -218,8 +218,8 @@ public void testBeginCreate() { } // test contains a data-plane call - // https://learn.microsoft.com/azure/container-instances/container-instances-volume-azure-files // Azure Storage doesn't support SMB mounting of file share using managed identity + // https://learn.microsoft.com/azure/container-instances/container-instances-volume-azure-files @DoNotRecord(skipInPlayback = true) @Test @Disabled("This request was denied due to internal policy. Local authentication methods are not allowed.") From ebbe31e9c86f318a5fe51f816ee2ff98f1801e06 Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Tue, 6 Jan 2026 15:38:07 +0800 Subject: [PATCH 06/11] chore(containerinstance): update assets tag Update the assets tag from a0e5cc2e29 to bd0493a9b7 in azure-resourcemanager-containerinstance module --- .../azure-resourcemanager-containerinstance/assets.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/assets.json b/sdk/containerinstance/azure-resourcemanager-containerinstance/assets.json index b1a68ecfaf1a..60882dab630e 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/assets.json +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/assets.json @@ -2,5 +2,5 @@ "AssetsRepo": "Azure/azure-sdk-assets", "AssetsRepoPrefixPath": "java", "TagPrefix": "java/containerinstance/azure-resourcemanager-containerinstance", - "Tag": "java/containerinstance/azure-resourcemanager-containerinstance_a0e5cc2e29" + "Tag": "java/containerinstance/azure-resourcemanager-containerinstance_bd0493a9b7" } From 170b5c7741e8b0a0f05462ea73c1c0f64a06ac25 Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Tue, 6 Jan 2026 15:51:35 +0800 Subject: [PATCH 07/11] refactor(tests): remove unused storage manager from container instance tests - Removed storage account creation logic from ContainerGroupTest - Eliminated unused StorageManager dependency from ContainerInstanceManagementTest - Cleaned up related imports and fields that were no longer needed - Simplified test setup by removing unnecessary storage account initialization --- .../containerinstance/ContainerGroupTest.java | 18 ------------------ .../ContainerInstanceManagementTest.java | 3 --- 2 files changed, 21 deletions(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index 8a8a14a5cb3c..c336ac168746 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -249,24 +249,6 @@ public void testBeginCreateWithEmptyDirectoryVolume() { String containerGroupName = generateRandomResourceName("container", 20); Region region = Region.US_WEST3; - StorageAccount storageAccount = storageManager.storageAccounts() - .define(containerGroupName) - .withRegion(region) - .withNewResourceGroup(rgName) - .disableSharedKeyAccess() - .create(); - - StorageAccountKey key = storageAccount.getKeys().get(0); - - String shareName = "share1"; - String fileEndpoint = String.format("https://%s.file.core.windows.net", storageAccount.name()); - ShareServiceAsyncClient shareServiceAsyncClient = new ShareServiceClientBuilder() - .endpoint(fileEndpoint) - .credential(new DefaultAzureCredentialBuilder().build()) - .httpClient(containerInstanceManager.httpPipeline().getHttpClient()) - .buildAsyncClient(); - shareServiceAsyncClient.createShare(shareName).block(); - // create storage account (and virtual network), before create container group Accepted acceptedContainerGroup = containerInstanceManager.containerGroups() .define(containerGroupName) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerInstanceManagementTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerInstanceManagementTest.java index a0ab0705d4b1..684736c28122 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerInstanceManagementTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerInstanceManagementTest.java @@ -12,7 +12,6 @@ import com.azure.core.management.profile.AzureProfile; import com.azure.resourcemanager.resources.fluentcore.utils.HttpPipelineProvider; import com.azure.resourcemanager.resources.fluentcore.utils.ResourceManagerUtils; -import com.azure.resourcemanager.storage.StorageManager; import com.azure.resourcemanager.test.ResourceManagerTestProxyTestBase; import com.azure.resourcemanager.test.utils.TestDelayProvider; import com.azure.resourcemanager.test.utils.TestIdentifierProvider; @@ -23,7 +22,6 @@ public class ContainerInstanceManagementTest extends ResourceManagerTestProxyTestBase { protected ContainerInstanceManager containerInstanceManager; protected String rgName = ""; - protected StorageManager storageManager; @Override protected HttpPipeline buildHttpPipeline(TokenCredential credential, AzureProfile profile, @@ -39,7 +37,6 @@ protected void initializeClients(HttpPipeline httpPipeline, AzureProfile profile internalContext.setIdentifierFunction(name -> new TestIdentifierProvider(testResourceNamer)); ResourceManagerUtils.InternalRuntimeContext.setDelayProvider(new TestDelayProvider(!isPlaybackMode())); containerInstanceManager = buildManager(ContainerInstanceManager.class, httpPipeline, profile); - storageManager = buildManager(StorageManager.class, httpPipeline, profile); setInternalContext(internalContext, containerInstanceManager); } From e470d05838c25c58a3ac914c706471805a2b5fe3 Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Tue, 6 Jan 2026 16:28:56 +0800 Subject: [PATCH 08/11] chore(containerinstance): update assets tag Update the assets tag in assets.json from bd0493a9b7 to cade898991 for azure-resourcemanager-containerinstance package. --- .../azure-resourcemanager-containerinstance/assets.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/assets.json b/sdk/containerinstance/azure-resourcemanager-containerinstance/assets.json index 60882dab630e..5ae2803d305d 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/assets.json +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/assets.json @@ -2,5 +2,5 @@ "AssetsRepo": "Azure/azure-sdk-assets", "AssetsRepoPrefixPath": "java", "TagPrefix": "java/containerinstance/azure-resourcemanager-containerinstance", - "Tag": "java/containerinstance/azure-resourcemanager-containerinstance_bd0493a9b7" + "Tag": "java/containerinstance/azure-resourcemanager-containerinstance_cade898991" } From a7a101ea0065e54e2a1c35d6d214a683a762a4ee Mon Sep 17 00:00:00 2001 From: Hui Zhu Date: Tue, 6 Jan 2026 16:33:27 +0800 Subject: [PATCH 09/11] refactor(ContainerGroupTest): remove unused imports - Removed unused Azure identity imports (AzureAuthorityHosts, ClientSecretCredential, DefaultAzureCredentialBuilder) - Removed unused storage related imports (StorageAccount, StorageAccountKey, ShareServiceAsyncClient, ShareServiceClientBuilder) --- .../containerinstance/ContainerGroupTest.java | 7 ------- 1 file changed, 7 deletions(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index c336ac168746..a0eb8d1df37a 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -5,9 +5,6 @@ import com.azure.core.management.Region; import com.azure.core.test.annotation.DoNotRecord; -import com.azure.identity.AzureAuthorityHosts; -import com.azure.identity.ClientSecretCredential; -import com.azure.identity.DefaultAzureCredentialBuilder; import com.azure.resourcemanager.containerinstance.models.Container; import com.azure.resourcemanager.containerinstance.models.ContainerAttachResult; import com.azure.resourcemanager.containerinstance.models.ContainerExec; @@ -21,10 +18,6 @@ import com.azure.resourcemanager.network.models.Network; import com.azure.resourcemanager.resources.fluentcore.model.Accepted; import com.azure.resourcemanager.resources.fluentcore.utils.ResourceManagerUtils; -import com.azure.resourcemanager.storage.models.StorageAccount; -import com.azure.resourcemanager.storage.models.StorageAccountKey; -import com.azure.storage.file.share.ShareServiceAsyncClient; -import com.azure.storage.file.share.ShareServiceClientBuilder; import org.junit.jupiter.api.Assertions; import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; From 557589a68378835b8ac5dd863a93c22bdc571ae1 Mon Sep 17 00:00:00 2001 From: v-huizhu2 Date: Tue, 6 Jan 2026 16:50:55 +0800 Subject: [PATCH 10/11] Update sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- .../resourcemanager/containerinstance/ContainerGroupTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index a0eb8d1df37a..06778ba77086 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -242,7 +242,7 @@ public void testBeginCreateWithEmptyDirectoryVolume() { String containerGroupName = generateRandomResourceName("container", 20); Region region = Region.US_WEST3; - // create storage account (and virtual network), before create container group + // create virtual network before creating container group Accepted acceptedContainerGroup = containerInstanceManager.containerGroups() .define(containerGroupName) .withRegion(region) From c90929755659bb09729d6de19d31c0b78ecd9d75 Mon Sep 17 00:00:00 2001 From: v-huizhu2 Date: Tue, 6 Jan 2026 17:47:49 +0800 Subject: [PATCH 11/11] Update sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java Co-authored-by: Xiaofei Cao <92354331+XiaofeiCao@users.noreply.github.com> --- .../resourcemanager/containerinstance/ContainerGroupTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java index 06778ba77086..6a597a99d78b 100644 --- a/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java +++ b/sdk/containerinstance/azure-resourcemanager-containerinstance/src/test/java/com/azure/resourcemanager/containerinstance/ContainerGroupTest.java @@ -215,7 +215,7 @@ public void testBeginCreate() { // https://learn.microsoft.com/azure/container-instances/container-instances-volume-azure-files @DoNotRecord(skipInPlayback = true) @Test - @Disabled("This request was denied due to internal policy. Local authentication methods are not allowed.") + @Disabled("This request was denied due to internal policy. Container Group requires that Storage Accounts is authenticated with managed identity, but Azure Storage doesn't support SMB mounting of file share using managed identity. ") public void testBeginCreateWithFileShareVolume() { String containerGroupName = generateRandomResourceName("container", 20); Region region = Region.US_WEST3;