-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add required packages #1513
Add required packages #1513
Conversation
I suspect that this is due to the change from I assume It might be more robust to switch to |
That would make sense, although I thought there was a headless variant rather than it being the default... In any case, the missing libraries do seem to indicate that.
Will give that a try! |
Marking this PR as draft, to avoid this being included in the daily message to the team via https://github.com/guardian/actions-prnouncer. Please undo this if I'm mistaken! |
Closing as we've preferred an alternative solution (using a different version of java) than adding these packages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Co-authored-by: David Furey <david.furey@guardian.co.uk>
What does this change?
Related to this prodmon PR and enables it to work, essentially - otherwise this will lead to an
UnsatisfiedLinkError
:I haven't dug deeply, but from what I can tell these used to get installed as part of the java 8 role, but for some reason don't anymore after switching to java 11 - which actually can be confirmed from a couple of bakes of the browser-testing-jammy-ARM-WITH-cdk-base-java11: this first one was accidentally still set to use java 8, but when I baked a second one correctly using java 11, the package changes log shows those have not been installed.
From what I can tell they are required to enable selenium/chromedriver to work, so it feels reasonable to include them in the role directly.
How to test
The prodmon PR has temporarily been set to use an AMI baked in code with this role - deploy that to prodmon secondary, and the app should keep working as expected.