Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

As of #133030 Elasticsearch uses S3's support for conditional writes to
protect against repository corruption. It is possible that some other
storage system may claim to be fully S3-compatible despite rejecting
such requests.

This commit adds a repository setting that allows to make all writes
unconditional, unsafely disabling the corruption protection, but
allowing users some time to work with their storage supplier to address
the incompatibility.

Backport of #137185 and #138406 to 9.2

As of elastic#133030 Elasticsearch uses S3's support for conditional writes to
protect against repository corruption. It is possible that some other
storage system may claim to be fully S3-compatible despite rejecting
such requests.

This commit adds a repository setting that allows to make all writes
unconditional, unsafely disabling the corruption protection, but
allowing users some time to work with their storage supplier to address
the incompatibility.

Backport of elastic#137185 and elastic#138406 to 9.2
@DaveCTurner
Copy link
Contributor Author

I thought that #133030 landed in 9.3.0 but I was remembering wrong, it actually took effect in 9.2.0, so we need to backport this change to the 9.2 series too.

@DaveCTurner DaveCTurner requested a review from mhl-b November 27, 2025 19:00
Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@DaveCTurner DaveCTurner merged commit cdc1b4c into elastic:9.2 Nov 28, 2025
34 checks passed
@DaveCTurner DaveCTurner deleted the 2025/11/27/s3-conditional-writes-option-92 branch November 28, 2025 07:28
DaveCTurner added a commit to DaveCTurner/elastic-docs-content that referenced this pull request Nov 28, 2025
This change was backported to 9.2.3 in elastic/elasticsearch#138735,
updating the `applies_to` marker to match.
DaveCTurner added a commit to elastic/docs-content that referenced this pull request Nov 28, 2025
)

This change was backported to 9.2.3 in elastic/elasticsearch#138735,
updating the `applies_to` marker to match.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants