search

Cacti / cacti

Cacti ™
http://www.cacti.net
GNU General Public License v2.0
1.63k stars 404 forks source link

cmd poller - add availability test for devices without poller items #5641

Closed xmacan closed 7 months ago

xmacan commented 8 months ago

with cmd poller devices without items in poller table aren't tested for up/down state. It is different behavior compared to spine.

From our perspective it is wrong. We have devices when we need to know only status. It causes confusion when user switch between cmd and spine.

TheWitness commented 8 months ago

I have to check this query at scale. Also need a CHANGELOG.

xmacan commented 8 months ago

Changelog added

xmacan commented 8 months ago

I lost my old repository so I cannot fix small issue.

There is error in my code on line 527: $ping->host = $host['id']; fix: $ping->host = $host;

@TheWitness we can discuss about this PR at the weekend :-)

TheWitness commented 8 months ago

Weekend went to crap per my PM.

TheWitness commented 7 months ago

Just waiting for checks to complete, then merging this one.