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

Update windows builds to golang v1.17 #3010

Merged
merged 12 commits into from
Sep 10, 2021

Conversation

vsiddharth
Copy link
Contributor

Summary

Update agent builds for windows to use golang v1.17

Implementation details

  • Update agent builds for windows to use go v1.17
  • Updategolang.org/x/sys v0.0.0-20190830141801-acfa387b8d69
  • Vendor dependencies
  • Temporarily skip failing integration tests only on WS2016 platform

Testing

An EC2 container instance was setup manually:

  • Unit and integration tests pass on WS 2019
  • Unit and integration tests pass on WS 2016

New tests cover the changes: Yes

Description for the changelog

Enhancement - Update windows builds to golang version v1.17

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
@angelcar
Copy link
Contributor

angelcar commented Sep 1, 2021

TestHandlerDoesntLeakGoroutines Unit Test fails consistently

Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
@vsiddharth
Copy link
Contributor Author

TestHandlerDoesntLeakGoroutines Unit Test fails consistently

This test has been quite flaky but after the recent changes, this test appears to be fine.
There have been a couple of other failures which I've been looking into.

Signed-off-by: Siddharth Vinothkumar <sidvin@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants