Skip to content

Conversation

@squadgazzz
Copy link
Contributor

It is not possible to use latest versions without updating cargo lockfile in the cowprotocol/services repo, which caused memory consumption issues, so this PR downgrades aws libs to versions that don't require that anymore.

Copilot AI review requested due to automatic review settings September 4, 2025 08:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR downgrades AWS SDK libraries to avoid memory consumption issues that occur when using the latest versions without updating the Cargo lockfile in the cowprotocol/services repository.

  • Downgrades aws-config from version 1.8.6 to 1.5.1
  • Downgrades aws-sdk-kms from version 1.86 to 1.5.1

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@squadgazzz squadgazzz merged commit 8817f4b into main Sep 4, 2025
6 checks passed
@squadgazzz squadgazzz deleted the update-aws-libs-2 branch September 4, 2025 08:38
squadgazzz added a commit to cowprotocol/services that referenced this pull request Sep 4, 2025
Replicates #3599 after reverting it in #3627, which caused some memory
issues after updating the lockfile. This version uses downgraded aws lib
versions to avoid updating the lockfile.

Depends on cowprotocol/ethcontract-rs#982

Tested on mainnet-shadow, looks fine.
xdecentralix pushed a commit to xdecentralix/services that referenced this pull request Sep 25, 2025
Replicates cowprotocol#3599 after reverting it in cowprotocol#3627, which caused some memory
issues after updating the lockfile. This version uses downgraded aws lib
versions to avoid updating the lockfile.

Depends on cowprotocol/ethcontract-rs#982

Tested on mainnet-shadow, looks fine.
xdecentralix pushed a commit to xdecentralix/services that referenced this pull request Sep 25, 2025
Replicates cowprotocol#3599 after reverting it in cowprotocol#3627, which caused some memory
issues after updating the lockfile. This version uses downgraded aws lib
versions to avoid updating the lockfile.

Depends on cowprotocol/ethcontract-rs#982

Tested on mainnet-shadow, looks fine.
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.

3 participants