Skip to content

Conversation

@tlrx
Copy link
Member

@tlrx tlrx commented Oct 9, 2019

A stupid test bug make it through the tests in #47721 and will trigger CI failures with NPEs.

@tlrx tlrx added >test Issues or PRs that are addressing/adding tests :Distributed Indexing/CCR Issues around the Cross Cluster State Replication features v8.0.0 labels Oct 9, 2019
@tlrx tlrx requested a review from dnhatn October 9, 2019 15:24
@elasticmachine
Copy link
Collaborator

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

Copy link
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

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

LGTM

@tlrx tlrx merged commit 7c5cd5f into elastic:master Oct 9, 2019
@tlrx
Copy link
Member Author

tlrx commented Oct 9, 2019

Thanks for the fast review @dnhatn!

@tlrx tlrx deleted the fix-autofollowtests branch October 9, 2019 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Indexing/CCR Issues around the Cross Cluster State Replication features >test Issues or PRs that are addressing/adding tests v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants