)]}'
{"id":"openstack%2Fnova~12192","triplet_id":"openstack%2Fnova~master~I44d2e513eb0f9f207dc795860d642c9399e4ad61","project":"openstack/nova","branch":"master","topic":"master","hashtags":[],"change_id":"I44d2e513eb0f9f207dc795860d642c9399e4ad61","subject":"Add support for create volume from snapshot.","status":"ABANDONED","created":"2012-08-30 14:09:35.000000000","updated":"2012-09-11 14:27:52.000000000","total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"215d180027d994b8f7c029c2e0849e062dd160b4","_number":12192,"virtual_id_number":12192,"owner":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"all":[{"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"value":1,"date":"2012-09-05 18:42:49.000000000","_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},{"value":1,"date":"2012-09-05 18:45:47.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1043,"name":"John Griffith","inactive":true}],"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":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":-1,"date":"2012-09-05 20:22:03.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},{"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":1043,"name":"John Griffith","inactive":true}],"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":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1043,"name":"John Griffith","inactive":true}],"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":1},"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":1043,"name":"John Griffith","inactive":true}],"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":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},{"_account_id":1043,"name":"John Griffith","inactive":true},{"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},{"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-08-30 14:32:52.000000000","updated_by":{"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},"reviewer":{"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},"state":"REVIEWER"},{"updated":"2012-08-30 14:49:54.000000000","updated_by":{"_account_id":1043,"name":"John Griffith","inactive":true},"reviewer":{"_account_id":1043,"name":"John Griffith","inactive":true},"state":"REVIEWER"},{"updated":"2012-09-05 18:42:49.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":"2012-09-05 18:45:47.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2012-09-05 20:22:03.000000000","updated_by":{"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},"reviewer":{"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},"state":"REVIEWER"}],"messages":[{"id":"de4d177e85d5de7987ba3bbd5d782e6a4de743d0","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-08-30 14:27:47.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nSmokeStack Results (patch set 1):\n\tUnit Failed:Nova pep8 failed.  http://smokestack.openstack.org/?go\u003d/jobs/32386\n\tLibvirt (Fedora 17) Failed:Failed to build nova packages. http://smokestack.openstack.org/?go\u003d/jobs/32384\n\tXenServer 5.6 (F17 domU) Failed:Failed to build nova packages. http://smokestack.openstack.org/?go\u003d/jobs/32385\n","accounts_in_message":[],"_revision_number":1},{"id":"81591a7cfba21dc6d7b925dabe0cead2a3d2a43a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-08-30 14:29:49.000000000","message":"Patch Set 1: Doesn\u0027t seem to work\n\nBuild failed\n\n- http://logs.openstack.org/12192/1/check/gate-nova-merge/7921 : SUCCESS\n- http://logs.openstack.org/12192/1/check/gate-nova-docs/3725 : SUCCESS\n- http://logs.openstack.org/12192/1/check/gate-nova-pep8/9037 : FAILURE\n- http://logs.openstack.org/12192/1/check/gate-nova-python26/5191 : SUCCESS\n- http://logs.openstack.org/12192/1/check/gate-nova-python27/5247 : SUCCESS\n- http://logs.openstack.org/12192/1/check/gate-tempest-devstack-vm/9800 : FAILURE\n- http://logs.openstack.org/12192/1/check/gate-nova-pylint/306 : FAILURE (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"ce54d7a7e754c954f7dd3067a737e28fdc893baa","author":{"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},"date":"2012-08-30 14:32:52.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nsee inline. any chance we have a test for this?\n\nalso what\u0027s the feature freeze policy for backporting from cinder?","accounts_in_message":[],"_revision_number":1},{"id":"6fc29a62d95f44ec4d6dd4c0138685e2a03d900e","author":{"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},"date":"2012-08-30 15:57:05.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nThanks for porting this over Chuck!\n\nI think it should really match the cinder commit (Change-Id: If4105e7af7ba33f09a15103b53ad675aceb2ebb4) though, so the code doesn\u0027t get too out of sync.\n\nIt needs the snapshot protect/unprotect changes from that commit to work anyway (see http://ceph.com/docs/master/dev/rbd-layering/ for details).","accounts_in_message":[],"_revision_number":1},{"id":"f9c73d9d04f4af17aea80e8c1a242d080f8e19fa","author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"date":"2012-09-05 18:23:03.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"f70e0f1749d615160a933b6981c8376a49baaebd","author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"date":"2012-09-05 18:23:25.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"51bf2a864c8c27ed494145cd49fa6e7b37d88bb8","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-09-05 18:42:49.000000000","message":"Patch Set 2: Works for me\n\nSmokeStack Results (patch set 2):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/33215\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/33213\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/33214\n","accounts_in_message":[],"_revision_number":2},{"id":"94eb0557f828f8fb69b8cb58408ebc91dea42caf","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-09-05 18:45:47.000000000","message":"Patch Set 2: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/12192/2/check/gate-nova-merge/8212 : SUCCESS\n- http://logs.openstack.org/12192/2/check/gate-nova-docs/4011 : SUCCESS\n- http://logs.openstack.org/12192/2/check/gate-nova-pep8/9323 : SUCCESS\n- http://logs.openstack.org/12192/2/check/gate-nova-python26/5474 : SUCCESS\n- http://logs.openstack.org/12192/2/check/gate-nova-python27/5533 : SUCCESS\n- http://logs.openstack.org/12192/2/check/gate-tempest-devstack-vm/10491 : SUCCESS\n- http://logs.openstack.org/12192/2/check/gate-tempest-devstack-vm-cinder/472 : SUCCESS\n- http://logs.openstack.org/12192/2/check/gate-nova-pylint/505 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"9a14650f906562e38537b891c9803bf48f79fd6e","author":{"_account_id":1107,"name":"Josh Durgin","email":"jdurgin@redhat.com","username":"jdurgin"},"date":"2012-09-05 20:22:03.000000000","message":"Patch Set 2: I would prefer that you didn\u0027t merge this\n\nThis won\u0027t work without the snapshot protect/unprotect additions I mentioned in my last review.","accounts_in_message":[],"_revision_number":2},{"id":"518e50111daf61120b30e28f0ce8fda59582c2fe","author":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"date":"2012-09-11 14:27:52.000000000","message":"Patch Set 2: Abandoned\n\nthis has been merged already.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"db76fcb5b7e109784f77da5346e1d27a9b2fffda","revisions":{"83c517f41a245d3f7e807516b5d5cd293b7ae581":{"kind":"REWORK","_number":1,"created":"2012-08-30 14:09:35.000000000","uploader":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"ref":"refs/changes/92/12192/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/92/12192/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/92/12192/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/92/12192/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/92/12192/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/92/12192/1"}}},"commit":{"parents":[{"commit":"4adbb96b5530184e3f42047a7416d6f315a14150","subject":"Merge \"Add a new exception for live migration.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4adbb96b5530184e3f42047a7416d6f315a14150"}]}],"author":{"name":"Chuck Short","email":"chuck.short@canonical.com","date":"2012-08-30 14:07:53.000000000","tz":-300},"committer":{"name":"Chuck Short","email":"chuck.short@canonical.com","date":"2012-08-30 14:09:30.000000000","tz":-300},"subject":"Add support for create volume from snapshot.","message":"Add support for create volume from snapshot.\n\nAdd create volume from snapshot to the RBD driver,\nthis is already in cinder, so we just port this code\nover from cinder.\n\nLP: #1032405\n\nChange-Id: I44d2e513eb0f9f207dc795860d642c9399e4ad61\nSigned-off-by: Chuck Short \u003cchuck.short@canonical.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/83c517f41a245d3f7e807516b5d5cd293b7ae581"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/83c517f41a245d3f7e807516b5d5cd293b7ae581"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"4adbb96b5530184e3f42047a7416d6f315a14150","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"db76fcb5b7e109784f77da5346e1d27a9b2fffda":{"kind":"REWORK","_number":2,"created":"2012-09-05 18:23:25.000000000","uploader":{"_account_id":24,"name":"Chuck Short","email":"charles.short@windriver.com","username":"zulcss"},"ref":"refs/changes/92/12192/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/92/12192/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/92/12192/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/92/12192/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/92/12192/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/92/12192/2"}}},"commit":{"parents":[{"commit":"fb84ed7b32520b11d4a6273eb002b1f1abd2fbe8","subject":"Merge \"fix issues with Nova security groups and Quantum\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/fb84ed7b32520b11d4a6273eb002b1f1abd2fbe8"}]}],"author":{"name":"Chuck Short","email":"chuck.short@canonical.com","date":"2012-08-30 14:07:53.000000000","tz":-300},"committer":{"name":"Chuck Short","email":"chuck.short@canonical.com","date":"2012-09-05 18:23:21.000000000","tz":-300},"subject":"Add support for create volume from snapshot.","message":"Add support for create volume from snapshot.\n\nAdd create volume from snapshot to the RBD driver,\nthis is already in cinder, so we just port this code\nover from cinder.\n\nLP: #1032405\n\nChange-Id: I44d2e513eb0f9f207dc795860d642c9399e4ad61\nSigned-off-by: Chuck Short \u003cchuck.short@canonical.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/db76fcb5b7e109784f77da5346e1d27a9b2fffda"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/db76fcb5b7e109784f77da5346e1d27a9b2fffda"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"fb84ed7b32520b11d4a6273eb002b1f1abd2fbe8","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}