On 20/02/2025 12:57, Ilya Maximets wrote:
> On 2/18/25 12:03, Roi Dayan wrote:
>> From: Dima Chumak <[email protected]>
>>
>> The 'fdb/show' command now supports machine-readable JSON output in
>> addition to the plain-text output for humans. An example would be:
>>
>> ovs-appctl --format=json --pretty fdb/show br-phy
>> [
>> {
>> "age": "static",
>> "mac": "e4:8c:07:08:00:02",
>> "port": 2,
>> "vlan": 0
>> },
>> {
>> "age": 14,
>> "mac": "e4:8c:07:08:00:03",
>> "port": 3,
>> "vlan": 0
>> }
>> ]
>
> Instead of returning different type values for the age, should we
> perhaps return "static" as a separate boolean field and not report
> the "age" in this case?
>
> Alternative might be to report age as a string always. But that
> doesn't sound particularly great.
>
> It may be very inconvenient to write parsers in some languages if
> the type of the value can be both int and the string.
>
> Best regards, Ilya Maximets.
right. Fixed for v4.
thanks
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev