)]}'
{"id":"openstack%2Fpython-keystoneclient~167543","triplet_id":"openstack%2Fpython-keystoneclient~master~Ie1a20a1e5e958a3133f0b86d1580a833e8928a43","project":"openstack/python-keystoneclient","branch":"master","topic":"bug/1417189","hashtags":[],"change_id":"Ie1a20a1e5e958a3133f0b86d1580a833e8928a43","subject":"Now keystone enables listing of user by name","status":"ABANDONED","created":"2015-03-25 10:18:25.000000000","updated":"2016-03-04 21:39:03.000000000","total_comment_count":14,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"5c22e788fd73758f70d015e93411ec2f1de7d408","_number":167543,"virtual_id_number":167543,"owner":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},{"value":-1,"date":"2016-01-07 03:59:54.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"date":"2015-05-29 16:08:16.000000000","_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},{"date":"2015-05-29 16:08:36.000000000","_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"}],"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":{"rejected":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"disliked":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-1,"date":"2015-09-15 09:13:29.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},{"value":-2,"date":"2015-04-14 14:14:41.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"value":-1,"date":"2015-04-14 09:31:17.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"}],"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":{"all":[{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"}],"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":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2015-03-28 16:03:38.000000000","updated_by":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"reviewer":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"state":"REVIEWER"},{"updated":"2015-03-28 16:04:12.000000000","updated_by":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"reviewer":{"_account_id":8978,"name":"Marek Denis","email":"marek.denis+openstack@gmail.com","username":"marek-denis"},"state":"REVIEWER"},{"updated":"2015-04-01 20:45:14.000000000","updated_by":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"reviewer":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"state":"REVIEWER"},{"updated":"2015-04-08 16:05:14.000000000","updated_by":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"reviewer":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"state":"REVIEWER"},{"updated":"2015-04-14 09:31:17.000000000","updated_by":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"reviewer":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"state":"REVIEWER"},{"updated":"2015-04-14 14:14:41.000000000","updated_by":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"reviewer":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"state":"REVIEWER"},{"updated":"2015-05-29 16:08:16.000000000","updated_by":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"reviewer":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"state":"REVIEWER"},{"updated":"2015-05-29 16:08:36.000000000","updated_by":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"reviewer":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"state":"REVIEWER"},{"updated":"2015-09-15 09:13:29.000000000","updated_by":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"reviewer":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"state":"REVIEWER"},{"updated":"2016-01-07 03:59:54.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"274cbe105b34173850cedef841015e5e45fe44ca","author":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"date":"2015-03-25 10:18:25.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a8be77909cd696a079be25b1ff4cc0e8b702894d","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2015-03-25 10:29:23.000000000","message":"Patch Set 1: Code-Review-1\n\nSo the keystone client cli is deprecated (and fozen), with all new developments going into the generalised openstack client (osc) - see: https://wiki.openstack.org/wiki/OpenStackClient/Commands#user\n\nI would advise looking this command to see if you still need improvements - and if so, please feel free to contribute to its code.","accounts_in_message":[],"_revision_number":1},{"id":"d88b46254804b55a64b25ac7f76817c8a8f1c586","author":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"date":"2015-03-25 10:45:07.000000000","message":"Patch Set 1:\n\nI could not see this feature in openstack client too. So, what should i do now? should i make changes into openstack client or keystone client and then to openstack client.","accounts_in_message":[],"_revision_number":1},{"id":"afdabd4d53d787c803f86ab5463ca1c897d16fc4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-25 11:10:37.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-keystoneclient-python26 http://logs.openstack.org/43/167543/1/check/gate-python-keystoneclient-python26/d242b76/ : SUCCESS in 2m 02s\n- gate-python-keystoneclient-pep8 http://logs.openstack.org/43/167543/1/check/gate-python-keystoneclient-pep8/72d992e/ : SUCCESS in 2m 26s\n- gate-python-keystoneclient-docs http://docs-draft.openstack.org/43/167543/1/check/gate-python-keystoneclient-docs/b6d3b6d//doc/build/html/ : SUCCESS in 3m 12s\n- gate-python-keystoneclient-python27 http://logs.openstack.org/43/167543/1/check/gate-python-keystoneclient-python27/b303c32/ : SUCCESS in 2m 39s\n- gate-python-keystoneclient-python34 http://logs.openstack.org/43/167543/1/check/gate-python-keystoneclient-python34/1f6fc93/ : SUCCESS in 3m 56s\n- gate-tempest-dsvm-neutron-src-python-keystoneclient-juno http://logs.openstack.org/43/167543/1/check/gate-tempest-dsvm-neutron-src-python-keystoneclient-juno/6a2bd80/ : SUCCESS in 45m 41s\n- gate-tempest-dsvm-neutron-src-python-keystoneclient http://logs.openstack.org/43/167543/1/check/gate-tempest-dsvm-neutron-src-python-keystoneclient/594a0d5/ : SUCCESS in 47m 27s\n- check-keystoneclient-dsvm-functional http://logs.openstack.org/43/167543/1/check/check-keystoneclient-dsvm-functional/f1c4547/ : SUCCESS in 19m 44s","accounts_in_message":[],"_revision_number":1},{"id":"49b8963554b098d5042f27942feaa3d345c6b5c4","author":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"date":"2015-03-25 13:36:27.000000000","message":"Patch Set 1: Code-Review-1\n\nSimilar situation to the feature proposed at https://review.openstack.org/#/c/140962/ You should provide this fix only into openstack-client, as keystone-client is deprecated now.","accounts_in_message":[],"_revision_number":1},{"id":"f6742a9072235007288e1ef2687d135b143cf036","author":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"date":"2015-03-26 11:02:45.000000000","message":"Patch Set 1:\n\nOpenstackclient uses keystoneclient\u0027s keystoneclient/v2_0/users.py , if i make changes only in openstackclient even then it won\u0027t work. I think, first, i need to change keystoneclient and once it is done then i should log a bug for openstackclient and then we can make changes to openstack client.","accounts_in_message":[],"_revision_number":1},{"id":"7d8b8b7ca766341746bb3bd4b310ac8481f635e4","author":{"_account_id":13161,"name":"Kamil Rykowski","email":"kamil.rykowski@gmail.com","username":"krykowski"},"date":"2015-03-26 13:39:01.000000000","message":"Patch Set 1:\n\n@rajiv: You are right! Only the CLI for keystoneclient is deprecated, and not the keystoneclient itself.","accounts_in_message":[],"_revision_number":1},{"id":"918d996f84b3911c835bea3b55558eea6afb64e8","author":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"date":"2015-03-27 03:17:48.000000000","message":"Patch Set 1:\n\ncan i please get it going? Can you please review and provide comments.","accounts_in_message":[],"_revision_number":1},{"id":"80f574ba4328aae88e8f8cd87ae6eeb2f3a4d939","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2015-03-29 12:10:44.000000000","message":"Patch Set 1:\n\nSo can I just check what the actual requirement here is: Do you want to do a \"get user\" or \"list users\"?  If you want to do get_user specifying a name, then you need to be aware that this is a V2 API call which is on the cusp of being deprecated in favour of v3.  If you want to do a list_users call in V3, then the API: GET /users?name\u003dxyz id the REST API you want to effectively issue.\n\nNow the openstack client, I think does actually support the V2 get_user_by_name call (at least that\u0027s what the code implies, I have not tried it).  The V3 list users call in openstack client does not appear to accept the filtering by user name....which would be a great addition...and do do that you would need to add it to the keystonecleint support (the \"library\" not the CLI)","accounts_in_message":[],"_revision_number":1},{"id":"dfdbda85e9378d22996b0e1861efa7221f391404","author":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"date":"2015-03-30 04:30:51.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"2f382a6ec64f7b3eac6655faadc55c47744e5ae3","author":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"date":"2015-03-30 04:37:18.000000000","message":"Patch Set 2:\n\nI have removes cli specific part and i have kept only library part. once these changes has been accepted, i will change openstackclient. I need to discuss one more thing, what if user specify both the tenant and name for listing. what should be the expected outcome in that case.","accounts_in_message":[],"_revision_number":2},{"id":"0d8ae53fc6d07183e5cb28bdbee2e2ad147aff65","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-03-30 05:21:33.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-keystoneclient-python26 http://logs.openstack.org/43/167543/2/check/gate-python-keystoneclient-python26/4a0533a/ : SUCCESS in 2m 19s\n- gate-python-keystoneclient-pep8 http://logs.openstack.org/43/167543/2/check/gate-python-keystoneclient-pep8/eb4e086/ : SUCCESS in 2m 06s\n- gate-python-keystoneclient-docs http://docs-draft.openstack.org/43/167543/2/check/gate-python-keystoneclient-docs/864d43b//doc/build/html/ : SUCCESS in 2m 54s\n- gate-python-keystoneclient-python27 http://logs.openstack.org/43/167543/2/check/gate-python-keystoneclient-python27/abf14f2/ : SUCCESS in 2m 03s\n- gate-python-keystoneclient-python34 http://logs.openstack.org/43/167543/2/check/gate-python-keystoneclient-python34/627f2f4/ : SUCCESS in 3m 47s\n- gate-tempest-dsvm-neutron-src-python-keystoneclient-juno http://logs.openstack.org/43/167543/2/check/gate-tempest-dsvm-neutron-src-python-keystoneclient-juno/540cac8/ : SUCCESS in 50m 38s\n- gate-tempest-dsvm-neutron-src-python-keystoneclient http://logs.openstack.org/43/167543/2/check/gate-tempest-dsvm-neutron-src-python-keystoneclient/84901cd/ : SUCCESS in 50m 35s\n- check-keystoneclient-dsvm-functional http://logs.openstack.org/43/167543/2/check/check-keystoneclient-dsvm-functional/8c4a7b3/ : SUCCESS in 18m 49s","accounts_in_message":[],"_revision_number":2},{"id":"42222e3274148b8c4bf06f5ccdac36f4d139db66","author":{"_account_id":7191,"name":"Jamie Lennox","email":"jamielennox@gmail.com","username":"jamielennox"},"date":"2015-04-01 20:45:14.000000000","message":"Patch Set 2: Code-Review-1\n\n(3 comments)\n\nWill need some tests as well. Thanks.","accounts_in_message":[],"_revision_number":2},{"id":"f76249cbd190db331116b6dcdcffbf529070f783","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2015-04-02 10:05:57.000000000","message":"Patch Set 2: Code-Review-1\n\nI am still unsure as to why we are doing this in the v2 client bit...why not in the v3 client bit....the v2 API will deprecated real soon....","accounts_in_message":[],"_revision_number":2},{"id":"4d888d451a9be0ca599ba822f7c0698501ecf4b6","author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"date":"2015-04-08 16:05:14.000000000","message":"Patch Set 2:\n\n(8 comments)","accounts_in_message":[],"_revision_number":2},{"id":"84d3af6c24e83119110263069b6e21d3c2e4fe3a","author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"date":"2015-04-08 16:05:21.000000000","message":"Patch Set 2: Code-Review-1","accounts_in_message":[],"_revision_number":2},{"id":"0d9eae54f6352f036998a7cddaf8cb0e3097c406","author":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"date":"2015-04-13 13:42:31.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"ae4ded854d8594cd3538550587da6fd23a26a939","author":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"date":"2015-04-13 13:44:56.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"9b77769591a6c3aedd4c5093b3e7fc72770421e9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2015-04-13 14:45:35.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-keystoneclient-python26 http://logs.openstack.org/43/167543/4/check/gate-python-keystoneclient-python26/b12829e/ : SUCCESS in 2m 08s\n- gate-python-keystoneclient-pep8 http://logs.openstack.org/43/167543/4/check/gate-python-keystoneclient-pep8/4906fa9/ : SUCCESS in 2m 09s\n- gate-python-keystoneclient-docs http://docs-draft.openstack.org/43/167543/4/check/gate-python-keystoneclient-docs/6b0f6bd//doc/build/html/ : SUCCESS in 2m 21s\n- gate-python-keystoneclient-python27 http://logs.openstack.org/43/167543/4/check/gate-python-keystoneclient-python27/a768da9/ : SUCCESS in 2m 25s\n- gate-python-keystoneclient-python34 http://logs.openstack.org/43/167543/4/check/gate-python-keystoneclient-python34/20611b0/ : SUCCESS in 3m 40s\n- gate-tempest-dsvm-neutron-src-python-keystoneclient http://logs.openstack.org/43/167543/4/check/gate-tempest-dsvm-neutron-src-python-keystoneclient/d63e7f8/ : SUCCESS in 58m 20s\n- check-keystoneclient-dsvm-functional http://logs.openstack.org/43/167543/4/check/check-keystoneclient-dsvm-functional/b158732/ : SUCCESS in 18m 55s","accounts_in_message":[],"_revision_number":4},{"id":"4f3744d75d49b06b498999704c646a16a64754d3","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2015-04-14 09:31:17.000000000","message":"Patch Set 4: Code-Review-1\n\nI still think we should not be doing this for v2.  We want people to use the v3 API....and that\u0027s where all the work for the openstackclient (osc) is going.  Is there really a requirement to use the v2 cli part of osc as well?","accounts_in_message":[],"_revision_number":4},{"id":"f677ad9d96f89ec3e96b700071b6a070e57917c2","author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"date":"2015-04-14 14:14:41.000000000","message":"Patch Set 4: Code-Review-2\n\nWe havea freeze on changes to the KC CLI.  PLease make this change in python-openstackclient","accounts_in_message":[],"_revision_number":4},{"id":"af748f35950b35365b6a3a3aabc76bb5127d8b05","author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"date":"2015-05-29 16:08:16.000000000","message":"Patch Set 4:\n\nAdam, can you please unblock? This is only client changes. NOT CLI.","accounts_in_message":[],"_revision_number":4},{"id":"271aeb248ff82dfa5404b5d4782b95bcb0bdc309","author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"date":"2015-05-29 16:08:36.000000000","message":"Patch Set 4:\n\nSince this change is not for the CLI part of the code I think the freeze doesn\u0027t apply","accounts_in_message":[],"_revision_number":4},{"id":"809e150b3c58d84aacc44f30c089629908c39716","author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"date":"2015-05-29 16:50:59.000000000","message":"Patch Set 4:\n\nI concur with others, this is not changing the old CLI so we shouldn\u0027t be blocking for that.  However, my earlier question has not been answered - which is why would be bother to change v2 API support...surely just providing this in the v3 commands in osc is sufficient?","accounts_in_message":[],"_revision_number":4},{"id":"f63e90f34ef7858e248098c8776dce6de73211fe","author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"date":"2015-05-29 17:08:37.000000000","message":"Patch Set 4:\n\nHenry, there\u0027s a huge performance cost right now if we don\u0027t do lookup by name at the client side. There are deployment tools out there that are still using V2 for initial user provisioning as the bug had indicated\n\nhttps://github.com/openstack/os-cloud-config/blob/master/os_cloud_config/keystone.py#L490\n\nI totally agreed that user should move to V3 ASAP. But I also think we have the obligation to support V2 till it is officially deprecated and removed.","accounts_in_message":[],"_revision_number":4},{"id":"a4dbeb85f8941e657b789f5a4decd56f7abd845b","author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"date":"2015-09-15 09:13:29.000000000","message":"Patch Set 4: Code-Review-1\n\n(3 comments)\n\nI really hope the API doesn\u0027t work this way. The behavior this client appears to be written against is not documented and would make zero sense.","accounts_in_message":[],"_revision_number":4},{"id":"c1b81bd56a7ccb7ff58fbd492e586e9025a2d643","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-01-07 03:59: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":"7f37bdeceeb15f5fc0782f607271b483712341b1","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-03-04 21:39:03.000000000","message":"Abandoned\n\nabandoning this because it has a bunch of negative reviews and no movement in 5 months.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"1502d7fb37626dba11e935ddfbf702d69f55d448","revisions":{"ed5e27ab2e991fbadfeac419f9a51506ff2f181b":{"kind":"REWORK","_number":1,"created":"2015-03-25 10:18:25.000000000","uploader":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"ref":"refs/changes/43/167543/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/43/167543/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/1"}}},"commit":{"parents":[{"commit":"a84b5ac115dd5775f7380a204811aa939fd5c242","subject":"Merge \"Add a FederatedBase v3 plugin\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/a84b5ac115dd5775f7380a204811aa939fd5c242"}]}],"author":{"name":"Rajiv Kumar","email":"rajiv.kumar@nectechnologies.in","date":"2015-03-25 10:02:50.000000000","tz":0},"committer":{"name":"Rajiv Kumar","email":"rajiv.kumar@nectechnologies.in","date":"2015-03-25 10:18:21.000000000","tz":0},"subject":"Now keystone enables listing of user by name","message":"Now keystone enables listing of user by name\n\nAPI support listing of user by name but keystone client does\nnot. It was required by user in some scenario(as described in bug\ndescription).\n\nFor supporting this feature, user option need to be added for help\nand displaying to user. It makes me to change shell.py.\n\ndo_user_list method uses list method of user.py, which need to be\nupdated according to the keystone API. In case of list of users,\nkeystone returns a dictionary having \u0027users\u0027 as key and list of\nall users as the value. But in case of display by name, we get,\na dictionary having \u0027user\u0027 as key and user details as value.\nWhich makes it necessary to pass \u0027user\u0027 as reponse key.\n\nIn base.py\u0027s _list method, in our case, we get a dictionary, which\nneeds to be convert to list before proceeding further. On the\nbasis of reponse_key, this decision is being made.\n\nChange-Id: Ie1a20a1e5e958a3133f0b86d1580a833e8928a43\nCloses-Bug: #1417189\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/ed5e27ab2e991fbadfeac419f9a51506ff2f181b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/ed5e27ab2e991fbadfeac419f9a51506ff2f181b"}]},"branch":"refs/heads/master"},"9ae038ad115b36a48ecde387ab7b12a5723794e1":{"kind":"REWORK","_number":2,"created":"2015-03-30 04:30:51.000000000","uploader":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"ref":"refs/changes/43/167543/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/43/167543/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/2"}}},"commit":{"parents":[{"commit":"a84b5ac115dd5775f7380a204811aa939fd5c242","subject":"Merge \"Add a FederatedBase v3 plugin\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/a84b5ac115dd5775f7380a204811aa939fd5c242"}]}],"author":{"name":"Rajiv Kumar","email":"rajiv.kumar@nectechnologies.in","date":"2015-03-25 10:02:50.000000000","tz":0},"committer":{"name":"Rajiv Kumar","email":"rajiv.kumar@nectechnologies.in","date":"2015-03-30 04:30:16.000000000","tz":330},"subject":"Now keystone enables listing of user by name","message":"Now keystone enables listing of user by name\n\nAPI support listing of user by name but keystone client does\nnot. It was required by user in some scenario(as described in bug\ndescription).\n\nFor supporting this feature, user option need to be added for help\nand displaying to user. It makes me to change shell.py.\n\ndo_user_list method uses list method of user.py, which need to be\nupdated according to the keystone API. In case of list of users,\nkeystone returns a dictionary having \u0027users\u0027 as key and list of\nall users as the value. But in case of display by name, we get,\na dictionary having \u0027user\u0027 as key and user details as value.\nWhich makes it necessary to pass \u0027user\u0027 as reponse key.\n\nIn base.py\u0027s _list method, in our case, we get a dictionary, which\nneeds to be convert to list before proceeding further. On the\nbasis of reponse_key, this decision is being made.\n\nChange-Id: Ie1a20a1e5e958a3133f0b86d1580a833e8928a43\nCloses-Bug: #1417189\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/9ae038ad115b36a48ecde387ab7b12a5723794e1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/9ae038ad115b36a48ecde387ab7b12a5723794e1"}]},"branch":"refs/heads/master"},"492876a4c9365d471a7502bd4ca14e5809e1785a":{"kind":"REWORK","_number":3,"created":"2015-04-13 13:42:31.000000000","uploader":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"ref":"refs/changes/43/167543/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/43/167543/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/3"}}},"commit":{"parents":[{"commit":"a84b5ac115dd5775f7380a204811aa939fd5c242","subject":"Merge \"Add a FederatedBase v3 plugin\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/a84b5ac115dd5775f7380a204811aa939fd5c242"}]}],"author":{"name":"Rajiv Kumar","email":"rajiv.kumar@nectechnologies.in","date":"2015-03-25 10:02:50.000000000","tz":0},"committer":{"name":"Rajiv Kumar","email":"rajiv.kumar@nectechnologies.in","date":"2015-04-13 13:38:52.000000000","tz":330},"subject":"Now keystone enables listing of user by name","message":"Now keystone enables listing of user by name\n\nThe API supports listing of user by name but keystone client\ndoes not. It was required by a user in some scenario(as\ndescribed in bug description).\n\nTo supporting this feature, the user option needs to be added for\nhelp and displaying to user.\n\ndo_user_list method uses list method of user.py, which needs to be\nupdated according to the keystone API. In the case of list of users,\nkeystone returns a dictionary having \u0027users\u0027 as key and list of\nall users as the value. But in case of display by name, we get,\na dictionary having \u0027user\u0027 as key and user details as value.\nWhich makes it necessary to pass \u0027user\u0027 as reponse key.\n\nIn base.py\u0027s _list method, in our case, we get a dictionary, which\nneeds to be converted to list before proceeding further. On the\nbasis of response_key, this decision is being made.\n\nChange-Id: Ie1a20a1e5e958a3133f0b86d1580a833e8928a43\nCloses-Bug: #1417189\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/492876a4c9365d471a7502bd4ca14e5809e1785a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/492876a4c9365d471a7502bd4ca14e5809e1785a"}]},"branch":"refs/heads/master"},"1502d7fb37626dba11e935ddfbf702d69f55d448":{"kind":"REWORK","_number":4,"created":"2015-04-13 13:44:56.000000000","uploader":{"_account_id":13912,"name":"rajiv","email":"thenakliman@gmail.com","username":"rajiv.kumar"},"ref":"refs/changes/43/167543/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-keystoneclient","ref":"refs/changes/43/167543/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-keystoneclient refs/changes/43/167543/4"}}},"commit":{"parents":[{"commit":"a84b5ac115dd5775f7380a204811aa939fd5c242","subject":"Merge \"Add a FederatedBase v3 plugin\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/a84b5ac115dd5775f7380a204811aa939fd5c242"}]}],"author":{"name":"Rajiv Kumar","email":"rajiv.kumar@nectechnologies.in","date":"2015-03-25 10:02:50.000000000","tz":0},"committer":{"name":"Rajiv Kumar","email":"rajiv.kumar@nectechnologies.in","date":"2015-04-13 13:44:33.000000000","tz":330},"subject":"Now keystone enables listing of user by name","message":"Now keystone enables listing of user by name\n\nThe API supports listing of user by name but keystone client\ndoes not. It was required by a user in some scenario(as\ndescribed in bug description).\n\nTo supporting this feature, the user option needs to be added for\nhelp and displaying to user.\n\ndo_user_list method uses list method of user.py, which needs to be\nupdated according to the keystone API. In the case of list of users,\nkeystone returns a dictionary having \u0027users\u0027 as key and list of\nall users as the value. But in case of display by name, we get,\na dictionary having \u0027user\u0027 as key and user details as value.\nWhich makes it necessary to pass \u0027user\u0027 as reponse key.\n\nIn base.py\u0027s _list method, in our case, we get a dictionary, which\nneeds to be converted to list before proceeding further. On the\nbasis of response_key, this decision is being made.\n\nChange-Id: Ie1a20a1e5e958a3133f0b86d1580a833e8928a43\nCloses-Bug: #1417189\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/1502d7fb37626dba11e935ddfbf702d69f55d448"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-keystoneclient/commit/1502d7fb37626dba11e935ddfbf702d69f55d448"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}