diff options
author | thiagoftsm <thiagoftsm@gmail.com> | 2019-09-03 18:30:26 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-09-03 18:30:26 +0000 |
commit | 0798426c7dbfc290b1f6f6e397063a09ee8a3447 (patch) | |
tree | 943caf03981652bdbd7cc3faead2205ee66ef748 /tests/Makefile.am | |
parent | 4d975c271258eaaef6b4aceabb2ff5f3212be879 (diff) |
Fix clear notification missing (#6638)
* alarm_clear: Mapping
In this PR I mapped all the necessary steps to discover the solution for the ISSUE 6581
* alarm_clear: Documentation and fixes
This commit fixes the problem that were present in Netdata and it also updates
the documentation of the functions and Netdata.
* alarm_clear: shell script
The original implementation did not have a shell script, here I begin to fix this
* alarm_clear: shell script
It is necessay to verify why make is not producing the same binary than cmake and finish the changes in the script
* alarm_clear: adjust in health.c
I rewrote the health.c to be more readable, but I discovered the problem I had in the last few hours
were due kernel update
* alarm_clear: script changes
In this commit I am bringing the final version of the script that
test the alarm repetition
* alarm_clear: script fix and remove comments
IN this commit I am fixing the shellcheck errors and removing some debug messages
that were present in the code while I was developing
* alarm_clear: Format
The health.c had wrong tabulation, this PR brings back the pattern of space as tab for this file
* alarm_clear: Script
The script was using killlall that is not more present in all Linux distribution
this commit removes this and bring the new way to stop Netdata
* alarm_clear: return to previous tabulation
I am bringing back the old tabulation here and I will create a new PR
exclusively for this
* alarm_clear: Remove comments
I am removing comments from this PR to keep the focus in the major problem
* alarm_clear: Remove comments 2
I forgot one comment
* alarm_clear: New variable
I am appending a new variable in the check before the rebase, because the health.c changed in other file
has a direct relationship with what I did here until now
* alarm_clear: Fix clear repetition
With this last commit, I am bringing a new way to raise the clear alarm, but it is not repeating more
with this fix, it displayed one time when it is cleaned and it will display the message again, if and only if,
the alarm was raised.
Diffstat (limited to 'tests/Makefile.am')
-rw-r--r-- | tests/Makefile.am | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tests/Makefile.am b/tests/Makefile.am index 0aa5af247f..179b04864b 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -7,6 +7,7 @@ CLEANFILES = \ health_mgmtapi/health-cmdapi-test.sh \ acls/acl.sh \ urls/request.sh \ + alarm_repetition/alarm.sh \ $(NULL) include $(top_srcdir)/build/subst.inc @@ -26,12 +27,14 @@ dist_noinst_DATA = \ health_mgmtapi/health-cmdapi-test.sh.in \ acls/acl.sh.in \ urls/request.sh.in \ + alarm_repetition/alarm.sh.in \ $(NULL) dist_plugins_SCRIPTS = \ health_mgmtapi/health-cmdapi-test.sh \ acls/acl.sh \ urls/request.sh \ + alarm_repetition/alarm.sh \ $(NULL) dist_noinst_SCRIPTS = \ |