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

Work around missing nodejs bug #66

Closed
jasonaowen opened this issue Oct 6, 2021 · 2 comments · Fixed by #69
Closed

Work around missing nodejs bug #66

jasonaowen opened this issue Oct 6, 2021 · 2 comments · Fixed by #69
Assignees
Labels
bug Something isn't working

Comments

@jasonaowen
Copy link
Contributor

We previously discovered a bug in nodesource, and reported it upstream: nodesource/distributions#1262 . They haven't fixed that bug yet, so in the meantime we need to work around it.

Test for the presence of /usr/bin/nodejs during image building, and if it is missing, create it with

update-alternatives --quiet --install /usr/bin/nodejs nodejs \
    /usr/bin/node 50 \
    --slave /usr/share/man/man1/nodejs.1.gz nodejs.1.gz \
    /usr/share/man/man1/node.1.gz
@jasonaowen jasonaowen added the bug Something isn't working label Oct 6, 2021
@xmunoz xmunoz self-assigned this Oct 14, 2021
@xmunoz
Copy link
Contributor

xmunoz commented Oct 14, 2021

@jasonaowen which images need node? All of them? Just backend?

@jasonaowen
Copy link
Contributor Author

@xmunoz yes, just backend, but I think our image building step is for all images in all environments, and the terraform steps are for each entire environment.

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.

2 participants