====== Checking before the patch ====== ====== Checking the tree with the patch ====== New warnings added --- /tmp/tmp.VBNorWMjIV 2026-05-01 08:35:58.712547437 -0400 +++ /tmp/tmp.jjoUpsCINL 2026-05-01 08:36:00.249533905 -0400 @@ -5 +5 @@ -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1:0: C0302: Too many lines in module (2775/1000) (too-many-lines) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1:0: C0302: Too many lines in module (2961/1000) (too-many-lines) @@ -194,106 +194,107 @@ -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1639:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1715:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1840:0: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1845:4: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1845:4: C0103: Class name "ovs_packet_msg" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1845:4: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1865:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1880:30: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1840:0: R0903: Too few public methods (1/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1885:0: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1889:4: C0103: Class name "dp_cmd_msg" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1906:8: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1906:8: C0103: Class name "dpstats" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1906:8: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1914:8: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1914:8: C0103: Class name "megaflowstats" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1914:8: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1889:4: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1927:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1948:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1949:22: C0103: Argument name "shouldUpcall" doesn't conform to snake_case naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1949:42: C0103: Argument name "versionStr" doesn't conform to snake_case naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1971:16: W0612: Unused variable 'i' (unused-variable) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1991:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2013:0: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2020:4: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2020:4: C0103: Class name "ovs_vport_msg" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2034:8: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2034:8: C0103: Class name "vportopts" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2034:8: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2041:8: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2041:8: C0103: Class name "vportstats" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2041:8: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2020:4: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2053:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2053:4: E0213: Method 'type_to_str' should have "self" as first argument (no-self-argument) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2054:8: R1705: Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2064:25: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2066:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2066:4: E0213: Method 'str_to_type' should have "self" as first argument (no-self-argument) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2067:8: R1705: Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2077:25: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2084:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2109:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2109:4: R0913: Too many arguments (6/5) (too-many-arguments) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2109:4: R0917: Too many positional arguments (6/5) (too-many-positional-arguments) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2123:8: C0103: Variable name "TUNNEL_DEFAULTS" doesn't conform to snake_case naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2167:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2176:11: C0121: Comparison 'p == None' should be 'p is None' (singleton-comparison) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2192:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2213:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2217:0: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2218:4: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2218:4: C0103: Class name "ovs_flow_msg" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2233:8: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2233:8: C0103: Class name "flowstats" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2233:8: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2239:8: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2244:20: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2281:29: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2294:29: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2308:8: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2309:12: C0103: Variable name "OVS_UFID_F_OMIT_KEY" doesn't conform to snake_case naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2310:12: C0103: Variable name "OVS_UFID_F_OMIT_MASK" doesn't conform to snake_case naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2311:12: C0103: Variable name "OVS_UFID_F_OMIT_ACTIONS" doesn't conform to snake_case naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2326:12: C0103: Variable name "uuidRawObj" doesn't conform to snake_case naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2308:41: W0613: Unused argument 'dpidx' (unused-argument) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2452:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2462:14: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2464:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2464:22: W0613: Unused argument 'packetmsg' (unused-argument) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2467:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2467:21: W0613: Unused argument 'packetmsg' (unused-argument) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2471:0: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2471:0: C0103: Class name "psample_sample" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2491:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2496:30: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2498:30: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2501:30: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2503:23: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2505:16: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2471:0: R0903: Too few public methods (1/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2508:0: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2508:0: C0103: Class name "psample_msg" doesn't conform to PascalCase naming style (invalid-name) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2508:0: R0903: Too few public methods (0/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2517:0: C0115: Missing class docstring (missing-class-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2522:4: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2517:0: R0903: Too few public methods (1/2) (too-few-public-methods) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2532:0: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2532:0: R0914: Too many local variables (16/15) (too-many-locals) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2539:10: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2541:8: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2544:10: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2548:8: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2552:10: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2555:14: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2566:28: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2568:16: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2578:0: C0116: Missing function or method docstring (missing-function-docstring) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2578:0: R0914: Too many local variables (25/15) (too-many-locals) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2706:23: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2711:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2713:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2721:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2726:14: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2738:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1685:33: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1697:33: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1701:16: W0707: Consider explicitly re-raising using 'except ValueError as exc' and 'raise ValueError("vlan(): invalid value '%s' for key '%s'" % (val, key)) from exc' (raise-missing-from) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1701:33: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1710:37: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1720:37: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1730:37: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1745:33: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1645:4: R0912: Too many branches (31/12) (too-many-branches) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1645:4: R0915: Too many statements (89/50) (too-many-statements) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1773:29: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1785:41: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1813:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:1899:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2026:0: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2031:4: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2031:4: C0103: Class name "ovs_packet_msg" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2031:4: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2051:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2066:30: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2026:0: R0903: Too few public methods (1/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2071:0: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2075:4: C0103: Class name "dp_cmd_msg" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2092:8: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2092:8: C0103: Class name "dpstats" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2092:8: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2100:8: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2100:8: C0103: Class name "megaflowstats" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2100:8: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2075:4: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2113:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2134:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2135:22: C0103: Argument name "shouldUpcall" doesn't conform to snake_case naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2135:42: C0103: Argument name "versionStr" doesn't conform to snake_case naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2157:16: W0612: Unused variable 'i' (unused-variable) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2177:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2199:0: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2206:4: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2206:4: C0103: Class name "ovs_vport_msg" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2220:8: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2220:8: C0103: Class name "vportopts" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2220:8: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2227:8: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2227:8: C0103: Class name "vportstats" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2227:8: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2206:4: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2239:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2239:4: E0213: Method 'type_to_str' should have "self" as first argument (no-self-argument) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2240:8: R1705: Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2250:25: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2252:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2252:4: E0213: Method 'str_to_type' should have "self" as first argument (no-self-argument) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2253:8: R1705: Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2263:25: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2270:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2295:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2295:4: R0913: Too many arguments (6/5) (too-many-arguments) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2295:4: R0917: Too many positional arguments (6/5) (too-many-positional-arguments) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2309:8: C0103: Variable name "TUNNEL_DEFAULTS" doesn't conform to snake_case naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2353:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2362:11: C0121: Comparison 'p == None' should be 'p is None' (singleton-comparison) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2378:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2399:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2403:0: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2404:4: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2404:4: C0103: Class name "ovs_flow_msg" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2419:8: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2419:8: C0103: Class name "flowstats" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2419:8: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2425:8: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2430:20: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2467:29: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2480:29: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2494:8: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2495:12: C0103: Variable name "OVS_UFID_F_OMIT_KEY" doesn't conform to snake_case naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2496:12: C0103: Variable name "OVS_UFID_F_OMIT_MASK" doesn't conform to snake_case naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2497:12: C0103: Variable name "OVS_UFID_F_OMIT_ACTIONS" doesn't conform to snake_case naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2512:12: C0103: Variable name "uuidRawObj" doesn't conform to snake_case naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2494:41: W0613: Unused argument 'dpidx' (unused-argument) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2638:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2648:14: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2650:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2650:22: W0613: Unused argument 'packetmsg' (unused-argument) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2653:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2653:21: W0613: Unused argument 'packetmsg' (unused-argument) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2657:0: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2657:0: C0103: Class name "psample_sample" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2677:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2682:30: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2684:30: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2687:30: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2689:23: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2691:16: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2657:0: R0903: Too few public methods (1/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2694:0: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2694:0: C0103: Class name "psample_msg" doesn't conform to PascalCase naming style (invalid-name) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2694:0: R0903: Too few public methods (0/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2703:0: C0115: Missing class docstring (missing-class-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2708:4: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2703:0: R0903: Too few public methods (1/2) (too-few-public-methods) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2718:0: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2718:0: R0914: Too many local variables (16/15) (too-many-locals) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2725:10: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2727:8: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2730:10: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2734:8: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2738:10: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) @@ -301,7 +302,18 @@ -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2752:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2756:29: R1719: The if expression can be replaced with 'test' (simplifiable-if-expression) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2760:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2768:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2578:0: R0912: Too many branches (34/12) (too-many-branches) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2578:0: R0915: Too many statements (121/50) (too-many-statements) -tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2578:9: W0613: Unused argument 'argv' (unused-argument) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2752:28: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2754:16: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2764:0: C0116: Missing function or method docstring (missing-function-docstring) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2764:0: R0914: Too many local variables (25/15) (too-many-locals) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2892:23: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2897:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2899:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2907:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2912:14: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2924:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2927:14: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2938:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2942:29: R1719: The if expression can be replaced with 'test' (simplifiable-if-expression) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2946:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2954:18: C0209: Formatting a regular string which could be an f-string (consider-using-f-string) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2764:0: R0912: Too many branches (34/12) (too-many-branches) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2764:0: R0915: Too many statements (121/50) (too-many-statements) +tools/testing/selftests/net/openvswitch/ovs-dpctl.py:2764:9: W0613: Unused argument 'argv' (unused-argument) @@ -311 +323 @@ -Your code has been rated at 7.48/10 (previous run: 7.48/10, -0.01) +Your code has been rated at 7.61/10 (previous run: 7.48/10, +0.14)