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

Migrate ec2 package to aws-sdk-to-v2 #4446

Merged
merged 11 commits into from
Dec 10, 2024
Merged

Conversation

tinnywang
Copy link
Contributor

Summary

Migrate the EC2 and EC2 metadata clients in the ec2 package to aws-sdk-go-v2.

Implementation details

Testing

New tests cover the changes: no

Description for the changelog

Migrate ec2 package to aws-sdk-to-v2.

Licensing

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

@tinnywang tinnywang marked this pull request as ready for review December 3, 2024 19:35
@tinnywang tinnywang requested a review from a team as a code owner December 3, 2024 19:35
@tinnywang tinnywang force-pushed the aws-sdk-go-v2/ec2 branch 3 times, most recently from b985a4d to c95cc58 Compare December 4, 2024 22:48
@mye956
Copy link
Contributor

mye956 commented Dec 4, 2024

Thanks a bunch for making this change!

ecs-agent/ec2/ec2_client.go Outdated Show resolved Hide resolved
ecs-agent/ec2/ec2_client.go Outdated Show resolved Hide resolved
ecs-agent/ec2/ec2_metadata_client.go Show resolved Hide resolved
ecs-agent/ec2/ec2_metadata_client.go Outdated Show resolved Hide resolved
@tinnywang tinnywang force-pushed the aws-sdk-go-v2/ec2 branch 2 times, most recently from 14ff880 to 58d51ed Compare December 6, 2024 08:34
@tinnywang tinnywang merged commit dd3ad7e into aws:dev Dec 10, 2024
40 checks passed
@tinnywang tinnywang deleted the aws-sdk-go-v2/ec2 branch December 10, 2024 10:27
BinSquare pushed a commit to BinSquare/amazon-ecs-agent that referenced this pull request Dec 23, 2024
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.

4 participants