)]}'
{"id":"openstack%2Fpython-cinderclient~641310","triplet_id":"openstack%2Fpython-cinderclient~master~Ice7d1c40dd794eef3a4b9744b661a5a690d5b911","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1808621","hashtags":[],"change_id":"Ice7d1c40dd794eef3a4b9744b661a5a690d5b911","subject":"Correct the \u0027get resource\u0027 behavior for non-admin user","status":"ABANDONED","created":"2019-03-06 10:37:50.000000000","updated":"2019-03-07 18:42:58.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0a3ed8b5f0353e31a0fa671a0ff5bea5e4b8638a","_number":641310,"virtual_id_number":641310,"owner":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":1,"date":"2019-03-06 12:21:22.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-03-06 12:21:22.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"cf7e3d118dc6ec2b96165fc02e6abe44d5700f2a","author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"date":"2019-03-06 10:37:50.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"2f1b2309993a6fd3f501c7157c70fee7e85a317e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-03-06 12:21:22.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full http://logs.openstack.org/10/641310/1/check/tempest-full/65d7218/ : SUCCESS in 1h 31m 34s\n- tempest-full-py3 http://logs.openstack.org/10/641310/1/check/tempest-full-py3/57399b6/ : SUCCESS in 1h 36m 21s\n- openstack-tox-cover http://logs.openstack.org/10/641310/1/check/openstack-tox-cover/e5ae232/cover/ : SUCCESS in 3m 41s\n- openstack-tox-lower-constraints http://logs.openstack.org/10/641310/1/check/openstack-tox-lower-constraints/fa14dd9/ : SUCCESS in 4m 00s\n- openstack-tox-pep8 http://logs.openstack.org/10/641310/1/check/openstack-tox-pep8/022a486/ : SUCCESS in 3m 19s\n- openstack-tox-py27 http://logs.openstack.org/10/641310/1/check/openstack-tox-py27/69a55e3/ : SUCCESS in 3m 25s\n- openstack-tox-py36 http://logs.openstack.org/10/641310/1/check/openstack-tox-py36/13cf3c6/ : SUCCESS in 3m 09s\n- openstack-tox-py37 http://logs.openstack.org/10/641310/1/check/openstack-tox-py37/f77be6e/ : SUCCESS in 4m 39s\n- openstack-tox-docs http://logs.openstack.org/10/641310/1/check/openstack-tox-docs/5769c20/html/ : SUCCESS in 2m 31s\n- cinderclient-dsvm-functional http://logs.openstack.org/10/641310/1/check/cinderclient-dsvm-functional/0bc2525/ : SUCCESS in 30m 14s (non-voting)\n- openstack-tox-pylint http://logs.openstack.org/10/641310/1/check/openstack-tox-pylint/9d4e08c/ : SUCCESS in 5m 29s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"017822b3341d48e9f844f9c7e6fd228774995f4e","author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"date":"2019-03-07 18:42:58.000000000","message":"Abandoned\n\nHandled at server side","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"aa5c13e2ce7d3c305c47a52be4683c223f7cce13","revisions":{"aa5c13e2ce7d3c305c47a52be4683c223f7cce13":{"kind":"REWORK","_number":1,"created":"2019-03-06 10:37:50.000000000","uploader":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"ref":"refs/changes/10/641310/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/10/641310/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/10/641310/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/10/641310/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/10/641310/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/10/641310/1"}}},"commit":{"parents":[{"commit":"8de1f789c5e4eef10c6757f4f1a6415deb6433a3","subject":"Drop py35 jobs","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8de1f789c5e4eef10c6757f4f1a6415deb6433a3"}]}],"author":{"name":"whoami-rajat","email":"rajatdhasmana@gmail.com","date":"2019-03-06 09:55:02.000000000","tz":330},"committer":{"name":"whoami-rajat","email":"rajatdhasmana@gmail.com","date":"2019-03-06 09:55:02.000000000","tz":330},"subject":"Correct the \u0027get resource\u0027 behavior for non-admin user","message":"Correct the \u0027get resource\u0027 behavior for non-admin user\n\nWhile performing any action on a resource (volume, snapshot etc),\nthe resource is fetched first with \u0027all_tenants\u003d1\u0027 filter.\nFor non-admin user this results in \u0027BadRequest invalid filters\u0027\nwhich is not the correct behaviour.\nThe correct behaviour should be the following :\n* The resource should be searched in the same\nproject(for non-admin users)\n* If the resource is not found, a NotFound exception should be\ngenerated rather than BadRequest\nEg: For revert-to-snapshot action, the response message should be\nsnapshot with name \u0027xyz\u0027 not found.\nThis patch adds the code for searching the resource again without\n\u0027all_tenants\u0027 option if BadRequest occurs.\n\nChange-Id: Ice7d1c40dd794eef3a4b9744b661a5a690d5b911\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/aa5c13e2ce7d3c305c47a52be4683c223f7cce13"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/aa5c13e2ce7d3c305c47a52be4683c223f7cce13"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
