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

Handle wrapped IO exceptions in test cluster distribution syncing #82288

Conversation

mark-vieira
Copy link
Contributor

Third attempt at sorting out this problem after #82154 didn't do the trick. The issue is that FileTreeIterator wraps the root cause exception in an UncheckedIOException so to detect this scenario properly we actually have to inspect the cause.

@mark-vieira mark-vieira added >non-issue :Delivery/Build Build or test infrastructure labels Jan 5, 2022
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Jan 5, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@mark-vieira mark-vieira added auto-backport-and-merge Automatically create backport pull requests and merge when ready auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v7.16.3 v7.17.0 v8.0.0 and removed v8.1.0 labels Jan 5, 2022
@mark-vieira
Copy link
Contributor Author

@elasticmachine update branch

@elasticsearchmachine elasticsearchmachine merged commit 01d2b78 into elastic:master Jan 6, 2022
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Jan 6, 2022
…astic#82288)

Third attempt at sorting out this problem after elastic#82154 didn't do the
trick. The issue is that `FileTreeIterator` wraps the root cause
exception in an `UncheckedIOException` so to detect this scenario
properly we actually have to inspect the cause.
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Jan 6, 2022
…astic#82288)

Third attempt at sorting out this problem after elastic#82154 didn't do the
trick. The issue is that `FileTreeIterator` wraps the root cause
exception in an `UncheckedIOException` so to detect this scenario
properly we actually have to inspect the cause.
mark-vieira added a commit to mark-vieira/elasticsearch that referenced this pull request Jan 6, 2022
…astic#82288)

Third attempt at sorting out this problem after elastic#82154 didn't do the
trick. The issue is that `FileTreeIterator` wraps the root cause
exception in an `UncheckedIOException` so to detect this scenario
properly we actually have to inspect the cause.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.0
7.17
7.16

elasticsearchmachine pushed a commit that referenced this pull request Jan 6, 2022
…2288) (#82300)

Third attempt at sorting out this problem after #82154 didn't do the
trick. The issue is that `FileTreeIterator` wraps the root cause
exception in an `UncheckedIOException` so to detect this scenario
properly we actually have to inspect the cause.
elasticsearchmachine pushed a commit that referenced this pull request Jan 6, 2022
…2288) (#82298)

Third attempt at sorting out this problem after #82154 didn't do the
trick. The issue is that `FileTreeIterator` wraps the root cause
exception in an `UncheckedIOException` so to detect this scenario
properly we actually have to inspect the cause.
elasticsearchmachine pushed a commit that referenced this pull request Jan 6, 2022
…2288) (#82299)

Third attempt at sorting out this problem after #82154 didn't do the
trick. The issue is that `FileTreeIterator` wraps the root cause
exception in an `UncheckedIOException` so to detect this scenario
properly we actually have to inspect the cause.
@mark-vieira mark-vieira deleted the testcluster_tmp_file_fix_10000 branch January 6, 2022 22:54
astefan pushed a commit to astefan/elasticsearch that referenced this pull request Jan 7, 2022
…astic#82288)

Third attempt at sorting out this problem after elastic#82154 didn't do the
trick. The issue is that `FileTreeIterator` wraps the root cause
exception in an `UncheckedIOException` so to detect this scenario
properly we actually have to inspect the cause.
astefan pushed a commit to astefan/elasticsearch that referenced this pull request Jan 7, 2022
…astic#82288)

Third attempt at sorting out this problem after elastic#82154 didn't do the
trick. The issue is that `FileTreeIterator` wraps the root cause
exception in an `UncheckedIOException` so to detect this scenario
properly we actually have to inspect the cause.
@pugnascotia pugnascotia removed the v8.0.0 label Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport-and-merge Automatically create backport pull requests and merge when ready auto-merge Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v7.16.3 v7.17.0 v8.0.0-rc2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants