Handle root as a deployment user better
Currently if you don't specify --deployment-user and for whatever reason it's run as root (e.g. ansible as root using sudo), the deployment fails because it attempts to write out to /home/root/. This change updates the user home root lookup for clouds.yaml and throws a warning about setting --deployment-user if it's root. Change-Id: I284b374cc5e6d6a147286f0485832258f93f038f
This commit is contained in:
parent
03da56c949
commit
ff56863b26
|
@ -2327,7 +2327,7 @@ def copy_clouds_yaml(user):
|
|||
:param user: deployment user
|
||||
"""
|
||||
clouds_etc_file = '/etc/openstack/clouds.yaml'
|
||||
clouds_home_dir = os.path.join('/home', user)
|
||||
clouds_home_dir = os.path.expanduser("~{}".format(user))
|
||||
clouds_config_dir = os.path.join(clouds_home_dir, '.config')
|
||||
clouds_openstack_config_dir = os.path.join(clouds_config_dir,
|
||||
'openstack')
|
||||
|
|
|
@ -1432,6 +1432,12 @@ class Deploy(command.Command):
|
|||
|
||||
def take_action(self, parsed_args):
|
||||
self.log.debug("take_action(%s)" % parsed_args)
|
||||
if parsed_args.deployment_user == 'root':
|
||||
self.log.warning(
|
||||
_("[WARNING] Deployment user is set to 'root'. This may cause "
|
||||
"some deployment files to be located in /root. Please use "
|
||||
"--deployment-user to specify the user you are deploying "
|
||||
"with."))
|
||||
try:
|
||||
self._standalone_deploy(parsed_args)
|
||||
except Exception as ex:
|
||||
|
|
Loading…
Reference in New Issue