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

Downgrade ecs-agent dependency versions to match agent #3768

Closed
wants to merge 2 commits into from

Conversation

amogh09
Copy link
Contributor

@amogh09 amogh09 commented Jun 27, 2023

Summary

Downgrading the versions of the following two ecs-agent dependencies to match agent module versions.

  • github.com/docker/docker downgraded from v24.0.2 to v20.10.24.
  • Transitive dependency github.com/docker/go-units downgraded from v0.5.0 to v0.4.0.

ecs-agent became dependent on these modules in #3754. I confirmed with the author of that PR that we don't need the latest versions of the above dependencies and that the versions used by agent module are fine. Without this change go mod tidy on agent module bumps the versions of these dependencies for agent module which we don't have a need for at the moment and would require additional qualification testing.

Rest of the changes in this PR are a result of go mod tidy && go mod vendor.

Implementation details

  1. Updated the dependencies as summarized above.
  2. Ran go mod tidy && go mod vendor in ecs-agent module.
  3. Ran go mod tidy && go mod vendor in agent module.

Testing

New tests cover the changes: NA

Description for the changelog

Downgrade ecs-agent dependency versions to match agent

Licensing

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

@amogh09 amogh09 marked this pull request as ready for review June 27, 2023 00:05
@amogh09 amogh09 requested a review from a team as a code owner June 27, 2023 00:05
@amogh09
Copy link
Contributor Author

amogh09 commented Jun 28, 2023

@ohsoo took care of this in #3767

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.

2 participants