)]}'
{"id":"openstack%2Fpython-cinderclient~446860","triplet_id":"openstack%2Fpython-cinderclient~master~I5baa810b2b2eeff80c8a391c5b3159864aca3dba","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1673670","hashtags":[],"change_id":"I5baa810b2b2eeff80c8a391c5b3159864aca3dba","subject":"Fix cinder xxx-list subcmds sort-keys problem","status":"ABANDONED","created":"2017-03-17 06:47:08.000000000","updated":"2017-06-24 06:56:38.000000000","total_comment_count":13,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"81279daad17ef2ac30e04dd20fcd18aa8b3e4902","_number":446860,"virtual_id_number":446860,"owner":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2017-06-24 06:56:38.000000000","_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":-1,"date":"2017-05-25 10:42:22.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":24883,"name":"wangzhenyu","email":"wangzy@fiberhome.com","username":"wangzhenyu"},{"_account_id":24564,"name":"qtlu","email":"qtlu@fiberhome.com","username":"qtlu"},{"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"},{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"}],"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":{"recommended":{"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"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":24883,"name":"wangzhenyu","email":"wangzy@fiberhome.com","username":"wangzhenyu"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":24564,"name":"qtlu","email":"qtlu@fiberhome.com","username":"qtlu"},{"value":1,"date":"2017-03-21 01:21:38.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},{"value":1,"date":"2017-04-10 03:41:18.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"}],"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":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":24883,"name":"wangzhenyu","email":"wangzy@fiberhome.com","username":"wangzhenyu"},{"_account_id":24564,"name":"qtlu","email":"qtlu@fiberhome.com","username":"qtlu"},{"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"}],"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":-1,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":24883,"name":"wangzhenyu","email":"wangzy@fiberhome.com","username":"wangzhenyu"},{"_account_id":24564,"name":"qtlu","email":"qtlu@fiberhome.com","username":"qtlu"},{"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"},{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"}],"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":3,"name":"Jenkins","username":"jenkins"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},{"_account_id":24564,"name":"qtlu","email":"qtlu@fiberhome.com","username":"qtlu"},{"_account_id":24883,"name":"wangzhenyu","email":"wangzy@fiberhome.com","username":"wangzhenyu"},{"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-03-17 07:24:53.000000000","updated_by":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"reviewer":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"state":"REVIEWER"},{"updated":"2017-03-17 07:24:57.000000000","updated_by":{"_account_id":24883,"name":"wangzhenyu","email":"wangzy@fiberhome.com","username":"wangzhenyu"},"reviewer":{"_account_id":24883,"name":"wangzhenyu","email":"wangzy@fiberhome.com","username":"wangzhenyu"},"state":"REVIEWER"},{"updated":"2017-03-17 07:25:03.000000000","updated_by":{"_account_id":24564,"name":"qtlu","email":"qtlu@fiberhome.com","username":"qtlu"},"reviewer":{"_account_id":24564,"name":"qtlu","email":"qtlu@fiberhome.com","username":"qtlu"},"state":"REVIEWER"},{"updated":"2017-03-17 19:24:29.000000000","updated_by":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"reviewer":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"state":"REVIEWER"},{"updated":"2017-03-21 01:21:38.000000000","updated_by":{"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"},"reviewer":{"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"},"state":"REVIEWER"},{"updated":"2017-04-10 03:41:18.000000000","updated_by":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"reviewer":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"state":"REVIEWER"},{"updated":"2017-05-25 10:42:22.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2017-06-24 06:56:38.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"}],"messages":[{"id":"cb2c0b943fbb65aa46dc601b75d213f0f32bf3d1","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-17 06:47:08.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a8b534fe952b2e044e6bab00bbffeefa732a3713","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-17 07:24:43.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"dd97b08656bef64b20ee92895cb972101108ed26","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-17 08:07:36.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/60/446860/1/check/gate-python-cinderclient-docs-ubuntu-xenial/ca61bee//doc/build/html/ : SUCCESS in 3m 08s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/60/446860/1/check/gate-python-cinderclient-pep8-ubuntu-xenial/807846a/ : SUCCESS in 2m 35s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/60/446860/1/check/gate-python-cinderclient-python27-ubuntu-xenial/7f74367/ : SUCCESS in 2m 11s\n- gate-python-cinderclient-python35 http://logs.openstack.org/60/446860/1/check/gate-python-cinderclient-python35/8240b25/ : SUCCESS in 2m 04s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/60/446860/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/70db8e6/ : SUCCESS in 1h 19m 33s\n- gate-python-cinderclient-pypy http://logs.openstack.org/60/446860/1/check/gate-python-cinderclient-pypy/b1703e8/ : SUCCESS in 4m 08s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/60/446860/1/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/ec7c37b/ : SUCCESS in 33m 52s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/60/446860/1/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/c89d19e/ : SUCCESS in 26m 14s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/60/446860/1/check/gate-python-cinderclient-pylint-ubuntu-xenial/5cc897a/ : FAILURE in 3m 19s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"aa8ec7c5dcf9711dc62ddc663c6a7a5a439d6879","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-17 08:22:09.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"ca6973b96953ba53159b80f263e8c75eac24257f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-17 09:29:35.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/60/446860/2/check/gate-python-cinderclient-docs-ubuntu-xenial/23a6d4d//doc/build/html/ : SUCCESS in 2m 05s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/60/446860/2/check/gate-python-cinderclient-pep8-ubuntu-xenial/cd738e6/ : SUCCESS in 2m 47s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/60/446860/2/check/gate-python-cinderclient-python27-ubuntu-xenial/304150d/ : SUCCESS in 2m 47s\n- gate-python-cinderclient-python35 http://logs.openstack.org/60/446860/2/check/gate-python-cinderclient-python35/0a68837/ : SUCCESS in 2m 25s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/60/446860/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/4f3786b/ : SUCCESS in 1h 06m 50s\n- gate-python-cinderclient-pypy http://logs.openstack.org/60/446860/2/check/gate-python-cinderclient-pypy/5d49a17/ : SUCCESS in 4m 09s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/60/446860/2/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/fb3946a/ : SUCCESS in 55m 51s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/60/446860/2/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/08e3fe3/ : SUCCESS in 25m 46s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/60/446860/2/check/gate-python-cinderclient-pylint-ubuntu-xenial/3128a83/ : FAILURE in 3m 50s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"c47cd6931cc726ed657e1fd3edf3348b0704cfa0","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2017-03-17 19:24:29.000000000","message":"Patch Set 2: Code-Review-1\n\n(3 comments)\n\nI think there are a number of issues with this patch.  Why did test_snapshot_actions get moved from v2 to v3?  Also code missing from the shell that I don\u0027t know was supposed to be removed.  Please look.  Thanks!","accounts_in_message":[],"_revision_number":2},{"id":"f298147334b4de2eb07aed90c65142697b57559e","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-19 01:50:41.000000000","message":"Patch Set 2:\n\n(1 comment)\n\n\u003e (3 comments)\n \u003e \n \u003e I think there are a number of issues with this patch.  Why did\n \u003e test_snapshot_actions get moved from v2 to v3?  Also code missing\n \u003e from the shell that I don\u0027t know was supposed to be removed. \n \u003e Please look.  Thanks!\n\nIn v2/volume_snapshots.py, class SnapshotManager just implements  list_manageable method. In v3/volume_snapshots.py class SnapshotManager implement all methods, include list_manageable. So I think it\u0027s reasonable to move test_snapshot_actions.py under v3 tests.\n\nAs to other tow problems, please let me deal with them on Monday. Thanks~","accounts_in_message":[],"_revision_number":2},{"id":"8d67bde620bbccae94b30f0ce0251a4bb4006fdc","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-20 00:03:04.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"176e41b1926cf9d03145798054f6a2fcaae58fc0","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-20 00:07:37.000000000","message":"Uploaded patch set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"63ebc558846a2739e1a9d6cab2120253dd09c8ec","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-20 00:09:43.000000000","message":"Patch Set 3:\n\n\u003e (3 comments)\n \u003e \n \u003e I think there are a number of issues with this patch.  Why did\n \u003e test_snapshot_actions get moved from v2 to v3?  Also code missing\n \u003e from the shell that I don\u0027t know was supposed to be removed. \n \u003e Please look.  Thanks!\n\n \u003e (3 comments)\n \u003e \n \u003e I think there are a number of issues with this patch.  Why did\n \u003e test_snapshot_actions get moved from v2 to v3?  Also code missing\n \u003e from the shell that I don\u0027t know was supposed to be removed. \n \u003e Please look.  Thanks!\n\nCommit msg and _( problems have been dealt with. Please review again. Thanks~","accounts_in_message":[],"_revision_number":3},{"id":"ab415a82b3a857f652139cce4b89ab159caa0dd8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-20 01:17:57.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/60/446860/4/check/gate-python-cinderclient-docs-ubuntu-xenial/96b8af3//doc/build/html/ : SUCCESS in 2m 49s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/60/446860/4/check/gate-python-cinderclient-pep8-ubuntu-xenial/aeca615/ : SUCCESS in 2m 53s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/60/446860/4/check/gate-python-cinderclient-python27-ubuntu-xenial/ba01e28/ : SUCCESS in 2m 08s\n- gate-python-cinderclient-python35 http://logs.openstack.org/60/446860/4/check/gate-python-cinderclient-python35/a78a356/ : SUCCESS in 3m 47s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/60/446860/4/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/eab4eff/ : SUCCESS in 1h 10m 08s\n- gate-python-cinderclient-pypy http://logs.openstack.org/60/446860/4/check/gate-python-cinderclient-pypy/ba801c5/ : SUCCESS in 4m 29s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/60/446860/4/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/8c39ea6/ : SUCCESS in 17m 30s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/60/446860/4/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/6dde21f/ : SUCCESS in 17m 12s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/60/446860/4/check/gate-python-cinderclient-pylint-ubuntu-xenial/1203201/ : FAILURE in 2m 12s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"3a5d9781e58d73cb38e271e0dfc42c5786089819","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-03-20 02:13:37.000000000","message":"Patch Set 4: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":4},{"id":"7b32f863a32ff08921d76d6c5f0cbe13a250c53a","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-03-20 02:32:55.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"bfefe41f30beda42da6f416e595a931a6f0ab20e","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-20 02:37:08.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"2481bd87ad2a540d99be3c510458ea03172c6809","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-20 07:00:33.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"92985ab9349f8dd9d95de443c47924bb93706f2f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-20 08:07:06.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/60/446860/5/check/gate-python-cinderclient-docs-ubuntu-xenial/a57e15e//doc/build/html/ : SUCCESS in 1m 56s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/60/446860/5/check/gate-python-cinderclient-pep8-ubuntu-xenial/625d90a/ : SUCCESS in 2m 57s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/60/446860/5/check/gate-python-cinderclient-python27-ubuntu-xenial/603f06a/ : SUCCESS in 2m 02s\n- gate-python-cinderclient-python35 http://logs.openstack.org/60/446860/5/check/gate-python-cinderclient-python35/2242ce7/ : SUCCESS in 3m 30s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/60/446860/5/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/0a93316/ : SUCCESS in 1h 06m 16s\n- gate-python-cinderclient-pypy http://logs.openstack.org/60/446860/5/check/gate-python-cinderclient-pypy/49f7c4a/ : SUCCESS in 3m 50s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/60/446860/5/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/6af5d9f/ : SUCCESS in 26m 25s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/60/446860/5/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/250b148/ : SUCCESS in 26m 05s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/60/446860/5/check/gate-python-cinderclient-pylint-ubuntu-xenial/28e74d0/ : FAILURE in 2m 19s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"a639991ce31986d756adee647a34721ce24bf02a","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-03-20 12:29:45.000000000","message":"Patch Set 5: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"3a682a3d29d4a1aafc40ca9fff80167f6d3deb74","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-21 00:00:36.000000000","message":"Patch Set 5:\n\n(1 comment)","accounts_in_message":[],"_revision_number":5},{"id":"225c23314226bf5de72c6266491dc518fd1a1368","author":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"date":"2017-03-21 00:05:34.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"31c4ab85689ac041ed6e3f1586aae47f6c8aacf0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-21 01:20:11.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-cinderclient-docs-ubuntu-xenial http://docs-draft.openstack.org/60/446860/6/check/gate-python-cinderclient-docs-ubuntu-xenial/3d77162//doc/build/html/ : SUCCESS in 1m 49s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/60/446860/6/check/gate-python-cinderclient-pep8-ubuntu-xenial/540bb1c/ : SUCCESS in 2m 27s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/60/446860/6/check/gate-python-cinderclient-python27-ubuntu-xenial/ae76674/ : SUCCESS in 6m 35s\n- gate-python-cinderclient-python35 http://logs.openstack.org/60/446860/6/check/gate-python-cinderclient-python35/b90ef8b/ : SUCCESS in 2m 06s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/60/446860/6/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/cb5810c/ : SUCCESS in 1h 12m 59s\n- gate-python-cinderclient-pypy http://logs.openstack.org/60/446860/6/check/gate-python-cinderclient-pypy/bae23de/ : SUCCESS in 2m 32s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/60/446860/6/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/da044d8/ : SUCCESS in 26m 46s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/60/446860/6/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/43afee7/ : SUCCESS in 26m 28s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/60/446860/6/check/gate-python-cinderclient-pylint-ubuntu-xenial/cc86f55/ : FAILURE in 4m 53s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"efa951e86e2f579910d66098c88672afa4e12a60","author":{"_account_id":25080,"name":"WangBinbin","email":"bbwang5827@fiberhome.com","username":"WangBinbin"},"date":"2017-03-21 01:21:38.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"7a3e07eee82902a94def6f43d48f3d7cf07ccca1","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-04-10 03:41:18.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"cd3662fa1d48a5e33eeff73e3030577caa85164f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-05-25 10:42:22.000000000","message":"Patch Set 6: 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":6},{"id":"daa35a333fd35c918cb874c1bae1d6cb2c49ef54","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2017-06-24 06:56:38.000000000","message":"Abandoned\n\nThis review is \u003e 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a \u0027recheck\u0027 comment to get fresh test results.","accounts_in_message":[],"_revision_number":6},{"id":"734479bcdcb09c6c32ec50c2007639309f6c8508","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2017-06-24 06:56:38.000000000","message":"Patch Set 6:\n\nThis review is \u003e 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a \u0027recheck\u0027 comment to get fresh test results.","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"02149f1edb703a63f736528df5e63af76b16b08e","revisions":{"fca29b76ddf420d0a84b07c8dc88dbadfcd804cd":{"kind":"REWORK","_number":1,"created":"2017-03-17 06:47:08.000000000","uploader":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"ref":"refs/changes/60/446860/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/60/446860/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/1"}}},"commit":{"parents":[{"commit":"29d29a7cd4c31fd96157beae90cb483403afe1af","subject":"Merge \"Add cinder tests for cinder snapshot create commands with parameters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/29d29a7cd4c31fd96157beae90cb483403afe1af"}]}],"author":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-17 06:31:45.000000000","tz":480},"committer":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-17 06:45:08.000000000","tz":480},"subject":"Fix cinder xxx-list subcmds sort-keys problem","message":"Fix cinder xxx-list subcmds sort-keys problem\n\nFollowing cinder xxx-list subcommands with \"--sort \u003ckey\u003e\" optional\nargument have problems in help massages and valid sort-keys you\ncan use.\n1.cinder list\n2.cinder --os-volume-api-version 3.8 manageable-list\n3.cinder --os-volume-api-version 3.8 snapshot-manageable-list\n4.cinder --os-volume-api-version 3.5 message-list\n5.cinder --os-volume-api-version 3.27 attachment-list\n6.cinder backup-list\n\nThese subcommands\u0027 valid sort-keys are different, but now in\npython-cinderclient, it just simplely use a SORT_KEY_VALUES dict.\n\nThe patch( https://review.openstack.org/#/c/445774/ ) already fix\nmanageable-list and snapshot-manageable-list subcommands problem.\n\nBut this patch uses a common way to resolve this problem in all problem\nsubcommands. Set different valid sort-keys according to resource-type.\nAnd add some ut cases.\n\nChange-Id: I5baa810b2b2eeff80c8a391c5b3159864aca3dba\nCloses-Bug: #1673670\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/fca29b76ddf420d0a84b07c8dc88dbadfcd804cd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/fca29b76ddf420d0a84b07c8dc88dbadfcd804cd"}]},"branch":"refs/heads/master"},"40898d0734db3ee28e3eb08e512ef831e334fb91":{"kind":"REWORK","_number":2,"created":"2017-03-17 08:22:09.000000000","uploader":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"ref":"refs/changes/60/446860/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/60/446860/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/2"}}},"commit":{"parents":[{"commit":"29d29a7cd4c31fd96157beae90cb483403afe1af","subject":"Merge \"Add cinder tests for cinder snapshot create commands with parameters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/29d29a7cd4c31fd96157beae90cb483403afe1af"}]}],"author":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-17 06:31:45.000000000","tz":480},"committer":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-17 08:20:42.000000000","tz":480},"subject":"Fix cinder xxx-list subcmds sort-keys problem","message":"Fix cinder xxx-list subcmds sort-keys problem\n\nFollowing cinder xxx-list subcommands with \"--sort \u003ckey\u003e\" optional\nargument have problems in help massages and valid sort-keys you\ncan use.\n1.cinder list\n2.cinder --os-volume-api-version 3.8 manageable-list\n3.cinder --os-volume-api-version 3.8 snapshot-manageable-list\n4.cinder --os-volume-api-version 3.5 message-list\n5.cinder --os-volume-api-version 3.27 attachment-list\n6.cinder backup-list\n\nThese subcommands\u0027 valid sort-keys are different, but now in\npython-cinderclient, it just simplely use a SORT_KEY_VALUES dict.\n\nThe patch( https://review.openstack.org/#/c/445774/ ) already fix\nmanageable-list and snapshot-manageable-list subcommands problem.\n\nBut this patch uses a common way to resolve this problem in all problem\nsubcommands. Set different valid sort-keys according to resource-type.\nAnd add some ut cases.\n\nChange-Id: I5baa810b2b2eeff80c8a391c5b3159864aca3dba\nCloses-Bug: #1673670\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/40898d0734db3ee28e3eb08e512ef831e334fb91"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/40898d0734db3ee28e3eb08e512ef831e334fb91"}]},"branch":"refs/heads/master"},"41411e7616909e099011b0bfb51fe0e10d0151cb":{"kind":"REWORK","_number":3,"created":"2017-03-20 00:03:04.000000000","uploader":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"ref":"refs/changes/60/446860/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/60/446860/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/3"}}},"commit":{"parents":[{"commit":"29d29a7cd4c31fd96157beae90cb483403afe1af","subject":"Merge \"Add cinder tests for cinder snapshot create commands with parameters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/29d29a7cd4c31fd96157beae90cb483403afe1af"}]}],"author":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-17 06:31:45.000000000","tz":480},"committer":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-20 00:01:19.000000000","tz":480},"subject":"Fix cinder xxx-list subcmds sort-keys problem","message":"Fix cinder xxx-list subcmds sort-keys problem\n\nFollowing cinder xxx-list subcommands with \"--sort \u003ckey\u003e\" optional\nargument have problems in help massages and valid sort-keys you\ncan use.\n1.cinder list\n2.cinder --os-volume-api-version 3.8 manageable-list\n3.cinder --os-volume-api-version 3.8 snapshot-manageable-list\n4.cinder --os-volume-api-version 3.5 message-list\n5.cinder --os-volume-api-version 3.27 attachment-list\n6.cinder backup-list\n\nThese subcommands\u0027 valid sort-keys are different, but now in\npython-cinderclient, it just simply use a SORT_KEY_VALUES dict.\n\nThe patch( https://review.openstack.org/#/c/445774/ ) already fix\nmanageable-list and snapshot-manageable-list subcommands problem.\n\nBut this patch uses a common way to resolve this problem in all problem\nsubcommands. Set different valid sort-keys according to resource-type.\nAnd add some ut cases.\n\nChange-Id: I5baa810b2b2eeff80c8a391c5b3159864aca3dba\nCloses-Bug: #1673670\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/41411e7616909e099011b0bfb51fe0e10d0151cb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/41411e7616909e099011b0bfb51fe0e10d0151cb"}]},"branch":"refs/heads/master"},"722b7a24649be7c0c0e05f46bf8369e25ffd1c4b":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2017-03-20 00:07:37.000000000","uploader":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"ref":"refs/changes/60/446860/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/60/446860/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/4"}}},"commit":{"parents":[{"commit":"29d29a7cd4c31fd96157beae90cb483403afe1af","subject":"Merge \"Add cinder tests for cinder snapshot create commands with parameters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/29d29a7cd4c31fd96157beae90cb483403afe1af"}]}],"author":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-17 06:31:45.000000000","tz":480},"committer":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-20 00:06:51.000000000","tz":480},"subject":"Fix cinder xxx-list subcmds sort-keys problem","message":"Fix cinder xxx-list subcmds sort-keys problem\n\nFollowing cinder xxx-list subcommands with \"--sort \u003ckey\u003e\" optional\nargument have problems in help massages and valid sort-keys you\ncan use.\n1.cinder list\n2.cinder --os-volume-api-version 3.8 manageable-list\n3.cinder --os-volume-api-version 3.8 snapshot-manageable-list\n4.cinder --os-volume-api-version 3.5 message-list\n5.cinder --os-volume-api-version 3.27 attachment-list\n6.cinder backup-list\n\nThese subcommands\u0027 valid sort-keys are different, but now in\npython-cinderclient, it just simply uses a SORT_KEY_VALUES dict.\n\nThe patch( https://review.openstack.org/#/c/445774/ ) already fix\nmanageable-list and snapshot-manageable-list subcommands problem.\n\nBut this patch uses a common way to resolve this problem in all problem\nsubcommands. Set different valid sort-keys according to resource-type.\nAnd add some ut cases.\n\nChange-Id: I5baa810b2b2eeff80c8a391c5b3159864aca3dba\nCloses-Bug: #1673670\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/722b7a24649be7c0c0e05f46bf8369e25ffd1c4b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/722b7a24649be7c0c0e05f46bf8369e25ffd1c4b"}]},"branch":"refs/heads/master"},"78aad06ececa418e671653051ca5ac95a9fdf0a3":{"kind":"REWORK","_number":5,"created":"2017-03-20 07:00:33.000000000","uploader":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"ref":"refs/changes/60/446860/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/60/446860/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/5"}}},"commit":{"parents":[{"commit":"29d29a7cd4c31fd96157beae90cb483403afe1af","subject":"Merge \"Add cinder tests for cinder snapshot create commands with parameters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/29d29a7cd4c31fd96157beae90cb483403afe1af"}]}],"author":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-17 06:31:45.000000000","tz":480},"committer":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-20 06:46:05.000000000","tz":480},"subject":"Fix cinder xxx-list subcmds sort-keys problem","message":"Fix cinder xxx-list subcmds sort-keys problem\n\nFollowing cinder xxx-list subcommands with \"--sort \u003ckey\u003e\" optional\nargument have problems in help massages and valid sort-keys you\ncan use.\n1.cinder list\n2.cinder --os-volume-api-version 3.8 manageable-list\n3.cinder --os-volume-api-version 3.8 snapshot-manageable-list\n4.cinder --os-volume-api-version 3.5 message-list\n5.cinder --os-volume-api-version 3.27 attachment-list\n6.cinder backup-list\n\nThese subcommands\u0027 valid sort-keys are different, but now in\npython-cinderclient, it just simply uses a SORT_KEY_VALUES dict.\n\nThe patch( https://review.openstack.org/#/c/445774/ ) already fix\nmanageable-list and snapshot-manageable-list subcommands problem.\n\nBut this patch uses a common way to resolve this problem in all problem\nsubcommands. Set different valid sort-keys according to resource-type.\nAnd add some ut cases.\n\nChange-Id: I5baa810b2b2eeff80c8a391c5b3159864aca3dba\nCloses-Bug: #1673670\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/78aad06ececa418e671653051ca5ac95a9fdf0a3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/78aad06ececa418e671653051ca5ac95a9fdf0a3"}]},"branch":"refs/heads/master"},"02149f1edb703a63f736528df5e63af76b16b08e":{"kind":"REWORK","_number":6,"created":"2017-03-21 00:05:34.000000000","uploader":{"_account_id":22299,"name":"lihaijing","email":"lihaijing@fiberhome.com","username":"lihaijing"},"ref":"refs/changes/60/446860/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/60/446860/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/60/446860/6"}}},"commit":{"parents":[{"commit":"29d29a7cd4c31fd96157beae90cb483403afe1af","subject":"Merge \"Add cinder tests for cinder snapshot create commands with parameters\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/29d29a7cd4c31fd96157beae90cb483403afe1af"}]}],"author":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-17 06:31:45.000000000","tz":480},"committer":{"name":"lihaijing","email":"lihaijing@fiberhome.com","date":"2017-03-21 00:04:53.000000000","tz":480},"subject":"Fix cinder xxx-list subcmds sort-keys problem","message":"Fix cinder xxx-list subcmds sort-keys problem\n\nFollowing cinder xxx-list subcommands with \"--sort \u003ckey\u003e\" optional\nargument have problems in help massages and valid sort-keys you\ncan use.\n1.cinder list\n2.cinder --os-volume-api-version 3.8 manageable-list\n3.cinder --os-volume-api-version 3.8 snapshot-manageable-list\n4.cinder --os-volume-api-version 3.5 message-list\n5.cinder --os-volume-api-version 3.27 attachment-list\n6.cinder backup-list\n\nThese subcommands\u0027 valid sort-keys are different, but now in\npython-cinderclient, it just simply uses a SORT_KEY_VALUES dict.\n\nThe patch( https://review.openstack.org/#/c/445774/ ) already fix\nmanageable-list and snapshot-manageable-list subcommands problem.\n\nBut this patch uses a common way to resolve this problem in all problem\nsubcommands. Set different valid sort-keys according to resource-type.\nAnd add some ut cases.\n\nChange-Id: I5baa810b2b2eeff80c8a391c5b3159864aca3dba\nCloses-Bug: #1673670\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/02149f1edb703a63f736528df5e63af76b16b08e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/02149f1edb703a63f736528df5e63af76b16b08e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}