Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wip] vendor: update aws-sdk-go to v1.12.5 #1006

Closed
wants to merge 1 commit into from
Closed

Conversation

adnxn
Copy link
Contributor

@adnxn adnxn commented Oct 5, 2017

Summary

updating aws-sdk-go to v1.12.5: b489cce

Implementation details

n/a

Testing

  • Builds on Linux (make release)
  • Builds on Windows (go build -out amazon-ecs-agent.exe ./agent)
  • Unit tests on Linux (make test) pass
  • Unit tests on Windows (go test -timeout=25s ./agent/...) pass
  • Integration tests on Linux (make run-integ-tests) pass
  • Integration tests on Windows (.\scripts\run-integ-tests.ps1) pass
  • Functional tests on Linux (make run-functional-tests) pass
  • Functional tests on Windows (.\scripts\run-functional-tests.ps1) pass

New tests cover the changes:
no

Description for the changelog

n/a

Licensing

This contribution is under the terms of the Apache 2.0 License:
yes

@adnxn adnxn requested review from aaithal and petderek October 5, 2017 20:02
@adnxn adnxn mentioned this pull request Oct 5, 2017
8 tasks
"Rev": "b489cce1dd1ada60f519fe0dc70c096379a42214"
},
{
"ImportPath": "github.com/aws/aws-sdk-go/vendor/github.com/go-ini/ini",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should not be picked up and appears to be an error; nested vendors should be excluded by godep.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hrm apparently it's a known issue tools/godep#498 (comment). ill try the work around described there.

@adnxn adnxn changed the title vendor: update aws-sdk-go to v1.12.5 [wip] vendor: update aws-sdk-go to v1.12.5 Oct 11, 2017
@adnxn
Copy link
Contributor Author

adnxn commented Oct 11, 2017

blocked on dep migration #1009

@aaithal
Copy link
Contributor

aaithal commented Oct 11, 2017

blocked on dep migration #1009

@adnxn why? The two should be independent of each other

@adnxn adnxn added this to the 1.15.3 milestone Nov 8, 2017
@adnxn
Copy link
Contributor Author

adnxn commented Nov 15, 2017

closing since aws-sdk-go was updated with the cgroup changes.

@adnxn adnxn closed this Nov 15, 2017
@nmeyerhans nmeyerhans modified the milestones: 1.15.3, 1.16.0 Nov 15, 2017
@adnxn adnxn deleted the sdk-update branch March 14, 2018 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants