Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add directory itself to keep list #23442

Merged
merged 3 commits into from
May 8, 2015

Conversation

clan
Copy link
Contributor

@clan clan commented May 7, 2015

if not fixed, following sls will delete /root/a/b

 /root/a:
   file.directory:
     - user: root
     - group: root
     - mode: 0755
     - clean: True
     - require:
         - file: /root/a/b

 /root/a/b:
   file.directory:
     - user: root
     - group: root
     - mode: 0755

if not fixed, following sls will delete /root/a/b

 /root/a:
   file.directory:
     - user: root
     - group: root
     - mode: 0755
     - clean: True
     - require:
         - file: /root/a/b

 /root/a/b:
   file.directory:
     - user: root
     - group: root
     - mode: 0755
@jfindlay
Copy link
Contributor

jfindlay commented May 7, 2015

@clan, thanks for the fix. In the future, please feel free to submit fixes directly to the oldest, supported release branch applicable, see https://docs.saltstack.com/en/latest/topics/development/contributing.html#which-salt-branch.

@jfindlay jfindlay added the bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch label May 7, 2015
for following sls:

 /root/a:
   file.directory:
     - user: root
     - group: root
     - mode: 0755
     - clean: True
     - require:
         - file: /root/a/b
         - file: /root/a/c

 /root/a/b:
   file.directory:
     - user: root
     - group: root
     - mode: 0755

 id_test:
   file.managed:
     - name: /root/a/c
     - user: root
     - group: root
     - mode: 0755
     - replace: False
@clan
Copy link
Contributor Author

clan commented May 7, 2015

ok, will submit to the oldest branch next time.

thatch45 added a commit that referenced this pull request May 8, 2015
@thatch45 thatch45 merged commit 57f6c72 into saltstack:develop May 8, 2015
@clan clan deleted the file_directory_require branch May 9, 2015 04:19
@rallytime rallytime added ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. and removed bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch labels May 12, 2015
thatch45 added a commit that referenced this pull request May 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants