)]}'
{"id":"openstack%2Fnova~22609","triplet_id":"openstack%2Fnova~master~I7a8aaea5772707b8bd6313249e74b1302eedefe8","project":"openstack/nova","branch":"master","topic":"bug/1131264","hashtags":[],"change_id":"I7a8aaea5772707b8bd6313249e74b1302eedefe8","subject":"Fix crash in quantumapi if no network or port id is specified","status":"MERGED","created":"2013-02-21 21:48:55.000000000","updated":"2013-02-28 21:54:48.000000000","submitted":"2013-02-28 21:54:46.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":18,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"78ff610818fa8267401907e0454f7fa6673e73a7","_number":22609,"virtual_id_number":22609,"owner":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"recommended":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},{"value":0,"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"value":1,"date":"2013-02-26 15:19:32.000000000","_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"value":0,"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"value":2,"date":"2013-02-28 21:54:46.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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"recommended":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"all":[{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"date":"2013-02-26 15:52:04.000000000","permitted_voting_range":{"min":0,"max":2},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":1,"date":"2013-02-27 01:04:06.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},{"value":1,"date":"2013-02-27 07:30:43.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":2,"date":"2013-02-27 17:50:54.000000000","_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2013-02-28 21:17:46.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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"all":[{"value":0,"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"date":"2013-02-26 15:52:04.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},{"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,"date":"2013-02-27 17:50:54.000000000","_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2013-02-28 21:17:46.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":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"value":0,"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},{"value":0,"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},{"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":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"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":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-02-25 18:42:32.000000000","updated_by":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"reviewer":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"state":"REVIEWER"},{"updated":"2013-02-26 15:19:32.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-02-27 01:04:06.000000000","updated_by":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"reviewer":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"state":"REVIEWER"},{"updated":"2013-02-27 07:30:43.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-02-27 17:50:54.000000000","updated_by":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"reviewer":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"state":"REVIEWER"},{"updated":"2013-02-28 21:17:46.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":"2013-02-28 21:54:46.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"dec48ac6e771b559a6235ab0076d93055b3787ae","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-21 22:08:18.000000000","message":"Patch Set 1: Works for me\n\nSmokeStack Results (patch set 1):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/73806\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/73807\n","accounts_in_message":[],"_revision_number":1},{"id":"897e4a38b17a14d42bec6a8dffc10ad6b65c3b32","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-02-21 22:27:26.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nThe unittests emit a NameError: global name \u0027time\u0027 is not defined exception in a thread but don\u0027t fail. Is this expected? if so can the Exception be asserted so that it does not add noise to the test output?","accounts_in_message":[],"_revision_number":1},{"id":"5f8bc988aac676d82069283fd99820088d250c4f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-21 22:43:52.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/22609/1/check/gate-nova-pep8/21333 : SUCCESS\n- http://logs.openstack.org/22609/1/check/gate-nova-docs/15892 : SUCCESS\n- http://logs.openstack.org/22609/1/check/gate-nova-python26/17322 : SUCCESS\n- http://logs.openstack.org/22609/1/check/gate-nova-python27/17391 : SUCCESS\n- http://logs.openstack.org/22609/1/check/gate-tempest-devstack-vm-full/4949 : SUCCESS\n- http://logs.openstack.org/22609/1/check/gate-tempest-devstack-vm-postgres-full/4668 : SUCCESS\n- http://logs.openstack.org/22609/1/check/gate-tempest-devstack-vm-quantum/9512 : SUCCESS\n- http://logs.openstack.org/22609/1/check/gate-nova-pylint/8698 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":1},{"id":"92ed997791b3bc11366f0c037dbbcd37a0f6ab1d","author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"date":"2013-02-21 22:51:06.000000000","message":"Patch Set 1: No score\n\nThe time import error is corrected in https://review.openstack.org/#/c/22626/","accounts_in_message":[],"_revision_number":1},{"id":"db5eff45f302b393adb8990e03edd9257c8a26fb","author":{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},"date":"2013-02-25 18:42:32.000000000","message":"Patch Set 1: (1 inline comment)\n\nA small comment inline.","accounts_in_message":[],"_revision_number":1},{"id":"774e49f680fdb075f1a592c87f481810821af366","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-02-25 18:48:30.000000000","message":"Patch Set 1: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"036c9752c0b1469e760dfeee83a140ffbfadf962","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2013-02-26 07:15:26.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(4 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"87591bbfc85a5b38dcbd7de243a0d449975f7836","author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"date":"2013-02-26 10:31:52.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"106f1a1a533e868a1bec120530350e8026c76c37","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-02-26 14:34:20.000000000","message":"Patch Set 1: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":1},{"id":"829d14e67697084b2a770da93c62fb9f48c7becb","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-02-26 14:58:03.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"adf5647bcf6673e18b141ffed3a12603f199704e","author":{"_account_id":2166,"name":"SmokeStack CI","username":"smokestack","tags":["SERVICE_USER"]},"date":"2013-02-26 15:19:32.000000000","message":"Patch Set 2: Works for me\n\nSmokeStack Results (patch set 2):\n\tLibvirt (Fedora 17) Success: http://smokestack.openstack.org/?go\u003d/jobs/74866\n\tXenServer 5.6 (F17 domU) Success: http://smokestack.openstack.org/?go\u003d/jobs/74867\n","accounts_in_message":[],"_revision_number":2},{"id":"12d1a25dd18561a9cba8bc0e2cf7af41a4bdd002","author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"date":"2013-02-26 15:33:18.000000000","message":"Patch Set 2: (3 inline comments)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"32c43ad3a222cb1fd1a6a95ef246ac9f4d04e577","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-02-26 15:52:04.000000000","message":"Patch Set 2: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"504508adcea330b2460418610411a1f4cd7d33f0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-26 15:53:32.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- http://logs.openstack.org/22609/2/check/gate-nova-pep8/21577 : SUCCESS\n- http://logs.openstack.org/22609/2/check/gate-nova-docs/16107 : SUCCESS\n- http://logs.openstack.org/22609/2/check/gate-nova-python26/17537 : SUCCESS\n- http://logs.openstack.org/22609/2/check/gate-nova-python27/17605 : SUCCESS\n- http://logs.openstack.org/22609/2/check/gate-tempest-devstack-vm-full/5505 : SUCCESS\n- http://logs.openstack.org/22609/2/check/gate-tempest-devstack-vm-postgres-full/5224 : SUCCESS\n- http://logs.openstack.org/22609/2/check/gate-tempest-devstack-vm-quantum/10199 : SUCCESS\n- http://logs.openstack.org/22609/2/check/gate-nova-pylint/8855 : SUCCESS (non-voting)\n","accounts_in_message":[],"_revision_number":2},{"id":"1060fc9ca2db35e2a52b5822f3e15d64f6725344","author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"date":"2013-02-26 23:32:03.000000000","message":"Patch Set 2: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"28ed943438cea14bf4c5741b83dceb79053515d3","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2013-02-26 23:44:42.000000000","message":"Patch Set 2: (2 inline comments)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"b6f91702cd4ea459bbf0e529aa92f3a206e12057","author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"date":"2013-02-27 01:04:06.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"9fcef82755dd33445acfb96ae7249f41184f6065","author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"date":"2013-02-27 07:30:43.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":2},{"id":"81978d0cc0cbd021396ad8827c8b6cadec7fc76f","author":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"date":"2013-02-27 17:50:54.000000000","message":"Patch Set 2: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":2},{"id":"2a839fc8620705f27f93f32077052e8546a9d503","author":{"_account_id":67,"name":"Vish Ishaya","email":"vishvananda@gmail.com","username":"vishvananda"},"date":"2013-02-28 21:17:46.000000000","message":"Patch Set 2: Looks good to me (core reviewer); Approved\n\n","accounts_in_message":[],"_revision_number":2},{"id":"f18c8f551fe7c4e41f085d6e491f712c5e17b483","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-28 21:17:49.000000000","message":"Patch Set 2: No score\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"cc4588e0cb4670a96bf8b7e801173af069aa3fc8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-28 21:54:46.000000000","message":"Patch Set 2: Verified\n\nBuild succeeded.\n\n- http://logs.openstack.org/22609/2/gate/gate-nova-docs/16321 : SUCCESS\n- http://logs.openstack.org/22609/2/gate/gate-nova-pep8/21834 : SUCCESS\n- http://logs.openstack.org/22609/2/gate/gate-nova-python26/17749 : SUCCESS\n- http://logs.openstack.org/22609/2/gate/gate-nova-python27/17819 : SUCCESS\n- http://logs.openstack.org/22609/2/gate/gate-tempest-devstack-vm-full/6151 : SUCCESS\n- http://logs.openstack.org/22609/2/gate/gate-tempest-devstack-vm-postgres-full/5870 : SUCCESS\n- http://logs.openstack.org/22609/2/gate/gate-tempest-devstack-vm-quantum/10954 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"f01c1a2c85781ba168e2aecff2b35f2ddc44847a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-02-28 21:54:48.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"cdf5e432de2cf1c0c5466470a6a0bccf999fcdd4","revisions":{"4241bd645b3654b5c600c867b8bcdaa28c2776cf":{"kind":"REWORK","_number":1,"created":"2013-02-21 21:48:55.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/09/22609/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/09/22609/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/09/22609/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/09/22609/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/09/22609/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/09/22609/1"}}},"commit":{"parents":[{"commit":"0f0a8c6bcce1b230a5719b35089f25e55a1359a8","subject":"Merge \"Fix instance can not be deleted after soft reboot\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/0f0a8c6bcce1b230a5719b35089f25e55a1359a8"}]}],"author":{"name":"Dan Smith","email":"danms@us.ibm.com","date":"2013-02-21 18:15:23.000000000","tz":-300},"committer":{"name":"Dan Smith","email":"danms@us.ibm.com","date":"2013-02-21 21:48:34.000000000","tz":-300},"subject":"Fix crash in quantumapi if no network or port id is specified","message":"Fix crash in quantumapi if no network or port id is specified\n\nThe existing code assumes that if no port id or fixed ip was specified,\nthat the network id passed in is valid. This is not necessarily the case,\nand results in adding None to the list of network ids, which later\ncauses a check to see if the network id list is empty to fail.\n\nThis only adds the network id to the list if it was specified, and\nadds a more explicit check to the region that previously fell through\nwithout finding a matching network so that a similar case would be\neasier to debug in the future.\n\nAddresses bug #1131264\n\nChange-Id: I7a8aaea5772707b8bd6313249e74b1302eedefe8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4241bd645b3654b5c600c867b8bcdaa28c2776cf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/4241bd645b3654b5c600c867b8bcdaa28c2776cf"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"0f0a8c6bcce1b230a5719b35089f25e55a1359a8","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"cdf5e432de2cf1c0c5466470a6a0bccf999fcdd4":{"kind":"REWORK","_number":2,"created":"2013-02-26 14:58:03.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/09/22609/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/09/22609/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/09/22609/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/09/22609/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/09/22609/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/09/22609/2"}}},"commit":{"parents":[{"commit":"a8fef8e5103973d8b9747bca3784c98e4075e454","subject":"Merge \"Minor code optimization in _compute_topic\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a8fef8e5103973d8b9747bca3784c98e4075e454"}]}],"author":{"name":"Dan Smith","email":"danms@us.ibm.com","date":"2013-02-21 18:15:23.000000000","tz":-300},"committer":{"name":"Dan Smith","email":"danms@us.ibm.com","date":"2013-02-26 14:57:17.000000000","tz":-300},"subject":"Fix crash in quantumapi if no network or port id is specified","message":"Fix crash in quantumapi if no network or port id is specified\n\nThe existing code assumes that if no port id or fixed ip was specified,\nthat the network id passed in is valid. This is not necessarily the case,\nand results in adding None to the list of network ids, which later\ncauses a check to see if the network id list is empty to fail.\n\nThis only adds the network id to the list if it was specified, and\nadds a more explicit check to the region that previously fell through\nwithout finding a matching network so that a similar case would be\neasier to debug in the future.\n\nAddresses bug #1131264\n\nChange-Id: I7a8aaea5772707b8bd6313249e74b1302eedefe8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/cdf5e432de2cf1c0c5466470a6a0bccf999fcdd4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/cdf5e432de2cf1c0c5466470a6a0bccf999fcdd4"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"a8fef8e5103973d8b9747bca3784c98e4075e454","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}