)]}'
{"id":"openstack%2Fnova~50668","triplet_id":"openstack%2Fnova~master~I5ec44f84267a700da16b350e686eee886e0d1183","project":"openstack/nova","branch":"master","topic":"remove-compute-api-class-opt","hashtags":[],"change_id":"I5ec44f84267a700da16b350e686eee886e0d1183","subject":"Remove compute_api_class config option","status":"MERGED","created":"2013-10-09 15:11:05.000000000","updated":"2013-10-16 20:57:09.000000000","submitted":"2013-10-16 20:57:06.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":5,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"282451291810df7ebfa59a26fe1680a5ed3aff07","_number":50668,"virtual_id_number":50668,"owner":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2013-10-16 20:57:06.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":1,"date":"2013-10-09 15:38:08.000000000","_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"}],"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":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"disliked":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2013-10-13 07:37:19.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":1,"date":"2013-10-13 01:13:52.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":2,"date":"2013-10-16 19:07:42.000000000","_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":2,"date":"2013-10-13 00:29:04.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"}],"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":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"all":[{"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":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":1,"date":"2013-10-16 19:08:22.000000000","_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"date":"2013-10-13 00:29:04.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"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":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},{"value":0,"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-09 15:38:08.000000000","updated_by":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"reviewer":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2013-10-13 00:29:04.000000000","updated_by":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"reviewer":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"state":"REVIEWER"},{"updated":"2013-10-13 01:13:52.000000000","updated_by":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"reviewer":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"state":"REVIEWER"},{"updated":"2013-10-13 07:37:19.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-10-16 19:08:22.000000000","updated_by":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"reviewer":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"state":"REVIEWER"},{"updated":"2013-10-16 20:57:06.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"43902e30835aa983775db923a9e8f6a6740a8512","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-10-09 15:38:08.000000000","message":"Patch Set 1: Works for me\n\nSmokeStack Results (patch set 1):\n- Libvirt (Fedora 19) https://smokestack.openstack.org/?go\u003d/jobs/142502 : SUCCESS \n- Libvirt (Centos 6.4) https://smokestack.openstack.org/?go\u003d/jobs/142504 : SUCCESS \n- XenServer (Fedora 18) https://smokestack.openstack.org/?go\u003d/jobs/142503 : SUCCESS \n","accounts_in_message":[],"_revision_number":1},{"id":"e03ecb4cb1479bf4667350fb0ea9a9e91b12a5a5","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-09 15:53:14.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- gate-nova-pep8 http://logs.openstack.org/68/50668/1/check/gate-nova-pep8/77c6e7e : SUCCESS in 4m 14s\n- gate-nova-docs http://docs-draft.openstack.org/68/50668/1/check/gate-nova-docs/7ce4df1/doc/build/html/ : SUCCESS in 3m 38s\n- gate-nova-python26 http://logs.openstack.org/68/50668/1/check/gate-nova-python26/26abf09 : SUCCESS in 19m 03s\n- gate-nova-python27 http://logs.openstack.org/68/50668/1/check/gate-nova-python27/7c1e1b2 : SUCCESS in 15m 59s\n- check-tempest-devstack-vm-full http://logs.openstack.org/68/50668/1/check/check-tempest-devstack-vm-full/0b2aeca : SUCCESS in 41m 44s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/68/50668/1/check/check-tempest-devstack-vm-postgres-full/c8aa5e4 : SUCCESS in 39m 12s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/68/50668/1/check/check-tempest-devstack-vm-neutron/5f3c215 : SUCCESS in 24m 07s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/68/50668/1/check/gate-tempest-devstack-vm-large-ops/56ad997 : SUCCESS in 12m 04s\n- check-grenade-devstack-vm http://logs.openstack.org/68/50668/1/check/check-grenade-devstack-vm/cf059e6 : SUCCESS in 35m 04s\n- gate-nova-pylint http://logs.openstack.org/68/50668/1/check/gate-nova-pylint/4b59237 : SUCCESS in 13m 09s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"f8a2abfc1283c4bf4993974d56cd874ef5c7a055","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-10-13 00:29:04.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"3b1b8825f37339c271eaf1a0cdc429291149d0db","author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"date":"2013-10-13 01:13:52.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"c8e8a999e8452344914f2e660d9d286ad0c338cb","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2013-10-13 07:37:19.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nHi,\nCan you please clarify why you removed the validation check for the cell type?\nThanks\nGary","accounts_in_message":[],"_revision_number":1},{"id":"0b79a306d092c89c767921e501c5d6ab11f60a06","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2013-10-13 07:39:15.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"dc7f65e2e4c255508b15d8e65122f21ef8d57899","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-10-14 15:07:00.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"0e362b7879b0b1c63bf1b40872845f515d2034da","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-10-14 15:09:39.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"6a72ae2529477da71ceaa4b5e428e4404eb0579e","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-10-16 17:12:31.000000000","message":"Patch Set 1:\n\ncheck experimental","accounts_in_message":[],"_revision_number":1},{"id":"d2d52588cf6057478b68937bb24e3b6afd783b20","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-10-16 17:13:58.000000000","message":"Patch Set 1: (1 inline comment)\n\nlgtm, but waiting for cells devstack gate.","accounts_in_message":[],"_revision_number":1},{"id":"c15432fe3b92c8b8a79ae50916dadb3f0691466e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-16 18:02:01.000000000","message":"Patch Set 1:\n\nBuild succeeded.\n\n- gate-devstack-vm-cells http://logs.openstack.org/68/50668/1/experimental/gate-devstack-vm-cells/e4a3dc2 : SUCCESS in 9m 55s (non-voting)\n- gate-tempest-devstack-vm-neutron-full http://logs.openstack.org/68/50668/1/experimental/gate-tempest-devstack-vm-neutron-full/26e1851 : FAILURE in 48m 20s (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"d0274918e4752c8a4a8c22b45e6b4a88874ea91a","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-10-16 19:07:42.000000000","message":"Patch Set 1: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"d0fc1a0dc8198512e5d5bfc79bda133dd0f68398","author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"date":"2013-10-16 19:08:22.000000000","message":"Patch Set 1: Approved\n\n","accounts_in_message":[],"_revision_number":1},{"id":"9036f0ad285a73207f8eb2bd4cebeb94f0f0b4c6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-16 19:08:24.000000000","message":"Patch Set 1: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"d4ce2aad762607931dce5905da11dc84614c626c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-16 20:57:06.000000000","message":"Patch Set 1: Verified\n\nBuild succeeded.\n\n- gate-nova-docs http://docs-draft.openstack.org/68/50668/1/gate/gate-nova-docs/b9614e3/doc/build/html/ : SUCCESS in 4m 18s\n- gate-nova-pep8 http://logs.openstack.org/68/50668/1/gate/gate-nova-pep8/3f55cff : SUCCESS in 3m 50s\n- gate-nova-python26 http://logs.openstack.org/68/50668/1/gate/gate-nova-python26/25fa7d2 : SUCCESS in 20m 01s\n- gate-nova-python27 http://logs.openstack.org/68/50668/1/gate/gate-nova-python27/4015ed7 : SUCCESS in 16m 00s\n- gate-tempest-devstack-vm-full http://logs.openstack.org/68/50668/1/gate/gate-tempest-devstack-vm-full/dfebfa9 : SUCCESS in 41m 22s\n- gate-tempest-devstack-vm-postgres-full http://logs.openstack.org/68/50668/1/gate/gate-tempest-devstack-vm-postgres-full/070cafe : SUCCESS in 45m 17s\n- gate-tempest-devstack-vm-neutron http://logs.openstack.org/68/50668/1/gate/gate-tempest-devstack-vm-neutron/9643d17 : SUCCESS in 18m 16s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/68/50668/1/gate/gate-tempest-devstack-vm-large-ops/ba40a38 : SUCCESS in 15m 44s\n- gate-tempest-devstack-vm-neutron-large-ops http://logs.openstack.org/68/50668/1/gate/gate-tempest-devstack-vm-neutron-large-ops/e138f6a : SUCCESS in 15m 40s (non-voting)\n- gate-grenade-devstack-vm http://logs.openstack.org/68/50668/1/gate/gate-grenade-devstack-vm/516c7b5 : SUCCESS in 23m 36s\n","accounts_in_message":[],"_revision_number":1},{"id":"9728628068c0fc06106bf83ec00ffbfbe2efec72","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-16 20:57:09.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"167e1589bbb37b9b54f4d114cbae8b657206b599","revisions":{"167e1589bbb37b9b54f4d114cbae8b657206b599":{"kind":"REWORK","_number":1,"created":"2013-10-09 15:11:05.000000000","uploader":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"ref":"refs/changes/68/50668/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/68/50668/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/68/50668/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/68/50668/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/68/50668/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/68/50668/1"}}},"commit":{"parents":[{"commit":"b0f6fdbc763854ef091136d5f75de123d1a1344f","subject":"Merge \"fix the an Unexpected API Error issue in flavor API\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b0f6fdbc763854ef091136d5f75de123d1a1344f"}]}],"author":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2013-10-09 15:06:46.000000000","tz":-240},"committer":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2013-10-09 15:09:08.000000000","tz":-240},"subject":"Remove compute_api_class config option","message":"Remove compute_api_class config option\n\nThis option was deprecated in Havana and can now be removed.  Its only\nuse was for changing out the API class if cells was in use.\nTechnically, it used to be possible to configure an arbitrary compute\nAPI class.  However, that would have been an unsupported setup, and has\nbeen completely broken since 9f902c7e17249cf5af1504f31d80c0247a02fe20\nanyway.\n\nAs a part of this change, the cell_type configuration option needed a\ndefault value.  I set it to \u0027compute\u0027.  My reasoning was that you would\nhave more compute cells than API cells, so might as well set it to the\nmore common case.\n\nChange-Id: I5ec44f84267a700da16b350e686eee886e0d1183\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/167e1589bbb37b9b54f4d114cbae8b657206b599"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/167e1589bbb37b9b54f4d114cbae8b657206b599"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}