just-containers / socklog-overlay

Small syslog add-on for s6-overlay
Other
20 stars 3 forks source link

socklog does not work out of the box with read-only rootfs #6

Open mikroskeem opened 4 years ago

mikroskeem commented 4 years ago

When using docker's --read-only and s6-overlay's S6_READ_ONLY_ROOT=1, then /var/log directory needs to be created before socklog can be used. Otherwise socklog yields error similar to #2 nonstop.

Considering that this is an add-on to s6-overlay, then perhaps this overlay could install an additional script setting up /var/log directory in case of S6_READ_ONLY_ROOT is enabled?

cat > /etc/cont-init.d/00-init-varlog <<EOF
#!/usr/bin/execlineb -P
mkdir -p /var/log
EOF
jprjr commented 4 years ago

There's already a cont-init.d script in this repo that's calling mkdir -p for /var/log/socklog/etc...

Are you running with a writable volume at /var?

On Mon, Apr 27, 2020 at 04:07:07PM -0700, Mark Vainomaa wrote:

When using docker's --read-only and s6-overlay's S6_READ_ONLY_ROOT=1, then /var/log directory needs to be created before socklog can be used. Otherwise socklog yields error similar to #2 nonstop.

Considering that this is an add-on to s6-overlay, then perhaps this overlay could install an additional script setting up /var/log directory in case of S6_READ_ONLY_ROOT is enabled?

cat > /etc/cont-init.d/00-init-varlog <<EOF
#!/usr/bin/execlineb -P
mkdir -p /var/log
EOF

-- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/just-containers/socklog-overlay/issues/6

mikroskeem commented 4 years ago

Yes, /var is writable (it's tmpfs, I followed https://github.com/just-containers/s6-overlay#read-only-root-filesystem)