[ovs-dev] [PATCH ovn branch-21.03 1/2] Support vlan-passthru for tag=0 logical switch ports

0-day Robot robot at bytheb.org
Thu Jul 29 19:36:00 UTC 2021


Bleep bloop.  Greetings Ihar Hrachyshka, 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:
ERROR: Remove Gerrit Change-Id's before submitting upstream.
21: Change-Id: I875fdacf05e131f0e0a3e3a3a67c0025b92226ad

WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers: Numan Siddique <numans at ovn.org>
WARNING: Line is 84 characters long (recommended limit is 79)
#69 FILE: northd/ovn-northd.c:3012:
            if (smap_get_bool(&op->od->nbs->other_config, "vlan-passthru", false)) {

WARNING: Line is 80 characters long (recommended limit is 79)
#88 FILE: ovn-nb.xml:568:
        localnet ports, fabric traffic that belongs to other tagged networks may

Lines checked: 237, Warnings: 3, Errors: 1


Please check this out.  If you feel there has been an error, please email aconole at redhat.com

Thanks,
0-day Robot


More information about the dev mailing list