)]}'
{"id":"openstack%2Fdevstack~72412","triplet_id":"openstack%2Fdevstack~master~I28794c3acacc6daa9f698f8031b58d1ee13c3bad","project":"openstack/devstack","branch":"master","topic":"bp/add-basic-marconi-tests","hashtags":[],"change_id":"I28794c3acacc6daa9f698f8031b58d1ee13c3bad","subject":"Add marconi to enabled services","status":"MERGED","created":"2014-02-10 16:26:27.000000000","updated":"2014-02-12 20:45:30.000000000","submitted":"2014-02-12 20:45:29.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c8fcd123482d9acef30810b620b64af466826064","_number":72412,"virtual_id_number":72412,"owner":{"_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":2,"date":"2014-02-12 20:45:29.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"}],"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":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"recommended":{"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},"all":[{"value":1,"date":"2014-02-10 20:25:43.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},{"value":2,"date":"2014-02-11 08:57:49.000000000","_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"},{"value":2,"date":"2014-02-10 20:21:15.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":1,"date":"2014-02-12 14:45:27.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"}],"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":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},{"value":0,"date":"2014-02-11 08:57:49.000000000","_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2014-02-10 18:03:29.000000000","_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"},{"value":1,"date":"2014-02-12 16:28:29.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},{"value":0,"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"}],"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":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},{"_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-02-10 20:25:43.000000000","updated_by":{"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},"reviewer":{"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},"state":"REVIEWER"},{"updated":"2014-02-11 08:57:49.000000000","updated_by":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"reviewer":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"state":"REVIEWER"},{"updated":"2014-02-12 14:45:27.000000000","updated_by":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"reviewer":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"state":"REVIEWER"},{"updated":"2014-02-12 16:28:29.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":"2014-02-12 20:45:29.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"7086d2f094b0ff04a01278b8a64db03eccb2d2e8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-10 16:26:27.000000000","message":"Patch Set 1:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"1ccdf524e029303e810f9ebbb1798996db99cd79","author":{"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},"date":"2014-02-10 16:29:15.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"04eb8b41589e242033c312076ef6a265e129cc1a","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-10 16:36:36.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nThis is the opposite of what we want to do...\n\nhttps://review.openstack.org/#/c/69497/ begins the process of eliminating these special-cases in is_service_enabled().  Once that merges you can add an is_marconi_enabled() function to lib/marconi to achieve the same result.\n\nActually, you could add the function now and when 69497 merges you\u0027ll be all set...","accounts_in_message":[],"_revision_number":1},{"id":"3af372efbba905f5b8bd9102581b55888cd5f64e","author":{"_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"},"date":"2014-02-10 18:03:29.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"cd952097733a54ca226ed0ff648d5e1a1d56adbc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-10 18:03:30.000000000","message":"Patch Set 2:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"b6f88a3091b3ebb981c1a4628e4c7735f0456776","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-10 19:48:15.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/12/72412/2/check/gate-devstack-bash8/c0fef90 : SUCCESS in 57s\n- check-devstack-dsvm-cells http://logs.openstack.org/12/72412/2/check/check-devstack-dsvm-cells/31f9539 : SUCCESS in 14m 28s\n- check-tempest-dsvm-full http://logs.openstack.org/12/72412/2/check/check-tempest-dsvm-full/cb13f8c : SUCCESS in 55m 11s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/12/72412/2/check/check-tempest-dsvm-postgres-full/0329dc4 : SUCCESS in 53m 48s\n- check-tempest-dsvm-neutron http://logs.openstack.org/12/72412/2/check/check-tempest-dsvm-neutron/9660a45 : SUCCESS in 34m 57s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/12/72412/2/check/gate-tempest-dsvm-large-ops/f227a60 : SUCCESS in 13m 37s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/12/72412/2/check/gate-tempest-dsvm-neutron-large-ops/648730e : SUCCESS in 14m 03s\n- check-grenade-dsvm http://logs.openstack.org/12/72412/2/check/check-grenade-dsvm/a56af24 : SUCCESS in 33m 52s\n- check-swift-dsvm-functional http://logs.openstack.org/12/72412/2/check/check-swift-dsvm-functional/4e325bb : SUCCESS in 15m 39s\n- check-devstack-dsvm-neutron http://logs.openstack.org/12/72412/2/check/check-devstack-dsvm-neutron/6d780e6 : FAILURE in 17m 43s (non-voting)\n- check-tempest-dsvm-ironic-nv http://logs.openstack.org/12/72412/2/check/check-tempest-dsvm-ironic-nv/8802ab6 : SUCCESS in 31m 02s (non-voting)\n- check-tempest-dsvm-ironic-postgres-nv http://logs.openstack.org/12/72412/2/check/check-tempest-dsvm-ironic-postgres-nv/b63ba7d : SUCCESS in 31m 21s (non-voting)\n- check-tempest-dsvm-savanna-nv http://logs.openstack.org/12/72412/2/check/check-tempest-dsvm-savanna-nv/36e6e1b : SUCCESS in 29m 37s (non-voting)\n- check-tempest-dsvm-savanna-postgres-nv http://logs.openstack.org/12/72412/2/check/check-tempest-dsvm-savanna-postgres-nv/56f4642 : SUCCESS in 27m 32s (non-voting)\n- check-tempest-dsvm-savanna-neutron-nv http://logs.openstack.org/12/72412/2/check/check-tempest-dsvm-savanna-neutron-nv/4b9fd07 : SUCCESS in 41m 01s (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"8ce3fb025aced681fce693417c1ebf8b441ae395","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-10 20:21:15.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"9cf8657c8ec896549a6c1496ca69ce51421c523e","author":{"_account_id":6944,"name":"Allele Dev","email":"cpp.cabrera@gmail.com","username":"cpp-cabrera"},"date":"2014-02-10 20:25:43.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"e584e0189815a3fec674485df0406f4b0ad35ad3","author":{"_account_id":866,"name":"Chmouel Boudjnah","email":"chmouel@chmouel.com","username":"chmouel"},"date":"2014-02-11 08:57:49.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"70a850c707bb4546b383b32900d026fc2192240d","author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"date":"2014-02-12 14:45:27.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"bd780beea8dd4fd34746a99424d637abda9b4c54","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2014-02-12 16:28:29.000000000","message":"Patch Set 2: Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"8c1643a5d421b1ba6546ea1ca09db667c6e1352c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-12 16:34:21.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"eb104d31cc333dd93844a126e667d654247fa25b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-12 20:45:29.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- gate-devstack-bash8 http://logs.openstack.org/12/72412/2/gate/gate-devstack-bash8/212699a : SUCCESS in 25s\n- gate-devstack-dsvm-cells http://logs.openstack.org/12/72412/2/gate/gate-devstack-dsvm-cells/a258c75 : SUCCESS in 14m 30s\n- gate-tempest-dsvm-full http://logs.openstack.org/12/72412/2/gate/gate-tempest-dsvm-full/8d448c3 : SUCCESS in 46m 29s\n- gate-tempest-dsvm-postgres-full http://logs.openstack.org/12/72412/2/gate/gate-tempest-dsvm-postgres-full/f0a5c57 : SUCCESS in 47m 08s\n- gate-tempest-dsvm-neutron http://logs.openstack.org/12/72412/2/gate/gate-tempest-dsvm-neutron/a9f7a9a : SUCCESS in 34m 04s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/12/72412/2/gate/gate-tempest-dsvm-large-ops/d52d36f : SUCCESS in 17m 17s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/12/72412/2/gate/gate-tempest-dsvm-neutron-large-ops/4158341 : SUCCESS in 18m 05s\n- gate-grenade-dsvm http://logs.openstack.org/12/72412/2/gate/gate-grenade-dsvm/6dc09c4 : SUCCESS in 40m 53s\n- gate-swift-dsvm-functional http://logs.openstack.org/12/72412/2/gate/gate-swift-dsvm-functional/218d137 : SUCCESS in 16m 41s\n","accounts_in_message":[],"_revision_number":2},{"id":"aed0956952a7616b3f2c327120e08b2c5b31c2ac","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-02-12 20:45:30.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"9972ec23c43cea1be6ee5174e72c06e32f295212","revisions":{"f35b78f32663f31b0bbc1fa991891281847d8613":{"kind":"REWORK","_number":1,"created":"2014-02-10 16:26:27.000000000","uploader":{"_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"},"ref":"refs/changes/12/72412/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/12/72412/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/72412/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/72412/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/72412/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/12/72412/1"}}},"commit":{"parents":[{"commit":"fb87714c211d8a8c43605af0f1dd9050934e8e3f","subject":"Merge \"Fix misspellings in devstack\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fb87714c211d8a8c43605af0f1dd9050934e8e3f"}]}],"author":{"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","date":"2014-02-10 16:22:39.000000000","tz":-300},"committer":{"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","date":"2014-02-10 16:28:15.000000000","tz":-300},"subject":"Add marconi to enabled services","message":"Add marconi to enabled services\n\nThis patch adds marconi to enabled services. This is needed to run\nthe tempest experimental job for marconi.\n\nChange-Id: I28794c3acacc6daa9f698f8031b58d1ee13c3bad\nImplements: blueprint add-basic-marconi-tests\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f35b78f32663f31b0bbc1fa991891281847d8613"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/f35b78f32663f31b0bbc1fa991891281847d8613"}]},"branch":"refs/heads/master"},"9972ec23c43cea1be6ee5174e72c06e32f295212":{"kind":"REWORK","_number":2,"created":"2014-02-10 18:03:29.000000000","uploader":{"_account_id":7498,"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","username":"malini-pk"},"ref":"refs/changes/12/72412/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/12/72412/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/72412/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/72412/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/12/72412/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/12/72412/2"}}},"commit":{"parents":[{"commit":"fb87714c211d8a8c43605af0f1dd9050934e8e3f","subject":"Merge \"Fix misspellings in devstack\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/fb87714c211d8a8c43605af0f1dd9050934e8e3f"}]}],"author":{"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","date":"2014-02-10 16:22:39.000000000","tz":-300},"committer":{"name":"Malini Kamalambal","email":"malini.kamalambal@rackspace.com","date":"2014-02-10 18:05:04.000000000","tz":-300},"subject":"Add marconi to enabled services","message":"Add marconi to enabled services\n\nThis patch adds marconi to enabled services. This is needed to run\nthe tempest experimental job for marconi.\n\nChange-Id: I28794c3acacc6daa9f698f8031b58d1ee13c3bad\nImplements: blueprint add-basic-marconi-tests\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9972ec23c43cea1be6ee5174e72c06e32f295212"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/9972ec23c43cea1be6ee5174e72c06e32f295212"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}