)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"f3752b72924569ca9da4d845f714e016ab9abb63","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Andy Shi \u003candy_s@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-06-09 14:04:01 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"getting rid of /r"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id:I09573ea2d124efbcb99785cf1a8563b240597ff6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"1ae5cdf2_3a3976f2","line":7,"updated":"2014-06-11 18:24:08.000000000","message":"it\u0027s better to have a more meaningful summary","commit_id":"7f521926a6708e38df70a33759284cf04e5a29b4"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"3b37555b560dbed18ca1cc402bf16b9c9b6bf77a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Andy Shi \u003candy_s@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-06-11 12:33:52 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Dell ml2 Mechanism Driver:clean up all the \"\\r\" characters at the ends of the lines"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id:I09573ea2d124efbcb99785cf1a8563b240597ff6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1ae5cdf2_2b076a19","line":7,"updated":"2014-06-11 19:41:18.000000000","message":"\"Dell ml2 Mechanism Driver\" alone is better.\n\nthe first line of commit message is used as a summary.\neg. https://review.openstack.org/#/","commit_id":"935d1486d207a751829f474fbb1e155cad2fcb70"}],"specs/juno/ml2-dell-afc-mech-driver.rst":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"3b37555b560dbed18ca1cc402bf16b9c9b6bf77a","unresolved":false,"context_lines":[{"line_number":107,"context_line":"::"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"  [ml2]"},{"line_number":110,"context_line":"  mechanism_drivers \u003d openvswitch,dell"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"Developer impact"},{"line_number":113,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"1ae5cdf2_0ba2aefc","line":110,"updated":"2014-06-11 19:41:18.000000000","message":"does this use unmodified OVS agent on compute/network nodes?","commit_id":"935d1486d207a751829f474fbb1e155cad2fcb70"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"https://blueprints.launchpad.net/neutron/+spec/dell-mechanism-driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This blueprint is to add mechanism driver to OpenStack"},{"line_number":10,"context_line":"Neutron ML2 plugin. The mechanism driver will work with Dell AFC SDN controller."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_6001c738","line":9,"updated":"2014-06-18 00:10:23.000000000","message":"add -\u003e add a\n\nOpenStack -\u003e the OpenStack\n\nPlease make sure that references to nouns such as \u0027plugin\u0027, \u0027controller\u0027 and \u0027rest api\u0027 are prefixed with the definite article \u0027the\u0027 as appropriate.\n\nReference: http://en.wikipedia.org/wiki/Article_(grammar)#Definite_article","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Dell Access Fabric Controller(AFC) provides multi-tenancy network connectivity"},{"line_number":17,"context_line":"for servers and VMs connected to the active fabric. It manages a group of Dell"},{"line_number":18,"context_line":"Switches (currently supporting Dell 4810, 4820, MXL, S6000) that use Openflow"},{"line_number":19,"context_line":"protocol to communicate with the AFC controller. The Dell ML2 mechanism driver"},{"line_number":20,"context_line":"proposed will serve as the channel to pass down orchestration from Openstack"},{"line_number":21,"context_line":"to AFC controller. The communication is conducted via AFC Rest API. The mechanism"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_99fbaa46","line":18,"updated":"2014-06-18 00:10:23.000000000","message":"Openflow -\u003e the Openflow","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":17,"context_line":"for servers and VMs connected to the active fabric. It manages a group of Dell"},{"line_number":18,"context_line":"Switches (currently supporting Dell 4810, 4820, MXL, S6000) that use Openflow"},{"line_number":19,"context_line":"protocol to communicate with the AFC controller. The Dell ML2 mechanism driver"},{"line_number":20,"context_line":"proposed will serve as the channel to pass down orchestration from Openstack"},{"line_number":21,"context_line":"to AFC controller. The communication is conducted via AFC Rest API. The mechanism"},{"line_number":22,"context_line":"driver will provide information needed by AFC to configure the Dell switches,"},{"line_number":23,"context_line":"such as vlan and port. It will also provide synchronization functionality with"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_79833ea9","line":20,"updated":"2014-06-18 00:10:23.000000000","message":"pass down -\u003e communicate","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":20,"context_line":"proposed will serve as the channel to pass down orchestration from Openstack"},{"line_number":21,"context_line":"to AFC controller. The communication is conducted via AFC Rest API. The mechanism"},{"line_number":22,"context_line":"driver will provide information needed by AFC to configure the Dell switches,"},{"line_number":23,"context_line":"such as vlan and port. It will also provide synchronization functionality with"},{"line_number":24,"context_line":"AFC controller when there is a disconnection between Openstack and AFC controller."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_79589e57","line":23,"updated":"2014-06-18 00:10:23.000000000","message":"synchronization functionality with -\u003e the ability to synchronize with the","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":26,"context_line":"Proposed change"},{"line_number":27,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Dell mechanism driver will implement the following functions following the"},{"line_number":30,"context_line":"mechanism driver framework:"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"- create_network_postcommit"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_791fbe70","line":29,"updated":"2014-06-18 00:10:23.000000000","message":"following -\u003e from","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Other than the standard information passed down by ml2 plugin, Dell mechanism"},{"line_number":40,"context_line":"driver will provide AFC controller with information needed to form the abstract"},{"line_number":41,"context_line":"fabric view. These information include the hypervisor information, the vm"},{"line_number":42,"context_line":"information and the tenant information."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"In addition to these functions, the Dell mechanism driver also has a thread that"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_540c893b","line":41,"updated":"2014-06-18 00:10:23.000000000","message":"Note entirely sure of your intent, but would this make more sense?\n\nThese information include... -\u003e This information would include details about hypervisors, VMs, and tenants.","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":41,"context_line":"fabric view. These information include the hypervisor information, the vm"},{"line_number":42,"context_line":"information and the tenant information."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"In addition to these functions, the Dell mechanism driver also has a thread that"},{"line_number":45,"context_line":"will check the AFC controller start up time. And it will decide whether or not"},{"line_number":46,"context_line":"to synchronize based on that."},{"line_number":47,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_54b729f5","line":44,"updated":"2014-06-18 00:10:23.000000000","message":"Please rephrase to indicate that a \u0027periodic task\u0027 will check for whether the driver has fallen out of sync with the AFC controller.","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":42,"context_line":"information and the tenant information."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"In addition to these functions, the Dell mechanism driver also has a thread that"},{"line_number":45,"context_line":"will check the AFC controller start up time. And it will decide whether or not"},{"line_number":46,"context_line":"to synchronize based on that."},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Alternatives"}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_b4c78d63","line":45,"updated":"2014-06-18 00:10:23.000000000","message":"I would like to see more detail about how the synchronization will be implemented, since distributed state management has been a persistent source of problems with other plugins.","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"},{"author":{"_account_id":2035,"name":"Maru Newby","email":"marun@redhat.com","username":"maru"},"change_message_id":"a3ca9adffe8df776014db9e02bf548059d024f41","unresolved":false,"context_lines":[{"line_number":128,"context_line":"----------"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"- The mechanism driver basic functions"},{"line_number":131,"context_line":"- The additional information of hypervisor, vm and tenant"},{"line_number":132,"context_line":"- The synchronization function"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1ae5cdf2_d4a7b9a5","line":131,"updated":"2014-06-18 00:10:23.000000000","message":"Do you mean sourcing these details from Neutron and transforming them into the format needed by the controller?","commit_id":"ca0183546cf72967cbdc225c9ab853fb94da5930"}]}