{"push_id":846561096,"size":35,"distinct_size":27,"ref":"refs/heads/master","head":"86e9804898a6cb373553d36940effead38183cdd","before":"5a21d9c2b1da69240bae1902319e628a42bc35b7","commits":[{"sha":"2ddf7558f2a726a440bfbfecce5045ec3ebcef08","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"ovn-controller: Un-inline get_chassis_by_name().\n\nI don't know of any reason to inline this.\n\nAlso rename for consistency with get_bridge().\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":false,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/2ddf7558f2a726a440bfbfecce5045ec3ebcef08"},{"sha":"78aab811be52cecb6dea3dc937bb9bc4e558e422","author":{"email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@nicira.com","name":"Justin Pettit"},"message":"ovn: Add stateful ACL support.\n\nAdd support for the \"allow-related\" ACL action.  This is dependent on\nthe OVS conntrack functionality, which is not available on all platforms\nor kernel versions.\n\nHere is a sample policy that will allow all tenants in logical switch\n\"ls0\" to SSH to each other.  Anyone can make an HTTP request to \"lp0\".\nAll other IP traffic is dropped:\n\n  ovn-nbctl acl-add ls0 from-lport 100 ip allow-related\n  ovn-nbctl acl-add ls0 to-lport 100 tcp.dst==22 allow-related\n  ovn-nbctl acl-add ls0 to-lport 100 \"outport == \\\"lp0\\\" \\\n            && tcp.dst==80\" allow-related\n  ovn-nbctl acl-add ls0 to-lport 1 ip drop\n\nNote: Kernel conntrack support is checked into the mainline Linux\nkernel, but hasn't been backported to the main OVS repo yet.\n\nSigned-off-by: Justin Pettit <jpettit@nicira.com>\nAcked-by: Ben Pfaff <blp@nicira.com>","distinct":false,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/78aab811be52cecb6dea3dc937bb9bc4e558e422"},{"sha":"89c87563dce8224cae9e5b189c9257c1ff02e8f7","author":{"email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@nicira.com","name":"Justin Pettit"},"message":"ovn-controller.8: Correct reference to \"ovn-encap-type\".\n\nSigned-off-by: Justin Pettit <jpettit@nicira.com>\nAcked-by: Ben Pfaff <blp@nicira.com>","distinct":false,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/89c87563dce8224cae9e5b189c9257c1ff02e8f7"},{"sha":"3c6535332029ff6a03121ce9a392c2cb627bea27","author":{"email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@nicira.com","name":"Justin Pettit"},"message":"ovn-sbctl: Add multiple encaps with \"chassis-add\".\n\nSigned-off-by: Justin Pettit <jpettit@nicira.com>\nAcked-by: Ben Pfaff <blp@nicira.com>","distinct":false,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/3c6535332029ff6a03121ce9a392c2cb627bea27"},{"sha":"379109949ece0c818e49e153d95f3a9f4057c68e","author":{"email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@nicira.com","name":"Justin Pettit"},"message":"ovn-controller: Support VXLAN enapsulation.\n\nSigned-off-by: Justin Pettit <jpettit@nicira.com>\nAcked-by: Ben Pfaff <blp@nicira.com>","distinct":false,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/379109949ece0c818e49e153d95f3a9f4057c68e"},{"sha":"bf388125f6b294224d31366b81c69908543d2671","author":{"email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@nicira.com","name":"Justin Pettit"},"message":"ovn-controller: Support multiple encaps simultaneously.\n\nSigned-off-by: Justin Pettit <jpettit@nicira.com>\nAcked-by: Ben Pfaff <blp@nicira.com>","distinct":false,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/bf388125f6b294224d31366b81c69908543d2671"},{"sha":"eb6b08eb4ad85de001abf36be83301d71a20345a","author":{"email":"d9b4b2b26c9ee6f83b716cfbeaaf4ce39c1ac41c@nicira.com","name":"Justin Pettit"},"message":"ovn.at: Add test for gateway.\n\nThis test exposed a problem that ovn-controller-vtep doesn't properly\nset up the \"Mcast_Macs_Remote\" table, which prevents broadcasts from\nbeing sourced from the physical side of the VTEP.  That issue needs to\nbe resolved, and then the full set of gateway traffic patterns can run.\n\nSigned-off-by: Justin Pettit <jpettit@nicira.com>\nAcked-by: Ben Pfaff <blp@nicira.com>","distinct":false,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/eb6b08eb4ad85de001abf36be83301d71a20345a"},{"sha":"3986cae69621bd82a8945345f6345edc2407b9fd","author":{"email":"3292324943adc2eb4010fbebf00756fa1d724198@netronome.com","name":"Simon Horman"},"message":"ofproto: Correct encoding and decoding of group desc properties.\n\n* encode: if properties are present include their length in\n          value of the length field of the group desc\n* decode: use the value of the length field to calculate the length of\n          properties rather than assuming that the rest of the message\n          is properties. This assumption is not correct as a message\n          may contain multiple group descs.\n\nFixes: 18ac06d3546e (\"ofp-util: Encoding and decoding of (draft) OpenFlow 1.5 group messages.\")\nSigned-off-by: Simon Horman <simon.horman@netronome.com>\nSigned-off-by: Ben Pfaff <blp@nicira.com>","distinct":false,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/3986cae69621bd82a8945345f6345edc2407b9fd"},{"sha":"3feeb8b198d48e5e65b2fc0d7cfb2b037677dd6a","author":{"email":"c4089df073b4e0e1342dd7f47682381cc38caa5c@nicira.com","name":"Gurucharan Shetty"},"message":"windefs: Redefine INET6_ADDRSTRLEN for Windows.\n\nWindows has INET6_ADDRSTRLEN defined as 65 whereas\nPOSIX has it as 46. This difference causes a unit test\nfailure as the test 'tunnel_push_pop' was looking at o/p\nformat based on the length of INET6_ADDRSTRLEN.\n\nSigned-off-by: Gurucharan Shetty <gshetty@nicira.com>\nAcked-by: Joe Stringer <joestringer@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/3feeb8b198d48e5e65b2fc0d7cfb2b037677dd6a"},{"sha":"419e9afa445ee481d09e95411e07a0edda0fafac","author":{"email":"c4089df073b4e0e1342dd7f47682381cc38caa5c@nicira.com","name":"Gurucharan Shetty"},"message":"atlocal.in: Hide a harmless error on Windows.\n\nSigned-off-by: Gurucharan Shetty <gshetty@nicira.com>\nAcked-by: Joe Stringer <joestringer@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/419e9afa445ee481d09e95411e07a0edda0fafac"},{"sha":"c7e69808bdb4beb5df2f69b964d448487e775926","author":{"email":"59969d4426ca8e68344c28a9fcbac155693c4b00@nicira.com","name":"Joe Stringer"},"message":"ofp-actions: Fix build on earlier GCC.\n\nGCC4.4 gets confused by anonymous fields + flexible fields, complains:\nlib/ofp-actions.h:510: error: flexible array member in otherwise empty\nstruct\nlib/ofp-actions.h:512: error: bit-field ‘build_assert_failed’ width not\nan integer constant\nlib/ofp-actions.h:514: error: bit-field ‘build_assert_failed’ width not\nan integer constant\n\nFix the problem by specifying the actions length as zero.\n\nReported-by: Pravin B Shelar <pshelar@nicira.com>\nSigned-off-by: Joe Stringer <joestringer@nicira.com>\nAcked-by: Pravin B Shelar <pshelar@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/c7e69808bdb4beb5df2f69b964d448487e775926"},{"sha":"00ea19e49636e06edbd799d5e0255f507f1e7152","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"ovn-sb: Document \"ip.ttl--;\" action.\n\nThis was implemented by commits a while back, but the documentation was\naccidentally omitted.\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/00ea19e49636e06edbd799d5e0255f507f1e7152"},{"sha":"69a832cfc033338101db192c6db0aa07397acaaf","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"ovn: Update TODO, ovn-northd flow table design, ovn-architecture for L3.\n\nThis is a proposed plan for logical L3 in OVN.  It is not entirely\ncomplete but it includes many important details and I believe that it moves\nplanning forward.\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/69a832cfc033338101db192c6db0aa07397acaaf"},{"sha":"a1d210a64f2b5a2661256aa50c200bf6ac1f3373","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"ovn-controller: Factor patch port management into new \"patch\" module.\n\nUpcoming patches will introduce new extensive use of patch ports and it\nseems reasonable to put it into its own file.\n\nThis is mostly code motion.  Code changes are limited to those necessary\nto make the separated code compile, except for renaming\ninit_bridge_mappings() to patch_run().\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/a1d210a64f2b5a2661256aa50c200bf6ac1f3373"},{"sha":"cb08edc691d863dbac4327fc6c85c6fc2a408e1c","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"patch: Bail out earlier if OVS IDL transactions cannot be executed.\n\nThe whole point of this module is side effects on the Open vSwitch\ndatabase, so the whole thing can be skipped if those are impossible.\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/cb08edc691d863dbac4327fc6c85c6fc2a408e1c"},{"sha":"58bcc67cd136bd1734d108c50b2f91c530d2cb1b","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"patch: Refactor to better support new kinds of patches.\n\nUntil now, the code here lumped together what was necessary to create and\ndestroy patch ports, with what was necessary to identify the patch ports\nthat were needed.  An upcoming patch will add new reasons to create patch\nports, so this commit more cleanly separates those two functions.\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/58bcc67cd136bd1734d108c50b2f91c530d2cb1b"},{"sha":"e51d619c27effc3f9b40c510c78eec44681b6077","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"patch: Allow client to determine port names.\n\nCalculating the patch port names from the bridge names makes sense when\nthere's only one pair of patch ports between a pair of bridges, but that\nwon't be the case for an upcoming use of patch ports.\n\nThis changes makes it easy to check for existing patch ports in\ncreate_patch_port(), instead of in its caller, and since that seems like a\nmore sensible place this change also moves it there.\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/e51d619c27effc3f9b40c510c78eec44681b6077"},{"sha":"14f82efde4592e8f062d00688978dc781b84f5d1","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"ovn-controller: Rename \"ovn-patch-port\" to \"ovn-localnet-port\".\n\nAn upcoming patch will introduce a different use for patch ports, so\novn-patch-port would become an ambiguous name.\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/14f82efde4592e8f062d00688978dc781b84f5d1"},{"sha":"d387d24dca19988de1d8935c036a04fc2ed9a8c4","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"ovn: Implement logical patch ports.\n\nThis implementation is suboptimal for several reasons.  First, it\ncreates an OVS port for every OVN logical patch port, not just for the\nones that are actually useful on this hypervisor.  Second, it's\nwasteful to create an OVS patch port per OVN logical patch port, when\nreally there's no benefit to them beyond a way to identify how a\npacket ingressed into a logical datapath.\n\nThere are two obvious ways to improve the situation here, by modifying\nOVS:\n\n    1. Add a way to configure in OVS which fields are preserved on a\n       hop across an OVS patch port.  If MFF_LOG_DATAPATH and\n       MFF_LOG_INPORT were preserved, then only a single pair of OVS\n       patch ports would be required regardless of the number of OVN\n       logical patch ports.\n\n    2. Add a new OpenFlow extension action modeled on \"resubmit\" that\n       also saves and restores the packet data and metadata (the\n       inability to do this is the only reason that \"resubmit\" can't\n       be used already).  Or add OpenFlow extension actions to\n       otherwise save and restore packet data and metadata.\n\nWe should probably choose one of those in the medium to long term, but\nI don't know which one.\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/d387d24dca19988de1d8935c036a04fc2ed9a8c4"},{"sha":"a20111174e368aba0e6d6869b8bbcd432cb38159","author":{"email":"6f50513b21952446fbc8c95a8f361b7aa434b7c7@nicira.com","name":"Ben Pfaff"},"message":"ovn: Add new predicates for matching broadcast and multicast packets.\n\nIn my opinion, \"eth.mcast\" is a bit more readable than \"eth.dst[40]\", and\nso on.\n\nSigned-off-by: Ben Pfaff <blp@nicira.com>\nAcked-by: Justin Pettit <jpettit@nicira.com>","distinct":true,"url":"https://api.github.com/repos/muraliran/ovs-reviews/commits/a20111174e368aba0e6d6869b8bbcd432cb38159"}]}