If hardware changes (or charm changes) and we want to remove a check, the charm doesn't do so

Bug #1819801 reported by Xav Paice
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
hw-health-charm
Won't Fix
Medium
Unassigned

Bug Description

When fixing lp1819733, I came about the situation where install() provides a different list of tools to the one already stored by kv.set('tools', list(tools)) on a previous iteration. New checks would have been added, but the one I wanted removed wasn't removed. We need to include some read of the 'old' value and check if there's a difference, if there is then we want to remove the old checks.

Suggest adding a kv.get('tools', set()) just before the kv.set, and then checking the difference, using remove_nrpe_checks() to remove the differences.

Xav Paice (xavpaice)
Changed in hw-health-charm:
importance: Undecided → Medium
Revision history for this message
Eric Chen (eric-chen) wrote :

Close very old issue

Changed in charm-hw-health:
status: New → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.