)]}'
{"id":"openstack%2Fdevstack~37963","triplet_id":"openstack%2Fdevstack~master~Ib16efac13440b5d50658a0e6be35bc735510a262","project":"openstack/devstack","branch":"master","topic":"bug/1200767","hashtags":[],"change_id":"Ib16efac13440b5d50658a0e6be35bc735510a262","subject":"Update the ML2 configuration in devstack","status":"MERGED","created":"2013-07-19 20:28:27.000000000","updated":"2013-07-31 03:48:14.000000000","submitted":"2013-07-31 03:48:14.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":14,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2414789e1ed67145609b3f8b9b7319be0df286b9","_number":37963,"virtual_id_number":37963,"owner":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},{"value":2,"date":"2013-07-31 03:48:14.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":2888,"name":"Mathieu Rohon","email":"mathieu.rohon@gmail.com","username":"mathieu-rohon"},{"value":0,"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"value":0,"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"value":0,"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},"all":[{"value":1,"date":"2013-07-30 20:03:53.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":2888,"name":"Mathieu Rohon","email":"mathieu.rohon@gmail.com","username":"mathieu-rohon"},{"value":1,"date":"2013-07-30 22:13:30.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"value":0,"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":2,"date":"2013-07-30 20:24:07.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2013-07-31 01:14:42.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"value":1,"date":"2013-07-30 18:20:52.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2013-07-30 20:59:13.000000000","_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":2888,"name":"Mathieu Rohon","email":"mathieu.rohon@gmail.com","username":"mathieu-rohon"},{"value":0,"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"value":0,"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":1,"date":"2013-07-31 02:06:36.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"value":0,"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"value":0,"date":"2013-07-31 02:18:50.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":2888,"name":"Mathieu Rohon","email":"mathieu.rohon@gmail.com","username":"mathieu-rohon"},{"value":0,"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"value":0,"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"value":0,"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"value":0,"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":2888,"name":"Mathieu Rohon","email":"mathieu.rohon@gmail.com","username":"mathieu-rohon"},{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-07-25 12:17:01.000000000","updated_by":{"_account_id":2888,"name":"Mathieu Rohon","email":"mathieu.rohon@gmail.com","username":"mathieu-rohon"},"reviewer":{"_account_id":2888,"name":"Mathieu Rohon","email":"mathieu.rohon@gmail.com","username":"mathieu-rohon"},"state":"REVIEWER"},{"updated":"2013-07-30 18:20:52.000000000","updated_by":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"reviewer":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"state":"REVIEWER"},{"updated":"2013-07-30 20:03:53.000000000","updated_by":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"reviewer":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"state":"REVIEWER"},{"updated":"2013-07-30 20:59:13.000000000","updated_by":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"reviewer":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"state":"REVIEWER"},{"updated":"2013-07-30 22:13:30.000000000","updated_by":{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},"reviewer":{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},"state":"REVIEWER"},{"updated":"2013-07-31 01:14:42.000000000","updated_by":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"reviewer":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"state":"REVIEWER"},{"updated":"2013-07-31 02:06:36.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-07-31 02:18:50.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2013-07-31 03:48:14.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"156a721fdb884021fd82e8c21abad1694d3143e6","author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"date":"2013-07-19 20:44:13.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(2 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"229cfc6d26237d8a4efd5fe37bf3746b351dc71d","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-07-19 20:51:01.000000000","message":"Patch Set 1: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"14ec7b115d7700a59ea53699874b786d8ea5a932","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-07-19 21:08:57.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"9a183d9dc6f24433a7d80f72a2df291fdc202361","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-19 21:58:25.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/63/37963/2/check/gate-tempest-devstack-vm-full/33969 : SUCCESS in 46m 24s\n- http://logs.openstack.org/63/37963/2/check/gate-tempest-devstack-vm-cells-full/1629 : FAILURE in 15m 35s (non-voting)\n- http://logs.openstack.org/63/37963/2/check/gate-tempest-devstack-vm-testr-full/481 : SUCCESS in 28m 32s (non-voting)\n- http://logs.openstack.org/63/37963/2/check/gate-tempest-devstack-vm-postgres-full/33659 : SUCCESS in 46m 29s\n- http://logs.openstack.org/63/37963/2/check/gate-tempest-devstack-vm-neutron/5746 : FAILURE in 19m 52s (non-voting)\n- http://logs.openstack.org/63/37963/2/check/gate-grenade-devstack-vm/19691 : SUCCESS in 22m 10s\n","accounts_in_message":[],"_revision_number":2},{"id":"fd5fc55e6df18e1d072155968256ad93c474b498","author":{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},"date":"2013-07-22 16:09:35.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nI have not yet reviewed this in detail, but I\u0027m concerned about the approach being taken of removing the simple user interface variables. I agree the EXTRA_OPTS stuff in needed to allow tweaking and experimentation (especially for features not yet explicitly supported by devstack), but we should try to make common use cases as simple to configure as possible.\n\nAlso, doesn\u0027t devstack need to support enabling vxlan tunnels in the openvswitch agent when used with ml2, and maybe also when used with the openvswitch plugin?","accounts_in_message":[],"_revision_number":2},{"id":"2ff4da4065cf3084723d539964fd441215f255d9","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-07-22 16:47:16.000000000","message":"Patch Set 2: (1 inline comment)\n\nBob, there is apparently a disconnect between the devstack and neutron folks. When I proposed adding direct VXLAN support into devstack, Sean pushed back and said he would no longer accept patches into devstack to proliferate neutron options. Thus, I went with a more generic way. So you can enable VXLAN tunnels using that generic approach. The same will have to apply ML2 unless someone can convince Sean of the merits of going against his previous decision. Please his comments to patchset one here. [1]\n\n[1] https://review.openstack.org/#/c/30315/","accounts_in_message":[],"_revision_number":2},{"id":"726e0fadb390e05652a9405f20225b8971402227","author":{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},"date":"2013-07-22 18:03:43.000000000","message":"Patch Set 2:\n\nDefinitely a disconnect - I had not seen the referenced review. I\u0027m fine with using the EXTRA_OPTS approach for advanced configuration, and agree with Sean that devstack shouldn\u0027t need separate code added for pass-through of each individual config item. \n\nBut I\u0027m still concerned about making sure basic devstack/neutron use cases don\u0027t require in-depth knowledge of lots of neutron configuration details. \n\nThe most basic devstack use case with ml2, as with the openvswitch or linuxbridge plugins, is a single-node deployment with local tenant networks. I believe this should remain the default. I also think its useful to warn the devstack user in this case that they are not getting cross-node connectivity.\n\nWhen someone wants to use vlan tenant networks instead of local tenant networks, I guess they could accomplish this using EXTRA_OPTS variables to configure the ml2 plugin\u0027s tenant_network_types, the vlan driver\u0027s network_vlan_ranges, and the agent\u0027s bridge or interface mappings. They\u0027d need to know all the relevant details for the specific plugin and agent being used. \n\nRight now, they can simply set ENABLE_TENANT_VLANS, TENANT_VLAN_RANGE, PHYSICAL_NETWORK, and OVS_PHYSICAL_BRIDGE, which are all documented in devstack/lib/neutron. The first three of these work exactly the same across the openvswitch, linuxbridge, and ml2 plugins, and only OVS_PHYSICAL_BRIDGE or LB_PHYSICAL_INTERFACE needs to be set based on the agent being used (either can be used with ml2).\n\nAre we really sure we want to remove the explicit support for a basic use case such as vlan tenant networks? If so, do we want to remove this from the openvswitch and linuxbridge plugins\u0027 devstack support as well? I\u0027d hate to see basic use cases be less well supported by devstack for ml2 than for the other plugins. My point here is that these are not just pass-through independent config items - they are individual plugin-independant decisions made by the devstack user that together effect multiple configuration items, which vary somewhat between plugins.\n\nWhatever we decide to do for vlan support in ml2 (and the other plugins), we probably want to do something similar for tunnel networks. Personally, I think setting ENABLE_TENANT_TUNNELS should be sufficient, resulting in the proper ml2 or openvswitch plugin config and the proper openvswitch agent config for GRE tunnels. A simple plugin-independent way to use VXLAN instead of GRE also seems to make sense. Again, I don\u0027t see these as pass-through config items, but as user decisions effecting multiple config items.","accounts_in_message":[],"_revision_number":2},{"id":"59acb76a434513990ade66de87822181106064b7","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-07-22 20:49:05.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"82abc32a6fa0b750fca50689929847e4e1ca9d74","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-22 21:50:20.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/63/37963/3/check/gate-tempest-devstack-vm-full/34512 : SUCCESS in 47m 33s\n- http://logs.openstack.org/63/37963/3/check/gate-tempest-devstack-vm-cells-full/2066 : FAILURE in 17m 07s (non-voting)\n- http://logs.openstack.org/63/37963/3/check/gate-tempest-devstack-vm-testr-full/912 : FAILURE in 32m 34s (non-voting)\n- http://logs.openstack.org/63/37963/3/check/gate-tempest-devstack-vm-postgres-full/34202 : SUCCESS in 1h 00m 57s\n- http://logs.openstack.org/63/37963/3/check/gate-tempest-devstack-vm-neutron/6336 : SUCCESS in 24m 24s (non-voting)\n- http://logs.openstack.org/63/37963/3/check/gate-grenade-devstack-vm/20194 : SUCCESS in 26m 25s\n","accounts_in_message":[],"_revision_number":3},{"id":"b6792f36adbee0bb75322b3fd705f98766758aca","author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"date":"2013-07-23 13:52:30.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"fc9258775d90f30d38f90474084417c984782a84","author":{"_account_id":2888,"name":"Mathieu Rohon","email":"mathieu.rohon@gmail.com","username":"mathieu-rohon"},"date":"2013-07-25 12:17:01.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\n(2 inline comments)\n\ni think it\u0027s a shame that it\u0027s so difficult to activate vxlan option while using devstack.\nin the agent, tunnel_types\u003dvxlan,gre should be set as soon as vxlan and gre are in the ml2 type_drivers.\nMore generally, I think that Q_ML2_TENANT_NETWORK_TYPE could be useful. ENABLE_TENANT_TUNNELS and ENABLE_TENANT_VLANS can be kept for backward compatibility, as a shortcut to activate only one tenant_network_type, but Q_ML2_TENANT_NETWORK_TYPE could be used to activate several network types for tenant, with no need to use provider networks. I think it\u0027s a shame not having this functionality in Devstack (or with a tricky localrc), because it\u0027s the powerful of ML2.","accounts_in_message":[],"_revision_number":3},{"id":"57ced30ae64e16b24053e0d72fa1f89b804747ee","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-07-29 18:17:01.000000000","message":"Patch Set 3: (2 inline comments)\n\nThanks for the review Mathieu. I\u0027m taking into account what you suggest now and hope to post a new patch later today.","accounts_in_message":[],"_revision_number":3},{"id":"55896ccf1f36e2c5d08979e600a98f4cf0cfaec1","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-07-29 20:35:26.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"49509d253a3a887f817ff25f4503d3c8151e4994","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-29 21:28:53.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/63/37963/4/check/gate-tempest-devstack-vm-full/37099 : SUCCESS in 43m 11s\n- http://logs.openstack.org/63/37963/4/check/gate-tempest-devstack-vm-cells-full/3826 : FAILURE in 15m 46s (non-voting)\n- http://logs.openstack.org/63/37963/4/check/gate-tempest-devstack-vm-testr-full/2644 : SUCCESS in 27m 35s (non-voting)\n- http://logs.openstack.org/63/37963/4/check/gate-tempest-devstack-vm-postgres-full/36775 : SUCCESS in 53m 17s\n- http://logs.openstack.org/63/37963/4/check/gate-tempest-devstack-vm-neutron/9168 : FAILURE in 16m 37s (non-voting)\n- http://logs.openstack.org/63/37963/4/check/gate-grenade-devstack-vm/22503 : SUCCESS in 23m 40s\n","accounts_in_message":[],"_revision_number":4},{"id":"2c0026d87c1d9253e5df54dbc48e57b4b31e3686","author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"date":"2013-07-29 23:52:08.000000000","message":"Patch Set 4: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"16a53b30553da35e0c321062453f85667acf808c","author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"date":"2013-07-30 01:50:26.000000000","message":"Patch Set 4: (1 inline comment)\n\nFound a spelling error. The ml2 script LGTM.","accounts_in_message":[],"_revision_number":4},{"id":"ddd1809ef3b9b44d8ed89f0d11e4c0a9dc1fed29","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-07-30 14:00:21.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"ed6575fb30a84de8876a3ac863491542d4d561f8","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-07-30 14:02:57.000000000","message":"Patch Set 4: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"f7b65902a75a6a87f9ea45c2fdab9a6276ceadfc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-30 14:55:16.000000000","message":"Patch Set 5: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/63/37963/5/check/gate-tempest-devstack-vm-full/37358 : SUCCESS in 54m 49s\n- http://logs.openstack.org/63/37963/5/check/gate-tempest-devstack-vm-cells-full/4031 : FAILURE in 26m 11s (non-voting)\n- http://logs.openstack.org/63/37963/5/check/gate-tempest-devstack-vm-testr-full/2847 : FAILURE in 32m 12s (non-voting)\n- http://logs.openstack.org/63/37963/5/check/gate-tempest-devstack-vm-postgres-full/37034 : SUCCESS in 44m 35s\n- http://logs.openstack.org/63/37963/5/check/gate-tempest-devstack-vm-neutron/9464 : FAILURE in 18m 19s (non-voting)\n- http://logs.openstack.org/63/37963/5/check/gate-grenade-devstack-vm/22731 : SUCCESS in 22m 33s\n","accounts_in_message":[],"_revision_number":5},{"id":"332ca2f27067653f119e73e6c7d4f81638d60106","author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"date":"2013-07-30 18:20:52.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":5},{"id":"8a48da0e1d31ed777492c5452e1751901e0f257e","author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"date":"2013-07-30 20:03:53.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\nthanks for taking care of my comments","accounts_in_message":[],"_revision_number":5},{"id":"931ea8905af5e55f4d5c0a927aefb308bf3bfbd3","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-30 20:24:07.000000000","message":"Patch Set 5: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":5},{"id":"abf6f0929faa7d2206b4d27cab4841445db0872b","author":{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},"date":"2013-07-30 22:13:30.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":5},{"id":"f9e01301c9b79888a99e1b2a30a71ff7d914cf3e","author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"date":"2013-07-31 01:14:42.000000000","message":"Patch Set 5: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":5},{"id":"765eb26677ceb23d422abba23d6cfad8fb0c952a","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-07-31 02:06:36.000000000","message":"Patch Set 5: Approved\n\n","accounts_in_message":[],"_revision_number":5},{"id":"8008d01af8e689fd93b78e7878528d785e57a0ad","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-31 02:06:38.000000000","message":"Patch Set 5: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":5},{"id":"909eaa65fcb31200915d7cdbb18892d60799095c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-31 03:48:14.000000000","message":"Patch Set 5: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/63/37963/5/gate/gate-tempest-devstack-vm-full/37604 : SUCCESS in 55m 18s\n- http://logs.openstack.org/63/37963/5/gate/gate-tempest-devstack-vm-postgres-full/37280 : SUCCESS in 44m 48s\n- http://logs.openstack.org/63/37963/5/gate/gate-tempest-devstack-vm-neutron/9744 : FAILURE in 19m 44s (non-voting)\n- http://logs.openstack.org/63/37963/5/gate/gate-grenade-devstack-vm/22945 : SUCCESS in 23m 16s\n","accounts_in_message":[],"_revision_number":5},{"id":"2f6da0ea429f14a4c76f937de206b54c3dfd7b12","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-07-31 03:48:14.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"b772659f9df8153caa3b0fa155614d2d6504541e","revisions":{"4b74d26423c5dcbda2f6983a9d9997b4a091b0d1":{"kind":"REWORK","_number":1,"created":"2013-07-19 20:28:27.000000000","uploader":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"ref":"refs/changes/63/37963/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/63/37963/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/63/37963/1"}}},"commit":{"parents":[{"commit":"80baf9a1993d3149fc259e95b6ee1cbb9adb2cc3","subject":"Merge \"Configure tempauth along keystoneauth.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/80baf9a1993d3149fc259e95b6ee1cbb9adb2cc3"}]}],"author":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-19 14:26:53.000000000","tz":0},"committer":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-19 14:28:21.000000000","tz":0},"subject":"Update the ML2 configuration in devstack","message":"Update the ML2 configuration in devstack\n\nThis commit adds support to devstack to set ML2 options including MechanismDrivers,\nTypeDrivers, and TypeDriver specific configuration for ML2 when running with\ndevstack.\n\nFixes bug 1200767\n\nChange-Id: Ib16efac13440b5d50658a0e6be35bc735510a262\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4b74d26423c5dcbda2f6983a9d9997b4a091b0d1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/4b74d26423c5dcbda2f6983a9d9997b4a091b0d1"}]},"branch":"refs/heads/master"},"5f67971d53d78600a89063afb834e9c647cdf1ce":{"kind":"REWORK","_number":2,"created":"2013-07-19 21:08:57.000000000","uploader":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"ref":"refs/changes/63/37963/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/63/37963/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/63/37963/2"}}},"commit":{"parents":[{"commit":"80baf9a1993d3149fc259e95b6ee1cbb9adb2cc3","subject":"Merge \"Configure tempauth along keystoneauth.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/80baf9a1993d3149fc259e95b6ee1cbb9adb2cc3"}]}],"author":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-19 14:26:53.000000000","tz":0},"committer":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-19 15:08:40.000000000","tz":0},"subject":"Update the ML2 configuration in devstack","message":"Update the ML2 configuration in devstack\n\nThis commit adds support to devstack to set ML2 options including MechanismDrivers,\nTypeDrivers, and TypeDriver specific configuration for ML2 when running with\ndevstack.\n\nFixes bug 1200767\n\nChange-Id: Ib16efac13440b5d50658a0e6be35bc735510a262\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5f67971d53d78600a89063afb834e9c647cdf1ce"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5f67971d53d78600a89063afb834e9c647cdf1ce"}]},"branch":"refs/heads/master"},"87a45878232d981e2b402cabb1cd47563cfb830b":{"kind":"REWORK","_number":3,"created":"2013-07-22 20:49:05.000000000","uploader":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"ref":"refs/changes/63/37963/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/63/37963/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/63/37963/3"}}},"commit":{"parents":[{"commit":"80baf9a1993d3149fc259e95b6ee1cbb9adb2cc3","subject":"Merge \"Configure tempauth along keystoneauth.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/80baf9a1993d3149fc259e95b6ee1cbb9adb2cc3"}]}],"author":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-19 14:26:53.000000000","tz":0},"committer":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-22 14:41:50.000000000","tz":0},"subject":"Update the ML2 configuration in devstack","message":"Update the ML2 configuration in devstack\n\nThis commit adds support to devstack to set ML2 options including MechanismDrivers,\nTypeDrivers, and TypeDriver specific configuration for ML2 when running with\ndevstack.\n\nFixes bug 1200767\n\nChange-Id: Ib16efac13440b5d50658a0e6be35bc735510a262\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/87a45878232d981e2b402cabb1cd47563cfb830b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/87a45878232d981e2b402cabb1cd47563cfb830b"}]},"branch":"refs/heads/master"},"b5eb2524f057648bb799685eaff2304648e1f2c7":{"kind":"REWORK","_number":4,"created":"2013-07-29 20:35:26.000000000","uploader":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"ref":"refs/changes/63/37963/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/63/37963/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/63/37963/4"}}},"commit":{"parents":[{"commit":"3f95a3fec8eaed32c88b61e757e2094709ec1b44","subject":"Merge \"xenapi: Setup and Rotate text console logs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3f95a3fec8eaed32c88b61e757e2094709ec1b44"}]}],"author":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-19 14:26:53.000000000","tz":0},"committer":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-29 07:56:37.000000000","tz":0},"subject":"Update the ML2 configuration in devstack","message":"Update the ML2 configuration in devstack\n\nThis commit adds support to devstack to set ML2 options including\nMechanismDrivers, TypeDrivers, and TypeDriver specific configuration for ML2\nwhen running with devstack.\n\nThis also allows for simple configuration of VLAN, GRE, and VXLAN ML2 devstack\nsetups with the OVS agent as follows:\n\n    # VLAN configuration\n    Q_PLUGIN\u003dml2\n    ENABLE_TENANT_VLANS\u003dTrue\n\n    # GRE tunnel configuration\n    Q_PLUGIN\u003dml2\n    ENABLE_TENANT_TUNNELS\u003dTrue\n\n    # VXLAN tunnel configuration\n    Q_PLUGIN\u003dml2\n    Q_ML2_TENANT_NETWORK_TYPE\u003dvxlan\n\nFixes bug 1200767\n\nChange-Id: Ib16efac13440b5d50658a0e6be35bc735510a262\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b5eb2524f057648bb799685eaff2304648e1f2c7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b5eb2524f057648bb799685eaff2304648e1f2c7"}]},"branch":"refs/heads/master"},"b772659f9df8153caa3b0fa155614d2d6504541e":{"kind":"REWORK","_number":5,"created":"2013-07-30 14:00:21.000000000","uploader":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"ref":"refs/changes/63/37963/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/63/37963/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/63/37963/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/63/37963/5"}}},"commit":{"parents":[{"commit":"3f95a3fec8eaed32c88b61e757e2094709ec1b44","subject":"Merge \"xenapi: Setup and Rotate text console logs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/3f95a3fec8eaed32c88b61e757e2094709ec1b44"}]}],"author":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-19 14:26:53.000000000","tz":0},"committer":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-07-30 01:24:42.000000000","tz":0},"subject":"Update the ML2 configuration in devstack","message":"Update the ML2 configuration in devstack\n\nThis commit adds support to devstack to set ML2 options including\nMechanismDrivers, TypeDrivers, and TypeDriver specific configuration for ML2\nwhen running with devstack.\n\nThis also allows for simple configuration of VLAN, GRE, and VXLAN ML2 devstack\nsetups with the OVS agent as follows:\n\n    # VLAN configuration\n    Q_PLUGIN\u003dml2\n    ENABLE_TENANT_VLANS\u003dTrue\n\n    # GRE tunnel configuration\n    Q_PLUGIN\u003dml2\n    ENABLE_TENANT_TUNNELS\u003dTrue\n\n    # VXLAN tunnel configuration\n    Q_PLUGIN\u003dml2\n    Q_ML2_TENANT_NETWORK_TYPE\u003dvxlan\n\nFixes bug 1200767\n\nChange-Id: Ib16efac13440b5d50658a0e6be35bc735510a262\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b772659f9df8153caa3b0fa155614d2d6504541e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/b772659f9df8153caa3b0fa155614d2d6504541e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}