-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: remove "web files" options leftovers #12403
chore: remove "web files" options leftovers #12403
Conversation
@@ -1,9 +1,6 @@ | |||
--- system/netdata.conf.orig 2018-05-10 19:44:49.000000000 +0300 | |||
+++ system/netdata.conf 2018-05-10 19:45:14.000000000 +0300 | |||
@@ -22,3 +22,7 @@ | |||
[web] | |||
web files owner = root | |||
web files group = netdata | |||
+ | |||
+[health] | |||
+ # script for sles 11, mail notifications only |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That is likely wrong and I need properly to regenerate the patch @Ferroin, correct? If so I just undo the commit 馃槃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SLES 11 is EOL. Maybe we just remove the patches, @Ferroin?
Summary
Should have been removed in #12339, but managed to survive 馃槃
@thiagoftsm @vlvkobal there are 2 more lines
If it is easy to remove and you think it is worth time (probably not), feel free to push additional commits in this branch.
Test Plan
Not needed
Additional Information