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

WIP for adding optional etc configurations and mount using drive UID #18

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

Conversation

cdbattags
Copy link

@cdbattags cdbattags commented May 28, 2021

Hi @iximiuz,

Just wanted to get this up and viewable for my current progress. By no means is this ready for a merge but I'd like to discuss how we'd like to optionally package bridge packages in a separate Dockerfile.bridge as a multi-stage build step.

Also, we could add a separate context for specifying these sorts of etc config files.

A few questions to start:

  • Do you have anything against moving to ext4 format?
  • Any way to specify the raw image to have a size rounded to nearest 100MB instead of the 1GB default? This is just based off the linux.dir folder size, right?

Let me know what you think. Thank ya.

As is now, this will properly boot on Digital Ocean without the bridge.

@iximiuz
Copy link
Owner

iximiuz commented May 31, 2021

Hi Christian!

Many thanks for your contribution! It looks really impressive and I'm glad to learn that you were able to make it work on DigitalOcean!

Regarding your questions...

  • ext3 -> ext4 is already done, please see this commit. ext3 probably shouldn't even have been used in the first place.
  • for the image size, indeed, 1 GB default is there because of the biggest distr size I tried to build. It's a good idea to make it configurable in the Makefile.

@iximiuz
Copy link
Owner

iximiuz commented May 31, 2021

FYI, there is another PR #14 with an attempt to add networking support. Probably these two PRs could be consolidated in one to avoid competing efforts.

Luis (@pathcl), you may find this PR interesting.

@iximiuz
Copy link
Owner

iximiuz commented May 31, 2021

As I already mentioned in #14, it'd be great to keep the bare minimal image as-is and allow it to be easily built with a simple Makefile target. That's great to have it as a starting point for further endeavors and also for educational purposes.

Multistage Docker builds can be leveraged to add the networking-related packages and configs, however, it'd complicate the base Dockerfile. I'm more inclined to add a separate Dockerfile.network starting FROM the base Docker image and adjusting the Makefile to use it when needed. WDYT about such approach?

@cdbattags
Copy link
Author

Multistage Docker builds can be leveraged to add the networking-related packages and configs, however, it'd complicate the base Dockerfile. I'm more inclined to add a separate Dockerfile.network starting FROM the base Docker image and adjusting the Makefile to use it when needed. WDYT about such approach?

I think that's the best bet as well. We'll just need to sync on how we want the tagging done for the bases.

@iximiuz
Copy link
Owner

iximiuz commented Jun 2, 2021

Currently, it's just docker-to-linux/debian (or any other distr name). We may consider renaming it to docker-to-linux/debian-base, and add a new layer for networking docker-to-linux/debian-network. Maybe eventually there will be more tree-shaped layers.

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.

2 participants