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

adding dockerfile #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shanehughes1990
Copy link

This adds a simple docker file for container deployment, Adding in the redwood-config for a default setup, users will need to create the cert and mount into the container but otherwise works.

A sample docker run command to get going, this requires you to have a cert.pem/key.pem in your current working directory.

docker run -d -p 6502:6502 -p 6510:6510 \
	-v $PWD/cert.pem:/etc/redwood/root.pem \
	-v $PWD/key.pem:/etc/redwood/root_key.pem \
	docker.io/library/redwood:latest

@andybalholm
Copy link
Owner

Adding a dockerfile sounds like a good idea. It should probably go in the startup directory along with the init scripts. It would be good to provide some documentation (maybe in a README-docker.md file, not just in the pull request comment).

@shanehughes1990
Copy link
Author

perfect I can do that, I wasn't sure how/where you wanted it all Ill move things around and create a doc for it.

@elico
Copy link
Contributor

elico commented Oct 16, 2023

The docker file is built for amd64 platforms only but should be built in such a way that would allow cross platform builds.
I want to build a container for ARM and ARM64 devices such as Mikrotik RouterOS devices which I already have.

@shanehughes1990
Copy link
Author

yeah you build with buildx this is just something I figured I would put a PR in for because I needed one for our use case.

@elico
Copy link
Contributor

elico commented Nov 1, 2023

@shanehughes1990 I am working on a buildx compatible Dockerfile, there is no reason to use a global Dockerfile with CPU specific build instructions.

@elico
Copy link
Contributor

elico commented Nov 1, 2023

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.

3 participants