Re: [ovs-dev] [PATCH ovn v2] QoS: Properly set qos when ovs db is read only

2023-09-16 Thread Xavier Simonart
Hi Numan, Ales Thanks for the review. While the patch covers a similar issue than bz https://bugzilla.redhat.com/2234349, I think that the bz is a different issue: QoS does not work anymore (even w/ the patch) on tunnel interfaces (see NEWS from commit 87514ac0). So I do not think that the tag sho

Re: [ovs-dev] [PATCH ovn v2] QoS: Properly set qos when ovs db is read only

2023-09-15 Thread Numan Siddique
On Fri, Sep 15, 2023 at 3:09 AM Ales Musil wrote: > > On Fri, Sep 15, 2023 at 8:21 AM Xavier Simonart wrote: > > > > QoS was not configured in OVS db when db was read only: the configuration > > was just ignored and not done later when OVS db became writable. > > It was sometimes set later, if/wh

Re: [ovs-dev] [PATCH ovn v2] QoS: Properly set qos when ovs db is read only

2023-09-15 Thread Ales Musil
On Fri, Sep 15, 2023 at 8:21 AM Xavier Simonart wrote: > > QoS was not configured in OVS db when db was read only: the configuration > was just ignored and not done later when OVS db became writable. > It was sometimes set later, if/when a recompute happened. > This is now fixed: when OVS db is re

[ovs-dev] [PATCH ovn v2] QoS: Properly set qos when ovs db is read only

2023-09-14 Thread Xavier Simonart
QoS was not configured in OVS db when db was read only: the configuration was just ignored and not done later when OVS db became writable. It was sometimes set later, if/when a recompute happened. This is now fixed: when OVS db is read only, the ports on which qos must be applied are stored and qos