Add zone dynamic and includes as service dependency

saltstack-formulas / bind-formula

http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html

Other

29 stars 117 forks source link

Add zone dynamic and includes as service dependency #106

Open Poil opened 6 years ago

Poil commented 6 years ago

I propose these changes :

Add the parameter "dynamic" to allow to not manage a zone if the file already exists
Add as service dependencies the "includes" parameter

javierbertoli commented 6 years ago

@Poil did you have any time to check @aanriot 's comments? Also, perhaps using Salt's dynamic dns update makes sense for zones marked as dynamic, so we can add a state named dynamic.sls where we can manage records without breaking the zone's journal? (I mentioned this in the related issue #104)

aanriot commented 5 years ago

@Poil , were you able to check this PR again? Thanks.

Poil commented 5 years ago

Sorry I had no time, I will try in the next week to check if it is possible or not to include a subfile for record, but I think it is not possible.