search

saltstack-formulas / lvm-formula

8 stars 13 forks source link

fix(travis): install lvm2 package #13

Closed noelmcloughlin closed 5 years ago

noelmcloughlin commented 5 years ago

Workaround Travis failure: https://travis-ci.org/noelmcloughlin/iscsi-formula/jobs/586775130

n-rodriguez commented 5 years ago

Weird because both of them are in default pillars : https://github.com/saltstack-formulas/lvm-formula/blob/master/lvm/defaults.yaml#L5

noelmcloughlin commented 5 years ago

Some formulas have broken map.jinja where os mapping is overwriting instead of merging dicts. Docker-formula is an example. I suspect this formula has bad map.jinja too - I have no time to investigate. This PR is a workaround I guess.

n-rodriguez commented 5 years ago

Some formulas have broken map.jinja where os mapping is overwriting instead of merging dicts. Docker-formula is an example. I suspect this formula has bad map.jinja too

it's the case