Skip to content

Fix ordering problem in add or renew lease test#38280

Merged
jasontedor merged 1 commit intoelastic:masterfrom
jasontedor:add-or-renew-ordering-fix
Feb 3, 2019
Merged

Fix ordering problem in add or renew lease test#38280
jasontedor merged 1 commit intoelastic:masterfrom
jasontedor:add-or-renew-ordering-fix

Conversation

@jasontedor
Copy link
Copy Markdown
Member

@jasontedor jasontedor commented Feb 3, 2019

We have to set the primary term before we add a retention lease, otherwise we can not assert the correct primary term.

Closes #38245
Relates #38275

We have to set the primary term before we add a retention lease,
otherwise we can not assert the correct primary term.
@jasontedor jasontedor added >test-failure Triaged test failures from CI v7.0.0 :Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. v6.7.0 labels Feb 3, 2019
@jasontedor jasontedor requested a review from dnhatn February 3, 2019 13:46
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

Copy link
Copy Markdown
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.

@jasontedor jasontedor merged commit d2cc145 into elastic:master Feb 3, 2019
@jasontedor jasontedor deleted the add-or-renew-ordering-fix branch February 3, 2019 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Distributed A catch all label for anything in the Distributed Area. Please avoid if you can. >test-failure Triaged test failures from CI v6.7.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants