1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00

Add detection of uid,gid,mode changes when deciding whether an archive needs to be unarchived again.

This commit is contained in:
Toshio Kuratomi 2015-01-14 19:12:35 -08:00 committed by Matt Clay
parent 65d3b167e6
commit 1728f9bba7

View file

@ -104,7 +104,7 @@ class ZipFile(object):
return self._files_in_archive return self._files_in_archive
def is_unarchived(self): def is_unarchived(self, mode, owner, group):
return dict(unarchived=False) return dict(unarchived=False)
def unarchive(self): def unarchive(self):
@ -148,10 +148,32 @@ class TgzFile(object):
self._files_in_archive.append(filename) self._files_in_archive.append(filename)
return self._files_in_archive return self._files_in_archive
def is_unarchived(self): def is_unarchived(self, mode, owner, group):
cmd = '%s -v -C "%s" --diff -%sf "%s"' % (self.cmd_path, self.dest, self.zipflag, self.src) cmd = '%s -C "%s" --diff -%sf "%s"' % (self.cmd_path, self.dest, self.zipflag, self.src)
rc, out, err = self.module.run_command(cmd) rc, out, err = self.module.run_command(cmd)
unarchived = (rc == 0) unarchived = (rc == 0)
if not unarchived:
# Check whether the differences are in something that we're
# setting anyway
# What will be set
to_be_set = set()
for perm in (('Mode', mode), ('Gid', group), ('Uid', owner)):
if perm[1] is not None:
to_be_set.add(perm[0])
# What is different
changes = set()
difference_re = re.compile(r': (.*) differs$')
for line in out.splitlines():
match = difference_re.search(line)
if not match:
# Unknown tar output. Assume we have changes
return dict(unarchived=unarchived, rc=rc, out=out, err=err, cmd=cmd)
changes.add(match.groups()[0])
if changes and changes.issubset(to_be_set):
unarchived = True
return dict(unarchived=unarchived, rc=rc, out=out, err=err, cmd=cmd) return dict(unarchived=unarchived, rc=rc, out=out, err=err, cmd=cmd)
def unarchive(self): def unarchive(self):
@ -242,7 +264,8 @@ def main():
res_args = dict(handler=handler.__class__.__name__, dest=dest, src=src) res_args = dict(handler=handler.__class__.__name__, dest=dest, src=src)
# do we need to do unpack? # do we need to do unpack?
res_args['check_results'] = handler.is_unarchived() res_args['check_results'] = handler.is_unarchived(file_args['mode'],
file_args['owner'], file_args['group'])
if res_args['check_results']['unarchived']: if res_args['check_results']['unarchived']:
res_args['changed'] = False res_args['changed'] = False
else: else: