Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-06 Thread Dumitru Ceara
On 7/2/21 8:39 PM, Han Zhou wrote: > On Fri, Jul 2, 2021 at 2:08 AM Dumitru Ceara wrote: >> >> Time how long change/run handlers take and log this at debug level. >> I-P engine debug logs are not so verbose so enabling them is quite >> common when debugging scale/control plane latency related issu

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Han Zhou
On Fri, Jul 2, 2021 at 2:08 AM Dumitru Ceara wrote: > > Time how long change/run handlers take and log this at debug level. > I-P engine debug logs are not so verbose so enabling them is quite > common when debugging scale/control plane latency related issues. > > One of the major missing pieces t

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Mark Gray
On 02/07/2021 17:02, Dumitru Ceara wrote: > On 7/2/21 5:51 PM, Mark Gray wrote: >> On 02/07/2021 10:07, Dumitru Ceara wrote: >>> Time how long change/run handlers take and log this at debug level. >>> I-P engine debug logs are not so verbose so enabling them is quite >>> common when debugging scale

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Dumitru Ceara
On 7/2/21 5:51 PM, Mark Gray wrote: > On 02/07/2021 10:07, Dumitru Ceara wrote: >> Time how long change/run handlers take and log this at debug level. >> I-P engine debug logs are not so verbose so enabling them is quite >> common when debugging scale/control plane latency related issues. >> >> One

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Mark Gray
On 02/07/2021 10:07, Dumitru Ceara wrote: > Time how long change/run handlers take and log this at debug level. > I-P engine debug logs are not so verbose so enabling them is quite > common when debugging scale/control plane latency related issues. > > One of the major missing pieces though was a

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Mark Gray
On 02/07/2021 14:56, Dumitru Ceara wrote: > Based on the above I won't be sending a v2 for now and I'll move back > the v1 in patchwork to "New" to continue the discussion here. > > Thanks, > Dumitru Thanks for doing the due diligence and investigating this possibility __

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Dumitru Ceara
On 7/2/21 1:36 PM, Dumitru Ceara wrote: > On 7/2/21 1:32 PM, Mark Gray wrote: >> On 02/07/2021 10:07, Dumitru Ceara wrote: >>> Time how long change/run handlers take and log this at debug level. >>> I-P engine debug logs are not so verbose so enabling them is quite >>> common when debugging scale/c

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Dumitru Ceara
On 7/2/21 1:32 PM, Mark Gray wrote: > On 02/07/2021 10:07, Dumitru Ceara wrote: >> Time how long change/run handlers take and log this at debug level. >> I-P engine debug logs are not so verbose so enabling them is quite >> common when debugging scale/control plane latency related issues. >> >> One

Re: [ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Mark Gray
On 02/07/2021 10:07, Dumitru Ceara wrote: > Time how long change/run handlers take and log this at debug level. > I-P engine debug logs are not so verbose so enabling them is quite > common when debugging scale/control plane latency related issues. > > One of the major missing pieces though was a

[ovs-dev] [PATCH ovn] inc-proc-eng: Improve debug logging.

2021-07-02 Thread Dumitru Ceara
Time how long change/run handlers take and log this at debug level. I-P engine debug logs are not so verbose so enabling them is quite common when debugging scale/control plane latency related issues. One of the major missing pieces though was a log about how long each I-P node change/run handler