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
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:
- 3517217-ci-using-stale
changes, plain diff MR !11725
Comments
Comment #3
maxwhiteheadComment #4
andypostComment #5
fjgarlin commentedRTBC. 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...
Comment #6
andypostWould be great to have a follow-up to decide on policy to update the images as browsers has a very fast release cycle
Comment #11
longwaveBackported down to all active branches to keep things in sync.
Comment #14
longwaveReverted from 10.5.x and 10.4.x as this caused a failure in a FunctionalJavascript test.
Comment #15
longwave