From e041d1a2c162d014cf3c936f9f84a14da36b2d00 Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Mon, 5 Sep 2022 09:41:28 +0200 Subject: [PATCH] integration: TestUpdateContainerResources_MemoryLimit: remove TODO comment This comment was added in 09a0c9471bf687ebcc7d8f8594606d127b3a803e when the Windows integration tests were enabled. The PR (microsoft/hcsshim#931) was merged, and part of hcsshim v0.9.0, and support for resource limits on Windows was added in 2bc77b8a2865efe83bc50225223555a919191f54, so it looks like this comment is no longer current. Signed-off-by: Sebastiaan van Stijn --- integration/container_update_resources_test.go | 1 - 1 file changed, 1 deletion(-) diff --git a/integration/container_update_resources_test.go b/integration/container_update_resources_test.go index e5f67cfc4..a2f122a12 100644 --- a/integration/container_update_resources_test.go +++ b/integration/container_update_resources_test.go @@ -230,7 +230,6 @@ func TestUpdateContainerResources_MemorySwap(t *testing.T) { } func TestUpdateContainerResources_MemoryLimit(t *testing.T) { - // TODO(claudiub): Make this test work once https://github.com/microsoft/hcsshim/pull/931 merges. t.Log("Create a sandbox") sb, sbConfig := PodSandboxConfigWithCleanup(t, "sandbox", "update-container-resources")