{"push_id":1281113442,"size":4,"distinct_size":0,"ref":"refs/heads/master","head":"22ecc560c8937bb0992eca0600c99bf919f65ed5","before":"784bf5d4eb3ce41c93927991d57ef6522363cd76","commits":[{"sha":"c65a31e2622402bcb1b0b8df0ab928b83a0ed4a0","author":{"name":"Ben Pfaff","email":"b30031e87fea8687a432a5f632461bee2630c54c@ovn.org"},"message":"learn: Fix iteration over learning specs.\n\nstruct ofpact_learn_spec is variable-length.  The 'n_specs' member of\nstruct ofpact_learn counted the number of specs, but the iteration loops\nover struct ofpact_learn_spec only iterated as far as the *minimum* length\nof 'n_specs' specs.\n\nThis fixes the problem, which exhibited as consistent failures for test 431\n(learning action - TCPv6 port learning), seemingly only on i386 since it\nshows up for my personal development machine but appears to not happen for\nanyone else.\n\nFixes: dfe191d5faa6 (\"ofp-actions: Waste less memory in learn actions.\")\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Jarno Rajahalme <jarno@ovn.org>","distinct":false,"url":"https://api.github.com/repos/blp/ovs-reviews/commits/c65a31e2622402bcb1b0b8df0ab928b83a0ed4a0"},{"sha":"7478b5a2b73be87d3b42e0c2ed7b09b1efeff0fe","author":{"name":"Daniele Di Proietto","email":"bdb660ec9fa1735e5882a9192984ca478941235f@vmware.com"},"message":"Revert \"ofproto: Always set MTU for new internal ports.\"\n\nThis reverts commit 47bf118665a3d0f3c153d1fe80e9af02ac9a4e9c.\nWhile the commit tries to make it more consistent, it breaks some system\ntests.  The assumptions made on the tests are probably made by many\nusers, so it's better to revert it.\n\nSigned-off-by: Daniele Di Proietto <diproiettod@vmware.com>\nAcked-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/blp/ovs-reviews/commits/7478b5a2b73be87d3b42e0c2ed7b09b1efeff0fe"},{"sha":"3a414a0a4f1901ba015ec80b917b9fb206f3c74f","author":{"name":"Daniele Di Proietto","email":"bdb660ec9fa1735e5882a9192984ca478941235f@vmware.com"},"message":"ofproto: Honor mtu_request even for internal ports.\n\nBy default Open vSwitch tries to configure internal interfaces MTU to\nmatch the bridge minimum, overriding any attempt by the user to\nconfigure it through standard system tools, or the database.\n\nWhile this works in many simple cases (there are probably many users\nthat rely on this) it may create problems for more advanced use cases\n(like any overlay networks).\n\nThis commit allows the user to override the default behavior by\nproviding an explict MTU in the mtu_request column in the Interface\ntable.\n\nThis means that Open vSwitch will now treat differently database MTU\nrequests from standard system tools MTU requests (coming from `ip link`\nor `ifconfig`), but this seems the best way to remain compatible with\nold users while providing a more powerful interface.\n\nSuggested-by: Darrell Ball <dlu998@gmail.com>\nSigned-off-by: Daniele Di Proietto <diproiettod@vmware.com>\nAcked-by: Ben Pfaff <blp@ovn.org>\nTested-by: Joe Stringer <joe@ovn.org>","distinct":false,"url":"https://api.github.com/repos/blp/ovs-reviews/commits/3a414a0a4f1901ba015ec80b917b9fb206f3c74f"},{"sha":"22ecc560c8937bb0992eca0600c99bf919f65ed5","author":{"name":"Ben Pfaff","email":"b30031e87fea8687a432a5f632461bee2630c54c@ovn.org"},"message":"tests: Fix recently broken sFlow tests.\n\nA recent improvement to the promptness of sFlow reporting caused some of\nthe sFlow tests to fail (because the output was reported sooner).  This\nfixes up sequence numbers in the expected output to match the new behavior.\nIt also reduces the amount of (virtual) time that the test waits since it's\nno longer necessary to wait as long.\n\nFixes: 784bf5d4eb3c (\"sflow-agent: Flush freshly-polled sFlow counters promptly.\")\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":false,"url":"https://api.github.com/repos/blp/ovs-reviews/commits/22ecc560c8937bb0992eca0600c99bf919f65ed5"}]}
{"actor":{"display_login":"blp"}}