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

fixed running docker compose up ui-mapper on MacOS by upgrading node version #2103

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

DanielJDufour
Copy link
Contributor

@DanielJDufour DanielJDufour commented Jan 23, 2025

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Related Issue

I was running into a Segmentation Fault issue when I was trying to run docker compose up --build ui-mapper.

Describe this PR

Basically just upgrades from Node 20 to Node 23

Screenshots

Screenshot of it working on a MacOS
Screenshot 2025-01-22 at 10 23 24β€―PM

Alternative Approaches Considered

Tried installing pnpm with npm install -g pnpm@version which seemed to help a little but still ran into seg fault issues when running vite. I think the issue is a fundamental issue with older versions of NodeJS.

Review Guide

Notes for the reviewer. How to test this change?
I ran the docker compose command and the application seemed to run. I haven't done any further testing.

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

@github-actions github-actions bot added the bug Something isn't working label Jan 23, 2025
@DanielJDufour DanielJDufour changed the title fixed running docker compose up ui-mapper on MacOS by upgrading node … fixed running docker compose up ui-mapper on MacOS by upgrading node version Jan 23, 2025
@DanielJDufour
Copy link
Contributor Author

DanielJDufour commented Jan 23, 2025

Are we limited to an older version of NodeJS on production? Or should we updated that too?

@spwoodcock
Copy link
Member

spwoodcock commented Jan 23, 2025

Are we limited to an older version of NodeJS on production? Or should we updated that too?

Nope, any version! ❀️docker.

Thanks for getting to the bottom of this πŸ™

@spwoodcock spwoodcock self-requested a review January 23, 2025 13:02
Copy link
Member

@spwoodcock spwoodcock left a comment

Choose a reason for hiding this comment

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

Prod dockerfile too πŸ‘

@spwoodcock
Copy link
Member

Does this work with node 22 by any chance?

That's the latest LTS version, so could be ideal to pin everything to.

But if only 23 works, then its not a big deal πŸ˜„

@DanielJDufour
Copy link
Contributor Author

DanielJDufour commented Jan 24, 2025

Updated both Dockerfiles to Node 22

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 this pull request may close these issues.

2 participants