From 1d02e964db7ed7864f9ec13338c39057f82c95dd Mon Sep 17 00:00:00 2001 From: Martin Schuppert Date: Wed, 12 Jun 2019 12:11:56 +0200 Subject: [PATCH] Fix log owner on computes during overcloud deploy runs [1] switched to run nova-manage discovery as non root user. In case of updates there can be already a nove-manage log owned by root from previous runs. This change make sure we change the owner of nova-manage log to nova:nova on overcloud deploy runs on the computes [1] https://review.opendev.org/#/c/652039/13/deployment/nova/nova-compute-container-puppet.yaml Closes-Bug: #1832482 Change-Id: Ie6d3f5262f7eb20ef92b07f2f359d88cb77b8752 (cherry picked from commit ccabc8f8e68c12321b98e4ada12c005d61b724a1) --- deployment/nova/nova-compute-container-puppet.yaml | 2 ++ ...ova_compute_fix_log_permissions-e866f91848d647fb.yaml | 9 +++++++++ 2 files changed, 11 insertions(+) create mode 100644 releasenotes/notes/nova_compute_fix_log_permissions-e866f91848d647fb.yaml diff --git a/deployment/nova/nova-compute-container-puppet.yaml b/deployment/nova/nova-compute-container-puppet.yaml index 5f1256af74..7ada5b8c7d 100644 --- a/deployment/nova/nova-compute-container-puppet.yaml +++ b/deployment/nova/nova-compute-container-puppet.yaml @@ -545,6 +545,8 @@ outputs: - {get_attr: [ContainersCommon, container_config_scripts]} - {get_attr: [NovaComputeCommon, container_config_scripts]} docker_config: + step_2: + get_attr: [NovaLogging, docker_config, step_2] step_3: nova_statedir_owner: image: &nova_compute_image {get_param: DockerNovaComputeImage} diff --git a/releasenotes/notes/nova_compute_fix_log_permissions-e866f91848d647fb.yaml b/releasenotes/notes/nova_compute_fix_log_permissions-e866f91848d647fb.yaml new file mode 100644 index 0000000000..9cd72dc514 --- /dev/null +++ b/releasenotes/notes/nova_compute_fix_log_permissions-e866f91848d647fb.yaml @@ -0,0 +1,9 @@ +--- +fixes: + - | + [1] switched to run nova-manage discovery as non root user. In case of + updates there can be already a nove-manage log owned by root from previous + runs. This change make sure we change the owner of nova-manage log to + nova:nova on overcloud deploy runs on the computes + + [1] https://review.opendev.org/#/c/652039/13/deployment/nova/nova-compute-container-puppet.yaml