)]}'
{"id":"openstack%2Fpython-ceilometerclient~170710","triplet_id":"openstack%2Fpython-ceilometerclient~master~I713ab2480d3e8cb2d1441a88405a9ffe8eedf447","project":"openstack/python-ceilometerclient","branch":"master","topic":"bug/1440463","hashtags":[],"change_id":"I713ab2480d3e8cb2d1441a88405a9ffe8eedf447","subject":"fix wrong required option tenant-id","status":"ABANDONED","created":"2015-04-05 09:43:37.000000000","updated":"2016-01-18 10:33:40.000000000","total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"27e883f5d104a6baa311a4668fbbb47cc4ac88f9","_number":170710,"virtual_id_number":170710,"owner":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":-1,"date":"2015-04-14 05:36:23.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"}],"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":{"disliked":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"value":-1,"date":"2015-04-09 03:40:23.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"}],"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":{"rejected":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"all":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2015-04-09 05:12:53.000000000","permitted_voting_range":{"min":-1,"max":0},"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-04-09 03:40:23.000000000","updated_by":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"reviewer":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"state":"REVIEWER"},{"updated":"2015-04-14 05:36:23.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"ef69ae397e3c7d8dcde2bcbd7859993fbd03ec75","author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"date":"2015-04-05 09:43:37.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"43758171d8ae168283ca7a0a878febeb570fde8e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-05 10:43:55.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-ceilometerclient-pep8 http://logs.openstack.org/10/170710/1/check/gate-python-ceilometerclient-pep8/7bbe16a/ : SUCCESS in 43s\n- gate-python-ceilometerclient-docs http://docs-draft.openstack.org/10/170710/1/check/gate-python-ceilometerclient-docs/34bd3dc//doc/build/html/ : SUCCESS in 1m 50s\n- gate-python-ceilometerclient-python27 http://logs.openstack.org/10/170710/1/check/gate-python-ceilometerclient-python27/6194aee/ : SUCCESS in 47s\n- gate-python-ceilometerclient-python34 http://logs.openstack.org/10/170710/1/check/gate-python-ceilometerclient-python34/1b66efc/ : SUCCESS in 2m 12s\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient-icehouse http://logs.openstack.org/10/170710/1/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient-icehouse/2ae1296/ : SUCCESS in 25m 03s\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient-juno http://logs.openstack.org/10/170710/1/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient-juno/1e49716/ : SUCCESS in 40m 48s\n- gate-python-ceilometerclient-pypy http://logs.openstack.org/10/170710/1/check/gate-python-ceilometerclient-pypy/877f469/ : SUCCESS in 1m 28s\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient http://logs.openstack.org/10/170710/1/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient/1a7b56e/ : SUCCESS in 1h 00m 11s\n- check-ceilometerclient-dsvm-functional http://logs.openstack.org/10/170710/1/check/check-ceilometerclient-dsvm-functional/4945d94/ : SUCCESS in 18m 28s","accounts_in_message":[],"_revision_number":1},{"id":"d8dbd1f2420648673e72266f82f045016b984bc8","author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"date":"2015-04-07 03:01:49.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"51884a5c4916697ab5653465f64ad1dbd4f5d972","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-07 03:55:35.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-ceilometerclient-pep8 http://logs.openstack.org/10/170710/2/check/gate-python-ceilometerclient-pep8/cdedb5a/ : SUCCESS in 50s\n- gate-python-ceilometerclient-docs http://docs-draft.openstack.org/10/170710/2/check/gate-python-ceilometerclient-docs/a6e3c65//doc/build/html/ : SUCCESS in 42s\n- gate-python-ceilometerclient-python27 http://logs.openstack.org/10/170710/2/check/gate-python-ceilometerclient-python27/5eb36a8/ : SUCCESS in 57s\n- gate-python-ceilometerclient-python34 http://logs.openstack.org/10/170710/2/check/gate-python-ceilometerclient-python34/3ff8703/ : SUCCESS in 1m 57s\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient-icehouse http://logs.openstack.org/10/170710/2/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient-icehouse/c1c86f7/ : SUCCESS in 27m 07s\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient-juno http://logs.openstack.org/10/170710/2/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient-juno/c4ea67b/ : SUCCESS in 41m 59s\n- gate-python-ceilometerclient-pypy http://logs.openstack.org/10/170710/2/check/gate-python-ceilometerclient-pypy/080adbc/ : SUCCESS in 1m 29s\n- gate-tempest-dsvm-neutron-src-python-ceilometerclient http://logs.openstack.org/10/170710/2/check/gate-tempest-dsvm-neutron-src-python-ceilometerclient/c2acf60/ : SUCCESS in 52m 19s\n- check-ceilometerclient-dsvm-functional http://logs.openstack.org/10/170710/2/check/check-ceilometerclient-dsvm-functional/02f0429/ : SUCCESS in 18m 40s","accounts_in_message":[],"_revision_number":2},{"id":"43a4629cfc6837fc03091a06e9e8effba1200064","author":{"_account_id":4491,"name":"Lianhao Lu","email":"llh_misc@outlook.com","username":"lianhao-lu"},"date":"2015-04-09 03:40:23.000000000","message":"Patch Set 2: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"3ce73b3c7712c51cb6ac875e8b719eecbcefa19c","author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"date":"2015-04-09 05:12:53.000000000","message":"Patch Set 2: Workflow-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"5bee4a49fa4e78858a92445515599709d87bd970","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-14 05:36:23.000000000","message":"Patch Set 2: Verified-1\n\nMerge Failed.\n\nThis change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset.","accounts_in_message":[],"_revision_number":2},{"id":"419a9ec155649c698380cebf8426fcd260bfcaff","author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"date":"2016-01-18 10:33:40.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"0465392a8f9df73d0f7d42fe69adc5941dad74bf","revisions":{"d3d8a12cf340f1104e9caaca51492754c5ab046d":{"kind":"REWORK","_number":1,"created":"2015-04-05 09:43:37.000000000","uploader":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"ref":"refs/changes/10/170710/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-ceilometerclient","ref":"refs/changes/10/170710/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/10/170710/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/10/170710/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/10/170710/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-ceilometerclient refs/changes/10/170710/1"}}},"commit":{"parents":[{"commit":"c40d67cf97d9e4bfdde4e6d3ee28a31d61c8cafe","subject":"Merge \"alarm: Use new gnocchi aggregation API\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/c40d67cf97d9e4bfdde4e6d3ee28a31d61c8cafe"}]}],"author":{"name":"ZhiQiang Fan","email":"zhiqiang.fan@huawei.com","date":"2015-04-05 09:36:07.000000000","tz":480},"committer":{"name":"ZhiQiang Fan","email":"zhiqiang.fan@huawei.com","date":"2015-04-05 09:43:22.000000000","tz":480},"subject":"WIP: fix wrong required option tenant-id","message":"WIP: fix wrong required option tenant-id\n\nKeystone API v2 has been deprecated, and we have already supported Keystone\nAPI v3, but when we run ceilometer CLI command with Keystone API v3, it\ncomplains that tenant id is required, however if we do provide tenant id,\nKeystone API v3 doesn\u0027t accept it.\n\nThis patch fixes the incorrect check for required options.\n\nChange-Id: I713ab2480d3e8cb2d1441a88405a9ffe8eedf447\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/d3d8a12cf340f1104e9caaca51492754c5ab046d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/d3d8a12cf340f1104e9caaca51492754c5ab046d"}]},"branch":"refs/heads/master"},"0465392a8f9df73d0f7d42fe69adc5941dad74bf":{"kind":"REWORK","_number":2,"created":"2015-04-07 03:01:49.000000000","uploader":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"ref":"refs/changes/10/170710/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-ceilometerclient","ref":"refs/changes/10/170710/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/10/170710/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/10/170710/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-ceilometerclient refs/changes/10/170710/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-ceilometerclient refs/changes/10/170710/2"}}},"commit":{"parents":[{"commit":"c40d67cf97d9e4bfdde4e6d3ee28a31d61c8cafe","subject":"Merge \"alarm: Use new gnocchi aggregation API\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/c40d67cf97d9e4bfdde4e6d3ee28a31d61c8cafe"}]}],"author":{"name":"ZhiQiang Fan","email":"zhiqiang.fan@huawei.com","date":"2015-04-05 09:36:07.000000000","tz":480},"committer":{"name":"ZhiQiang Fan","email":"zhiqiang.fan@huawei.com","date":"2015-04-07 02:59:47.000000000","tz":480},"subject":"fix wrong required option tenant-id","message":"fix wrong required option tenant-id\n\nKeystone API v2 has been deprecated, and we have already supported Keystone\nAPI v3, but when we use ceilometerclient with Keystone API v3, it\ncomplains that tenant id is required, even we\u0027ve already provide project information.\n\nWe should support Keystone API v3 expected parameter: project_id, or project_name\nwith domain information. And remove the required tenant-id in Keystone API v3\nscenario, because that is only used by Keystone API v2, and now we should encourage\nuser to use Keystone API v3.\n\nThis patch fixes:\n\n* the incorrect check for required options\n* support project related parameters for non-shell case\n* replace tenant parameters with project parameters in get_auth_plugin()\n\nChange-Id: I713ab2480d3e8cb2d1441a88405a9ffe8eedf447\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/0465392a8f9df73d0f7d42fe69adc5941dad74bf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-ceilometerclient/commit/0465392a8f9df73d0f7d42fe69adc5941dad74bf"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}