saltstack-formulas / influxdb-formula

Installs and configures the InfluxDB timeseries database
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Apache License 2.0
9 stars 34 forks source link

Drop dependency on pip-formula #19

Closed wwentland closed 7 years ago

wwentland commented 8 years ago

We do not want interformula dependency at this point and therefore just maintain a complete copy of the pip-formula settings in this formula.

Luckily the pip package name seems to be identical on, at least, Debian and RedHat and their derivatives.

wwentland commented 8 years ago

ping

wwentland commented 8 years ago

I'd like to point out that I don't like this change as I see absolutely no merit in duplicating states in every single formula and forcing us to maintain the same "formula" in multiple formulas.

The current consensus, however, seems to be that inter-formula dependencies are bad which is, I guess, primarily due to the fact that we cannot express them in a way that would ensure these other formulas are being installed.