[ovs-dev] [PATCH branch-2.8/2.7] ovsdb-monitor.at: Use correct perl scripts.

Ilya Maximets i.maximets at samsung.com
Wed Mar 6 07:57:18 UTC 2019


On 06.03.2019 0:01, Michael Santana Francisco wrote:
> On 3/5/19 3:30 PM, Aaron Conole wrote:
>> Ilya Maximets <i.maximets at samsung.com> writes:
>>
>>> Python scripts appeared only since OVS 2.9.
>>> This fixes unit test 'monitor-cond-change with many sessions pending'.
>>> Issue appered while backporting the following patch:
>>> e0f42d4a6548 ("monitor: Fix crash when monitor condition adds new columns.")
>>>
>>> Signed-off-by: Ilya Maximets <i.maximets at samsung.com>
>>> ---
>> Acked-by: Aaron Conole <aconole at redhat.com>
>> _______________________________________________
>> dev mailing list
>> dev at openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> Hi Ilya,

Hi Michael.

> 
> So funny story, I was working on fixing this exact same issue on 2.7 and 2.8, but I had not yet seen your patch until a couple of minutes ago. I made the same exact fix you made, tested it, made a patch, I was getting ready to submit the patch, then I saw your patch and realized you already submitted a fix for it. It works, thanks.

It happens from time to time for me too. Thanks for looking at the issue.

BTW, as you're working with tests and CI on older branches, you may found
following patch-set interesting:
  https://patchwork.ozlabs.org/project/openvswitch/list/?series=94247

Best regards, Ilya Maximets.


More information about the dev mailing list