)]}'
{"id":"openstack%2Fpython-novaclient~203416","triplet_id":"openstack%2Fpython-novaclient~master~Iba0cc1993f67351b11d034f372d7a5b98dc017f0","project":"openstack/python-novaclient","branch":"master","topic":"bug/1475973","hashtags":[],"change_id":"Iba0cc1993f67351b11d034f372d7a5b98dc017f0","subject":"hypervisor command can\u0027t use cell format id to show hypervisor","status":"MERGED","created":"2015-07-19 10:34:11.000000000","updated":"2015-07-27 11:04:01.000000000","submitted":"2015-07-27 11:03:59.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"2b63a9f766e5417633b81aa5a5923acc18bd1742","_number":203416,"virtual_id_number":203416,"owner":{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2015-07-27 11:03:59.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"date":"2015-07-22 10:18:01.000000000","_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"}],"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":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2015-07-24 21:37:59.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":2,"date":"2015-07-27 10:12:05.000000000","_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"}],"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":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":1,"date":"2015-07-27 10:12:05.000000000","_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"}],"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":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-07-24 21:37:59.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":"2015-07-27 10:12:05.000000000","updated_by":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"reviewer":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"state":"REVIEWER"},{"updated":"2015-07-27 11:03:59.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"927007ee0a3b5e16ed8e6d66c562d1d25e5963f7","author":{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"},"date":"2015-07-19 10:34:11.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"2843e8613b3d37a7ffc33b8f1378af1cc8ce1843","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-19 11:27:55.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-novaclient-python26 http://logs.openstack.org/16/203416/1/check/gate-python-novaclient-python26/ba02ce1/ : FAILURE in 1m 33s\n- gate-python-novaclient-pep8 http://logs.openstack.org/16/203416/1/check/gate-python-novaclient-pep8/f204aef/ : SUCCESS in 1m 07s\n- gate-python-novaclient-docs http://docs-draft.openstack.org/16/203416/1/check/gate-python-novaclient-docs/6c1a1e9//doc/build/html/ : SUCCESS in 1m 08s\n- gate-python-novaclient-python27 http://logs.openstack.org/16/203416/1/check/gate-python-novaclient-python27/498e038/ : FAILURE in 1m 19s\n- gate-python-novaclient-python34 http://logs.openstack.org/16/203416/1/check/gate-python-novaclient-python34/a3771d4/ : FAILURE in 2m 24s\n- gate-python-novaclient-pypy http://logs.openstack.org/16/203416/1/check/gate-python-novaclient-pypy/27bab32/ : FAILURE in 3m 34s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/16/203416/1/check/gate-tempest-dsvm-neutron-src-python-novaclient/b04f5fd/ : SUCCESS in 53m 34s\n- gate-novaclient-dsvm-functional http://logs.openstack.org/16/203416/1/check/gate-novaclient-dsvm-functional/f5dcb7b/ : SUCCESS in 27m 26s","accounts_in_message":[],"_revision_number":1},{"id":"942977c557cab3d525eb666e227f3cbab77bd74a","author":{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"},"date":"2015-07-20 04:44:31.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"517124653a05ab4d64df952d5e2ae42343b3414d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-20 05:33:24.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-python26 http://logs.openstack.org/16/203416/2/check/gate-python-novaclient-python26/61df652/ : SUCCESS in 1m 33s\n- gate-python-novaclient-pep8 http://logs.openstack.org/16/203416/2/check/gate-python-novaclient-pep8/3c5c422/ : SUCCESS in 1m 02s\n- gate-python-novaclient-docs http://docs-draft.openstack.org/16/203416/2/check/gate-python-novaclient-docs/8e9a5fa//doc/build/html/ : SUCCESS in 1m 29s\n- gate-python-novaclient-python27 http://logs.openstack.org/16/203416/2/check/gate-python-novaclient-python27/a4222a0/ : SUCCESS in 2m 40s\n- gate-python-novaclient-python34 http://logs.openstack.org/16/203416/2/check/gate-python-novaclient-python34/d93c4bd/ : SUCCESS in 2m 02s\n- gate-python-novaclient-pypy http://logs.openstack.org/16/203416/2/check/gate-python-novaclient-pypy/827bd21/ : SUCCESS in 2m 33s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/16/203416/2/check/gate-tempest-dsvm-neutron-src-python-novaclient/20c77d8/ : SUCCESS in 47m 23s\n- gate-novaclient-dsvm-functional http://logs.openstack.org/16/203416/2/check/gate-novaclient-dsvm-functional/8cb45ad/ : SUCCESS in 20m 47s","accounts_in_message":[],"_revision_number":2},{"id":"86571faeb5c2efd6fd51a33d144618c6361a4fc3","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-07-20 15:49:27.000000000","message":"Patch Set 2: Code-Review+1\n\nI am slightly concerned about the URL encoding; \u0027@\u0027 in particular shouldn\u0027t ever appear in a path, it should be replaced with %40.  That said, I presume from your commit message that you have already tested that this works against a real nova instance—which seems correct, as I would expect some of the underlying code to take care of the translations for you.  Please verify that you\u0027ve done that kind of testing, and I\u0027ll upgrade my +1 to a +2…","accounts_in_message":[],"_revision_number":2},{"id":"aa34257364cac5432ba165d754f5cc78350d5089","author":{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"},"date":"2015-07-22 10:18:01.000000000","message":"Patch Set 2:\n\nI had using devstack to build cells environments and using nova client to show hypervisor with cell id. And curl request url wouldn\u0027t been encoded. You can also try special character for flavor request. Try to create flavor named with special character like f@. Then you can using nova flavor-show \u0027f@\u0027 to request flavor. And curl request url will not replace @ with %40.","accounts_in_message":[],"_revision_number":2},{"id":"5d3e6b147282c7e46157070a08feb5aee0132d24","author":{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"},"date":"2015-07-22 10:26:40.000000000","message":"Patch Set 2:\n\nI\u0027m sorry. I was wrong, I will still test","accounts_in_message":[],"_revision_number":2},{"id":"92b10c685d9baa7e6e8d2034c03c06d6564f5a57","author":{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"},"date":"2015-07-22 10:58:48.000000000","message":"Patch Set 2:\n\n@Kevin L. Mitchell\nI had test using cell format id such as region!child@1 to send request. Nova client can get hypervisor information though http://ip:8774/os-hypervisors/region!child@1.","accounts_in_message":[],"_revision_number":2},{"id":"f16ebb6335895158b74dc443f42b11aa38f99956","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2015-07-24 21:37:59.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"4fc583f4ba9e3d2925c5504c7db6ef5875d79fc3","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2015-07-27 10:12:05.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\nIts a shame we need to do this, but this seems correct, sadly.","accounts_in_message":[],"_revision_number":2},{"id":"5a1e0b54259170684b9429bcdfb440a9c82fa390","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-27 10:12:10.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"aca211374ca0cee41815c50661b401ca4ce86998","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-27 11:03:59.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-novaclient-python26 http://logs.openstack.org/16/203416/2/gate/gate-python-novaclient-python26/d1de8d4/ : SUCCESS in 1m 40s\n- gate-python-novaclient-docs http://docs-draft.openstack.org/16/203416/2/gate/gate-python-novaclient-docs/fbcf110//doc/build/html/ : SUCCESS in 1m 57s\n- gate-python-novaclient-pep8 http://logs.openstack.org/16/203416/2/gate/gate-python-novaclient-pep8/e9e34d3/ : SUCCESS in 1m 07s\n- gate-python-novaclient-python27 http://logs.openstack.org/16/203416/2/gate/gate-python-novaclient-python27/2b5663d/ : SUCCESS in 1m 56s\n- gate-python-novaclient-python34 http://logs.openstack.org/16/203416/2/gate/gate-python-novaclient-python34/74c283a/ : SUCCESS in 2m 29s\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/16/203416/2/gate/gate-tempest-dsvm-neutron-src-python-novaclient/fabefeb/ : SUCCESS in 49m 40s\n- gate-novaclient-dsvm-functional http://logs.openstack.org/16/203416/2/gate/gate-novaclient-dsvm-functional/c51d9a1/ : SUCCESS in 21m 56s","accounts_in_message":[],"_revision_number":2},{"id":"196bb5a2343e29a51091e0678ac6baa35be84ac1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-07-27 11:04:01.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"c3a0c035dc6bd4676805d0618dfe334ed2d81228","revisions":{"3fff57dc404b3775ba8b524fa3c2c36701bb7834":{"kind":"REWORK","_number":1,"created":"2015-07-19 10:34:11.000000000","uploader":{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"},"ref":"refs/changes/16/203416/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/16/203416/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/16/203416/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/16/203416/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/16/203416/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/16/203416/1"}}},"commit":{"parents":[{"commit":"7c96ed73a05c9d7944a82d8068f9602ebb464af1","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/7c96ed73a05c9d7944a82d8068f9602ebb464af1"}]}],"author":{"name":"Chung Chih, Hung","email":"lyan.h@inwinstack.com","date":"2015-07-19 10:31:45.000000000","tz":480},"committer":{"name":"Chung Chih, Hung","email":"lyan.h@inwinstack.com","date":"2015-07-19 10:31:45.000000000","tz":480},"subject":"hypervisor command can\u0027t use cell format id to show hypervisor","message":"hypervisor command can\u0027t use cell format id to show hypervisor\n\nThis bug was occurred in cell mode.\n\nIn cell mode, compute node\u0027s id was not identified by integer type. It\nwas formatted with \"path!to!cell@ID\".\nTherefore we can\u0027t using the id of hypervisor-list output.\n\nFor example,\n\u003e nova hypervisor-list\n+----------------+--------------------------+-------+---------+\n| ID | Hypervisor hostname | State | Status |\n+----------------+--------------------------+-------+---------+\n| region!child@1 | vagrant-ubuntu-trusty-64 | up | enabled |\n+----------------+--------------------------+-------+---------+\n\nChange-Id: Iba0cc1993f67351b11d034f372d7a5b98dc017f0\nCloses-Bug: 1475973\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/3fff57dc404b3775ba8b524fa3c2c36701bb7834"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/3fff57dc404b3775ba8b524fa3c2c36701bb7834"}]},"branch":"refs/heads/master"},"c3a0c035dc6bd4676805d0618dfe334ed2d81228":{"kind":"REWORK","_number":2,"created":"2015-07-20 04:44:31.000000000","uploader":{"_account_id":14447,"name":"lyanchih","email":"lyan.h@inwinstack.com","username":"lyanchih"},"ref":"refs/changes/16/203416/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/16/203416/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/16/203416/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/16/203416/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/16/203416/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/16/203416/2"}}},"commit":{"parents":[{"commit":"7c96ed73a05c9d7944a82d8068f9602ebb464af1","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/7c96ed73a05c9d7944a82d8068f9602ebb464af1"}]}],"author":{"name":"Chung Chih, Hung","email":"lyan.h@inwinstack.com","date":"2015-07-19 10:31:45.000000000","tz":480},"committer":{"name":"lyanchih","email":"lyan.h@inwinstack.com","date":"2015-07-20 04:55:32.000000000","tz":0},"subject":"hypervisor command can\u0027t use cell format id to show hypervisor","message":"hypervisor command can\u0027t use cell format id to show hypervisor\n\nThis bug was occurred in cell mode.\n\nIn cell mode, compute node\u0027s id was not identified by integer type. It\nwas formatted with \"path!to!cell@ID\".\nTherefore we can\u0027t using the id of hypervisor-list output.\n\nFor example,\n\u003e nova hypervisor-list\n+----------------+--------------------------+-------+---------+\n| ID | Hypervisor hostname | State | Status |\n+----------------+--------------------------+-------+---------+\n| region!child@1 | vagrant-ubuntu-trusty-64 | up | enabled |\n+----------------+--------------------------+-------+---------+\n\nChange-Id: Iba0cc1993f67351b11d034f372d7a5b98dc017f0\nCloses-Bug: 1475973\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/c3a0c035dc6bd4676805d0618dfe334ed2d81228"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/c3a0c035dc6bd4676805d0618dfe334ed2d81228"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}