cinder, ovn and manila bundles should not run with --user=root

[Michele Baldessari]

This is unnecessary and not best practice. The reason this is the case is that other services (rabbit/galera/etc) drop to a non-root user by themselves, so this was overlooked.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/582456

[Michele Baldessari (michele)]

And https://review.openstack.org/#/c/580127

[Michele Baldessari (michele)]

And https://review.openstack.org/#/c/582500

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/582500
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=bebc5848da710ce6cfbbd495da47f4d70c3c86b1
Submitter: Zuul
Branch: master

commit bebc5848da710ce6cfbbd495da47f4d70c3c86b1
Author: Numan Siddique <email address hidden>
Date: Fri Jul 13 15:57:34 2018 +0530

    Remove setting root user for ovn dbs pacemaker bundle

    This patch removes the option "--user=root" from the docker options.
    Right now the kolla image - ovn-northd which is used for ovn dbs pacemaker
    bundle doesn't set the user 'openvswitch'. So even after this patch
    is merged, the ovn bundle will be started as root user.

    This patch is 1st step towards running ovn dbs pacemaker as a non root user.

    It requires changes in
     - in ovn-northd kolla image and
     - in the ovn-dbs.yaml pacemaker service file in T-H-T to give the proper
       permissions to the log and data folders
    to run the ovn pacemaker bundle as a non root user which will be handled
    in follow up patches.

    Related-bug: #1781534
    Change-Id: I32f37522b012967f12c459cdf4cf14c43c079231

[Juan Antonio Osorio Robles (juan-osorio-robles)]

Seems that some services are still missing. Is this still planned work?

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/queens
Review: https://review.opendev.org/710438

[OpenStack Infra (hudson-openstack)]

Change abandoned by Michele Baldessari (<email address hidden>) on branch: master
Review: https://review.opendev.org/582456

[OpenStack Infra (hudson-openstack)]

Change abandoned by Flavio Fernandes (<email address hidden>) on branch: stable/queens
Review: https://review.opendev.org/710438
Reason: .