{"push_id":2779017045,"size":37,"distinct_size":37,"ref":"refs/heads/master","head":"21f41519a2b7fe64b306f3ed6a516946b597d419","before":"7a21fb06efc8375dcdd4dbb4d05071350b8db90b","commits":[{"sha":"4d8f04b3e93d7a853619e664abde955d0dfe0c48","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"util: Fix abs_file_name() bugs on Windows.\n\nabs_file_name() believed that a file name that begins with / or contains :\nis absolute and that any other file name is relative.  On Windows, this is\nwrong in at least the following ways:\n\n   * / and \\ are interchangeable on Windows.\n\n   * A name that begins with \\\\ or // is also absolute.\n\n   * A name that begins with X: but not X:\\ is not absolute.\n\n   * A name with : in some position other than the second position is\n     not absolute (although it might not be valid either?).\n\nFurthermore, Windows has more than one current working directory (one per\nvolume letter), so trying to make a file name absolute by just prefixing\nthe current working directory for the current volume results in silliness.\n\nThis patch attempts to fix the problem.\n\nThis makes OVS link against shlwapi, which is needed to use\nPathIsRelative().\n\nFound by inspection.\n\nAcked-by: Alin Gabriel Serdean <aserdean@ovn.org>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/4d8f04b3e93d7a853619e664abde955d0dfe0c48"},{"sha":"4c36252dce687a53e39a37b2acbdbb20a965217d","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"unixctl: Use absolute paths on Windows too.\n\nWhen this case was adapted for Windows, asb_file_name() simply didn't work\nat all there.  Now, it should work OK, and it seems like the right thing\nto do, and it makes the code more straightforward too.\n\nAcked-by: Alin Gabriel Serdean <aserdean@ovn.org>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/4c36252dce687a53e39a37b2acbdbb20a965217d"},{"sha":"c8cd1307b2482df7df25e758b3cca6bebd76009f","author":{"name":"Martin Xu","email":"d739f56430f2fc9e8bf249951027992fc5ecdbc9@gmail.com"},"message":"rhel: support kmod build against multiple 7.2 kernels, rhel6\n\nThis patch extends commit 89dd5819cf18 (rhel: support kmod-openvswitch\nbuild against multiple kernels, rhel6) to support building kmod RPMs\nwith multiple minor revisions within 3.10.0-327 kernels. It was\ndiscovered for RHEL 7.2 that 41.3 minor revision introduced backward\nincompatible changes.\n\nVMware-BZ: #2170334\n\nSigned-off-by: Martin Xu <martinxu9.ovs@gmail.com>\nCC: Greg Rose <gvrose8192@gmail.com>\nCC: Ben Pfaff <blp@ovn.org>\nCC: Flavio Leitner <fbl@redhat.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Yi-Hung Wei <yihung.wei@gmail.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/c8cd1307b2482df7df25e758b3cca6bebd76009f"},{"sha":"c98638358053ee5a9f31f7acbcef20670698d0bc","author":{"name":"Martin Xu","email":"d739f56430f2fc9e8bf249951027992fc5ecdbc9@gmail.com"},"message":"rhel: bug fix kmod spec file, rhel6\n\nThis patch fixes a scenario not working for RHEL7.3 in commit 89dd5819cf18.\nWhen multiple versions passed into the kversion for the spec file, the\nvariable is used as is for the kernel module paths for command\nweak-modules --add-modules. Then the modules cannot be found.\n\nFixes: 89dd5819cf18 (rhel: support kmod-openvswitch build against\nmultiple kernels, rhel6)\n\nSigned-off-by: Martin Xu <martinxu9.ovs@gmail.com>\nCC: Greg Rose <gvrose8192@gmail.com>\nCC: Ben Pfaff <blp@ovn.org>\nCC: Flavio Leitner <fbl@redhat.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Yi-Hung Wei <yihung.wei@gmail.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/c98638358053ee5a9f31f7acbcef20670698d0bc"},{"sha":"5da3e1fe02d31144fb24cd187550434248aa7b1f","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"nx-match: Fix memory leak in oxm_pull_field_array() error case.\n\nReported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=9424\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/5da3e1fe02d31144fb24cd187550434248aa7b1f"},{"sha":"7ee9c6e03416613f8461b37d7fdc52205d3c742b","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"tests: Fix cluster torture test.\n\nA previous commit to improve timing also caused the cluster torture test to\nbe skipped (unless it failed early).  This is related to the shell \"while\"\nloop's use of a variable $phase to indicate how far it got in the test\nprocedure.  A very fast machine, or one on which the races went just the\nright way, might finish the test before all the torture properly starts, so\nthe code is designed to just skip the test if that happens.  However, a\ncommit to improve the accuracy ended up skipping it all the time.\n\nPrior to the timing commit, the loop looked something like this:\n\n    phase=0\n    while :; do\n        ...things that eventually increment $phase to 2...\n    done\n    AT_SKIP_IF([test $phase != 2])\n\nThis works fine.\n\nThe timing commit changed the \"while :\" to \"(...something...) | while\nread\".  This looks innocuous but it actually causes everything inside the\n\"while\" loop to run in a subshell.  Thus, the increments to $phase are not\nvisible after the loop ends, and the test always gets skipped.\n\nThis commit fixes the problem by storing the phase in a file instead of a\nshell variable.\n\nFixes: 0f03ae3754ec (\"ovsdb: Improve timing in cluster torture test.\")\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/7ee9c6e03416613f8461b37d7fdc52205d3c742b"},{"sha":"c7b5c534842884445a5cc1452fee45b3c9fd655e","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"tests: Fix use of variable in cluster torture test.\n\nremove_server() is supposed to deal with its argument $i, not $victim.  In\nthis case they happen to have the same value so the difference is moot,\nbut it's still best to be clear.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/c7b5c534842884445a5cc1452fee45b3c9fd655e"},{"sha":"f39f02102f669b85a37104e7bc60cb764cc6b2f3","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"tests: Suppress \"long poll interval\" messages for ovsdb-cluster tests.\n\nThe cluster torture tests can provoke these messages, especially if run in\nparallel or with valgrind, and they shouldn't cause a failure.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/f39f02102f669b85a37104e7bc60cb764cc6b2f3"},{"sha":"6233b87dbbc7a69f1226cab2d3f10208d16e4dd1","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ovsdb-tool: Fix logic for displaying server names in \"check-cluster\".\n\nWithout this patch, server names are displayed as UUIDs even when we know\nthe file name; file names are easier for the user to read.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/6233b87dbbc7a69f1226cab2d3f10208d16e4dd1"},{"sha":"6bf2e3f644dfe22a20584a15cc6df294e5efc4c5","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ovsdb-tool: Read server headers first, before full logs, in \"check-cluster\".\n\nHaving the headers available before reading the complete logs means that\nserver IDs can be associated with the server file names earlier, which can\nimprove error messages in some cases.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/6bf2e3f644dfe22a20584a15cc6df294e5efc4c5"},{"sha":"7073a83fc731fc997dd51bfdd8cd99dd29a663ab","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ovsdb-tool: Check for duplicate server IDs in \"check-cluster\".\n\nThe user shouldn't provide a given server's log more than once but this\ncheck makes sure.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/7073a83fc731fc997dd51bfdd8cd99dd29a663ab"},{"sha":"c2d71875f2a32665d9eaca5e964e76cae2795691","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ovsdb-tool: Only check leader completeness when we can, in \"check-cluster\".\n\nGenerally when we know the leader for a term, in \"check-cluster\", it's\nbecause we read that leader's log file.  In that case, we have the leader's\nlog_end because it told us.  However, taking a snapshot can discard that\ndata.  In that case, log_end is 0 and we should not try to check for leader\ncompleteness on that basis.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Mark Michelson <mmichels@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/c2d71875f2a32665d9eaca5e964e76cae2795691"},{"sha":"6c8dd8caaf1939bcd2f4d356850d877e1d9dc436","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"ovsdb-cluster: Add comment to test.\n\nI thought I had added this while revising a previous patch but oops.\n\nFixes: 7ee9c6e03416 (\"tests: Fix cluster torture test.\")\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/6c8dd8caaf1939bcd2f4d356850d877e1d9dc436"},{"sha":"c5358a967aa144db7ffddff7d192a0f52b57885d","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"meta-flow: Make mf_vl_mff_mf_from_nxm_header() require a valid field.\n\nAll the users of mf_vl_mff_mf_from_nxm_header() expect it to always obtain\na valid field or to report an error.  In practice, it did not report an\nerror in the case where the field was unknown (although it did report an\nerror in some other cases).  This commit fixes the problem.\n\nReported-at: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=9652\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Alin Gabriel Serdean <aserdean@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/c5358a967aa144db7ffddff7d192a0f52b57885d"},{"sha":"2f7662a10e3990e679c939506d4258339b348c66","author":{"name":"Ben Pfaff","email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@ovn.org"},"message":"pcap-file: Fix formatting of log message.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Alin Gabriel Serdean <aserdean@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/2f7662a10e3990e679c939506d4258339b348c66"},{"sha":"49c5ee21ead68be4fc4593206dd022ad9249b286","author":{"name":"Timothy Redaelli","email":"a4b59bb75648eb5009c48ef2c8b66a57977abb22@redhat.com"},"message":"utilities: don't use a bashism to check that datapath exists\n\n[[ ]] syntax is not supported, at least, by dash that Debian, Ubuntu and other\nlinux distributions may use instead of bash.\n\nThis commit uses, instead, a POSIX way that is compatible with any POSIX\nshell (bash, dash, busybox sh, etc).\n\nCC: Martin Xu <martinxu9.ovs@gmail.com>\nFixes: 9763d17fbd05 (\"utilities: check datapath exists before conntrack flush\")\nSigned-off-by: Timothy Redaelli <tredaelli@redhat.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Aaron Conole <aconole@redhat.com>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/49c5ee21ead68be4fc4593206dd022ad9249b286"},{"sha":"ab16d2c2871b82d1f71c652657791acd9ca51161","author":{"name":"Timothy Redaelli","email":"a4b59bb75648eb5009c48ef2c8b66a57977abb22@redhat.com"},"message":"stream-ssl: Don't enable new TLS versions by default\n\nCurrently protocol_flags is populated by the list of SSL and TLS\nprotocols by hand. This means that when a new TLS version is added to\nopenssl (in this case TLS v1.3 is added to openssl 1.1.1 beta)\novsdb-server automatically enable support to it with the default ciphers.\nThis can be a security problem (since other ciphers can be enabled) and it\nalso makes a test (SSL db: implementation) to fail.\n\nThis commit changes the 'protocol_flags' to use the list of all protocol\nflags as provided by openssl library (SSL_OP_NO_SSL_MASK) so there is no\nneed to keep the list updated by hand.\n\nSigned-off-by: Timothy Redaelli <tredaelli@redhat.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/ab16d2c2871b82d1f71c652657791acd9ca51161"},{"sha":"eb739be2ec3e063675ecfa5cb71f7459fbe6f725","author":{"name":"Darrell Ball","email":"5cb751bd4da145a3789c9f4291568adbd95b709c@gmail.com"},"message":"db-ctl-base: Fix build with gcc 7.3 with O3.\n\nSigned-off-by: Darrell Ball <dlu998@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/eb739be2ec3e063675ecfa5cb71f7459fbe6f725"},{"sha":"1ea9742269fcb959bd27d9aa188d9c6ce32b1ee6","author":{"name":"Timothy Redaelli","email":"a4b59bb75648eb5009c48ef2c8b66a57977abb22@redhat.com"},"message":"utilities: Install ovs-tcp{dump, undump} also when only Python3 is enabled\n\nSince commit 793bdb6c0500 (\"ovs-tcpdump: Fix incompatibilities with python3\")\nand commit 227abb77d3d1 (\"ovs-tcpundump: Fix incompatibilities with python3\")\novs-tcpdump and ovs-tcpdump works with Python3 as well.\n\nThis commit allows ovs-tcpdump and ovs-tcpundump to be installed also when\nonly Python3 is enabled.\n\nSigned-off-by: Timothy Redaelli <tredaelli@redhat.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/1ea9742269fcb959bd27d9aa188d9c6ce32b1ee6"},{"sha":"2f15f8f313366589ae1065b7689d9d2d3bf139e3","author":{"name":"Darrell Ball","email":"5cb751bd4da145a3789c9f4291568adbd95b709c@gmail.com"},"message":"dpctl: Simplify dpctl_flush_conntrack.\n\nThe function dpctl_flush_conntrack() and other such new functions with\nmultiple optional arguments can be simplified by reodering the checks\nfor optional parameters, where the datapath argument is checked for\nlast.\n\nSigned-off-by: Darrell Ball <dlu998@gmail.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/kevintraynor/ovs-dpdk-master/commits/2f15f8f313366589ae1065b7689d9d2d3bf139e3"}]}
{"actor":{"display_login":"kevintraynor"}}