diff options
| author | Chulmin Kang <chaman.kang@gmail.com> | 2017-07-02 16:09:30 +0900 |
|---|---|---|
| committer | Chulmin Kang <chaman.kang@gmail.com> | 2017-07-02 19:05:36 +0900 |
| commit | ea3d140cab23ce3016e3bea7d35e4f5fbb8611f6 (patch) | |
| tree | 56c014f3ee8910295b878af3c6cd4e7fb117124f | |
| parent | f26b1ba0b362e9091141f0fad2f7b91edc40ac11 (diff) | |
| download | kolla-ea3d140cab23ce3016e3bea7d35e4f5fbb8611f6.tar.gz kolla-ea3d140cab23ce3016e3bea7d35e4f5fbb8611f6.zip | |
Fix argument to handle_permissions
Fix the format of argument passed to handle_permissions function, so
that handle_permissions can handle permissions and ownerships of
config_files.
Change-Id: Ic07975a16973df22cdaa6685fc326620812f8f65
Closes-Bug: #1701876
| -rw-r--r-- | docker/base/set_configs.py | 5 | ||||
| -rw-r--r-- | tests/test_set_config.py | 7 |
2 files changed, 7 insertions, 5 deletions
diff --git a/docker/base/set_configs.py b/docker/base/set_configs.py index 229090817..e2005c5f7 100644 --- a/docker/base/set_configs.py +++ b/docker/base/set_configs.py @@ -121,8 +121,9 @@ class ConfigFile(object): os.chown(dest, stat.st_uid, stat.st_gid) def _set_properties_from_conf(self, path): - handle_permissions({'owner': self.owner, 'path': path, - 'perm': self.perm}) + config = {'permissions': + [{'owner': self.owner, 'path': path, 'perm': self.perm}]} + handle_permissions(config) def copy(self): diff --git a/tests/test_set_config.py b/tests/test_set_config.py index aff97b3bb..643b5ba54 100644 --- a/tests/test_set_config.py +++ b/tests/test_set_config.py @@ -143,9 +143,10 @@ class ConfigFileTest(base.BaseTestCase): config_file = copy.deepcopy(FAKE_CONFIG_FILE) config_file._set_properties_from_conf(config_file.dest) - mock_handle_permissions.assert_called_with({'owner': 'user1', - 'path': config_file.dest, - 'perm': '0644'}) + mock_handle_permissions.assert_called_with({'permissions': + [{'owner': 'user1', + 'path': config_file.dest, + 'perm': '0644'}]}) @mock.patch('glob.glob', return_value=[]) def test_copy_no_source_not_optional(self, |