opendev/system-config
Code Issues Proposed changes

Fix three nits

Browse Source 
These were caught in earlier reviews.

Change-Id: I026678fbbe871ffb138fd2c0dba9e788457da3a0
This commit is contained in:
Monty Taylor 2015-11-30 07:44:10 -06:00
parent 714c934d0c
commit b23025acaa
3 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
doc/source/sysadmin.rst
View File

@ -299,7 +299,7 @@ puppet at times when landing a change to the puppet repo would be either
unreasonable or impossible. unreasonable or impossible.
There are two sections in each file, `disabled` and `disabled:children`. Due There are two sections in each file, `disabled` and `disabled:children`. Due
to te multi-cloud nature of the ansible inventory, a hostname cannot be counted to the multi-cloud nature of the ansible inventory, a hostname cannot be counted
on to be unique, so each cloud instance is listed in the inventory by its on to be unique, so each cloud instance is listed in the inventory by its
UUID with a group created for its hostname. If you want to disable a cloud UUID with a group created for its hostname. If you want to disable a cloud
instance by name, you need to put its name in `disabled:children`. If you want instance by name, you need to put its name in `disabled:children`. If you want

3
launch/launch-node.py
View File

@ -298,7 +298,8 @@ def main():
dns.print_dns(client, options.name) dns.print_dns(client, options.name)
# Remove the ansible inventory cache so that next run finds the new # Remove the ansible inventory cache so that next run finds the new
# server # server
os.unlink('/var/cache/ansible-inventory.json') if os.path.exists('/var/cache/ansible-inventory/ansible-inventory.cache'):
os.unlink('/var/cache/ansible-inventory/ansible-inventory.cache')
if __name__ == '__main__': if __name__ == '__main__':
main() main()

2
modules/openstack_project/manifests/puppetmaster.pp
View File

@ -207,6 +207,6 @@ class openstack_project::puppetmaster (
ensure => present, ensure => present,
owner => 'root', owner => 'root',
group => 'root', group => 'root',
mode => '0444', mode => '0644',
} }
} }