Skip to content

Commit

Permalink
Fix failing windows tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Julien Duchesne committed Jul 26, 2019
1 parent 5d04c87 commit cd42477
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions agent/api/task/task_windows_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -259,8 +259,10 @@ func TestCPUPercentBasedOnUnboundedEnabled(t *testing.T) {

func TestWindowsMemoryReservationOption(t *testing.T) {
// Testing sending a task to windows overriding MemoryReservation value
rawHostConfigInput := docker.HostConfig{
MemoryReservation: nonZeroMemoryReservationValue,
rawHostConfigInput := dockercontainer.HostConfig{
Resources: dockercontainer.Resources{
MemoryReservation: nonZeroMemoryReservationValue,
},
}

rawHostConfig, err := json.Marshal(&rawHostConfigInput)
Expand All @@ -272,10 +274,10 @@ func TestWindowsMemoryReservationOption(t *testing.T) {
Arn: "arn:aws:ecs:us-east-1:012345678910:task/c09f0188-7f87-4b0f-bfc3-16296622b6fe",
Family: "myFamily",
Version: "1",
Containers: []*Container{
Containers: []*apicontainer.Container{
{
Name: "c1",
DockerConfig: DockerConfig{
DockerConfig: apicontainer.DockerConfig{
HostConfig: strptr(string(rawHostConfig)),
},
},
Expand Down

0 comments on commit cd42477

Please sign in to comment.