)]}'
{"id":"openstack%2Fnova~10164","triplet_id":"openstack%2Fnova~master~I15ceb7625425ab097eebd5b7dd3606a171329f97","project":"openstack/nova","branch":"master","topic":"bp/no-db-messaging","hashtags":[],"change_id":"I15ceb7625425ab097eebd5b7dd3606a171329f97","subject":"Don\u0027t use rpc to lock/unlock an instance.","status":"MERGED","created":"2012-07-23 19:53:41.000000000","updated":"2012-07-26 03:10:27.000000000","submitted":"2012-07-26 01:48:02.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":2,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0fe6c469641e88f7c3b21a91907200850db516de","_number":10164,"virtual_id_number":10164,"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":0,"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},{"value":0,"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":2,"date":"2012-07-26 01:48:02.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"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":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},"all":[{"value":0,"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},{"value":2,"date":"2012-07-26 00:54:17.000000000","_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2012-07-26 00:28:59.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"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":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},"all":[{"value":0,"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},{"value":1,"date":"2012-07-26 00:54:17.000000000","_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2012-07-26 00:28:59.000000000","_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},{"value":0,"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},{"value":0,"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"value":0,"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"}],"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":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},{"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},{"_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-07-25 12:59:23.000000000","updated_by":{"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},"reviewer":{"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},"state":"REVIEWER"},{"updated":"2012-07-25 14:13:02.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-07-26 00:28:59.000000000","updated_by":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"reviewer":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"state":"REVIEWER"},{"updated":"2012-07-26 00:54:17.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-07-26 01:48:02.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"8cd7e2fd915d61cd54bc6a26eb6b2085153a4ffc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-07-23 20:32:19.000000000","message":"Patch Set 1: Works for me\n\nBuild successful\n\n- https://jenkins.openstack.org/job/gate-nova-merge/5135/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-docs/1001/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-pep8/6314/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python26/2511/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python27/2523/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/4509/testReport : SUCCESS\n","accounts_in_message":[],"_revision_number":1},{"id":"f6c04a4feaa0c52b77e031057f852b67c1c525a9","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2012-07-23 23:08:12.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"43752c1c6f786686a976c026ac59cb28a8d808d8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-07-23 23:24:41.000000000","message":"Patch Set 2: Doesn\u0027t seem to work\n\nBuild failed\n\n- https://jenkins.openstack.org/job/gate-nova-merge/5182/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-docs/1046/consoleFull : FAILURE\n- https://jenkins.openstack.org/job/gate-nova-pep8/6358/consoleFull : FAILURE\n- https://jenkins.openstack.org/job/gate-nova-python26/2558/consoleFull : FAILURE\n- https://jenkins.openstack.org/job/gate-nova-python27/2568/consoleFull : FAILURE\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/4573/testReport : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"6d60669292cc6d9dc25c4ec5082a6656d91f7914","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-07-24 03:26:13.000000000","message":"Patch Set 2: Doesn\u0027t seem to work\n\nSmokeStack Results (patch set 2):\n\tUnit Failed:Nova pep8 failed.  http://smokestack.openstack.org/?go\u003d/jobs/25776\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/25775\n","accounts_in_message":[],"_revision_number":2},{"id":"0697fa36d4ab7f9793da6c2c3eeb77fffc62ad21","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2012-07-24 17:08:54.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"946f8771db0bc26590965365615beb432d2a0545","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-07-24 17:26:31.000000000","message":"Patch Set 3: Works for me\n\nBuild successful\n\n- https://jenkins.openstack.org/job/gate-nova-merge/5233/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-docs/1097/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-pep8/6409/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python26/2606/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python27/2619/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/4648/testReport : SUCCESS\n","accounts_in_message":[],"_revision_number":3},{"id":"29559fad46a56c63f7529a67e4f1d75b28d95e51","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2012-07-25 12:35:22.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"c27b7e0b8a1c16c83e93396162e013a8cc8ae2cc","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-07-25 12:54:16.000000000","message":"Patch Set 4: Works for me\n\nBuild successful\n\n- https://jenkins.openstack.org/job/gate-nova-merge/5306/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-docs/1171/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-pep8/6483/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python26/2681/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python27/2694/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/4765/testReport : SUCCESS\n","accounts_in_message":[],"_revision_number":4},{"id":"4bfca234c91f5df6f59c869c95dda41ec38d26ec","author":{"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},"date":"2012-07-25 12:59:23.000000000","message":"Patch Set 4: (1 inline comment)\n\nComment inline","accounts_in_message":[],"_revision_number":4},{"id":"07b4a63ae503e3773f025d40f10051ca423189d0","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2012-07-25 13:33:50.000000000","message":"Patch Set 4: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"fb23e0ffd7df07dfdc6b126d07acc7593b7d3c97","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2012-07-25 14:13:02.000000000","message":"Patch Set 4: Works for me\n\nSmokeStack Results (patch set 4):\n\tUnit Success: http://smokestack.openstack.org/?go\u003d/jobs/25963\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/25961\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/25962\n","accounts_in_message":[],"_revision_number":4},{"id":"3273a9c53ddc4e22fd212ec77601dcbdc24abde0","author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"date":"2012-07-26 00:12:35.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"89781ea922ae08dd97f5840502a737260ba5042d","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2012-07-26 00:28:59.000000000","message":"Patch Set 5: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":5},{"id":"3f8e96b2421ade73fea558e78fb9ba98abacff88","author":{"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},"date":"2012-07-26 00:54:17.000000000","message":"Patch Set 5: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":5},{"id":"c1088b0f166b65ae2c5ff0f6ee3938fa6e9eec48","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-07-26 00:54:22.000000000","message":"Patch Set 5:\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":5},{"id":"c9333a6ac80f6a20d6b14995a647fff2292c340e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-07-26 00:56:07.000000000","message":"Patch Set 5: Works for me\n\nBuild successful\n\n- https://jenkins.openstack.org/job/gate-nova-merge/5366/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-docs/1234/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-pep8/6546/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python26/2743/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python27/2756/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/4852/testReport : SUCCESS\n","accounts_in_message":[],"_revision_number":5},{"id":"3dc8e3f6ea150cbcb435c4e3ba94b3dd895a48cd","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-07-26 01:48:02.000000000","message":"Patch Set 5: Verified\n\nBuild successful\n\n- https://jenkins.openstack.org/job/gate-nova-merge/5384/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-docs/1249/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-pep8/6561/consoleFull : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python26/2758/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-nova-python27/2771/testReport : SUCCESS\n- https://jenkins.openstack.org/job/gate-tempest-devstack-vm/4870/testReport : SUCCESS\n","accounts_in_message":[],"_revision_number":5},{"id":"a677b6b5b92d8bdd5db2258b92aa22d19c39f59f","date":"2012-07-26 02:05:11.000000000","message":"Change could not be merged because of a missing dependency.\n\nThe following changes must also be submitted:\n\n* I45914ed55f2c037ee2efd7775bd57fbde36a2116\n","accounts_in_message":[],"_revision_number":5},{"id":"0001450c37b94428cdbe4993e80f54b191efbce4","date":"2012-07-26 02:59:24.000000000","message":"Change could not be merged because of a missing dependency.","accounts_in_message":[],"_revision_number":5},{"id":"2c8e89a6abef86f3d215a8d1315055c4ef2bcb56","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-07-26 03:10:27.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"f5289971b7da19111ca6a68bb46c1108ea46664b","revisions":{"7b95946819e1907ede6d7e36feb0e35a0058815e":{"kind":"REWORK","_number":1,"created":"2012-07-23 19:53:41.000000000","uploader":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"ref":"refs/changes/64/10164/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/64/10164/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/64/10164/1"}}},"commit":{"parents":[{"commit":"780d30edc0d9d4ce4755a295a69386d114ea6afc","subject":"Convert reboot_instance to take a full instance.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/780d30edc0d9d4ce4755a295a69386d114ea6afc"}]}],"author":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-20 17:50:00.000000000","tz":-240},"committer":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-23 19:12:53.000000000","tz":-240},"subject":"Don\u0027t use rpc to lock/unlock an instance.","message":"Don\u0027t use rpc to lock/unlock an instance.\n\nInstead of converting this method to send a full instance over rpc\ninstead of just an instance UUID, this patch removes the usage of rpc\nfor this operation entirely.  All it\u0027s doing is a database update.  RPC\nis expensive, so cut out the middle-man.\n\nOne functional difference with this approach is that the db update is\nnow synchronous on the API node, instead of kicking off an async message\nto a compute node to handle it.  This seems fine, though.\n\nPart of blueprint no-db-messaging.\n\nChange-Id: I15ceb7625425ab097eebd5b7dd3606a171329f97\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7b95946819e1907ede6d7e36feb0e35a0058815e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/7b95946819e1907ede6d7e36feb0e35a0058815e"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"780d30edc0d9d4ce4755a295a69386d114ea6afc","is_merged_in_target_branch":false,"change_id":"Id2c455f66966a9b446e5bbbe542ed3a2b0655289","change_number":10163,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"93feb86b863b11a1e3951c8a38b9536d1bdb5404":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2012-07-23 23:08:12.000000000","uploader":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"ref":"refs/changes/64/10164/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/64/10164/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/64/10164/2"}}},"commit":{"parents":[{"commit":"da842e15bd476bc29ef4fafb2e59407c53d9438c","subject":"Convert reboot_instance to take a full instance.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/da842e15bd476bc29ef4fafb2e59407c53d9438c"}]}],"author":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-20 17:50:00.000000000","tz":-240},"committer":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-23 23:08:03.000000000","tz":-240},"subject":"Don\u0027t use rpc to lock/unlock an instance.","message":"Don\u0027t use rpc to lock/unlock an instance.\n\nInstead of converting this method to send a full instance over rpc\ninstead of just an instance UUID, this patch removes the usage of rpc\nfor this operation entirely.  All it\u0027s doing is a database update.  RPC\nis expensive, so cut out the middle-man.\n\nOne functional difference with this approach is that the db update is\nnow synchronous on the API node, instead of kicking off an async message\nto a compute node to handle it.  This seems fine, though.\n\nPart of blueprint no-db-messaging.\n\nChange-Id: I15ceb7625425ab097eebd5b7dd3606a171329f97\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/93feb86b863b11a1e3951c8a38b9536d1bdb5404"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/93feb86b863b11a1e3951c8a38b9536d1bdb5404"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"da842e15bd476bc29ef4fafb2e59407c53d9438c","is_merged_in_target_branch":false,"change_id":"Id2c455f66966a9b446e5bbbe542ed3a2b0655289","change_number":10163,"patch_set_number":2,"change_status":"MERGED"}],"branch":"refs/heads/master"},"1c1fc919713da0c80ef3bcb7301f6c05a4670cca":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2012-07-24 17:08:54.000000000","uploader":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"ref":"refs/changes/64/10164/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/64/10164/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/64/10164/3"}}},"commit":{"parents":[{"commit":"f895a15a9adf8dc7a7e18567dc4f54f0603d8fc8","subject":"Convert reboot_instance to take a full instance.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f895a15a9adf8dc7a7e18567dc4f54f0603d8fc8"}]}],"author":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-20 17:50:00.000000000","tz":-240},"committer":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-24 17:08:46.000000000","tz":-240},"subject":"Don\u0027t use rpc to lock/unlock an instance.","message":"Don\u0027t use rpc to lock/unlock an instance.\n\nInstead of converting this method to send a full instance over rpc\ninstead of just an instance UUID, this patch removes the usage of rpc\nfor this operation entirely.  All it\u0027s doing is a database update.  RPC\nis expensive, so cut out the middle-man.\n\nOne functional difference with this approach is that the db update is\nnow synchronous on the API node, instead of kicking off an async message\nto a compute node to handle it.  This seems fine, though.\n\nPart of blueprint no-db-messaging.\n\nChange-Id: I15ceb7625425ab097eebd5b7dd3606a171329f97\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1c1fc919713da0c80ef3bcb7301f6c05a4670cca"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/1c1fc919713da0c80ef3bcb7301f6c05a4670cca"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"f895a15a9adf8dc7a7e18567dc4f54f0603d8fc8","is_merged_in_target_branch":false,"change_id":"Id2c455f66966a9b446e5bbbe542ed3a2b0655289","change_number":10163,"patch_set_number":3,"change_status":"MERGED"}],"branch":"refs/heads/master"},"f4a137b7c19e13807272bbcefd6d2295f980ceda":{"kind":"TRIVIAL_REBASE","_number":4,"created":"2012-07-25 12:35:22.000000000","uploader":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"ref":"refs/changes/64/10164/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/64/10164/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/64/10164/4"}}},"commit":{"parents":[{"commit":"f62c4e1601ffbfedd0a24fef77ffe3ab6185eeec","subject":"Convert reboot_instance to take a full instance.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f62c4e1601ffbfedd0a24fef77ffe3ab6185eeec"}]}],"author":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-20 17:50:00.000000000","tz":-240},"committer":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-25 12:27:41.000000000","tz":-240},"subject":"Don\u0027t use rpc to lock/unlock an instance.","message":"Don\u0027t use rpc to lock/unlock an instance.\n\nInstead of converting this method to send a full instance over rpc\ninstead of just an instance UUID, this patch removes the usage of rpc\nfor this operation entirely.  All it\u0027s doing is a database update.  RPC\nis expensive, so cut out the middle-man.\n\nOne functional difference with this approach is that the db update is\nnow synchronous on the API node, instead of kicking off an async message\nto a compute node to handle it.  This seems fine, though.\n\nPart of blueprint no-db-messaging.\n\nChange-Id: I15ceb7625425ab097eebd5b7dd3606a171329f97\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f4a137b7c19e13807272bbcefd6d2295f980ceda"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f4a137b7c19e13807272bbcefd6d2295f980ceda"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"f62c4e1601ffbfedd0a24fef77ffe3ab6185eeec","is_merged_in_target_branch":false,"change_id":"Id2c455f66966a9b446e5bbbe542ed3a2b0655289","change_number":10163,"patch_set_number":4,"change_status":"MERGED"}],"branch":"refs/heads/master"},"f5289971b7da19111ca6a68bb46c1108ea46664b":{"kind":"TRIVIAL_REBASE","_number":5,"created":"2012-07-26 00:12:35.000000000","uploader":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"ref":"refs/changes/64/10164/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/64/10164/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/64/10164/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/64/10164/5"}}},"commit":{"parents":[{"commit":"6aac5f1308aa3e360204bd8f4dcfe90522f6db2e","subject":"Convert reboot_instance to take a full instance.","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/6aac5f1308aa3e360204bd8f4dcfe90522f6db2e"}]}],"author":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-20 17:50:00.000000000","tz":-240},"committer":{"name":"Russell Bryant","email":"rbryant@redhat.com","date":"2012-07-25 23:52:14.000000000","tz":-240},"subject":"Don\u0027t use rpc to lock/unlock an instance.","message":"Don\u0027t use rpc to lock/unlock an instance.\n\nInstead of converting this method to send a full instance over rpc\ninstead of just an instance UUID, this patch removes the usage of rpc\nfor this operation entirely.  All it\u0027s doing is a database update.  RPC\nis expensive, so cut out the middle-man.\n\nOne functional difference with this approach is that the db update is\nnow synchronous on the API node, instead of kicking off an async message\nto a compute node to handle it.  This seems fine, though.\n\nPart of blueprint no-db-messaging.\n\nChange-Id: I15ceb7625425ab097eebd5b7dd3606a171329f97\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f5289971b7da19111ca6a68bb46c1108ea46664b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/f5289971b7da19111ca6a68bb46c1108ea46664b"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"6aac5f1308aa3e360204bd8f4dcfe90522f6db2e","is_merged_in_target_branch":true,"change_id":"Id2c455f66966a9b446e5bbbe542ed3a2b0655289","change_number":10163,"patch_set_number":5,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}