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

ci: update machine image #391

Merged
merged 2 commits into from
Jul 7, 2022
Merged

Conversation

pvlugter
Copy link
Member

@pvlugter pvlugter commented Jul 7, 2022

Latest proxy docker images (on temurin 17) are not usable on older docker. Update the machine image for CircleCI to get newer versions of everything. I'll manually check this.

@pvlugter pvlugter force-pushed the circleci-machine-image branch 2 times, most recently from 0f35804 to 3991430 Compare July 7, 2022 00:52
@pvlugter
Copy link
Member Author

pvlugter commented Jul 7, 2022

Docker version used now (20.10.14) looks fine for the temurin 17 on jammy images.

@pvlugter pvlugter mentioned this pull request Jul 7, 2022
@pvlugter
Copy link
Member Author

pvlugter commented Jul 7, 2022

That also bumped the provided node and npm versions, which then breaks some installs. Also add nvm install to the builds and tests that didn't have this already. Could make sense to do a bump to the next node LTS version as well (currently on the old maintenance LTS version 14, current LTS is 16).

@pvlugter
Copy link
Member Author

pvlugter commented Jul 7, 2022

As CI is passing now, with old proxy images, will merge now to try #388.

@pvlugter pvlugter merged commit fc563b1 into lightbend:main Jul 7, 2022
@pvlugter pvlugter deleted the circleci-machine-image branch July 7, 2022 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant