Skip to content
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.

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

[CI] Reactivate 3rd party tests on CI [2nd] #32353

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