Bleep bloop. Greetings Numan Siddique, 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 103 characters long (recommended limit is 79) #90 FILE: utilities/ovn-ctl:769: OVS_RUNDIR=${OVS_RUNDIR} start_ovn_daemon "$OVN_CONTROLLER_PRIORITY" "$OVN_CONTROLLER_WRAPPER" "$@" WARNING: Line is 81 characters long (recommended limit is 79) #176 FILE: utilities/ovn-ctl:1199: start_ovnbr_ovsdb start ovn bridge controller db ovsdb-server process WARNING: Line is 80 characters long (recommended limit is 79) #178 FILE: utilities/ovn-ctl:1201: stop_ovnbr_ovsdb stop ovn bridge controller db ovsdb-server process WARNING: Line is 83 characters long (recommended limit is 79) #180 FILE: utilities/ovn-ctl:1203: restart_ovnbr_ovsdb restart ovn bridge controller db ovsdb-server process WARNING: Line is 84 characters long (recommended limit is 79) #182 FILE: utilities/ovn-ctl:1205: status_ovnbr_ovsdb status ovn bridge controller db ovsdb-server processes WARNING: Line is 108 characters long (recommended limit is 79) #192 FILE: utilities/ovn-ctl:1285: --ovn-br-controller-priority=NICE set ovn-br-controller's niceness (default: $OVN_CONTROLLER_PRIORITY) WARNING: Line is 86 characters long (recommended limit is 79) #193 FILE: utilities/ovn-ctl:1286: --ovn-br-controller-wrapper=WRAPPER run with a wrapper like valgrind for debugging WARNING: Line is 80 characters long (recommended limit is 79) #194 FILE: utilities/ovn-ctl:1287: --ovn-br-controller-ssl-key=KEY OVN Bridge Controller SSL/TLS private key file WARNING: Line is 82 characters long (recommended limit is 79) #195 FILE: utilities/ovn-ctl:1288: --ovn-br-controller-ssl-cert=CERT OVN Bridge Controller SSL/TLS certificate file WARNING: Line is 88 characters long (recommended limit is 79) #196 FILE: utilities/ovn-ctl:1289: --ovn-br-controller-ssl-ca-cert=CERT OVN Bridge Controller SSL/TLS CA certificate file WARNING: Line is 111 characters long (recommended limit is 79) #197 FILE: utilities/ovn-ctl:1290: --ovn-br-controller-ssl-bootstrap-ca-cert=CERT Bootstrapped OVN Bridge Controller SSL/TLS CA certificate file WARNING: Line is 85 characters long (recommended limit is 79) #198 FILE: utilities/ovn-ctl:1291: --ovn-br-controller-ssl-protocols=PROTOCOLS OVN Bridge Controller SSL/TLS protocols WARNING: Line is 83 characters long (recommended limit is 79) #199 FILE: utilities/ovn-ctl:1292: --ovn-br-controller-ssl-ciphers=CIPHERS OVN Bridge Controller SSL/TLS cipher list WARNING: Line is 99 characters long (recommended limit is 79) #200 FILE: utilities/ovn-ctl:1293: --ovn-br-controller-ssl-ciphersuites=CIPHERSUITES OVN Bridge Controller TLSv1.3+ ciphersuite list WARNING: Line is 83 characters long (recommended limit is 79) #203 FILE: utilities/ovn-ctl:1296: --ovn-br-db-ssl-ca-cert=CERT OVN Bridge Controller DB SSL/TLS CA certificate file WARNING: Line is 80 characters long (recommended limit is 79) #204 FILE: utilities/ovn-ctl:1297: --ovn-br-db-ssl-protocols=PROTOCOLS OVN Bridge Controller DB SSL/TLS protocols WARNING: Line is 94 characters long (recommended limit is 79) #206 FILE: utilities/ovn-ctl:1299: --ovn-br-db-ssl-ciphersuites=CIPHERSUITES OVN Bridge Controller DB TLSv1.3+ ciphersuite list WARNING: Line is 108 characters long (recommended limit is 79) #207 FILE: utilities/ovn-ctl:1300: --ovn-br-controller-log=STRING ovn controller process logging params (default: $OVN_CONTROLLER_LOG) WARNING: Line is 109 characters long (recommended limit is 79) #208 FILE: utilities/ovn-ctl:1301: --ovn-br-db-log=STRING ovn brdb ovsdb-server processes logging params (default: $OVN_BR_DB_LOG) WARNING: Line is 83 characters long (recommended limit is 79) #217 FILE: utilities/ovn-ctl:1435: --db-ovnbr-sock=SOCKET OVN_Bridge_Controller db socket (default: $DB_OVNBR_SOCK) WARNING: Line is 81 characters long (recommended limit is 79) #218 FILE: utilities/ovn-ctl:1436: --db-ovnbr-file=FILE OVN_Bridge_Controller db file (default: $DB_OVNBR_FILE) WARNING: Line is 87 characters long (recommended limit is 79) #219 FILE: utilities/ovn-ctl:1437: --db-ovnbr-pidfile=FILE OVN_Bridge_Controller db pidfile (default: $DB_OVNBR_PIDFILE) WARNING: Line is 83 characters long (recommended limit is 79) #220 FILE: utilities/ovn-ctl:1438: --db-ovnbr-schema=FILE OVN_Bridge_Controller db file (default: $DB_OVNBR_SCHEMA) WARNING: Line is 89 characters long (recommended limit is 79) #221 FILE: utilities/ovn-ctl:1439: --db-ovnbr-addr=ADDR OVN_Bridge_Controller db ptcp address (default: $DB_OVNBR_ADDR) WARNING: Line is 86 characters long (recommended limit is 79) #222 FILE: utilities/ovn-ctl:1440: --db-ovnbr-port=PORT OVN_Bridge_Controller db ptcp port (default: $DB_OVNBR_PORT) WARNING: Line is 100 characters long (recommended limit is 79) #223 FILE: utilities/ovn-ctl:1441: --db-ovnbr-ctrl-sock=SOCKET OVN_Bridge_Controller db control socket (default: $DB_OVNBR_CTRL_SOCK) WARNING: Line is 87 characters long (recommended limit is 79) #224 FILE: utilities/ovn-ctl:1442: --ovn-ovnbr-logfile=FILE OVN_Bridge_Controller log file (default: $OVN_OVNBR_LOGFILE) WARNING: Line is 127 characters long (recommended limit is 79) #225 FILE: utilities/ovn-ctl:1443: --db-ovnbr-create-insecure-remote=yes|no Create ptcp OVN_Bridge_Controller remote (default: $DB_OVNBR_CREATE_INSECURE_REMOTE) WARNING: Line is 81 characters long (recommended limit is 79) #310 FILE: utilities/ovn-ctl.8.xml:117: <p><code>--ovn-br-controller-ssl-bootstrap-ca-cert=<var>CERT</var></code></p> Lines checked: 352, Warnings: 29, Errors: 0 Please check this out. If you feel there has been an error, please email acon...@redhat.com Thanks, 0-day Robot _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev