nagios-charmers team mailing list archive

[Paul Goins <paul.goins@xxxxxxxxxxxxx>]

Review: Approve

Based on discussion with Xav: +1.

Diff comments:

> diff --git a/.gitignore b/.gitignore
> index 8f6789a..3fbd340 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -17,3 +17,5 @@ __pycache__/
>  
>  # version data
>  src/repo-info
> +
> +tools.zip

Discussed; quite intentional.  Very well.

> diff --git a/src/lib/utils/hwdiscovery.py b/src/lib/utils/hwdiscovery.py
> index c144fa4..480fea2 100644
> --- a/src/lib/utils/hwdiscovery.py
> +++ b/src/lib/utils/hwdiscovery.py
> @@ -39,8 +39,6 @@ def get_tools():
>          tools.add('hpe')  # ['hponcfg', 'ilorest']
>      elif hw_vendor == 'HP':
>          tools.add('hp')  # ['hponcfg', 'hp-health', 'hpssacli']
> -    elif hw_vendor == 'Supermicro':
> -        tools.add('sas3ircu')

Discussed; this code apparently is some old legacy logic which probably should be done away with.  +1 on removal.

>  
>      # SW RAID?
>      if _supports_mdadm():


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