Skip to content

Conversation

@adomenech73
Copy link
Contributor

@adomenech73 adomenech73 commented Sep 27, 2016

I just slightly modified the arm Dockerfile to make it run on my Pine64 card cluster.
Just changed to the frozeneye nodejs repository, so in essence it works exactly as arm arch.

To run it on dedicated hardware I build the container as:

docker $(docker-machine config pine64-1) build -f Dockerfile.aarch64 -t visualizer-aarch64 .

And I run it as:

docker $(docker-machine config pine64-1) run -it -d --privileged -p 8080:8080 -e HOST=pine64-1 -v /var/run/docker.sock:/var/run/docker.sock --name visualizer-aarch64_box visualizer-aarch64

Note the "--privileged" parameter, needed to overpass permissions problems

Hope it's helpful for anyone besides me

@ManoMarks
Copy link
Contributor

@alexellis would you take a look as my ARM expert?

@@ -0,0 +1,29 @@
FROM frozeneye/aarch64-nodejs:latest
Copy link
Contributor

@alexellis alexellis Sep 30, 2016

Choose a reason for hiding this comment

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

Out of interest - do you know much about the frozeneye user on the Hub?

I saw this in their build log:

 ---> Running in 7639fbe42eba
Removing intermediate container 7639fbe42eba
rpc error: code = 2 desc = "oci runtime error: exec format error"

https://hub.docker.com/r/frozeneye/aarch64-nodejs/builds/b4q72yulmlpht9ggd3ytmlv/

I would say this looks good to me - but would like to know more about the frozeneye account / images.

Copy link
Contributor

Choose a reason for hiding this comment

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

No idea. @adomenech73 is that your account?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nope!! but dockerfile is here Github and I think (because the keys) it's based on official node dockerfiles Github, I didn't crosscheck every reference but seems fair enough for me.

The only difference I see, is that uses aarch64/debian repository Dockerhub that is the only source I found for aarch64 on dockerhub. I didn't audit everything on detail but seemed coherent to me.

Copy link
Member

Choose a reason for hiding this comment

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

Sounds risky if it's from an unknown source - that person could just as easily change the Dockerfile to anything else. Given that this image has to be run with access to the docker socket, that means the author of that image effectively has root access to your hosts

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's true, didn't realize. If it can help I can change the pull request to not make reference to this image and instead construct it directly from aarch64/debian

Copy link
Contributor

Choose a reason for hiding this comment

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

That may be a better option to derive from aarch64 (providing that account is also known). The concern about mounting the socket is also valid but not specific to this pull request.

Copy link
Member

Choose a reason for hiding this comment

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

Yes, the aarch64 repo are maintained by @tianon, and are basically the "official" images (in progress) https://hub.docker.com/r/aarch64/debian/, those images can be trusted

@adomenech73
Copy link
Contributor Author

adomenech73 commented Oct 1, 2016

@alexellis @thaJeztah What about something like this?

@alexellis
Copy link
Contributor

Liking that @adomenech73.

@thaJeztah as a follow-up maybe we can ask Tianon to create an official node base image under aarch64?

@thaJeztah
Copy link
Member

@alexellis perhaps he can help, but the official NodeJs images is not maintained by him, but by NodeJs upstream; https://github.com/nodejs/docker-node

@tianon
Copy link

tianon commented Oct 1, 2016 via email

@thaJeztah
Copy link
Member

Wow, thanks @tianon !

@adomenech73
Copy link
Contributor Author

@tianon Image simplifies a lot the readability. I used 4.6.0-slim tags because seems to be the most similar one to the main Dockerfile

@ManoMarks ManoMarks merged commit edfdc74 into dockersamples:master Oct 2, 2016
@adomenech73 adomenech73 deleted the aarch64-support branch October 2, 2016 21:33
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.

5 participants