Problem/Motivation

When migrating drupalci_environments images from dockerhub to gitlab.com registry, noticed that core CI is using drupalci `chromedriver` image.

At some point the drupalci image build processed changed, changes to chromedriver only go to the image titled `webdriver-chromedriver` (last touched 1 year ago), `chromedriver` image is last touched >2 years ago.

Doubtful this is causing any issues, but probably would make sense to update.

Steps to reproduce

Can compare timestamp of production tag of these two images on dockerhub (https://hub.docker.com/r/drupalci/webdriver-chromedriver/tags, https://hub.docker.com/r/drupalci/chromedriver/tags). Though we now use registry at https://gitlab.com/drupal-infrastructure/drupalci/drupalci-environments/... (but all images were synced over yesterday so timestamps don't communicate it).

Otherwise I reviewed drupalci_environments build CI, and it pushes to `webdriver-chromedriver`.

Proposed resolution

Update core CI references from `$_CONFIG_DOCKERHUB_ROOT/chromedriver:production` to `$_CONFIG_DOCKERHUB_ROOT/webdriver-chromedriver:production`

Remaining tasks

Update core CI references from `$_CONFIG_DOCKERHUB_ROOT/chromedriver:production` to `$_CONFIG_DOCKERHUB_ROOT/webdriver-chromedriver:production`

Release notes snippet

Fix CI chromedriver image reference to use updated image

Issue fork drupal-3517217

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

maxwhitehead created an issue. See original summary.

maxwhitehead’s picture

Status: Active » Needs review
andypost’s picture

fjgarlin’s picture

Status: Needs review » Reviewed & tested by the community

RTBC. The shorter syntax was an alias.

Registry images: https://gitlab.com/drupal-infrastructure/drupalci/drupalci-environments/...
Dockerfile inside the folders (the path is used to generate the image name): https://git.drupalcode.org/project/drupalci_environments/-/tree/dev/webd...

andypost’s picture

Would be great to have a follow-up to decide on policy to update the images as browsers has a very fast release cycle

  • longwave committed fd2d1df7 on 10.4.x
    Issue #3517217 by maxwhitehead, fjgarlin: CI: Using stale drupalci...

  • longwave committed e51f36f4 on 10.5.x
    Issue #3517217 by maxwhitehead, fjgarlin: CI: Using stale drupalci...

  • longwave committed bb1a18b7 on 11.1.x
    Issue #3517217 by maxwhitehead, fjgarlin: CI: Using stale drupalci...

  • longwave committed 82f500d0 on 11.x
    Issue #3517217 by maxwhitehead, fjgarlin: CI: Using stale drupalci...
longwave’s picture

Version: 11.x-dev » 10.4.x-dev
Status: Reviewed & tested by the community » Fixed

Backported down to all active branches to keep things in sync.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

longwave’s picture

Version: 10.4.x-dev » 11.0.x-dev

Reverted from 10.5.x and 10.4.x as this caused a failure in a FunctionalJavascript test.

longwave’s picture

Version: 11.0.x-dev » 11.1.x-dev