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

googlechrome.vm fails when Github action image updates to latest chrome #1138

Closed
emtuls opened this issue Sep 16, 2024 · 1 comment · Fixed by #1146
Closed

googlechrome.vm fails when Github action image updates to latest chrome #1138

emtuls opened this issue Sep 16, 2024 · 1 comment · Fixed by #1146
Assignees
Labels
🐛 bug Something isn't working

Comments

@emtuls
Copy link
Member

emtuls commented Sep 16, 2024

What's the problem?

As noted in this comment, when the image that is used by Github Actions updates to use the latest Google Chrome, an error occurs with the googlechrome.vm installation because the same version is already installed, thus causing the package to fail to install and any subsequent package that depends on it.

Second, when this package fails to install, it also causes an error for other packages that depend on googlechrome.vm, one of those being cyberchef.vm, and this raises the error seen in #1111.
When googlechrome.vm is not installed, the path to the browser is not in the location we expect, which also causes a failure for finding the icon in the path we are using.
This particular part of the issue will be resolved when we change the icon for CyberChef to be something other than Google Chrome in this PR: #1054

See error below for what I mean by googlechrome.vm failing to install in the latest run. The image was recently updated to use the latest chrome browser (https://github.com/actions/runner-images/releases/tag/win22%2F20240912.1) and this issue has been noted in other areas as an issue that can occur: yarnpkg/yarn#1647 (comment)
image

Steps to Reproduce

View our daily logs when Github Action images are updated to latest chrome as noted here: https://github.com/actions/runner-images/releases.

Environment

Environment Agnostic

Depends on Github Actions Images and when they update to latest chrome.

Additional Information

No response

Detected by test suite

No

@emtuls emtuls added the 🐛 bug Something isn't working label Sep 16, 2024
@emtuls emtuls changed the title googlechrome.vm package fails when Github action image updates to latest chrome. googlechrome.vm fails when Github action image updates to latest chrome. Sep 16, 2024
@emtuls emtuls changed the title googlechrome.vm fails when Github action image updates to latest chrome. googlechrome.vm fails when Github action image updates to latest chrome Sep 16, 2024
@Ana06
Copy link
Member

Ana06 commented Sep 30, 2024

It seems we are not checking valid exit codes (concretely the 1603 in this case) of the installer in the googlechrome package as we do in the VM-Install-With-Installer helper function and in the donkan.vm package.

@emtuls would you like to add the exit codes check to fix it? 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants