Comment 4 for bug 1546372

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

Reviewed: https://review.openstack.org/541630
Committed: https://git.openstack.org/cgit/openstack/trove/commit/?id=3b726f3013b0ba2a9f86b5b65278fadb981335cf
Submitter: Zuul
Branch: master

commit 3b726f3013b0ba2a9f86b5b65278fadb981335cf
Author: Zhao Chao <email address hidden>
Date: Wed Feb 7 15:37:30 2018 +0800

    report_root should always use context.user

    The RootHistory intends to record by whom and when the root user of the
    underlying datastore backend was enabled. So the "user" column should
    always be set to the context user, not the actual root user in the
    database. But report_root(and report_root_enabled in
    taskmanager.models) use the database user instead, this is not correct,
    This patch will fix.

    Also remove the "user" argument from Root.create() and
    HistoryRoot.create() because the context is already passed in as an
    argument when these methods are called.

    Closes-Bug: #1546372
    Change-Id: I3b4c8ee56c7e0876fb384f0c5841d2d391bd555d
    Signed-off-by: Zhao Chao <email address hidden>