Skip to content

Conversation

vladimirdolzhenko
Copy link
Contributor

Initial PR #32315 was not sufficient:

https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+third-party-tests/31/

It is not possible to reproduce it locally - therefore I ran CI for my branch and got green build:

https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+third-party-tests/33/

@vladimirdolzhenko vladimirdolzhenko added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v7.0.0 labels Jul 25, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

Copy link
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

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

LGTM

@vladimirdolzhenko
Copy link
Contributor Author

@elasticmachine test this please

1 similar comment
@vladimirdolzhenko
Copy link
Contributor Author

@elasticmachine test this please

@vladimirdolzhenko vladimirdolzhenko merged commit a254716 into elastic:master Jul 25, 2018
@vladimirdolzhenko
Copy link
Contributor Author

thanks @ywelsch

@vladimirdolzhenko vladimirdolzhenko deleted the reactive-third-party-tests-on-ci-2 branch July 25, 2018 14:34
dnhatn added a commit that referenced this pull request Jul 26, 2018
* master:
  [DOCS] Fix formatting error in Slack action
  Painless: Fix documentation links to use existing refs (#32335)
  Painless: Decouple PainlessLookupBuilder and Whitelists (#32346)
  [DOCS] Adds recommendation for xpack.security.enabled (#32345)
  [TEST] Mute ConvertProcessortTests.testConvertIntHexError
  [TEST] Fix failure due to exception message in java11 (#32321)
  [DOCS] Fixes typo in ML aggregations page
  [DOCS] Adds link from bucket_span property to common time units
  [ML][DOCS] Add documentation for detector rules and filters (#32013)
  Add opaque_id to index audit logging (#32260)
  Add 6.5.0 version to master
  fixes broken build for third-party-tests (#32353)
vladimirdolzhenko added a commit that referenced this pull request Nov 20, 2018
vladimirdolzhenko added a commit that referenced this pull request Nov 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >test Issues or PRs that are addressing/adding tests v7.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants