diff options
| author | Jeffrey Zhang <zhang.lei.fly@gmail.com> | 2017-05-09 11:18:39 +0800 |
|---|---|---|
| committer | Jeffrey Zhang <zhang.lei.fly@gmail.com> | 2017-05-09 21:41:57 +0800 |
| commit | 97467c38c4df5ccd0a35bfa1c7626b0aceea0806 (patch) | |
| tree | b2804393ce7e40c52b4951ffeac468dc5c597033 /docker/base/set_configs.py | |
| parent | 5841abe910c50d34fbbf4dcfde7b6e92dd82bdae (diff) | |
| download | kolla-97467c38c4df5ccd0a35bfa1c7626b0aceea0806.tar.gz kolla-97467c38c4df5ccd0a35bfa1c7626b0aceea0806.zip | |
Catch exception during calling os.chmod in set_configs.py
set_perms may fail in some case, but it is not critical. Just leave a
warning rather than exit.
Change-Id: I757c7b60611fb6c0547bd1b26ec8a8af03684e5f
Closes-Bug: 1689454
Diffstat (limited to 'docker/base/set_configs.py')
| -rw-r--r-- | docker/base/set_configs.py | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/docker/base/set_configs.py b/docker/base/set_configs.py index 9c3d1f1e4..eb4b7ac17 100644 --- a/docker/base/set_configs.py +++ b/docker/base/set_configs.py @@ -354,7 +354,13 @@ def handle_permissions(config): def set_perms(path, uid, gid): LOG.info('Setting permission for %s', path) - os.chown(path, uid, gid) + if not os.path.exists(path): + LOG.warning('file %s do not exist', path) + return + try: + os.chown(path, uid, gid) + except OSError: + LOG.exception('Set file permission failed for %s', path) for dest in glob.glob(path): set_perms(dest, uid, gid) |