)]}'
{"id":"openstack%2Fpython-novaclient~397815","triplet_id":"openstack%2Fpython-novaclient~master~Ie46b0e89723022c12ff2dae301f248c56178b2ad","project":"openstack/python-novaclient","branch":"master","topic":"bug/1645929","hashtags":[],"change_id":"Ie46b0e89723022c12ff2dae301f248c56178b2ad","subject":"Fix can\u0027t show other tenant\u0027s server error","status":"ABANDONED","created":"2016-11-15 16:25:50.000000000","updated":"2017-08-23 02:40:51.000000000","total_comment_count":11,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f2569d3c3dfba33137444b4be570a2ead0ceb921","_number":397815,"virtual_id_number":397815,"owner":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":1,"date":"2016-12-08 10:41:13.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":2},"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"}],"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":2271,"name":"Michael Still","email":"mikal@stillhq.com","username":"mikalstill"},{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-11-23 12:25:28.000000000","updated_by":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"reviewer":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"state":"REVIEWER"},{"updated":"2016-11-23 12:25:55.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":"2016-11-30 11:10:47.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":"2016-11-30 16:17: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":"2016-12-01 02:02:44.000000000","updated_by":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"reviewer":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"state":"REVIEWER"},{"updated":"2016-12-08 10:41:13.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"0c18276f4c4786cdc976d7f48af677dac3e7e3ee","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-11-15 16:25:50.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6039c14288ec7cd438d9f4e72882a6ad2e6eb03c","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-11-15 16:46:16.000000000","message":"Patch Set 1:\n\n(1 comment)\n\nLooks good to me; one observation noted in-line, though, that I\u0027d like to see your response to.  If there\u0027s a good reason to not use the prefix I suggested, I\u0027ll +2.","accounts_in_message":[],"_revision_number":1},{"id":"62d59347805b4b4f02e4dca3e20e81c83ee66f23","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-15 17:31:28.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/15/397815/1/check/gate-python-novaclient-docs-ubuntu-xenial/bf0ffa1//doc/build/html/ : SUCCESS in 1m 28s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/15/397815/1/check/gate-python-novaclient-pep8-ubuntu-xenial/79316d8/ : SUCCESS in 1m 45s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/15/397815/1/check/gate-python-novaclient-python27-ubuntu-xenial/b3c5916/ : SUCCESS in 1m 49s\n- gate-python-novaclient-python34 http://logs.openstack.org/15/397815/1/check/gate-python-novaclient-python34/437fb22/ : SUCCESS in 2m 51s\n- gate-python-novaclient-python35-nv http://logs.openstack.org/15/397815/1/check/gate-python-novaclient-python35-nv/d925883/ : SUCCESS in 2m 14s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/15/397815/1/check/gate-tempest-dsvm-neutron-src-python-novaclient/25584e3/ : SUCCESS in 44m 06s\n- gate-novaclient-dsvm-functional-mitaka-nv http://logs.openstack.org/15/397815/1/check/gate-novaclient-dsvm-functional-mitaka-nv/1af063d/ : SUCCESS in 25m 12s (non-voting)\n- gate-novaclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/15/397815/1/check/gate-novaclient-dsvm-functional-identity-v3-only-nv/d66951d/ : SUCCESS in 36m 52s (non-voting)\n- gate-novaclient-dsvm-functional-neutron http://logs.openstack.org/15/397815/1/check/gate-novaclient-dsvm-functional-neutron/5f79811/ : SUCCESS in 1h 05m 27s","accounts_in_message":[],"_revision_number":1},{"id":"29d47ffa649b6633eb3b55efbcabd57fdc1de9f2","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-11-22 16:10:34.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"e95085029c2813a28440c6b6e75a4d8a07fef2df","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-11-22 16:23:09.000000000","message":"Uploaded patch set 3: Commit message was updated.","accounts_in_message":[],"_revision_number":3},{"id":"39ae099ad2d9b0d17893f5944c4f27ed4312a513","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-22 17:31:11.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/15/397815/3/check/gate-python-novaclient-docs-ubuntu-xenial/d2b2a5b//doc/build/html/ : SUCCESS in 2m 09s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/15/397815/3/check/gate-python-novaclient-pep8-ubuntu-xenial/c2c33c9/ : SUCCESS in 1m 46s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/15/397815/3/check/gate-python-novaclient-python27-ubuntu-xenial/8f6983f/ : SUCCESS in 1m 43s\n- gate-python-novaclient-python34 http://logs.openstack.org/15/397815/3/check/gate-python-novaclient-python34/212a7a9/ : SUCCESS in 4m 37s\n- gate-python-novaclient-python35-nv http://logs.openstack.org/15/397815/3/check/gate-python-novaclient-python35-nv/cdf1575/ : SUCCESS in 1m 57s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/15/397815/3/check/gate-tempest-dsvm-neutron-src-python-novaclient/9e2eecc/ : SUCCESS in 1h 07m 49s\n- gate-novaclient-dsvm-functional-mitaka-nv http://logs.openstack.org/15/397815/3/check/gate-novaclient-dsvm-functional-mitaka-nv/89b5f53/ : SUCCESS in 27m 32s (non-voting)\n- gate-novaclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/15/397815/3/check/gate-novaclient-dsvm-functional-identity-v3-only-nv/a22f337/ : SUCCESS in 37m 25s (non-voting)\n- gate-novaclient-dsvm-functional-neutron http://logs.openstack.org/15/397815/3/check/gate-novaclient-dsvm-functional-neutron/5758fcc/ : SUCCESS in 36m 59s","accounts_in_message":[],"_revision_number":3},{"id":"3c4abeef2f16cbb751202374ab511c5bae595e81","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-11-22 17:39:01.000000000","message":"Patch Set 3: Code-Review+2\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"4967b16c80291d9b14d77a2835938e4ce35fad1b","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-11-22 17:49:50.000000000","message":"Patch Set 4: Published edit on patch set 3","accounts_in_message":[],"_revision_number":4},{"id":"78c30f7bb7957e11df699ce100333e2c35bb7afb","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-11-22 17:50:18.000000000","message":"Patch Set 4:\n\n\u003e (2 comments)\n\nThanks!","accounts_in_message":[],"_revision_number":4},{"id":"9ea66f812b053a1927e8eb45edbe6b5cd5bd7440","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-11-22 18:51:53.000000000","message":"Patch Set 4: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"459f2301cf364485ae9b2395577cc0aefaa51625","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-22 18:57:54.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/15/397815/4/check/gate-python-novaclient-docs-ubuntu-xenial/93c82ae//doc/build/html/ : SUCCESS in 2m 27s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/15/397815/4/check/gate-python-novaclient-pep8-ubuntu-xenial/f983666/ : SUCCESS in 2m 50s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/15/397815/4/check/gate-python-novaclient-python27-ubuntu-xenial/e20eefd/ : SUCCESS in 2m 55s\n- gate-python-novaclient-python34 http://logs.openstack.org/15/397815/4/check/gate-python-novaclient-python34/ba2f137/ : SUCCESS in 1m 49s\n- gate-python-novaclient-python35-nv http://logs.openstack.org/15/397815/4/check/gate-python-novaclient-python35-nv/1001c6f/ : SUCCESS in 2m 46s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/15/397815/4/check/gate-tempest-dsvm-neutron-src-python-novaclient/0fd9263/ : SUCCESS in 1h 05m 55s\n- gate-novaclient-dsvm-functional-mitaka-nv http://logs.openstack.org/15/397815/4/check/gate-novaclient-dsvm-functional-mitaka-nv/9084893/ : SUCCESS in 28m 41s (non-voting)\n- gate-novaclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/15/397815/4/check/gate-novaclient-dsvm-functional-identity-v3-only-nv/ff131b9/ : SUCCESS in 39m 49s (non-voting)\n- gate-novaclient-dsvm-functional-neutron http://logs.openstack.org/15/397815/4/check/gate-novaclient-dsvm-functional-neutron/035f5db/ : SUCCESS in 37m 14s","accounts_in_message":[],"_revision_number":4},{"id":"bcad6e25ecb6ef793c04bde67074fa5e9ace55a8","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-11-23 12:32:08.000000000","message":"Patch Set 4:\n\n\u003e (1 comment)\n\nThanks!","accounts_in_message":[],"_revision_number":4},{"id":"91b80e2698e2976773046d6217bdaf7ed233326c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-23 19:15:54.000000000","message":"Patch Set 4: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":4},{"id":"12c03f44ff73914dada07c80719fe0cd9ea2ed95","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-11-24 05:44:46.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"6aee93b08298a2a4c7fa1a16c62fdc073c9fcce3","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-11-24 05:47:53.000000000","message":"Patch Set 5:\n\n\u003e (1 comment)\n\n@Kevin L. Mitchell I fix merge conflict issue and also replace other environment `ALL_TENANTS` with `NOVA_ALL_TENANTS`, it may need you review again, thanks.","accounts_in_message":[],"_revision_number":5},{"id":"37a1e04c895a88511728a63bc9a5de10b9988e6c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-11-24 06:52:17.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/15/397815/5/check/gate-python-novaclient-docs-ubuntu-xenial/2990ec4//doc/build/html/ : SUCCESS in 1m 20s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/15/397815/5/check/gate-python-novaclient-pep8-ubuntu-xenial/ccec8b9/ : SUCCESS in 2m 15s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/15/397815/5/check/gate-python-novaclient-python27-ubuntu-xenial/58fb810/ : SUCCESS in 3m 31s\n- gate-python-novaclient-python34 http://logs.openstack.org/15/397815/5/check/gate-python-novaclient-python34/3890f68/ : SUCCESS in 3m 10s\n- gate-python-novaclient-python35-nv http://logs.openstack.org/15/397815/5/check/gate-python-novaclient-python35-nv/79cd8c3/ : SUCCESS in 1m 38s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient http://logs.openstack.org/15/397815/5/check/gate-tempest-dsvm-neutron-src-python-novaclient/363cb63/ : SUCCESS in 43m 22s\n- gate-novaclient-dsvm-functional-mitaka-nv http://logs.openstack.org/15/397815/5/check/gate-novaclient-dsvm-functional-mitaka-nv/ba33663/ : SUCCESS in 37m 15s (non-voting)\n- gate-novaclient-dsvm-functional-identity-v3-only-nv http://logs.openstack.org/15/397815/5/check/gate-novaclient-dsvm-functional-identity-v3-only-nv/9e44149/ : SUCCESS in 1h 07m 17s (non-voting)\n- gate-novaclient-dsvm-functional-neutron http://logs.openstack.org/15/397815/5/check/gate-novaclient-dsvm-functional-neutron/64de073/ : SUCCESS in 54m 38s","accounts_in_message":[],"_revision_number":5},{"id":"197fd84f4d070820b879d318baa06c0e0d6ebaa3","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-11-28 14:38:41.000000000","message":"Patch Set 5: Code-Review+2","accounts_in_message":[],"_revision_number":5},{"id":"9ae6b3f98f522d64fcc0dee67d07a2886912533e","author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"date":"2016-11-30 11:10:47.000000000","message":"Patch Set 5: Code-Review-1\n\n(4 comments)\n\nFor a new feature like this, can we please add a reno release note:\nhttp://docs.openstack.org/developer/nova/code-review.html#when-a-release-note-is-needed\n\nI don\u0027t really like how you list all servers, I believe what you really want to do is include the all_tenants\u003dTrue flag when you are searching for the named server:\nhttps://github.com/openstack/nova/blob/master/nova/api/openstack/compute/servers.py#L311\n\nWhen you have an admin token, the get for a specific uuid should work just fine without the need for additional flags. So I believe its only the name search change you need to add here.","accounts_in_message":[],"_revision_number":5},{"id":"a1d38028eebfd886abfc2a468a2f3caded0a61b4","author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"date":"2016-11-30 16:17:02.000000000","message":"Patch Set 5: -Code-Review\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"a4c94196fb99e8c5187e1ee203d02b1069022b9e","author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"date":"2016-12-01 02:02:44.000000000","message":"Patch Set 5: Code-Review-1\n\nI don\u0027t like this also. I talk with int32bit. The use-case he work on is that the deployer enable --all-tenant parameter to the non-admin user by policy config. But the policy config can\u0027t enable non-admin user to show other project\u0027s server. There is limit in the db layer, only admin can do that.\n\nSo this patch is trying to implement similar thing with list servers.\n\nI\u0027m really not sure this good way.","accounts_in_message":[],"_revision_number":5},{"id":"e430a9652f0b01d2e881e0859d5719b297d8be23","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-12-01 02:26:25.000000000","message":"Patch Set 5:\n\n(2 comments)\n\nI admit this is not a very good way but may be a last chance to get the server. It shouldn\u0027t be paged as both \u0027limit\u0027 and \u0027marker\u0027 are None in this place by default.\nWe create a readonly-admin role, all users in this role can list all servers in any project implemented by policy. But our users complain that  they can\u0027t show some servers they list.\nOur client can get all servers from NOVA API, why not get one?","accounts_in_message":[],"_revision_number":5},{"id":"5aa8eb5df6285a88ede0c364f2febef133869c55","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-07 17:50:32.000000000","message":"Patch Set 5: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":5},{"id":"870bf22280840ed3b0446dff6ed27ac3e0d13c21","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2016-12-08 09:30:46.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"7a56f267d2345f11be82d392de29b64d9c620eac","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-08 10:41:13.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-novaclient-docs-ubuntu-xenial http://docs-draft.openstack.org/15/397815/6/check/gate-python-novaclient-docs-ubuntu-xenial/c21da31//doc/build/html/ : SUCCESS in 2m 10s\n- gate-python-novaclient-pep8-ubuntu-xenial http://logs.openstack.org/15/397815/6/check/gate-python-novaclient-pep8-ubuntu-xenial/b65c034/ : SUCCESS in 2m 46s\n- gate-python-novaclient-python27-ubuntu-xenial http://logs.openstack.org/15/397815/6/check/gate-python-novaclient-python27-ubuntu-xenial/2d50513/ : SUCCESS in 2m 39s\n- gate-python-novaclient-python34 http://logs.openstack.org/15/397815/6/check/gate-python-novaclient-python34/a4c8d6a/ : SUCCESS in 1m 52s\n- gate-python-novaclient-python35-nv http://logs.openstack.org/15/397815/6/check/gate-python-novaclient-python35-nv/2186336/ : SUCCESS in 2m 59s (non-voting)\n- gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial http://logs.openstack.org/15/397815/6/check/gate-tempest-dsvm-neutron-src-python-novaclient-ubuntu-xenial/98753aa/ : SUCCESS in 48m 13s\n- gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv http://logs.openstack.org/15/397815/6/check/gate-novaclient-dsvm-functional-mitaka-ubuntu-trusty-nv/c893285/ : SUCCESS in 27m 24s (non-voting)\n- gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/15/397815/6/check/gate-novaclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/946b591/ : SUCCESS in 1h 08m 09s (non-voting)\n- gate-novaclient-dsvm-functional-neutron-ubuntu-xenial http://logs.openstack.org/15/397815/6/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/2b99fbc/ : SUCCESS in 1h 06m 25s","accounts_in_message":[],"_revision_number":6},{"id":"9164271ce0ffe3a43195f389b09cffc621c11435","author":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"date":"2017-08-23 02:40:51.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"3db9df1b05398dc422490a3576bac7ada0b1cc09","revisions":{"0d0cd3b70ff91fd4fee07d763c1c2bf2d0e2bc0b":{"kind":"REWORK","_number":1,"created":"2016-11-15 16:25:50.000000000","uploader":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"ref":"refs/changes/15/397815/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/15/397815/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/1"}}},"commit":{"parents":[{"commit":"ccae3f0951b9ad956c6fef0aa572dc1d6dd51659","subject":"Merge \"Replace oslo_utils.timeutils.isotime\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/ccae3f0951b9ad956c6fef0aa572dc1d6dd51659"}]}],"author":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-15 03:06:03.000000000","tz":480},"committer":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-15 16:25:46.000000000","tz":480},"subject":"Show non-owner project servers","message":"Show non-owner project servers\n\nThis feature adds the possibility to show a server which is not\ncurrent context project if permitted. The proposal is to add a new\nflag ``--all-tenants`` in ``nova show`` subcommand. If ``--all-tenants``\nis set to ``True``, we will find a server from all projects by id or\nname.\n\nbp show-server-all-tenants\n\nChange-Id: Ie46b0e89723022c12ff2dae301f248c56178b2ad\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/0d0cd3b70ff91fd4fee07d763c1c2bf2d0e2bc0b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/0d0cd3b70ff91fd4fee07d763c1c2bf2d0e2bc0b"}]},"branch":"refs/heads/master"},"efea242beb618c43ab7cc9fb6b51499baa5a5f47":{"kind":"REWORK","_number":2,"created":"2016-11-22 16:10:34.000000000","uploader":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"ref":"refs/changes/15/397815/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/15/397815/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/2"}}},"commit":{"parents":[{"commit":"ccae3f0951b9ad956c6fef0aa572dc1d6dd51659","subject":"Merge \"Replace oslo_utils.timeutils.isotime\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/ccae3f0951b9ad956c6fef0aa572dc1d6dd51659"}]}],"author":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-15 03:06:03.000000000","tz":480},"committer":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-22 16:10:12.000000000","tz":480},"subject":"Show non-owner project servers","message":"Show non-owner project servers\n\nThis feature adds the possibility to show a server which is not\ncurrent context project if permitted. The proposal is to add a new\nflag ``--all-tenants`` in ``nova show`` subcommand. If ``--all-tenants``\nis set to ``True``, we will find a server from all projects by id or\nname.\n\nbp show-server-all-tenants\n\nChange-Id: Ie46b0e89723022c12ff2dae301f248c56178b2ad\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/efea242beb618c43ab7cc9fb6b51499baa5a5f47"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/efea242beb618c43ab7cc9fb6b51499baa5a5f47"}]},"branch":"refs/heads/master"},"0144811235f15f01c5c35d89fba043c83cc41c73":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2016-11-22 16:23:09.000000000","uploader":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"ref":"refs/changes/15/397815/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/15/397815/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/3"}}},"commit":{"parents":[{"commit":"ccae3f0951b9ad956c6fef0aa572dc1d6dd51659","subject":"Merge \"Replace oslo_utils.timeutils.isotime\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/ccae3f0951b9ad956c6fef0aa572dc1d6dd51659"}]}],"author":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-15 03:06:03.000000000","tz":480},"committer":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-22 16:21:52.000000000","tz":480},"subject":"Show other project servers using --all-tenants option","message":"Show other project servers using --all-tenants option\n\nWe can list all servers using `--all-tenants`, but we can\u0027t show\nthe server we list if not in current project event though we have enough\npermisions. It seems inconsistent and weird.\n\nThis feature adds the possibility to show a server which is not\ncurrent context project if permitted. The proposal is to add a new\nflag ``--all-tenants`` in ``nova show`` subcommand. If ``--all-tenants``\nis present, we will find a server from all projects by id or name.\n\nbp show-server-all-tenants\n\nChange-Id: Ie46b0e89723022c12ff2dae301f248c56178b2ad\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/0144811235f15f01c5c35d89fba043c83cc41c73"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/0144811235f15f01c5c35d89fba043c83cc41c73"}]},"branch":"refs/heads/master"},"26077c507d8a2c716cf786ee0e1d6afa58b9f64a":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2016-11-22 17:49:50.000000000","uploader":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"ref":"refs/changes/15/397815/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/15/397815/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/4"}}},"commit":{"parents":[{"commit":"ccae3f0951b9ad956c6fef0aa572dc1d6dd51659","subject":"Merge \"Replace oslo_utils.timeutils.isotime\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/ccae3f0951b9ad956c6fef0aa572dc1d6dd51659"}]}],"author":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-15 03:06:03.000000000","tz":480},"committer":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-22 17:48:33.000000000","tz":0},"subject":"Show other project servers using --all-tenants option","message":"Show other project servers using --all-tenants option\n\nWe can list all servers using `--all-tenants`, but we can\u0027t show\nthe server we list if not in current project even though we have enough\npermision. It seems inconsistent and weird.\n\nThis feature adds the possibility to show a server which is not\ncurrent context project if permitted. The proposal is to add a new\nflag ``--all-tenants`` in ``nova show`` subcommand. If ``--all-tenants``\nis present, we will find a server from all projects by id or name.\n\nbp show-server-all-tenants\n\nChange-Id: Ie46b0e89723022c12ff2dae301f248c56178b2ad\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/26077c507d8a2c716cf786ee0e1d6afa58b9f64a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/26077c507d8a2c716cf786ee0e1d6afa58b9f64a"}]},"branch":"refs/heads/master"},"dc7967096992b560b5562888070c64cb5dc5a6ed":{"kind":"REWORK","_number":5,"created":"2016-11-24 05:44:46.000000000","uploader":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"ref":"refs/changes/15/397815/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/15/397815/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/5"}}},"commit":{"parents":[{"commit":"94aec599c64e7c7bcc14f6fca2f5131fd8fb67bf","subject":"Merge \"Remove unused test code\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/94aec599c64e7c7bcc14f6fca2f5131fd8fb67bf"}]}],"author":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-15 03:06:03.000000000","tz":480},"committer":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-24 05:43:06.000000000","tz":480},"subject":"Show other project servers using --all-tenants option","message":"Show other project servers using --all-tenants option\n\nWe can list all servers using `--all-tenants`, but we can\u0027t show\nthe server we list if not in current project event though we have enough\npermissions. It seems inconsistent and weird.\n\nThis feature adds the possibility to show a server which is not\ncurrent context project if permitted. The proposal is to add a new\nflag ``--all-tenants`` in ``nova show`` subcommand. If ``--all-tenants``\nis present, we will find a server from all projects by id or name.\n\nbp show-server-all-tenants\n\nChange-Id: Ie46b0e89723022c12ff2dae301f248c56178b2ad\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/dc7967096992b560b5562888070c64cb5dc5a6ed"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/dc7967096992b560b5562888070c64cb5dc5a6ed"}]},"branch":"refs/heads/master"},"3db9df1b05398dc422490a3576bac7ada0b1cc09":{"kind":"REWORK","_number":6,"created":"2016-12-08 09:30:46.000000000","uploader":{"_account_id":21479,"name":"int32bit","email":"krystism@gmail.com","username":"int32bit"},"ref":"refs/changes/15/397815/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-novaclient","ref":"refs/changes/15/397815/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-novaclient refs/changes/15/397815/6"}}},"commit":{"parents":[{"commit":"ac2beb20af16898f4d15dd37e75b9cde5026dfd1","subject":"Merge \"Updated from global requirements\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/ac2beb20af16898f4d15dd37e75b9cde5026dfd1"}]}],"author":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-11-15 03:06:03.000000000","tz":480},"committer":{"name":"int32bit","email":"krystism@gmail.com","date":"2016-12-08 09:30:12.000000000","tz":480},"subject":"Fix can\u0027t show other tenant\u0027s server error","message":"Fix can\u0027t show other tenant\u0027s server error\n\nWe can list all servers using `--all-tenants`, but we can\u0027t show\nthe server we list if not in current project event though we have enough\npermissions. It seems inconsistent and unreasonable.\n\nThis patch adds the possibility to show a server which is not\ncurrent context project if permitted. If ``--all-tenants`` option\nis present, we will find a server from all projects by id or name.\n\nCloses-Bug: #1645929\n\nChange-Id: Ie46b0e89723022c12ff2dae301f248c56178b2ad\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/3db9df1b05398dc422490a3576bac7ada0b1cc09"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-novaclient/commit/3db9df1b05398dc422490a3576bac7ada0b1cc09"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}