)]}'
{"id":"openstack%2Fdevstack~40882","triplet_id":"openstack%2Fdevstack~master~Iedff731fbbe319df1478db6f40a29d387747e554","project":"openstack/devstack","branch":"master","topic":"bp/ml2-devstack-mechanism-driver","hashtags":[],"change_id":"Iedff731fbbe319df1478db6f40a29d387747e554","subject":"Support per-MechanismDriver config with ML2","status":"ABANDONED","created":"2013-08-08 15:02:27.000000000","updated":"2013-10-03 06:03:06.000000000","total_comment_count":9,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f5344720f0c3047c4ab14627bbc150e5c6952c29","_number":40882,"virtual_id_number":40882,"owner":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2013-09-24 16:43:15.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},{"value":0,"date":"2013-09-25 20:12:06.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"value":-1,"date":"2013-09-25 19:54:27.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"}],"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":{"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"date":"2013-09-25 19:54:27.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},{"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"},{"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-08-08 15:02:50.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-08-08 16:45:55.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-08-20 20:05:58.000000000","updated_by":{"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"},"reviewer":{"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"},"state":"REVIEWER"},{"updated":"2013-09-09 10:05:20.000000000","updated_by":{"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},"reviewer":{"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},"state":"REVIEWER"},{"updated":"2013-09-24 16:43:15.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2013-09-25 19:54:27.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"}],"messages":[{"id":"d3c7ff158611a18c9a98b0cb4ff090fdda0d5804","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-08-08 16:06:59.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/82/40882/1/check/gate-tempest-devstack-vm-full/508d4b8 : SUCCESS in 43m 37s\n- http://logs.openstack.org/82/40882/1/check/gate-tempest-devstack-vm-cells-full/25d5f05 : FAILURE in 12m 50s (non-voting)\n- http://logs.openstack.org/82/40882/1/check/gate-tempest-devstack-vm-testr-full/ec15dee : SUCCESS in 29m 10s (non-voting)\n- http://logs.openstack.org/82/40882/1/check/gate-tempest-devstack-vm-postgres-full/f4ac034 : SUCCESS in 44m 03s\n- http://logs.openstack.org/82/40882/1/check/gate-tempest-devstack-vm-neutron/88e84ed : SUCCESS in 19m 09s\n- http://logs.openstack.org/82/40882/1/check/gate-grenade-devstack-vm/2242431 : SUCCESS in 27m 55s\n","accounts_in_message":[],"_revision_number":1},{"id":"efeb3106ac4fed7f08f223b9bee0e12b95e63ca1","author":{"_account_id":6558,"name":"Sukhdev Kapur","email":"sukhdevkapur@gmail.com","username":"sukhdev-8"},"date":"2013-08-20 20:05:58.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\nI tested this for Arista Mechanism Driver. Seems to do the right thing.","accounts_in_message":[],"_revision_number":1},{"id":"c72127d338042a66067d1f99229c92ff0f50882d","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-08-20 20:20:50.000000000","message":"Patch Set 1: Work In Progress\n\nNeed to address the use case where there are multiple configuration sections in a MechanismDriver .ini file, which isn\u0027t covered by the current patch.","accounts_in_message":[],"_revision_number":1},{"id":"d5777059bd033ee96c6431b330602e62b5cb400d","author":{"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},"date":"2013-09-09 10:05:20.000000000","message":"Patch Set 1: (5 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"06de6f695b5ae2d98d317525559e05ea4a4d09c8","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-09-09 13:44:58.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"88c4bcc037f994f3608d949ba55bba4e60d30c76","author":{"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},"date":"2013-09-09 14:29:19.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"3a9e1fcb4748f62c473caaadb6615192b2085eec","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-09-09 14:37:38.000000000","message":"Patch Set 1:\n\nFrancois, I filed this bug [1] to track addressing the issues you found unrelated to this patch. I also had stumbled onto those, so it will be good to clean them up.\n\n[1] https://bugs.launchpad.net/devstack/+bug/1222854","accounts_in_message":[],"_revision_number":1},{"id":"2d36754afe227313a52cf5ec9a618a1c7680d0db","author":{"_account_id":6820,"name":"Francois Eleouet","email":"f.eleouet@gmail.com","username":"fanchon"},"date":"2013-09-10 07:26:49.000000000","message":"Patch Set 1: (1 inline comment)\n\nWorks fine for me, very convenient to test newly introduced options. Just an inline suggestion on usage that could be made simpler IMHO.","accounts_in_message":[],"_revision_number":1},{"id":"5eec345fcc304b30f0ab74536c6f976b7662ea78","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-23 12:48:11.000000000","message":"Patch Set 1:\n\nrecheck no bug","accounts_in_message":[],"_revision_number":1},{"id":"fc8ad762107d370583ef2f7b9f137420d6b47665","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-09-23 12:48:12.000000000","message":"Patch Set 1:\n\nrechecking because review is older than 5 days, and is at risk of merge conflict.","accounts_in_message":[],"_revision_number":1},{"id":"c7f46e88c9fb65d07a14e8d5b2f02b884073b671","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-23 12:48:27.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\nThis change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"32d1d69bae8d3d80c864c461c1942d1ba705b5bf","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-09-24 15:48:22.000000000","message":"Patch Set 1: (1 inline comment)\n\nNew patch inbound with some small nits and rebased.","accounts_in_message":[],"_revision_number":1},{"id":"64d9fa0586183155f156740407f4a9e20bf5cdd6","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-09-24 15:48:34.000000000","message":"Patch Set 1: Ready For Review\n\nNew patch inbound in a few minutes.","accounts_in_message":[],"_revision_number":1},{"id":"8e59ba8488711ee995b4ea3090603947431a5ef5","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-09-24 16:14:38.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"2abc7b3838472e318a496bac53a220527cd9af18","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-09-24 16:43:15.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-tempest-devstack-vm-full http://logs.openstack.org/82/40882/2/check/gate-tempest-devstack-vm-full/4d0e62a : SUCCESS in 27m 48s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/82/40882/2/check/gate-tempest-devstack-vm-postgres-full/8deff32 : SUCCESS in 28m 34s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/82/40882/2/check/gate-tempest-devstack-vm-neutron/8c717a0 : SUCCESS in 17m 16s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/82/40882/2/check/gate-tempest-devstack-vm-large-ops/99d15f1 : SUCCESS in 11m 00s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/82/40882/2/check/gate-grenade-devstack-vm/f621f95 : SUCCESS in 24m 36s\n- gate-swift-devstack-vm-functional http://logs.openstack.org/82/40882/2/check/gate-swift-devstack-vm-functional/ebd318e : SUCCESS in 10m 13s\n","accounts_in_message":[],"_revision_number":2},{"id":"63ef8f56b16956e571151c7bc08b9e26733ee9e2","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-09-25 19:54:27.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nI am trying to add a general config file mechanism to eliminate all of the magic variables like you\u0027re doing here with _CONFIG_SECTIONS.  Have a look at https://review.openstack.org/#/c/46768/ and see if that solves this need.","accounts_in_message":[],"_revision_number":2},{"id":"aa60e5b9424d89f1df2156f0ae7fdb8b9da4aeb8","author":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"date":"2013-09-25 20:12:06.000000000","message":"Patch Set 2:\n\nThanks Dean, I was pointed to this by someone else this morning. The code in your patch looks great, I want to verify it, hope to get to that today. Then I\u0027ll mark this one abandoned. Thanks!","accounts_in_message":[],"_revision_number":2},{"id":"95e02cc5d2663d9db7a0ae243e4629092b3761db","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-10-03 06:03:06.000000000","message":"Patch Set 2:\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":2},{"id":"b9f3b698cf0ad489cbcd79b7aebe324e393d795f","author":{"_account_id":9,"name":"LaunchpadSync","username":"launchpadsync"},"date":"2013-10-03 06:03:06.000000000","message":"Patch Set 2: Abandoned\n\ncode review expired after 1 week of no activity after a negative review, it can be restored using the `Restore Change` button under the Patch Set on the web interface","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"6ab6a197fe89d17d901224d5ee2ee325dd2b3d2c","revisions":{"5b4f025850542f2dd235462f0b1313541f4530dc":{"kind":"REWORK","_number":1,"created":"2013-08-08 15:02:27.000000000","uploader":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"ref":"refs/changes/82/40882/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/82/40882/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/40882/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/40882/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/40882/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/82/40882/1"}}},"commit":{"parents":[{"commit":"d11ce17aa18fb766157ff34b3a938715a26f2192","subject":"Merge \"Correctly setup ML2 mechanism_drivers\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d11ce17aa18fb766157ff34b3a938715a26f2192"}]}],"author":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-08-08 08:55:57.000000000","tz":0},"committer":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-08-08 09:01:59.000000000","tz":0},"subject":"Support per-MechanismDriver config with ML2","message":"Support per-MechanismDriver config with ML2\n\nThis commit adds support to devstack to set per-MechanismDriver\nconfiguration when running with the ML2 Neutron plugin. Allows\nfor arbitrary MechanismDriver configuration with no additional\nchanges to devstack as MechanismDrivers are added to ML2.\n\nImplements blueprint ml2-devstack-mechanism-driver\n\nChange-Id: Iedff731fbbe319df1478db6f40a29d387747e554\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5b4f025850542f2dd235462f0b1313541f4530dc"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/5b4f025850542f2dd235462f0b1313541f4530dc"}]},"branch":"refs/heads/master"},"6ab6a197fe89d17d901224d5ee2ee325dd2b3d2c":{"kind":"REWORK","_number":2,"created":"2013-09-24 16:14:38.000000000","uploader":{"_account_id":105,"name":"Kyle Mestery","email":"mestery@mestery.com","username":"mestery"},"ref":"refs/changes/82/40882/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/82/40882/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/40882/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/40882/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/82/40882/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/82/40882/2"}}},"commit":{"parents":[{"commit":"d24674ed22687159ced65613c714a075d9804279","subject":"Merge \"Modified ceilometer alarm evaluator console script\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d24674ed22687159ced65613c714a075d9804279"}]}],"author":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-08-08 08:55:57.000000000","tz":0},"committer":{"name":"Kyle Mestery","email":"kmestery@cisco.com","date":"2013-09-19 23:30:30.000000000","tz":0},"subject":"Support per-MechanismDriver config with ML2","message":"Support per-MechanismDriver config with ML2\n\nThis commit adds support to devstack to set per-MechanismDriver\nconfiguration when running with the ML2 Neutron plugin. Allows\nfor arbitrary MechanismDriver configuration with no additional\nchanges to devstack as MechanismDrivers are added to ML2.\n\nImplements blueprint ml2-devstack-mechanism-driver\n\nChange-Id: Iedff731fbbe319df1478db6f40a29d387747e554\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6ab6a197fe89d17d901224d5ee2ee325dd2b3d2c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/6ab6a197fe89d17d901224d5ee2ee325dd2b3d2c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}