cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

[cosmetic] solaris wwn fix (#55154)

Browse source 
* Solaris WWN parsing cosmetic fix (commit 924f5b5467)

* simplify module.run_command by removing use_unsafe_shell and remove calling of external grep

* add changelog fragment
This commit is contained in:
Anatoly Pugachev 2019-04-18 23:31:40 +03:00 committed by Brian Coca
parent afb5e02c19
commit 701cf25891
2 changed files with 8 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
changelogs/fragments/fibre_channel_wwn_fact_solaris_minor.yaml Normal file
View file

@ -0,0 +1,2 @@
minor_changes:
- cosmetic change, simplify FC WWN facts gathering on Solaris

7
lib/ansible/module_utils/facts/network/fc_wwn.py
View file

@ -46,17 +46,18 @@ class FcWwnInitiatorFactCollector(BaseFactCollector):
elif sys.platform.startswith('sunos'): elif sys.platform.startswith('sunos'):
""" """
on solaris 10 or solaris 11 should use `fcinfo hba-port` on solaris 10 or solaris 11 should use `fcinfo hba-port`
on solaris 9, `prtconf -pv` TBD (not implemented): on solaris 9 use `prtconf -pv`
""" """
cmd = module.get_bin_path('fcinfo') cmd = module.get_bin_path('fcinfo')
cmd = cmd + " hba-port | grep 'Port WWN'" cmd = cmd + " hba-port"
rc, fcinfo_out, err = module.run_command(cmd, use_unsafe_shell=True) rc, fcinfo_out, err = module.run_command(cmd)
""" """
# fcinfo hba-port | grep "Port WWN" # fcinfo hba-port | grep "Port WWN"
HBA Port WWN: 10000090fa1658de HBA Port WWN: 10000090fa1658de
""" """
if fcinfo_out: if fcinfo_out:
for line in fcinfo_out.splitlines(): for line in fcinfo_out.splitlines():
if 'Port WWN' in line:
data = line.split(' ') data = line.split(' ')
fc_facts['fibre_channel_wwn'].append(data[-1].rstrip()) fc_facts['fibre_channel_wwn'].append(data[-1].rstrip())
elif sys.platform.startswith('aix'): elif sys.platform.startswith('aix'):