)]}'
{"id":"openstack%2Fnova~27537","triplet_id":"openstack%2Fnova~master~I5e7f944c1c29b2f211ece2ef86c0959c81e806df","project":"openstack/nova","branch":"master","topic":"bug/1172297","hashtags":[],"change_id":"I5e7f944c1c29b2f211ece2ef86c0959c81e806df","subject":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder","status":"MERGED","created":"2013-04-26 09:02:00.000000000","updated":"2013-05-15 19:50:23.000000000","submitted":"2013-05-15 19:50:20.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":7,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"7db18c73a3233123809548ed1e88884c9eb72e4b","_number":27537,"virtual_id_number":27537,"owner":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":2,"date":"2013-05-15 19:50:20.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"all":[{"value":0,"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},{"value":2,"date":"2013-05-14 08:56:18.000000000","_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":2,"date":"2013-05-15 19:04:02.000000000","_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"all":[{"value":0,"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},{"value":0,"date":"2013-05-14 08:56:18.000000000","_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":1,"date":"2013-05-15 19:04:02.000000000","_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},{"value":0,"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-04-29 21:27:57.000000000","updated_by":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"reviewer":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"state":"REVIEWER"},{"updated":"2013-04-30 14:32:15.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-05-08 13:40:42.000000000","updated_by":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"reviewer":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"state":"REVIEWER"},{"updated":"2013-05-14 08:56:18.000000000","updated_by":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"reviewer":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"state":"REVIEWER"},{"updated":"2013-05-15 19:04:02.000000000","updated_by":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"reviewer":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"state":"REVIEWER"},{"updated":"2013-05-15 19:50:20.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"5283cb4a4c8997cd28a068d8798a03ae6ab96768","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-04-26 09:20:58.000000000","message":"Patch Set 1: Works for me\n\nSmokeStack Results (patch set 1):\n\tLibvirt (Fedora 18) Success: http://smokestack.openstack.org/?go\u003d/jobs/84578\n\tXenServer 5.6 (F18 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/84579\n","accounts_in_message":[],"_revision_number":1},{"id":"1eda717deef1fd668fb0803015025d4a3665d383","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-26 09:22:51.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/GerritJenkinsGithub#Test_Failures\n\n- http://logs.openstack.org/27537/1/check/gate-nova-pep8/24940 : SUCCESS\n- http://docs-draft.openstack.org/27537/1/check/gate-nova-docs/19158/doc/build/html/ : SUCCESS\n- http://logs.openstack.org/27537/1/check/gate-nova-python26/20566 : SUCCESS\n- http://logs.openstack.org/27537/1/check/gate-nova-python26-rhel6/1067 : SUCCESS (non-voting)\n- http://logs.openstack.org/27537/1/check/gate-nova-python27/20654 : SUCCESS\n- http://logs.openstack.org/27537/1/check/gate-tempest-devstack-vm-full/13676 : FAILURE\n- http://logs.openstack.org/27537/1/check/gate-tempest-devstack-vm-postgres-full/13391 : FAILURE (non-voting)\n- http://logs.openstack.org/27537/1/check/gate-tempest-devstack-vm-quantum/19644 : FAILURE\n- http://logs.openstack.org/27537/1/check/gate-grenade-devstack-vm/1529 : FAILURE (non-voting)\n- http://logs.openstack.org/27537/1/check/gate-nova-pylint/10771 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"3826cd626edbe193ac1e4ba4b2c20716335de963","author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"date":"2013-04-26 11:51:54.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"73466782d3c42f5bc65251265a7476d76f388583","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-04-26 12:09:45.000000000","message":"Patch Set 2: Works for me\n\nSmokeStack Results (patch set 2):\n\tLibvirt (Fedora 18) Success: http://smokestack.openstack.org/?go\u003d/jobs/84589\n\tXenServer 5.6 (F18 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/84590\n","accounts_in_message":[],"_revision_number":2},{"id":"936f6e03544398cbfb2e3b816b52d07bfb4f5b4d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-26 12:39:24.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/27537/2/check/gate-nova-pep8/24942 : SUCCESS\n- http://docs-draft.openstack.org/27537/2/check/gate-nova-docs/19160/doc/build/html/ : SUCCESS\n- http://logs.openstack.org/27537/2/check/gate-nova-python26/20568 : SUCCESS\n- http://logs.openstack.org/27537/2/check/gate-nova-python26-rhel6/1068 : SUCCESS (non-voting)\n- http://logs.openstack.org/27537/2/check/gate-nova-python27/20656 : SUCCESS\n- http://logs.openstack.org/27537/2/check/gate-tempest-devstack-vm-full/13686 : SUCCESS\n- http://logs.openstack.org/27537/2/check/gate-tempest-devstack-vm-postgres-full/13401 : FAILURE (non-voting)\n- http://logs.openstack.org/27537/2/check/gate-tempest-devstack-vm-quantum/19656 : SUCCESS\n- http://logs.openstack.org/27537/2/check/gate-grenade-devstack-vm/1536 : FAILURE (non-voting)\n- http://logs.openstack.org/27537/2/check/gate-nova-pylint/10772 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"e76c1cc90b81c60312b8fdd0b7fe356d14fe908b","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2013-04-29 19:06:45.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n(2 inline comments)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"f144108bdc3161c9315daef25ded2d117f8c786b","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2013-04-29 21:27:39.000000000","message":"Patch Set 2: Do not merge\n\nFrom a quick look at this, it appears to be a non backwards copmatible API change.  Am I missing something?","accounts_in_message":[],"_revision_number":2},{"id":"dafd7c1d759ed3aed1ec83da4c6be7c78a84da25","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2013-04-29 21:31:28.000000000","message":"Patch Set 2:\n\nWell, take a close look at the changes made to the API outputs; I would say that, at least in the XML case, there is clearly a bug, and this is a reasonable way of resolving that problem.  That said, it probably wouldn\u0027t be that difficult to alter the API changes to continue to include the data, and just add an appropriate fix to the XML template to correct that issue…","accounts_in_message":[],"_revision_number":2},{"id":"42c41c14b36ff710830b935478ef2d25f2c0411c","author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"date":"2013-04-30 11:40:03.000000000","message":"Patch Set 2: (1 inline comment)\n\nI wouldn\u0027t say there is any Nova API change here: the magor change is in nova.volume.cinder.API class which is nothing more than refactoring. As for changed templates - this is what Cinder actually retuns on create snapshot request: http://docs.openstack.org/api/openstack-block-storage/2.0/content/Create_Snapshot.html (see volume_id is a string, not a dict)\n\nSo I think it was a small bug in tests - https://github.com/openstack/nova/blob/master/nova/tests/api/openstack/fakes.py#L604 - here you can see that stub_snapshot_create accepts volume_id while cinder.API.create_snapshot accepted volume object before my change. Thus it creates stub snapshot and assigns volume dict to the \"volume_id\" attribute. I think this bug forced to use \"wrong\" templates.","accounts_in_message":[],"_revision_number":2},{"id":"98066d423311b53b8bf31b1d10693b6850546df4","author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"date":"2013-04-30 14:15:03.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"9ae6e592e7c2a0244a712a94884f0f5abf79e8aa","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-04-30 14:32:15.000000000","message":"Patch Set 3: Works for me\n\nSmokeStack Results (patch set 3):\n\tLibvirt (Fedora 18) Success: http://smokestack.openstack.org/?go\u003d/jobs/85082\n","accounts_in_message":[],"_revision_number":3},{"id":"a1bae3675c28480c8b0b4272a90213dfe1e1cc6e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-04-30 14:59:13.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/27537/3/check/gate-nova-pep8/25081 : SUCCESS\n- http://docs-draft.openstack.org/27537/3/check/gate-nova-docs/19299/doc/build/html/ : SUCCESS\n- http://logs.openstack.org/27537/3/check/gate-nova-python26/20707 : SUCCESS\n- http://logs.openstack.org/27537/3/check/gate-nova-python26-rhel6/1165 : SUCCESS (non-voting)\n- http://logs.openstack.org/27537/3/check/gate-nova-python27/20795 : SUCCESS\n- http://logs.openstack.org/27537/3/check/gate-tempest-devstack-vm-full/14007 : SUCCESS\n- http://logs.openstack.org/27537/3/check/gate-tempest-devstack-vm-postgres-full/13722 : FAILURE (non-voting)\n- http://logs.openstack.org/27537/3/check/gate-tempest-devstack-vm-quantum/20050 : SUCCESS\n- http://logs.openstack.org/27537/3/check/gate-grenade-devstack-vm/1841 : FAILURE (non-voting)\n- http://logs.openstack.org/27537/3/check/gate-nova-pylint/10869 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":3},{"id":"5b70164b0fa8f12e30dea1204961d141795a197f","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2013-04-30 16:16:25.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\nLGTM.  Russell?  You convinced yet that this is fixing a bug in the API rather than changing it?","accounts_in_message":[],"_revision_number":3},{"id":"5788c63a326fc6285053dafe4565124abfc0db05","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2013-05-08 13:40:42.000000000","message":"Patch Set 3: No score\n\n","accounts_in_message":[],"_revision_number":3},{"id":"35c7f2044ca0160fc303a08fbd5b78fda23055e4","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-05-14 08:53:45.000000000","message":"Patch Set 3: (4 inline comments)\n\nOk, I\u0027ve split out the api samples change to make it clear to everyone that this is a bug in the tests. See Id006e3995c7696aa8f061a2b96123ea27e4b6d3f\n\nI\u0027ll rebase this commit onto that","accounts_in_message":[],"_revision_number":3},{"id":"02a11edaf7546fc146e9726e9d6174b3b91f4162","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-05-14 08:55:54.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"6942255fda8b4245ad5c59edf47aa631d5bddec6","author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"date":"2013-05-14 08:56:18.000000000","message":"Patch Set 4: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"1197c18b57c71ed5c95bd4f569d1be12f43b8fc2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-14 09:42:16.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/27537/4/check/gate-nova-pep8/25831 : SUCCESS in 00:03:19\n- http://docs-draft.openstack.org/27537/4/check/gate-nova-docs/20049/doc/build/html/ : SUCCESS in 00:02:40\n- http://logs.openstack.org/27537/4/check/gate-nova-python26/21457 : SUCCESS in 00:12:31\n- http://logs.openstack.org/27537/4/check/gate-nova-python27/21545 : SUCCESS in 00:08:40\n- http://logs.openstack.org/27537/4/check/gate-tempest-devstack-vm-full/15843 : SUCCESS in 00:44:39\n- http://logs.openstack.org/27537/4/check/gate-tempest-devstack-vm-postgres-full/15559 : FAILURE in 00:45:53 (non-voting)\n- http://logs.openstack.org/27537/4/check/gate-tempest-devstack-vm-quantum/22278 : SUCCESS in 00:21:36\n- http://logs.openstack.org/27537/4/check/gate-grenade-devstack-vm/3496 : FAILURE in 00:10:24 (non-voting)\n- http://logs.openstack.org/27537/4/check/gate-nova-pylint/11476 : SUCCESS in 00:08:04 (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"7424b795e15269d3fcef0abd059c766c0da86c1b","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2013-05-15 19:04:02.000000000","message":"Patch Set 4: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":4},{"id":"9816e9cfd9e9f9693eeb318607397aab99a99258","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-15 19:04:10.000000000","message":"Patch Set 4: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"e60400344b8db5dd98e26a54ecb14c64396d9c98","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-15 19:50:20.000000000","message":"Patch Set 4: Verified\n\nBuild succeeded.\n\n- http://docs-draft.openstack.org/27537/4/gate/gate-nova-docs/20159/doc/build/html/ : SUCCESS in 3m 34s\n- http://logs.openstack.org/27537/4/gate/gate-nova-pep8/25941 : SUCCESS in 3m 07s\n- http://logs.openstack.org/27537/4/gate/gate-nova-python26/21565 : SUCCESS in 14m 04s\n- http://logs.openstack.org/27537/4/gate/gate-nova-python27/21655 : SUCCESS in 8m 34s\n- http://logs.openstack.org/27537/4/gate/gate-tempest-devstack-vm-full/16143 : SUCCESS in 41m 37s\n- http://logs.openstack.org/27537/4/gate/gate-tempest-devstack-vm-postgres-full/15859 : FAILURE in 45m 30s (non-voting)\n- http://logs.openstack.org/27537/4/gate/gate-tempest-devstack-vm-quantum/22640 : SUCCESS in 23m 03s\n- http://logs.openstack.org/27537/4/gate/gate-grenade-devstack-vm/3761 : FAILURE in 10m 46s (non-voting)\n","accounts_in_message":[],"_revision_number":4},{"id":"ad6c63d3d9a5b07fe6ac5379a0569d26b79c0688","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-05-15 19:50:23.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"586e752e69ca891714f390bf59ad30d5081d4498","revisions":{"7a2e88e36cfc738ae841d46e6b61679ea67e1be3":{"kind":"REWORK","_number":1,"created":"2013-04-26 09:02:00.000000000","uploader":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"ref":"refs/changes/37/27537/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/37/27537/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/37/27537/1"}}},"commit":{"parents":[{"commit":"25141fd36c4885fb2fa0590d15db9b8d563f1afb","subject":"Merge \"Imported Translations from Transifex\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/25141fd36c4885fb2fa0590d15db9b8d563f1afb"}]}],"author":{"name":"Oleg Bondarev","email":"obondarev@mirantis.com","date":"2013-04-26 08:52:26.000000000","tz":240},"committer":{"name":"Oleg Bondarev","email":"obondarev@mirantis.com","date":"2013-04-26 09:00:08.000000000","tz":240},"subject":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder","message":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder\n\nMake cinder.API methods accept volume_id instead of the whole volume object.\nThis will remove redundant roundtrip to get the volume before\npassing it to other methods as in fact they only need the id.\n\nFixes bug 1172297\n\nChange-Id: I5e7f944c1c29b2f211ece2ef86c0959c81e806df\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7a2e88e36cfc738ae841d46e6b61679ea67e1be3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7a2e88e36cfc738ae841d46e6b61679ea67e1be3"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"25141fd36c4885fb2fa0590d15db9b8d563f1afb","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"d7f3537e810387ba051ca5e94aefe12665981a1e":{"kind":"REWORK","_number":2,"created":"2013-04-26 11:51:54.000000000","uploader":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"ref":"refs/changes/37/27537/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/37/27537/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/37/27537/2"}}},"commit":{"parents":[{"commit":"25141fd36c4885fb2fa0590d15db9b8d563f1afb","subject":"Merge \"Imported Translations from Transifex\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/25141fd36c4885fb2fa0590d15db9b8d563f1afb"}]}],"author":{"name":"Oleg Bondarev","email":"obondarev@mirantis.com","date":"2013-04-26 08:52:26.000000000","tz":240},"committer":{"name":"Oleg Bondarev","email":"obondarev@mirantis.com","date":"2013-04-26 11:29:55.000000000","tz":240},"subject":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder","message":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder\n\nMake cinder.API methods accept volume_id instead of the whole volume object.\nThis will remove redundant roundtrip to get the volume before\npassing it to other methods as in fact they only need the id.\n\nFixes bug 1172297\n\nChange-Id: I5e7f944c1c29b2f211ece2ef86c0959c81e806df\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d7f3537e810387ba051ca5e94aefe12665981a1e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/d7f3537e810387ba051ca5e94aefe12665981a1e"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"25141fd36c4885fb2fa0590d15db9b8d563f1afb","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"b417d83338a762f1ae0aeb8e1b88b8b18cc817c6":{"kind":"REWORK","_number":3,"created":"2013-04-30 14:15:03.000000000","uploader":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"ref":"refs/changes/37/27537/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/37/27537/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/37/27537/3"}}},"commit":{"parents":[{"commit":"25141fd36c4885fb2fa0590d15db9b8d563f1afb","subject":"Merge \"Imported Translations from Transifex\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/25141fd36c4885fb2fa0590d15db9b8d563f1afb"}]}],"author":{"name":"Oleg Bondarev","email":"obondarev@mirantis.com","date":"2013-04-26 08:52:26.000000000","tz":240},"committer":{"name":"Oleg Bondarev","email":"obondarev@mirantis.com","date":"2013-04-30 14:13:06.000000000","tz":240},"subject":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder","message":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder\n\nMake cinder.API methods accept volume_id instead of the whole volume object.\nThis will remove redundant roundtrip to get the volume before\npassing it to other methods as in fact they only need the id.\n\nFixes bug 1172297\n\nChange-Id: I5e7f944c1c29b2f211ece2ef86c0959c81e806df\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b417d83338a762f1ae0aeb8e1b88b8b18cc817c6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/b417d83338a762f1ae0aeb8e1b88b8b18cc817c6"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"25141fd36c4885fb2fa0590d15db9b8d563f1afb","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"586e752e69ca891714f390bf59ad30d5081d4498":{"kind":"REWORK","_number":4,"created":"2013-05-14 08:55:54.000000000","uploader":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"ref":"refs/changes/37/27537/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/37/27537/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/37/27537/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/37/27537/4"}}},"commit":{"parents":[{"commit":"28f0b01717f17ecc545a25f6deb0aa240e5aaf0d","subject":"Fix response from snapshot create stub","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/28f0b01717f17ecc545a25f6deb0aa240e5aaf0d"}]}],"author":{"name":"Oleg Bondarev","email":"obondarev@mirantis.com","date":"2013-04-26 08:52:26.000000000","tz":240},"committer":{"name":"Mark McLoughlin","email":"markmc@redhat.com","date":"2013-05-14 08:55:23.000000000","tz":60},"subject":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder","message":"Refactor nova.volume.cinder.API to reduce roundtrips with Cinder\n\nMake cinder.API methods accept volume_id instead of the whole volume object.\nThis will remove redundant roundtrip to get the volume before\npassing it to other methods as in fact they only need the id.\n\nFixes bug 1172297\n\nChange-Id: I5e7f944c1c29b2f211ece2ef86c0959c81e806df\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/586e752e69ca891714f390bf59ad30d5081d4498"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/586e752e69ca891714f390bf59ad30d5081d4498"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"28f0b01717f17ecc545a25f6deb0aa240e5aaf0d","is_merged_in_target_branch":true,"change_id":"Id006e3995c7696aa8f061a2b96123ea27e4b6d3f","change_number":29056,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}