Skip to content

Conversation

@findepi
Copy link
Member

@findepi findepi commented Nov 19, 2025

This attempts to reproduce a problem that was observed on a production
environment where S3 client side retries combined with
s3.exclusive-create=true flag (default) where believed to cause
failures during Iceberg maintenance procedures. This test did not
reproduce that problem, but might be useful anyway.

currently based on

- register resources with closer as early as possible
- use toxic name without spaces; it seems that names with spaces don't
  work correctly
- drop redundant container configuration (network alias)
Network chaos (toxiproxy) setup is specific to assertions going to be
performed.  While we could test different setups and assertions with
separate similar classes, this would be a lot of similar boilerplate
code. This commit refactor the test class so that the test method
configures network breakers.
This attempts to reproduce a problem that was observed on a production
environment where S3 client side retries combined with
`s3.exclusive-create=true` flag (default) where believed to cause
failures during Iceberg maintenance procedures.  This test did not
reproduce that problem, but might be useful anyway.
@cla-bot cla-bot bot added the cla-signed label Nov 19, 2025
@findepi findepi marked this pull request as draft November 19, 2025 14:42
@findepi findepi closed this Nov 20, 2025
@findepi findepi deleted the findepi/s3-test-retries branch November 20, 2025 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants