Skip to content

SearchableSnapshotsIntegTests.testSnapshotOfSearchableSnapshotIncludesNoDataButCanBeRestored() #66414

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 15, 2020
Merged

SearchableSnapshotsIntegTests.testSnapshotOfSearchableSnapshotIncludesNoDataButCanBeRestored() #66414

merged 1 commit into from
Dec 15, 2020

Conversation

@csoulios
Copy link
Contributor

@csoulios csoulios commented Dec 15, 2020

Mute test SearchableSnapshotsIntegTests.testSnapshotOfSearchableSnapshotIncludesNoDataButCanBeRestored()

Relates to #66411

@csoulios
Muted failing test
1e9a7ba 
SearchableSnapshotsIntegTests.
testSnapshotOfSearchableSnapshotIncludesNoDataButCanBeRestored
@csoulios csoulios added :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v8.0.0 >test-mute Use for PR that only mute tests v7.11.0 labels Dec 15, 2020
@elasticmachine elasticmachine added the Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. label Dec 15, 2020
@elasticmachine
Copy link
Collaborator

elasticmachine commented Dec 15, 2020

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

@csoulios csoulios mentioned this pull request Dec 15, 2020
Merged
@csoulios csoulios merged commit 7b74522 into elastic:master Dec 15, 2020
@csoulios csoulios deleted the mute-test-snapshot-master branch December 15, 2020 21:15
DaveCTurner added a commit that referenced this pull request Dec 15, 2020
@DaveCTurner
Revert "Muted failing SearchableSnapshotsIntegTests test (#66414)"
ed4ab8c 
This reverts commit 7b74522 from
`master`. The test fails for 7.x-specific reasons, see #66415 for the
fix.
@DaveCTurner
Copy link
Contributor

DaveCTurner commented Dec 15, 2020

Many thanks for the prompt action here @csoulios. This failure appears only to affect 7.x (it's a subtle change in default behaviour, see #66415) so I have reverted this AwaitsFix on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. >test-mute Use for PR that only mute tests v7.11.0 v8.0.0-alpha1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@csoulios @elasticmachine @DaveCTurner @jakelandis