Skip to content

Mute testDeleteActionDeletesSearchableSnapshot #54818

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
Apr 6, 2020
Merged

Mute testDeleteActionDeletesSearchableSnapshot #54818

merged 1 commit into from
Apr 6, 2020

Conversation

andreidan
Copy link
Contributor

@andreidan andreidan commented Apr 6, 2020 •
edited
Loading

No description provided.

@andreidan andreidan added :Data Management/ILM+SLM Index and Snapshot lifecycle management >test-mute Use for PR that only mute tests labels Apr 6, 2020
@andreidan andreidan requested a review from tlrx April 6, 2020 15:09
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/ILM+SLM)

tlrx
tlrx approved these changes Apr 6, 2020
View reviewed changes
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, maybe #54433 should unmute this?

@andreidan andreidan merged commit e1fb09a into elastic:master Apr 6, 2020
@tlrx tlrx mentioned this pull request Apr 7, 2020
Merged
tlrx added a commit that referenced this pull request Apr 7, 2020
@tlrx @DaveCTurner
@ywelsch @dakrone @andreidan
Merge feature/searchable-snapshots branch into 7.x (#54803) (#54825)
4d36917 
This is a backport of #54803 for 7.x.

This pull request cherry picks the squashed commit from #54803 with the additional commits:

    6f50c92 which adjusts master code to 7.x
    a114549 to mute a failing ILM test (#54818)
    48cbca1 and 50186b2 that cleans up and fixes the previous test
    aae12bb that adds a missing feature flag (#54861)
    6f330e3 that adds missing serialization bits (#54864)
    bf72c02 that adjust the version in YAML tests
    a51955f that adds some plumbing for the transport client used in integration tests

Co-authored-by: David Turner <david.turner@elastic.co>
Co-authored-by: Yannick Welsch <yannick@welsch.lu>
Co-authored-by: Lee Hinman <dakrone@users.noreply.github.com>
Co-authored-by: Andrei Dan <andrei.dan@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tlrx tlrx tlrx approved these changes

Assignees
No one assigned
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >test-mute Use for PR that only mute tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@andreidan @elasticmachine @tlrx