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

mimir-build-image: Upgrade to Node.js 20 #6540

Merged
merged 2 commits into from
Nov 2, 2023
Merged

mimir-build-image: Upgrade to Node.js 20 #6540

merged 2 commits into from
Nov 2, 2023

Conversation

aknuds1
Copy link
Contributor

@aknuds1 aknuds1 commented Nov 2, 2023

What this PR does

Upgrade build image to latest Node.js LTS, v20. We are currently on v14, which is deprecated and produces a huge warning while building the image.

Which issue(s) this PR fixes or relates to

Checklist

  • [na] Tests updated
  • [na] Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Copy link
Contributor

github-actions bot commented Nov 2, 2023

Building new version of mimir-build-image. After image is built and pushed to Docker Hub, a new commit will automatically be added to this PR with new image version `grafana/mimir-build-image:pr6540-8b3fe039e9`. This can take up to 1 hour.

Copy link
Member

@pstibrany pstibrany left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I hope it's as easy as bumping the version in setup script :)

@aknuds1
Copy link
Contributor Author

aknuds1 commented Nov 2, 2023

@pstibrany do you have any idea where we use Node.js? I couldn't figure that out (main reason why the PR is a draft).

@pstibrany
Copy link
Member

@pstibrany do you have any idea where we use Node.js? I couldn't figure that out (main reason why the PR is a draft).

RUN npm install -g [email protected]

This seems to be the only thing we need it for.

@aknuds1 aknuds1 marked this pull request as ready for review November 2, 2023 14:35
@aknuds1 aknuds1 requested review from wilfriedroset, vaxvms, bubu11e and a team as code owners November 2, 2023 14:35
Copy link
Contributor

github-actions bot commented Nov 2, 2023

Not building new version of mimir-build-image. This PR modifies `mimir-build-image/Dockerfile`, but the image `grafana/mimir-build-image:pr6540-8b3fe039e9` already exists.

@aknuds1 aknuds1 enabled auto-merge (squash) November 2, 2023 14:46
@aknuds1 aknuds1 merged commit 0f2715b into main Nov 2, 2023
@aknuds1 aknuds1 deleted the arve/node branch November 2, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants