Re: [ovs-dev] [PATCH ovn] ovn-ctl: Use the current user for default file permissions.

2024-04-07 Thread Ales Musil
On Fri, Apr 5, 2024 at 10:09 PM Numan Siddique wrote: > > > On Thu, Apr 4, 2024 at 4:01 PM Mark Michelson wrote: > >> Thanks Ales, >> >> Acked-by: Mark Michelson >> > > Thanks. I applied this patch to the main branch. Does this require a > backport ? > Hi Numan, yes please, it should be

Re: [ovs-dev] [PATCH ovn] ovn-ctl: Use the current user for default file permissions.

2024-04-05 Thread Numan Siddique
On Thu, Apr 4, 2024 at 4:01 PM Mark Michelson wrote: > Thanks Ales, > > Acked-by: Mark Michelson > 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

Re: [ovs-dev] [PATCH ovn] ovn-ctl: Use the current user for default file permissions.

2024-04-04 Thread Mark Michelson
Thanks Ales, Acked-by: Mark Michelson 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

[ovs-dev] [PATCH ovn] ovn-ctl: Use the current user for default file permissions.

2024-03-25 Thread Ales Musil
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