{"push_id":1310593714,"size":151,"distinct_size":151,"ref":"refs/heads/master","head":"baf9f02a42ce7d4a90a28aa654a945cde5164bf9","before":"f84b6e45365285a37465908c27a7f5823a9ab7d0","commits":[{"sha":"7cc36b8fe77610961498c96465bd7911ce50ff2d","author":{"name":"Justin Pettit","email":"d2a491ac77f0a8410a61da33b4e303a4175d2d1d@ovn.org"},"message":"OVN-Tutorial: Replace example with ASCII quotes.\n\nThe \"--ovn\" argument for SANDBOXFLAGS used unicode quotes, which when\ncopy and pasted made the command mysteriously fail.\n\nSigned-off-by: Justin Pettit <jpettit@ovn.org>\nAcked-by: Lance Richardson <lrichard@redhat.com>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/7cc36b8fe77610961498c96465bd7911ce50ff2d"},{"sha":"1da17ab52966e2eb06717f3614e39172a65ea02c","author":{"name":"Justin Pettit","email":"d2a491ac77f0a8410a61da33b4e303a4175d2d1d@ovn.org"},"message":"daemon: Minor tweaking of man page fragment.\n\nSigned-off-by: Justin Pettit <jpettit@ovn.org>\nAcked-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Ryan Moats <rmoats@us.ibm.com>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/1da17ab52966e2eb06717f3614e39172a65ea02c"},{"sha":"f850c889d9e74e4e190b5bbc50a33658798a04ec","author":{"name":"Justin Pettit","email":"d2a491ac77f0a8410a61da33b4e303a4175d2d1d@ovn.org"},"message":"ovn-trace: Minor cleanups.\n\nSigned-off-by: Justin Pettit <jpettit@ovn.org>\nAcked-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Ryan Moats <rmoats@us.ibm.com>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/f850c889d9e74e4e190b5bbc50a33658798a04ec"},{"sha":"4bb7f5680b19f6ffc1d7fc1d29959af1529a1318","author":{"name":"Liran Schour","email":"61d12354ac4a0869034c1f75fd9317453659ba31@il.ibm.com"},"message":"ovsdb: Fix reference to table's row on condition_add|remove_clause\n\nUse struct uuid * on [add|remove]_clause on columns which are references to\ntables. That prevents use-after-free errors.\n\nSigned-off-by: Liran Schour <lirans@il.ibm.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/4bb7f5680b19f6ffc1d7fc1d29959af1529a1318"},{"sha":"1b37392f567b174fa249a1266a4e35ac13211403","author":{"name":"Conner Herriges","email":"9d76f16ebb342265f6f11f4adcbd6209d187399c@ibm.com"},"message":"Add monitoring to ovsdb-server\n\nMonitoring is a valid option on POSIX based platforms. The monitor\ncreates an additional process to monitor the ovsdb-server daemon. If\nthe daemon dies due to a programming error, then the monitor process\nstarts a new copy of it. If the daemon dies or exits for another\nreason, the monitor process exits.\n\nThe monitor option is not currently enabled for either OVN Northbound or\nSouthbound databases. The simplest method for doing this is to add\n--monitor to the calling lines in the start_ovsdb function in ovn-ctl, which\nis where ovsdb-server for ovn is currently started. This is the method employed\nby this patch.\n\nThe second option would be to modify the code used by ovs-ctl to start\novsdb-server so that it will work for ovn, so that start_daemon from ovs-lib\nis used correctly and without modification. This option is certainly the\nmore complex option, especially considering the northbound and southbound\ndatabases in ovn.\n\nSigned-off-by: Conner Herriges <conner.herriges@ibm.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/1b37392f567b174fa249a1266a4e35ac13211403"},{"sha":"9dba43815bc6d09125ba033fb9a766fc243afb8e","author":{"name":"Ben Pfaff","email":"b30031e87fea8687a432a5f632461bee2630c54c@ovn.org"},"message":"AUTHORS: Add Conner Herriges.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/9dba43815bc6d09125ba033fb9a766fc243afb8e"},{"sha":"6e31816f719925b5b56a14ac564d3b6194f2df68","author":{"name":"Chandra S Vejendla","email":"c554db1f0722a45af70217fef39519fe430678b7@us.ibm.com"},"message":"ovn: Delete stale MAC_Bindings that result in Referential Integrity Violation\n\nThe MAC_Bindings have a strong reference to the Datapath_Binding. However the\nMAC_Bindings are never deleted anywhere, and when the Datapath (associated\nwith a MAC_Binding) is deleted, the ovsdb-server returns Referential\nIntegrity Violation. This prevents newer operations initiated from the CMS\nfrom being committed to the Southbound DB.\n\nThe patch fixes this  by deleting the MAC_Binding entry when the\nlogical_port referred in the mac_binding entry is deleted.\n\nSigned-off-by: Chandra Sekhar Vejendla <csvejend@us.ibm.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/6e31816f719925b5b56a14ac564d3b6194f2df68"},{"sha":"ce81a73f6a6deeb5e1c9de1be2dc3fe5a2fd3a56","author":{"name":"Ryan Moats","email":"e79a1704e070f6044f34a46a3f5277e8ea05871b@us.ibm.com"},"message":"ovn-controller: Fix memory leak when parsing lflow actions.\n\nParsing logical flow actions with ovnacts_parse* that include\nstring constants current leak memory.  Add calls to ovnacts_free\nto recapture said memory.\n\nSigned-off-by: Ryan Moats <rmoats@us.ibm.com>\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/ce81a73f6a6deeb5e1c9de1be2dc3fe5a2fd3a56"},{"sha":"36149af381df82568f5f6a3b15be44a7801ce6f6","author":{"name":"Paul Boca","email":"047d6c737d4c223ff1f9d054f567effdde325385@cloudbasesolutions.com"},"message":"tunnel tests: Skip ipsec_gre tests on Windows\n\nThe ipsec feature is not implemented on windows.\n\nSigned-off-by: Paul-Daniel Boca <pboca@cloudbasesolutions.com>\nSigned-off-by: Gurucharan Shetty <guru@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/36149af381df82568f5f6a3b15be44a7801ce6f6"},{"sha":"e0a47eae57690a8fc6e6f90405fa2e024d0830f4","author":{"name":"Ben Pfaff","email":"b30031e87fea8687a432a5f632461bee2630c54c@ovn.org"},"message":"ovn: Update TODO for items that have been completed.\n\nI think that all of these items are either done now or just simply don't\nneed this level of detail.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/e0a47eae57690a8fc6e6f90405fa2e024d0830f4"},{"sha":"03dabc347946d13f3dce71a7f14af636a7fc5ba9","author":{"name":"Ben Pfaff","email":"b30031e87fea8687a432a5f632461bee2630c54c@ovn.org"},"message":"ovn: Add TODO items proposed for 2.7 in OVN IRC meeting.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/03dabc347946d13f3dce71a7f14af636a7fc5ba9"},{"sha":"9ab989b7b6eaa781663f090dd2b366bf98dc02ee","author":{"name":"Ben Pfaff","email":"b30031e87fea8687a432a5f632461bee2630c54c@ovn.org"},"message":"ovn-northd: Avoid excessive work to find router ports.\n\nThe ovn_datapath for each logical switch maintains an array of its ports\nof type \"router-port\", but instead of iterating through it build_pre_acls()\niterated through all of the ports in the entire database, which is\nwasteful and duplicative work.  This commit switches to using the array of\nrouter ports.\n\nThis change is best viewed ignoring white space only changes.\n\nSigned-off-by: Ben Pfaff <blp@ovn.org>\nAcked-by: Ryan Moats <rmoats@us.ibm.com>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/9ab989b7b6eaa781663f090dd2b366bf98dc02ee"},{"sha":"54b78c9501f7887e1a2d6290ef1d44ee42a9222f","author":{"name":"Jarno Rajahalme","email":"34c96149e0f84eb25c9df6d97f4a94ff9adedc00@ovn.org"},"message":"vswitchd: Deprecate packet buffering in OVS 2.6.\n\nOVS implementation of buffering packets that are sent to the\ncontroller is not compliant with the OpenFlow specifications after\nOpenFlow 1.0.  OVS implementation executes the buffered packet against\nthe actions of the modified or added rule, whereas OpenFlow (since\n1.1) specifies that the packet should be matched against the flow\ntable 0 and processed accordingly.\n\nRather than fix this behavior, and potentially break OVS users, we\npropose to remove the feature altogether, starting in OVS 2.7.  This\npatch announces this in 'NEWS' for OVS 2.6, and adds detail to the FAQ\nquestion about backet buffering.\n\nSigned-off-by: Jarno Rajahalme <jarno@ovn.org>\nAcked-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/54b78c9501f7887e1a2d6290ef1d44ee42a9222f"},{"sha":"fef32d76d36ae1ec6092ce42e7ad97b4051ee5ab","author":{"name":"Paul Boca","email":"047d6c737d4c223ff1f9d054f567effdde325385@cloudbasesolutions.com"},"message":"python tests: Skip python tests specific to Linux.\n\nThere is a difference between POSIX pid and Windows pid, not all the\ntime are equal.  On Windows when a python script is started, a sh\ncommand is triggered as the parent for script.  So when we try to\nget the daemon pid with 'echo $!', this will get the pid of sh\nnot of python.exe as expected.  Some tests use undefined switches,\non Windows, for `kill` command.\n\nSigned-off-by: Paul-Daniel Boca <pboca@cloudbasesolutions.com>\nSigned-off-by: Gurucharan Shetty <guru@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/fef32d76d36ae1ec6092ce42e7ad97b4051ee5ab"},{"sha":"88e62998a2db1f8c9f3df2ef9f8d0cf8a2277262","author":{"name":"Paul Boca","email":"047d6c737d4c223ff1f9d054f567effdde325385@cloudbasesolutions.com"},"message":"datapath-windows: Fix connection state tracking in Conntrack\n\nA connection can be both a reply and related.\n\nSigned-off-by: Paul-Daniel Boca <pboca@cloudbasesolutions.com>\nAcked-by: Sairam Venugopal <vsairam@vmware.com>\nSigned-off-by: Gurucharan Shetty <guru@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/88e62998a2db1f8c9f3df2ef9f8d0cf8a2277262"},{"sha":"c184807ced554c1dc7b69b3cd4f59cd85575fdf1","author":{"name":"Jarno Rajahalme","email":"34c96149e0f84eb25c9df6d97f4a94ff9adedc00@ovn.org"},"message":"lib: Retire packet buffering feature.\n\nOVS implementation of buffering packets that are sent to the\ncontroller is not compliant with the OpenFlow specifications after\nOpenFlow 1.0, which is possibly true since OpenFlow 1.0 is not really\nspecifying the packet buffering behavior.\n\nOVS implementation executes the buffered packet against the actions of\nthe modified or added rule, whereas OpenFlow (since 1.1) specifies\nthat the packet should be matched against the flow table 0 and\nprocessed accordingly.\n\nRather than fix this behavior, and potentially break OVS users, the\npacket buffering feature is removed altogether.  After all, such\npacket buffering is an optional OpenFlow feature, and as such any\npossible users should continue to work without this feature.\n\nThis patch also makes OVS check the received 'buffer_id' values more\nrigorously, and fixes some internal users accordingly.\n\nFound by inspection.\n\nSigned-off-by: Jarno Rajahalme <jarno@ovn.org>\nAcked-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/c184807ced554c1dc7b69b3cd4f59cd85575fdf1"},{"sha":"5221d53eaf2da31ed4b594a353c8a1026d5673cf","author":{"name":"Lance Richardson","email":"529d81928dd60beeaf299499acfc13e39cc028fb@redhat.com"},"message":"upcall: reduce log level for \"no reference to recirc flow\" message\n\nReduce log level from \"warn\" to \"debug\" for \"upcall: no reference to\nrecirc flow\" log message.\n\nSuggested-by: Jarno Rajahalme <jarno@ovn.org>\nSigned-off-by: Lance Richardson <lrichard@redhat.com>\nAcked-by: Jarno Rajahalme <jarno@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/5221d53eaf2da31ed4b594a353c8a1026d5673cf"},{"sha":"0ccf23765853ba456febe96a578f9d92615adc69","author":{"name":"Jarno Rajahalme","email":"34c96149e0f84eb25c9df6d97f4a94ff9adedc00@ovn.org"},"message":"ofproto-dpif: Remove prototype for an undefined function.\n\nWe do not have a 'ofproto_dpif_refresh_rule()' function.\n\nSigned-off-by: Jarno Rajahalme <jarno@ovn.org>\nAcked-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/0ccf23765853ba456febe96a578f9d92615adc69"},{"sha":"f52a761fac28cfe3e6caddf34f75e032811cde7e","author":{"name":"Jarno Rajahalme","email":"34c96149e0f84eb25c9df6d97f4a94ff9adedc00@ovn.org"},"message":"ofproto: Remove stale comment.\n\nThe previous line tells that this comment is now stale. Remove it.\n\nSigned-off-by: Jarno Rajahalme <jarno@ovn.org>\nAcked-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/f52a761fac28cfe3e6caddf34f75e032811cde7e"},{"sha":"58026109d5ad0914d873e57df45c5e9dcc8e68da","author":{"name":"Jarno Rajahalme","email":"34c96149e0f84eb25c9df6d97f4a94ff9adedc00@ovn.org"},"message":"object-collection: Remove access to stub.\n\nBetter not use access to the *_collection_stub(), as it is an internal\nimplementation detail.\n\nSigned-off-by: Jarno Rajahalme <jarno@ovn.org>\nAcked-by: Ben Pfaff <blp@ovn.org>","distinct":true,"url":"https://api.github.com/repos/doonhammer/ovs/commits/58026109d5ad0914d873e57df45c5e9dcc8e68da"}]}
{"actor":{"display_login":"doonhammer"}}