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

Change the Volume for configuration to match the Volume in the titpetric/netdata Dockerfile #3

Merged
merged 1 commit into from May 7, 2018

Conversation

acrelle
Copy link
Contributor

@acrelle acrelle commented May 7, 2018

Change the default netdata configuration to be /etc/netdata/override.

Scripts placed in here are automatically copied into /etc/netdata during the container start script.

/etc/netdata/override is the volume exposed in the original Dockerfile:

https://hub.docker.com/r/titpetric/netdata/~/dockerfile/

@acrelle acrelle changed the title Change the default volume for configuration to match the volume in the titpetric/netdata Dockerfile Change the Volume for configuration to match the Volume in the titpetric/netdata Dockerfile May 7, 2018
@acrelle
Copy link
Contributor Author

acrelle commented May 7, 2018

Raised because I had a brand new install that wasn't triggering alarms, and I was new to netdata. After more time than I care to admit struggling as to why.. I found that overwriting /etc/netdata meant that the alerting and alarms configuration was broken. Inside the container, this folder was empty when using the default Unraid template.

… scripts placed in here are automatically copied into /etc/netdata during the container start script. /etc/netdata/override is the volume exposed in the original Dockerfile. Update Changelog.
@Data-Monkey
Copy link
Owner

Thanks for your contribution, merged!

@Data-Monkey Data-Monkey merged commit bf96289 into Data-Monkey:master May 7, 2018
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.

None yet

2 participants