Pin Version of google-chrome-stable
Package
#51975
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Right now, we have a couple different versions of this package installed across our servers. Checking with
apt-cache policy google-chrome-stable
reveals:The discrepancy between staging and test specifically is causing some problems; some apps code using newer browser features fails to build on test but passes on staging, which can lead to confusingly delayed DTT failures when staging builds the apps package at first and test only attempts to build some time after the offending PR has been merged.
To resolve, we explicitly specify which version we should target. I've chosen to target the version currently installed on staging because it's closest to the version we target for our UI tests, but we could alternatively choose to update to version 113.
Links
See threads at https://codedotorg.slack.com/archives/C0T0PNR0D/p1683573120446859 and https://codedotorg.slack.com/archives/C0T0PNTM3/p1684359034466229and for more context.
Testing story
Verified on an adhoc that we correctly install the targeted version.