diff options
| author | Yamato Tanaka <yatanaka@redhat.com> | 2022-03-29 11:35:12 +0900 |
|---|---|---|
| committer | Radosław Piliszek <radoslaw.piliszek@gmail.com> | 2022-04-05 08:34:01 +0000 |
| commit | 4c071fcc3a5efa01240602d6566035029fce2d5b (patch) | |
| tree | 723ade9f1d9cdc44699f3c02a6e9fa23686442e9 | |
| parent | c572fc61d02071a4fc5a950d12b783e9b8a998ae (diff) | |
| download | kolla-4c071fcc3a5efa01240602d6566035029fce2d5b.tar.gz kolla-4c071fcc3a5efa01240602d6566035029fce2d5b.zip | |
Emit log when copying file/directory permissions
Currently, kolla sets file permissions at _set_properties_from_file()
method, but no logs are emitted here. This can make it
difficult to investigate the cause of issues which are
caused by setting permissions.
This patch makes _set_properties_from_file() output a
log.
Closes-Bug: #1966825
Change-Id: I91116931aa30fd2c5e51ed1ac04663fcfa4a5f36
| -rw-r--r-- | docker/base/set_configs.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/docker/base/set_configs.py b/docker/base/set_configs.py index e624f29ba..1d1fc48ae 100644 --- a/docker/base/set_configs.py +++ b/docker/base/set_configs.py @@ -120,6 +120,7 @@ class ConfigFile(object): self._set_properties_from_conf(dest) def _set_properties_from_file(self, source, dest): + LOG.info('Copying permissions from %s onto %s', source, dest) shutil.copystat(source, dest) stat = os.stat(source) os.chown(dest, stat.st_uid, stat.st_gid) |