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 PR (microsoft/hcsshim#931) was
merged, and part of 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 authored and Kirtana Ashok committed Jan 18, 2023
1 parent 9a522ff commit 52bb846
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 52bb846

Please sign in to comment.