Add server / proxy byos and paygo images support to sumaform #1418
Conversation
maximenoel8
changed the title
| packages: ["salt-minion"] | ||
|
|
||
| %{ endif } | ||
|
|
||
| %{ if image == "sles15sp5-paygo" || image == "sles12sp5-paygo" || image == "sap15sp5-paygo" } | ||
| packages: ["salt-minion"] |
There was a problem hiding this comment.
I wonder if we can't follow the same strategy as others, include the corresponding repository, and install venv-salt-minion. Would that work?
There was a problem hiding this comment.
Do we have public repository having venv-salt-minion package available for those distributions.
By default, those distributions have product repositories and salt-minion is available.
This salt-minion is only to run the sumaform highstate.
There was a problem hiding this comment.
Using what is available by default will simplify our life on cloud public overall.
There was a problem hiding this comment.
I was more worry about the different salt paths and processes leftovers, using the same package we will reduce the leftovers because they will be overriden.
Anyway, approving.
What does this PR change?
Add support for new AWS images :