Comment 3 for bug 1350075

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to trove (master)

Reviewed: https://review.openstack.org/128781
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=1f84823fb1baea9ece5fe3df52c2fa317cadbcc5
Submitter: Jenkins
Branch: master

commit 1f84823fb1baea9ece5fe3df52c2fa317cadbcc5
Author: Amrith Kumar <email address hidden>
Date: Wed Oct 15 18:35:19 2014 -0400

    Eliminate use of sudo for two things that don't need it

    The MySQL restore strategy used exec_with_root_helper() which was a
    skinny wrapper on execute with sudo to find out whether MySQL was up
    and running or not. This is unnecessary as any user can run mysqladmin
    (by default) and similarly, any user can run ps (not only root).

    Since these two code paths were the only ones that used
    exec_with_root_helper() I've eliminated that as well. It didn't do
    something so profound that its loss will be missed anyway.

    Change-Id: Ia0452417b88416b0289f894b99a8632ab0328539
    Closes-Bug: #1350075