search

fboaventura / dckr-mrtg

:whale: Docker MRTG Instance that will configure monitoring from the environment variable.
https://hub.docker.com/r/fboaventura/dckr-mrtg/
MIT License
26 stars 17 forks source link

Small improvements #13

Closed TweakM closed 1 year ago

TweakM commented 1 year ago

This PR adds three things:

  1. Correct spelling error in mrtg.cfg
  2. Only run indexmaker if the index file does not exist. Without this, each container restart the index will be regenerated and appended to the existing file.
  3. Add the capability to specify a USERID and GROUPID for lighttpd.
TweakM commented 1 year ago

@fboaventura : Thank you! Will you also update the images on Docker Hub? (and maybe also the documentation there). That will allow everyone to use the improvements.

fboaventura commented 1 year ago

@TweakM, the image is already published. I'm dealing with a known issue with the auto-build that fails before publishing the documentation.