Compare commits

...

5 commits

Author SHA1 Message Date
Silejonu
19e89f1095
Merge 9ec1d668b4 into d3204df536 2025-07-21 17:06:36 +02:00
softwarefactory-project-zuul[bot]
d3204df536
Merge pull request #660 from saito-hideki/pr_azp_syntax
[AZP] Fixed syntax error in AZP settings

SUMMARY
Fixed syntax error in AZP settings
ISSUE TYPE

CI Pull Request

COMPONENT NAME

ansible.posix

ADDITIONAL INFORMATION
None
2025-07-17 03:39:40 +00:00
Hideki SAITO
162e72182f Fixed syntax error in azurepileline settings
Signed-off-by: Hideki SAITO <hsaito@macallan.home.fgrep.org>
2025-07-17 12:16:09 +09:00
Silejonu
9ec1d668b4 refactor(acl): improve execution speed 2025-06-05 08:54:42 +02:00
Silejonu
392fd0deaa fix(acl): correctly assert needed changes when recursive is true 2025-05-10 09:59:10 +02:00
5 changed files with 11 additions and 8 deletions

View file

@ -64,7 +64,7 @@ stages:
- template: templates/matrix.yml - template: templates/matrix.yml
parameters: parameters:
nameFormat: "{0}" nameFormat: "{0}"
testFormat: 2.19/{0OI} testFormat: 2.19/{0}
targets: targets:
- name: Sanity - name: Sanity
test: sanity test: sanity
@ -92,7 +92,6 @@ stages:
dependsOn: [] dependsOn: []
jobs: jobs:
- template: templates/matrix.yml - template: templates/matrix.yml
I
parameters: parameters:
nameFormat: "{0}" nameFormat: "{0}"
testFormat: 2.17/{0} testFormat: 2.17/{0}

View file

@ -0,0 +1,2 @@
bugfixes:
- acl - correctly assert needed changes when pointing to a directory and recursive is set to true.

View file

@ -1,3 +1,3 @@
~-- ---
trivial: trivial:
- Bump ansible-core version to 2.20 of devel branch and add 2.19 to CI - Bump ansible-core version to 2.20 of devel branch and add 2.19 to CI

View file

@ -0,0 +1,2 @@
trivial:
- AZP - fixed syntax error in CI test.

View file

@ -244,16 +244,16 @@ def acl_changed(module, cmd, entry, use_nfsv4_acls=False):
lines = run_acl(module, cmd) lines = run_acl(module, cmd)
counter = 0 counter = 0
for line in lines: for line in lines:
if line.endswith('*,*') and not use_nfsv4_acls: if not use_nfsv4_acls and not line.endswith('*,*'):
return False return True
# if use_nfsv4_acls and entry is listed # if use_nfsv4_acls and entry is listed
if use_nfsv4_acls and entry == line: if use_nfsv4_acls and entry == line:
counter += 1 counter += 1
# The current 'nfs4_setfacl --test' lists a new entry, # The current 'nfs4_setfacl --test' lists a new entry,
# which will be added at the top of list, followed by the existing entries. # which will be added at the top of the list, followed by the existing entries.
# So if the entry has already been registered, the entry should be find twice. # So if the entry has already been registered, the entry should be found twice.
if counter == 2: if not use_nfsv4_acls or counter == 2:
return False return False
return True return True