Skip to content

Conversation

@DaveCTurner
Copy link
Contributor

Reverts the test added in #123505 - this is not behaviour on which we
rely any more, and it does not apply with SDKv2 anyway.

Reverts the test added in elastic#123505 - this is not behaviour on which we
rely any more, and it does not apply with SDKv2 anyway.
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v9.1.0 labels Apr 2, 2025
@DaveCTurner DaveCTurner requested a review from tlrx April 2, 2025 07:31
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Coordination Meta label for Distributed Coordination team label Apr 2, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-distributed-coordination (Team:Distributed Coordination)

Copy link
Member

@tlrx tlrx 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 15899af into elastic:main Apr 2, 2025
17 checks passed
@DaveCTurner DaveCTurner deleted the 2025/04/02/remove-testWriteBlobWithExceptionThrownAtClosingTime branch April 2, 2025 08:43
andreidan pushed a commit to andreidan/elasticsearch that referenced this pull request Apr 9, 2025
Reverts the test added in elastic#123505 - this is not behaviour on which we
rely any more, and it does not apply with SDKv2 anyway.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed Coordination Meta label for Distributed Coordination team >test Issues or PRs that are addressing/adding tests v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants