On Thu, Apr 4, 2024 at 4:01 PM Mark Michelson <mmich...@redhat.com> wrote:
> Thanks Ales, > > Acked-by: Mark Michelson <mmich...@redhat.com> > Thanks. I applied this patch to the main branch. Does this require a backport ? Numan > On 3/25/24 06:40, Ales Musil wrote: > > The ovn-ctl utility was assuming that the user/group is always root, > > when not specified otherwise by the --ovn-user/--ovn-group options. > > This has the consequence of trying to change permissions of OVN > > directories to root:root even though the script might be run as > > completely different user. > > > > Take the current user and group instead of the hardcoded root. > > At the same time remove the ovs-user option as it was not used for > > anything and might be confusing. > > > > Reported-at: https://issues.redhat.com/browse/FDP-245 > > Signed-off-by: Ales Musil <amu...@redhat.com> > > --- > > utilities/ovn-ctl | 5 ++--- > > utilities/ovn-ctl.8.xml | 1 - > > utilities/ovn-lib.in | 4 ++-- > > 3 files changed, 4 insertions(+), 6 deletions(-) > > > > diff --git a/utilities/ovn-ctl b/utilities/ovn-ctl > > index 700efe35a..dae5e22f4 100755 > > --- a/utilities/ovn-ctl > > +++ b/utilities/ovn-ctl > > @@ -269,8 +269,8 @@ $cluster_remote_port > > # Set the owner of the ovn_dbdir (with -R option) to OVN_USER if > set. > > # This is required because the ovndbs are created with root > permission > > # if not present when create_cluster/upgrade_db is called. > > - INSTALL_USER="root" > > - INSTALL_GROUP="root" > > + INSTALL_USER="$(id -un)" > > + INSTALL_GROUP="$(id -gn)" > > [ "$OVN_USER" != "" ] && INSTALL_USER="${OVN_USER%:*}" > > [ "${OVN_USER##*:}" != "" ] && INSTALL_GROUP="${OVN_USER##*:}" > > > > @@ -1088,7 +1088,6 @@ Options: > > --ovn-ic-sb-db-ssl-protocols=PROTOCOLS OVN IC Southbound DB SSL > protocols > > --ovn-ic-sb-db-ssl-ciphers=CIPHERS OVN IC Southbound DB SSL cipher > list > > --ovn-user="user[:group]" pass the --user flag to the ovn > daemons > > - --ovs-user="user[:group]" pass the --user flag to ovs daemons > > --ovsdb-nb-wrapper=WRAPPER run with a wrapper like valgrind for > debugging > > --ovsdb-sb-wrapper=WRAPPER run with a wrapper like valgrind for > debugging > > --ovsdb-disable-file-column-diff=no|yes > > diff --git a/utilities/ovn-ctl.8.xml b/utilities/ovn-ctl.8.xml > > index 57712bfdc..c0fbb0792 100644 > > --- a/utilities/ovn-ctl.8.xml > > +++ b/utilities/ovn-ctl.8.xml > > @@ -70,7 +70,6 @@ > > <p><code>--ovsdb-nb-wrapper=<var>WRAPPER</var></code></p> > > <p><code>--ovsdb-sb-wrapper=<var>WRAPPER</var></code></p> > > <p><code>--ovn-user=<var>USER:GROUP</var></code></p> > > - <p><code>--ovs-user=<var>USER:GROUP</var></code></p> > > <p><code>-h</code> | <code>--help</code></p> > > > > <h1>File location options</h1> > > diff --git a/utilities/ovn-lib.in b/utilities/ovn-lib.in > > index 1e48ef28c..65cbfbcdc 100644 > > --- a/utilities/ovn-lib.in > > +++ b/utilities/ovn-lib.in > > @@ -48,8 +48,8 @@ LC_ALL=C; export LC_ALL > > ovn_install_dir () { > > DIR="$1" > > INSTALL_MODE="${2:-755}" > > - INSTALL_USER="root" > > - INSTALL_GROUP="root" > > + INSTALL_USER="$(id -un)" > > + INSTALL_GROUP="$(id -gn)" > > [ "$OVN_USER" != "" ] && INSTALL_USER="${OVN_USER%:*}" > > [ "${OVN_USER##*:}" != "" ] && INSTALL_GROUP="${OVN_USER##*:}" > > > > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev