Bleep bloop. Greetings Eli Britstein, I am a robot and I have tried out your
patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
checkpatch:
WARNING: Line is 98 characters long (recommended limit is 79)
#111 FILE: vswitchd/vswitch.xml:1863:
type='{"type": "string", "enum": ["set", ["omit",
"include-non-zero", "include"]]}'>
WARNING: Line is 90 characters long (recommended limit is 79)
#121 FILE: vswitchd/vswitch.xml:1880:
For <code>include-non-zero</code> Open vSwitch omits the 802.1Q
header on output
WARNING: Line is 85 characters long (recommended limit is 79)
#122 FILE: vswitchd/vswitch.xml:1881:
if both the VLAN ID and priority would be zero. Set to
<code>include</code>
Lines checked: 129, Warnings: 3, Errors: 0
Please check this out. If you feel there has been an error, please email
[email protected]
Thanks,
0-day Robot
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev