search

saltstack-formulas / lvm-formula

8 stars 13 forks source link

Fix three minor issues #8

Closed noelmcloughlin closed 5 years ago

noelmcloughlin commented 5 years ago

This PR has three updates.

1/ Hide STDERR from unless/onlyif conditionals.

[ERROR   ] Command '[u'pvdisplay', u'-c', u'/dev/loop0']' failed with return code: 5
[ERROR   ] stderr:   Failed to find physical volume "/dev/loop0".
[ERROR   ] retcode: 5
[ERROR   ] Command '[u'pvdisplay', u'-c', u'/dev/loop0']' failed with return code: 5
[ERROR   ] stderr:   Failed to find physical volume "/dev/loop0".
[ERROR   ] retcode: 5

2/ Create missing remove state documented in README.

    No matching sls found for 'lvm.remove' in env 'base'

3/ Remove config.sls from init.sls - its only for legacy pillar data.

[CRITICAL] Rendering SLS 'base:lvm.config' failed: Jinja variable 'salt.utils.odict.OrderedDict object' has no attribute 'devices'