[ovs-dev] [PATCH v3 ovn 2/2] ovn-northd: Support mixing stateless/stateful ACLs with Stateless_Filter.

0-day Robot robot at bytheb.org
Wed Sep 2 16:06:47 UTC 2020


Bleep bloop.  Greetings Dumitru Ceara, 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 lacks whitespace around operator
WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
#931 FILE: utilities/ovn-nbctl.c:605:
  [--type={switch | port-group}] [--may-exist]\n\

WARNING: Line lacks whitespace around operator
#932 FILE: utilities/ovn-nbctl.c:606:
  stateless-filter-add {SWITCH | PORTGROUP} PRIORITY MATCH \n\

WARNING: Line lacks whitespace around operator
#933 FILE: utilities/ovn-nbctl.c:607:
                            add a stateless filter to SWITCH/PORTGROUP\n\

WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
#934 FILE: utilities/ovn-nbctl.c:608:
  [--type={switch | port-group}]\n\

WARNING: Line lacks whitespace around operator
#935 FILE: utilities/ovn-nbctl.c:609:
  stateless-filter-del {SWITCH | PORTGROUP} [PRIORITY MATCH]\n\

WARNING: Line lacks whitespace around operator
#936 FILE: utilities/ovn-nbctl.c:610:
                            remove stateless filters from SWITCH/PORTGROUP\n\

WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
WARNING: Line lacks whitespace around operator
#937 FILE: utilities/ovn-nbctl.c:611:
  [--type={switch | port-group}]\n\

WARNING: Line lacks whitespace around operator
#938 FILE: utilities/ovn-nbctl.c:612:
  stateless-filter-list {SWITCH | PORTGROUP}\n\

Lines checked: 1204, Warnings: 14, Errors: 0


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