saltstack-formulas / shorewall-formula

Saltstack formula for managing shorewall
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
5 stars 21 forks source link

Dynamically generated params with mine #14

Closed daks closed 6 years ago

daks commented 6 years ago

Hello,

this MR propose a solution to be able to generate Shorewall params with a dynamic query to Salt Mine #12. Using pillar, you can specify which salt mine query you want, and also a default value, if this query returns no results.

The mine query can return a list of IP addresses, a list of lists of IPs, or a combination of those two formats and it'll be managed by the code.

daks commented 6 years ago

Did anyone had the possibility to verify this MR? It works well for us in production since several weeks.

aboe76 commented 6 years ago

@daks I don't use shorewall, on my machines...

daks commented 6 years ago

@aboe76 ok, so who can at least review it, and maybe then merge it? I think this functionnality is potentially interesting for many people.

aboe76 commented 6 years ago

@puneetk could you take a look and review the code?

daks commented 6 years ago

any news?

daks commented 6 years ago

ping

daks commented 6 years ago

Hi,

is there anyone which can look at this pull request, and merge it if it's ok?

The code:

n-rodriguez commented 6 years ago

ping @puneetk

daks commented 6 years ago

This pull request is standing here for 4 months. Is there any chance someone with commit rights will look at it?

aboe76 commented 6 years ago

@daks will merge it for you

daks commented 6 years ago

thanks