Skip to content
This repository was archived by the owner on Jan 5, 2025. It is now read-only.

Create docker-compose.yml #29

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

Conversation

gzcwnk
Copy link

@gzcwnk gzcwnk commented Apr 15, 2021

Docker-compose example.

This might assist ppl in writing a docker-compose.yml for themselves

Docker-compose example.
@robinsmidsrod
Copy link
Contributor

I think the example would be better served if it was part of the README instead of as a stand-alone file.

@cavazquez
Copy link
Contributor

cavazquez commented Feb 4, 2022

version: 3 is not necessary. you should delete
network: vlan is better than host.
latest is bad practice. change to v1.2.0
limit cpus and memory with cpus: 1 , mem_limit: "500M"
/home/dhcp change to REPLACE
TZ only work in Auckland

- /home/dhcp:/data
network_mode: "host"
environment:
- TZ=Pacific/Auckland
Copy link
Contributor

@yscialom yscialom Jul 5, 2022

Choose a reason for hiding this comment

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

Should not be part of the PR, as it is a personal setting.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can you mount read-only /etc/localtime instead and get the same behavior?

restart: unless-stopped
ports:
# Public dhcpd Port:
- "67:67/udp"
Copy link
Contributor

Choose a reason for hiding this comment

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

Port publishing is meaningless in the host network.

version: "3"
services:
dhcp:
image: networkboot/dhcpd:latest
Copy link
Contributor

Choose a reason for hiding this comment

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

Use a specific version instead of :latest.

# Public dhcpd Port:
- "67:67/udp"
volumes:
- /home/dhcp:/data
Copy link
Contributor

@yscialom yscialom Jul 5, 2022

Choose a reason for hiding this comment

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

Should not be part of the PR, as it is a personal setting.
Prefer a relative path, - ./data:/data maybe.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants