-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix owner of the /var/lib/openstack created by the neutron_dhcp role #409
Fix owner of the /var/lib/openstack created by the neutron_dhcp role #409
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve |
816ec93
to
4e48273
Compare
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
This directory may be created by libvirt role or neutron_dhcp role and it had wrong owner set (root) when was created by edpm_neutron_dhcp role. Because of that edpm_libvirt role which was run after edpm_neutron_dhcp was failing due to permission error to access /var/lib/openstack/config/ directory.
4e48273
to
e0a6307
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: booxter, rebtoor, slawqo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
b547875
into
openstack-k8s-operators:main
This directory may be created by libvirt role or neutron_dhcp role and it had wrong owner set (root) when was created by edpm_neutron_dhcp role. Because of that edpm_libvirt role which was run after edpm_neutron_dhcp was failing due to permission error to access /var/lib/openstack/config/ directory.