Hi Xavier,

On 12/5/25 2:57 PM, Xavier Simonart wrote:
>>> +AS_BOX([$(date +%H:%M:%S.%03N) Add/delete LSP while northd is sleeping])
>> Do we really need the date in the output?  Is this a left over from your
>> debugging?
>>
> In tests such as this one which repeat similar operations multiple times
> (e.g we delete
> three times ls1) I found it much easier to have a date in the
> testsuite.log, so we can easily
> find in other logs or db what happens at that time. So yes, it's debugging,
> but does not cost much
> and should not influence the dynamics of the test (as would a vlog/set dbg
> for instance).
> So I find it easier if we keep the date, but I can remove it if you find we
> should not have such debug info
> in the tests.

That's fine, let's keep it if it adds value.

Regards,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to