nagios-charmers team mailing list archive

[Chris Sanders <chris.sanders@xxxxxxxxxxxxx>]

I've answered the question in-line.

Diff comments:

> diff --git a/src/lib/hwhealth/tools.py b/src/lib/hwhealth/tools.py
> index e18b464..fd6692d 100644
> --- a/src/lib/hwhealth/tools.py
> +++ b/src/lib/hwhealth/tools.py
> @@ -270,6 +298,7 @@ class Sas3Ircu(VendorTool):
>      """
>      def __init__(self):
>          super().__init__(shortname='sas3ircu')
> +        self.checksums = ['f150eb37bb332668949a3eccf9636e0e03f874aecd17a39d586082c6be1386bd']

Including checksum in the charm for customer audit is the point of this merge and it's not optional. There is no record of changes if someone rebuilds the charm and pushes it to the charm store with a different set of files and no code changes to the charm.

>  
>  
>  class Sas2Ircu(VendorTool):


-- 
https://code.launchpad.net/~aluria/hw-health-charm/+git/hw-health-charm/+merge/369278
Your team Nagios Charm developers is subscribed to branch hw-health-charm:master.