openstack/charm-neutron-gateway
Code Issues Proposed changes

Change default mkdir permissions to 755 for deploy from source

Browse Source
This commit is contained in:
Corey Bryant 2015-04-17 12:04:02 +00:00
parent 84df1598d1
commit f8915dfc56
2 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hooks/quantum_utils.py
View File

@ -854,7 +854,7 @@ def git_pre_install():
add_user_to_group('neutron', 'neutron')
for d in dirs:
mkdir(d, owner='neutron', group='neutron', perms=0700, force=False)
mkdir(d, owner='neutron', group='neutron', perms=0755, force=False)
for l in logs:
write_file(l, '', owner='neutron', group='neutron', perms=0644)

14
unit_tests/test_quantum_utils.py
View File

@ -723,19 +723,19 @@ class TestQuantumAgentReallocation(CharmTestCase):
add_user_to_group.assert_called_with('neutron', 'neutron')
expected = [
call('/etc/neutron', owner='neutron',
group='neutron', perms=0700, force=False),
group='neutron', perms=0755, force=False),
call('/etc/neutron/rootwrap.d', owner='neutron',
group='neutron', perms=0700, force=False),
group='neutron', perms=0755, force=False),
call('/etc/neutron/plugins', owner='neutron',
group='neutron', perms=0700, force=False),
group='neutron', perms=0755, force=False),
call('/etc/nova', owner='neutron',
group='neutron', perms=0700, force=False),
group='neutron', perms=0755, force=False),
call('/var/lib/neutron', owner='neutron',
group='neutron', perms=0700, force=False),
group='neutron', perms=0755, force=False),
call('/var/lib/neutron/lock', owner='neutron',
group='neutron', perms=0700, force=False),
group='neutron', perms=0755, force=False),
call('/var/log/neutron', owner='neutron',
group='neutron', perms=0700, force=False),
group='neutron', perms=0755, force=False),
]
self.assertEquals(mkdir.call_args_list, expected)
expected = [