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

Revert chromedriver update #50324

Merged
merged 11 commits into from
Nov 12, 2019
Merged

Revert chromedriver update #50324

merged 11 commits into from
Nov 12, 2019

Conversation

flash1293
Copy link
Contributor

@flash1293 flash1293 commented Nov 12, 2019

Just a test whether this fails the tests.

Last week a chromedriver update got merged in and now there are some tests flaky on 7.5 and 7.x and also sometimes in master: https://github.com/elastic/kibana/issues?q=is%3Aopen+is%3Aissue+assignee%3Aflash1293+label%3Afailed-test

This PR is here to test whether those failures are caused by the update. If this is the case, the update should be reverted on all branches.

@bhavyarm
Copy link
Contributor

cc @dmlemeshko FYI. Thanks @flash1293

@flash1293
Copy link
Contributor Author

Ping @dmlemeshko - I checked the commit history and this one is close to the point in time things started to go south. Not sure whether that's just bad luck though.

Did you run the new version somewhere else a few times?
In the PR itself there is one failing run https://github.com/elastic/kibana/runs/281743734 but maybe that is caused by something else.

Also I'm not sure whether this is a 7.x or a master thing - most failures seem to be realted to 7.x but on master-related PRs there are also strange failures.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@flash1293
Copy link
Contributor Author

Jenkins, test this.

@elasticmachine
Copy link
Contributor

💔 Build Failed

@flash1293 flash1293 changed the title Revert chrome Revert chromedriver update Nov 12, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@jbudz jbudz marked this pull request as ready for review November 12, 2019 21:29
@jbudz jbudz added release_note:skip Skip the PR/issue when compiling release notes v7.5.0 v7.6.0 v8.0.0 labels Nov 12, 2019
@jbudz jbudz merged commit 5c6eb9c into elastic:master Nov 12, 2019
jbudz pushed a commit that referenced this pull request Nov 12, 2019
* deprecate include_type_name

* include_type_name

* remove doc from mappings

* Updated timelion mapping

* Updated spaces and uptime mapping

* monitoring apm mapping

* Updated more mappings

* 2 more mappings

* Updated reporting mappings after syncing with @gammon

* Revert "update chromedriver dependency to v78 (#49737)"

This reverts commit 4a696b9.
jbudz pushed a commit that referenced this pull request Nov 12, 2019
* deprecate include_type_name

* include_type_name

* remove doc from mappings

* Updated timelion mapping

* Updated spaces and uptime mapping

* monitoring apm mapping

* Updated more mappings

* 2 more mappings

* Updated reporting mappings after syncing with @gammon

* Revert "update chromedriver dependency to v78 (#49737)"

This reverts commit 4a696b9.
tylersmalley pushed a commit to tylersmalley/kibana that referenced this pull request Nov 13, 2019
kertal pushed a commit to kertal/kibana that referenced this pull request Nov 13, 2019
* deprecate include_type_name

* include_type_name

* remove doc from mappings

* Updated timelion mapping

* Updated spaces and uptime mapping

* monitoring apm mapping

* Updated more mappings

* 2 more mappings

* Updated reporting mappings after syncing with @gammon

* Revert "update chromedriver dependency to v78 (elastic#49737)"

This reverts commit 4a696b9.
jloleysens added a commit to jloleysens/kibana that referenced this pull request Nov 13, 2019
* upstream/master:
  Remove internal platform types exports (elastic#50427)
  [APM] Document `apm_oss.metricsIndices` and `apm_oss.sourcemap… (elastic#50312)
  [Telemetry] Server side fetcher (elastic#50015)
  [SIEM] Detection engine placeholders (elastic#50220)
  [Uptime] Donut chart loader position centered vertically  (elastic#50219)
  update telemetry banner notice text (elastic#50403)
  Fix aborting when searching without batching (elastic#49966)
  [Telemetry] Remove telemetry splash page and add conditional messaging (elastic#50189)
  Revert chromedriver update (elastic#50324)
  Remove deprecated argument include_type_name from ES calls (elastic#50285)
  [Maps] add settings to maps telemetry (elastic#50161)
  remove visualize loader (elastic#46910)
  Fix misuse of react-router and react-router-dom (elastic#50120)
This was referenced Nov 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v7.4.3 v7.5.0 v7.6.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants