Project

General

Profile

Bug #28028

chromedriver version mismatch with jenkins chromium version

Added by Tomer Brisker 11 months ago. Updated 9 days ago.

Status:
Closed
Priority:
Urgent
Assignee:
Category:
Tests
Target version:
-

Description

jenkins chromium got auto updated to 77 causing all integration tests to fail with:

Selenium::WebDriver::Error::SessionNotCreatedError: session not created: Chrome version must be between 71 and 75
  (Driver info: chromedriver=2.46.628388 (4a34a70827ac54148e092aafb70504c4ea7ae926),platform=Linux 3.10.0-1062.1.1.el7.x86_64 x86_64)
    test/integration_test_helper.rb:275:in `login_admin'
    test/integration_test_helper.rb:304:in `before_setup' (Selenium::WebDriver::Error::SessionNotCreatedError)
/usr/local/rvm/gems/ruby-2.5.1@test_develop_pr_core-3/gems/selenium-webdriver-3.142.6/lib/selenium/webdriver/remote/response.rb:34    


Related issues

Has duplicate Foreman - Refactor #29916: Remove chromedriver optional dependencyDuplicate

Associated revisions

Revision a45b0d04 (diff)
Added by Tomer Brisker 11 months ago

Refs #28028 - update chromedriver to match jenkins version

This is a stopgap to get tests running again, we should find a solution
that doesn't require manual update every time a new version of chromium
is released.

Revision a62400a1 (diff)
Added by Michael Moll 9 months ago

Refs #28028 - update chromedriver to match jenkins version

Revision 0576daff (diff)
Added by Michael Moll 4 months ago

Refs #28028 - update chromedriver to match jenkins version

Revision 4526bb9e (diff)
Added by Ewoud Kohl van Wijngaarden 4 months ago

Fixes #28028 - Drop chromedriver dependency

This prefers the NPM path it used to look at, but falls back to looking
in $PATH.

History

#1 Updated by The Foreman Bot 11 months ago

  • Assignee set to Tomer Brisker
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7099 added

#2 Updated by The Foreman Bot 9 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7327 added

#3 Updated by The Foreman Bot 7 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7450 added

#4 Updated by Tomer Brisker 7 months ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/7450)

#5 Updated by The Foreman Bot 4 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7690 added

#6 Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Has duplicate Refactor #29916: Remove chromedriver optional dependency added

#7 Updated by The Foreman Bot 4 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7695 added

#8 Updated by The Foreman Bot 4 months ago

  • Fixed in Releases 2.2.0 added

#9 Updated by Tomer Brisker 4 months ago

  • Fixed in Releases 2.0.1, 2.1.0 added
  • Fixed in Releases deleted (2.2.0)

#10 Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Status changed from Ready For Testing to Closed

#11 Updated by The Foreman Bot 9 days ago

  • Pull request https://github.com/theforeman/foreman/pull/7975 added

Also available in: Atom PDF