cherami-server icon indicating copy to clipboard operation
cherami-server copied to clipboard

clean-up/rename storageMonitor -> spaceMon

Open kirg opened this issue 6 years ago • 6 comments

  • Fixes multiple minor issues in the 'storageMonitor'
    • potential race enabling/disabling write (create/stoppage of channel)
    • new go-routine created every cycle
    • ticker not stopped
    • avoid logging when not in warn/alert condition
    • remove two-way dependency between storehost and spaceMon
    • minor annoyance: rename 'storagemonitor" to "spaceMon" to reduce file completion collision with "storehost.go" on 'stor< tab >' .. :-)

kirg avatar Nov 20 '17 21:11 kirg

Coverage Status

Coverage decreased (-0.2%) to 66.31% when pulling 87269bc88d8c50732b7d6ae07291864d4ce85948 on spaceMon into 27bdb2ba00f8f22cc97593526e2ec7920c316071 on master.

coveralls avatar Nov 20 '17 22:11 coveralls

Coverage Status

Changes Unknown when pulling 10cf0d2668f15431eefbb9bbc978233645c85e86 on spaceMon into ** on master**.

coveralls avatar Nov 21 '17 01:11 coveralls

Coverage Status

Changes Unknown when pulling 2954dd31196c7ff9873500701ce8aa22ecd8c2de on spaceMon into ** on master**.

coveralls avatar Nov 21 '17 03:11 coveralls

Coverage Status

Changes Unknown when pulling 6b01dc9c7ce9abe95dec9606a1c2e4e827aec355 on spaceMon into ** on master**.

coveralls avatar Nov 21 '17 06:11 coveralls

Coverage Status

Changes Unknown when pulling c22fe7edabcbbee7dc5496de758ae3e2d3f50753 on spaceMon into ** on master**.

coveralls avatar Nov 21 '17 07:11 coveralls

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


kiranrg seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jun 27 '19 13:06 CLAassistant