Skip to content

Commit

Permalink
integration: TestUpdateContainerResources_MemoryLimit: remove TODO co…
Browse files Browse the repository at this point in the history
…mment

This comment was added in 09a0c94 when the
Windows integration tests were enabled. The mentioned PR (microsoft/hcsshim#931)
which was in hcsshim v0.9.0, and support for resource limits on Windows was added
in 2bc77b8, so it looks like this comment is no
longer current.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
  • Loading branch information
thaJeztah committed Sep 5, 2022
1 parent edd59aa commit c058011
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion integration/container_update_resources_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -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")

Expand Down

0 comments on commit c058011

Please sign in to comment.