Problem running from inside a NodeJs app inside a Docker container

[realies]

Problem running from inside a NodeJs app inside a Docker container

[realies]

@Grumpier, why are you trying to run a container within a container?

[Grumpier]

Thanks for getting back to me. I don’t know how else to communicate with your image. Any suggestions? Sent from my iPhoneOn 9 Mar 2023, at 21:14, realies ***@***.***> wrote: @Grumpier, why are you trying to run a container within a container? —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[realies]

@Grumpier, any reason not to install the audiowaveform binary inside the container?

[Grumpier]

Yes, that’s the obvious solution, though I was hoping to keep the app as compartmentalized as possible and I’m a complete Docker newbie. Sent from my iPhoneOn 9 Mar 2023, at 21:48, realies ***@***.***> wrote: @Grumpier, any reason not to install the audiowaveform binary inside the container? —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[realies]

Running a container inside a container is not optimal. If there's no reason to do so, you can install the audiowaveform binary inside the nodejs app container.

[Grumpier]

Yes, thanks. Any tips on the best way to ensure my nodejs app has access to the audiowave binary inside the container? Do I still call it with a child process as I would on a host system? Do I need to assign an ENTRYPOINT for the built binary? As I said, I’m a newbie. Sent from my iPhoneOn 10 Mar 2023, at 14:22, realies ***@***.***> wrote: Running a container inside a container is not optimal. If there's no reason to do so, you can install the audiowaveform binary inside the nodejs app container. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[realies]

Install the audiowaveform binary in the container or bake it into the image to be able to run it from within your nodejs app.

[Grumpier]

But how do I "bake it in"? If I modify my Dockerfile to something like this: # syntax=docker/dockerfile:1 FROM realies/audiowaveform FROM node:18-alpine WORKDIR /app COPY ["package*.json", "./"] COPY ./ ./ RUN npm i CMD ["npm", "run", "start"] audiowaveform is still not found in the shell. outlandaudiotrackssite-server-1 | /bin/sh: audiowaveform: not found

…

On Mar 10, 2023, at 17:10, realies ***@***.***> wrote: Install the audiowaveform binary in the container or bake it into the image to be able to run it from within your nodejs app. — Reply to this email directly, view it on GitHub <#10 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADDAUBZSSPH6KWXA4NTZTZLW3NG6NANCNFSM6AAAAAAVMALED4>. You are receiving this because you were mentioned.

[realies]

FROM ubuntu:latest
RUN apt-get update && \
 apt-get install software-properties-common curl -y
RUN curl -fsSL https://deb.nodesource.com/setup_18.x | bash - && \
 apt-get install -y nodejs
RUN add-apt-repository ppa:chris-needham/ppa && \
 apt-get update && \
 apt-get install audiowaveform -y
COPY ./app /app
WORKDIR /app
RUN npm i
CMD ["npm", "start"]

You can uninstall software-properties-common and curl after they are called. You can then use audiowaveform in node with something like:

const util = require('util');
const exec = util.promisify(require('child_process').exec);
(async () => {
  console.log(await exec('audiowaveform --version'));
})();

You should have a look at more examples or read the docs: https://docs.docker.com/reference/; https://nodejs.org/api/child_process.html

[Grumpier]

Thanks! I’ll check this out. Sent from my iPhoneOn 10 Mar 2023, at 20:58, realies ***@***.***> wrote: FROM ubuntu:latest RUN apt-get update && \ apt-get install software-properties-common curl -y RUN curl -fsSL https://deb.nodesource.com/setup_18.x | bash - && \ apt-get install -y nodejs RUN add-apt-repository ppa:chris-needham/ppa && \ apt-get update && \ apt-get install audiowaveform -y COPY ./app /app WORKDIR /app RUN npm i CMD ["npm", "start"] You can uninstall software-properties-common and curl after they are called. You can then use audiowaveform in node with something like: const util = require('util'); const exec = util.promisify(require('child_process').exec); (async () => { console.log(await exec('audiowaveform --version')); })(); You should have a look at more examples or read the docs: https://docs.docker.com/reference/; https://nodejs.org/api/child_process.html —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[Grumpier]

Your additions to my Dockerfile worked perfectly. Thanks. I didn’t realize you can just apt-get install node into ubuntu. Duh!Sent from my iPhoneOn 10 Mar 2023, at 20:58, realies ***@***.***> wrote: FROM ubuntu:latest RUN apt-get update && \ apt-get install software-properties-common curl -y RUN curl -fsSL https://deb.nodesource.com/setup_18.x | bash - && \ apt-get install -y nodejs RUN add-apt-repository ppa:chris-needham/ppa && \ apt-get update && \ apt-get install audiowaveform -y COPY ./app /app WORKDIR /app RUN npm i CMD ["npm", "start"] You can uninstall software-properties-common and curl after they are called. You can then use audiowaveform in node with something like: const util = require('util'); const exec = util.promisify(require('child_process').exec); (async () => { console.log(await exec('audiowaveform --version')); })(); You should have a look at more examples or read the docs: https://docs.docker.com/reference/; https://nodejs.org/api/child_process.html —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>