librenms / docker

LibreNMS Docker image
MIT License
691 stars 278 forks source link

SNMPTRAPD: misspelling of variable causing the container not to run when correct spelling is used in envournment #220

Closed MarlinMr closed 3 years ago

MarlinMr commented 3 years ago

Problem was introduced in by #180.

SIDECAR_SNNMPTRAPD was spelled with 2 Ns. This request changes it to the correct 1 N.

This is probably the actual cause for #218 that was "fixed" by #219. The typo will cause the setup not to run, should correct spelling be used by people running it.

Because the spelling error was also put into examples, it is possible that people are running the container with the error, and this merge will break it. The documentation seems to be correct. I suspect @glance- was running it with the error.

MarlinMr commented 3 years ago

Rebase your branch, thx.

I think I fixed it