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

upgrade chromedriver 2.21.2 -> 2.22.1 #7859

Merged
merged 2 commits into from
Aug 3, 2016

Conversation

ppisljar
Copy link
Member

@ppisljar ppisljar commented Jul 27, 2016

tests involving alert or confirm boxes were failing for me with "unexpected alert box" error message.

it also happened at CI ... but cant find that log now.

@ppisljar
Copy link
Member Author

jenkins, test this

@ppisljar
Copy link
Member Author

@LeeDr can you reproduce this error on your end ?

@LeeDr
Copy link

LeeDr commented Jul 28, 2016

Here's a link to the jenkins test failure;
https://kibana-ci.elastic.co/job/elastic+kibana+pull-request+multijob-selenium/61/consoleFull

And a bit of that log in case it gets cleaned up;

>> PASS: chrome on any platform - settings app - index result field sort - sort by heading - name - should sort ascending (490ms)
14:17:07.040: tryForTime failure: [GET http://localhost:4444/wd/hub/session/220e6f2a10e9a3ca86a35ad97cf56482/alert_text] unknown error: cannot determine loading status
from unexpected alert open
  (Session info: chrome=51.0.2704.106)
  (Driver info: chromedriver=2.21.371461 (633e689b520b25f3e264a2ede6b74ccc23cb636a),platform=Linux 3.13.0-74-generic x86_64)
14:17:07.549: tryForTime failure: [POST http://localhost:4444/wd/hub/session/220e6f2a10e9a3ca86a35ad97cf56482/element / {"using":"css selector","value":"button.btn.btn-danger.ng-scope"}] unexpected alert open: {Alert text : Are you sure you want to remove this index pattern?}
  (Session info: chrome=51.0.2704.106)
  (Driver info: chromedriver=2.21.371461 (633e689b520b25f3e264a2ede6b74ccc23cb636a),platform=Linux 3.13.0-74-generic x86_64)
14:17:08.058: tryForTime failure: [POST http://localhost:4444/wd/hub/session/220e6f2a10e9a3ca86a35ad97cf56482/element / {"using":"css selector","value":"button.btn.btn-danger.ng-scope"}] unexpected alert open: {Alert text : Are you sure you want to remove this index pattern?}

I was able to reproduce the problem on a Ubuntu VM, but only about 1 in 5 runs of the management tests caused it. I've experimented with a couple of different changes to narrow down the problem, try to work-around it, and capture details to file an issue on Intern (or for chromedriver).
I don't have anything concrete yet...

FYI, I run the tests in a loop like this so that it stops when it fails;

# in one shell start the ui:server
npm run test:ui:server

# in another shell run just the management tests in a loop until they fail
export count=0; while ( time npm run test:ui:runner -- --appSuites=management ); do let count=count+1; echo $count; sleep 5; done

@LeeDr
Copy link

LeeDr commented Jul 29, 2016

jenkins, test this

@LeeDr
Copy link

LeeDr commented Aug 1, 2016

The tests passed on the last build. Test again. jenkins, test this

@LeeDr
Copy link

LeeDr commented Aug 1, 2016

jenkins, test this

@LeeDr LeeDr changed the title downgrading chromedriver downgrading chromedriver 2.21.2 -> 2.18.0 Aug 1, 2016
@LeeDr
Copy link

LeeDr commented Aug 1, 2016

@ppisljar Can you confirm you've run the management (or all the functional tests) a few times locally and they pass with chromedriver 2.18.0 ?

@LeeDr
Copy link

LeeDr commented Aug 1, 2016

Another test passed. Testing again.

@LeeDr
Copy link

LeeDr commented Aug 1, 2016

jenkins, test this

@ppisljar
Copy link
Member Author

ppisljar commented Aug 1, 2016

i can confirm. i ran tests 5 times in a row and they always passed

@LeeDr
Copy link

LeeDr commented Aug 1, 2016

LGTM

I tested with the downgraded chromedriver locally and tests are passing on jenkins-ci. In the last build/test on jenkins the tests passed but there was a timeout updating github;

Finished Build : #173 of Job : elastic+kibana+pull-request+multijob-selenium with status : SUCCESS
java.lang.NullPointerExceptionSetting status of 13d7db3ebef77102f9441f403cd7bcf2443acfe6 to SUCCESS with url https://kibana-ci.elastic.co/job/elastic-kibana-pull-request/277/ and message: 'Build finished. '
Using context: kibana-ci
Could not update commit status of the Pull Request on GitHub.
java.net.SocketTimeoutException: connect timed out

@ppisljar
Copy link
Member Author

ppisljar commented Aug 2, 2016

latest version of chromedriver seems to fix this issue ... upgrading to latest (2.22.1)

@Bargs
Copy link
Contributor

Bargs commented Aug 3, 2016

I ran into the same problem during testing yesterday, and upgrading to 2.22.1 worked for me as well. LGTM

@LeeDr LeeDr changed the title downgrading chromedriver 2.21.2 -> 2.18.0 upgrade chromedriver 2.21.2 -> 2.22.1 Aug 3, 2016
@LeeDr
Copy link

LeeDr commented Aug 3, 2016

LGTM

@LeeDr LeeDr merged commit a7942b1 into elastic:master Aug 3, 2016
@ppisljar ppisljar deleted the downgradeChromeDriver branch August 4, 2016 05:40
elastic-jasper added a commit that referenced this pull request Aug 17, 2016
---------

**Commit 1:**
downgrading chromedriver

* Original sha: 13d7db3
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-07-27T15:18:03Z

**Commit 2:**
upgrading chromedriver to latest version

* Original sha: 0a1838e
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-08-02T11:55:47Z
elastic-jasper added a commit that referenced this pull request Aug 17, 2016
---------

**Commit 1:**
downgrading chromedriver

* Original sha: 13d7db3
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-07-27T15:18:03Z

**Commit 2:**
upgrading chromedriver to latest version

* Original sha: 0a1838e
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-08-02T11:55:47Z
epixa added a commit that referenced this pull request Aug 18, 2016
---------

**Commit 1:**
downgrading chromedriver

* Original sha: 13d7db3
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-07-27T15:18:03Z

**Commit 2:**
upgrading chromedriver to latest version

* Original sha: 0a1838e
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-08-02T11:55:47Z
epixa added a commit that referenced this pull request Aug 18, 2016
[backport] PR #7859 to 4.x - Bump chromedriver
elastic-jasper added a commit that referenced this pull request Aug 18, 2016
---------

**Commit 1:**
downgrading chromedriver

* Original sha: 13d7db3
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-07-27T15:18:03Z

**Commit 2:**
upgrading chromedriver to latest version

* Original sha: 0a1838e
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-08-02T11:55:47Z
epixa added a commit that referenced this pull request Aug 18, 2016
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
upgrade chromedriver 2.21.2 -> 2.22.1

Former-commit-id: a7942b1
tkajtoch added a commit that referenced this pull request Jul 13, 2024
`v95.2.0`⏩`v95.3.0`

_[Questions? Please see our Kibana upgrade
FAQ.](https://github.com/elastic/eui/blob/main/wiki/eui-team-processes/upgrading-kibana.md#faq-for-kibana-teams)_

---

## [`v95.3.0`](https://github.com/elastic/eui/releases/v95.3.0)

- Updated `EuiThemeProvider`s to allow modifying/setting custom
`breakpoint`s in nested usage (as opposed to only at the top
`EuiProvider` level) ([#7862](elastic/eui#7862))

**Bug fixes**

- Fixed a Chrome/Edge CSS `mask-image` bug that was affecting scroll
overflow shadow utilties
([#7855](elastic/eui#7855))

**CSS-in-JS conversions**

- Converted `EuiColorPicker` to Emotion; Removed `$euiColorPickerWidth`
([#7845](elastic/eui#7845))
- Converted `EuiColorPickerSwatch` to Emotion
([#7853](elastic/eui#7853))
- Converted `EuiColorPalettePicker` and `EuiColorPaletteDisplay` to
Emotion ([#7854](elastic/eui#7854))
  - Removed `$euiColorPaletteDisplaySizes`
  - Removed `@mixin euiColorPaletteInnerBorder`
- Removed `$euiColorPickerValueRange0`, `$euiColorPickerValueRange1`,
`$euiColorPickerSaturationRange0`, `$euiColorPickerSaturationRange1`,
and `$euiColorPickerIndicatorSize`
([#7859](elastic/eui#7859))

**Accessibility**

- Updated the `aria-label` attribute for the `EuiFilePicker` remove file
button ([#7860](elastic/eui#7860))

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
cee-chen pushed a commit to cee-chen/kibana that referenced this pull request Sep 12, 2024
`v95.2.0`⏩`v95.3.0`

_[Questions? Please see our Kibana upgrade
FAQ.](https://github.com/elastic/eui/blob/main/wiki/eui-team-processes/upgrading-kibana.md#faq-for-kibana-teams)_

---

- Updated `EuiThemeProvider`s to allow modifying/setting custom
`breakpoint`s in nested usage (as opposed to only at the top
`EuiProvider` level) ([elastic#7862](elastic/eui#7862))

**Bug fixes**

- Fixed a Chrome/Edge CSS `mask-image` bug that was affecting scroll
overflow shadow utilties
([elastic#7855](elastic/eui#7855))

**CSS-in-JS conversions**

- Converted `EuiColorPicker` to Emotion; Removed `$euiColorPickerWidth`
([elastic#7845](elastic/eui#7845))
- Converted `EuiColorPickerSwatch` to Emotion
([elastic#7853](elastic/eui#7853))
- Converted `EuiColorPalettePicker` and `EuiColorPaletteDisplay` to
Emotion ([elastic#7854](elastic/eui#7854))
  - Removed `$euiColorPaletteDisplaySizes`
  - Removed `@mixin euiColorPaletteInnerBorder`
- Removed `$euiColorPickerValueRange0`, `$euiColorPickerValueRange1`,
`$euiColorPickerSaturationRange0`, `$euiColorPickerSaturationRange1`,
and `$euiColorPickerIndicatorSize`
([elastic#7859](elastic/eui#7859))

**Accessibility**

- Updated the `aria-label` attribute for the `EuiFilePicker` remove file
button ([elastic#7860](elastic/eui#7860))

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants