)]}'
{"id":"openstack%2Fpython-cinderclient~454159","triplet_id":"openstack%2Fpython-cinderclient~master~Ie38236db364d59ddab42cb925d0435777b0ffe86","project":"openstack/python-cinderclient","branch":"master","topic":"bug/1680444","hashtags":[],"change_id":"Ie38236db364d59ddab42cb925d0435777b0ffe86","subject":"Removes unicode \u0027u\u0027 response for \"cinder get-capabilities\"","status":"MERGED","created":"2017-04-06 12:39:23.000000000","updated":"2018-01-23 18:03:43.000000000","submitted":"2018-01-23 18:03:43.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":3,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"454159-1516730623764-35d18e55","meta_rev_id":"5a52870f1c1d9d6408ae9974cc4a0c6dc98f782e","_number":454159,"virtual_id_number":454159,"owner":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":2,"date":"2018-01-23 18:03:43.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"date":"2018-01-23 13:17:09.000000000","_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":2,"date":"2018-01-23 13:06:43.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2018-01-23 16:06:56.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2018-01-23 16:06:56.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"value":0,"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"}],"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":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-04-06 13:56:22.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2017-04-10 15:26:56.000000000","updated_by":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"reviewer":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"state":"REVIEWER"},{"updated":"2017-06-02 15:47:05.000000000","updated_by":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"reviewer":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"state":"REVIEWER"},{"updated":"2018-01-23 13:06:43.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"},{"updated":"2018-01-23 13:17:09.000000000","updated_by":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"reviewer":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"state":"REVIEWER"},{"updated":"2018-01-23 16:06:56.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":"2018-01-23 18:03:43.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"06af94af65213330658f757b15c34792581f751a","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2017-04-06 12:39:23.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"906c27e120f14a08dc148ab3e5da929c6e24e3f6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-04-06 13:56:22.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/59/454159/1/check/gate-python-cinderclient-docs-ubuntu-xenial/1381892//doc/build/html/ : SUCCESS in 2m 27s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/59/454159/1/check/gate-python-cinderclient-pep8-ubuntu-xenial/8ea7301/ : SUCCESS in 3m 05s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/59/454159/1/check/gate-python-cinderclient-python27-ubuntu-xenial/9617dac/ : SUCCESS in 2m 27s\n- gate-python-cinderclient-python35 http://logs.openstack.org/59/454159/1/check/gate-python-cinderclient-python35/97e93a9/ : SUCCESS in 3m 38s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/59/454159/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/0f48710/ : SUCCESS in 1h 14m 06s\n- gate-python-cinderclient-pypy http://logs.openstack.org/59/454159/1/check/gate-python-cinderclient-pypy/670c82b/ : SUCCESS in 4m 28s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/59/454159/1/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/5f01530/ : SUCCESS in 26m 22s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/59/454159/1/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/0dca084/ : SUCCESS in 31m 43s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/59/454159/1/check/gate-python-cinderclient-pylint-ubuntu-xenial/f72a7ec/ : SUCCESS in 3m 31s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"c819c5f5b9beea301a86957751c6765ea009061f","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2017-04-06 14:08:03.000000000","message":"Patch Set 1: Code-Review-1\n\nHave we verified that we do not need to specify unicode here?  I would like to get more eyes on this.  Adding Eric Harney as he gets this stuff better than I.","accounts_in_message":[],"_revision_number":1},{"id":"3762342f2e9fa72fd0d5d6b1e7b6947f15d3d05c","author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"date":"2017-04-10 15:26:56.000000000","message":"Patch Set 1: Code-Review-1\n\nAgree with Jay.\nAnd please, fix the commit message. I actually want to see not the result of the code execution here, but the description of changes you\u0027ve done","accounts_in_message":[],"_revision_number":1},{"id":"fa632d522cb32a4ae453456b30e8092400b1886b","author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"date":"2017-06-02 15:47:05.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"39efb07c09ac92a3b892d17b1415306872db0ef0","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2017-09-08 16:28:29.000000000","message":"Patch Set 1:\n\nHuanxuan Ao, can we get this updated based on Eric\u0027s recommendation?","accounts_in_message":[],"_revision_number":1},{"id":"6f6d91c5ab20a4b9afa27c8598f48025b0b15402","author":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"date":"2018-01-20 19:04:31.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"676b4e360980953364d08d765a7ec8d9fda33e3e","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-20 21:35:02.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 http://logs.openstack.org/59/454159/2/check/openstack-tox-pep8/106709e/ : FAILURE in 5m 21s\n- openstack-tox-py27 http://logs.openstack.org/59/454159/2/check/openstack-tox-py27/e0ff4ef/ : SUCCESS in 4m 27s\n- openstack-tox-py35 http://logs.openstack.org/59/454159/2/check/openstack-tox-py35/05379eb/ : SUCCESS in 4m 42s\n- build-openstack-sphinx-docs http://logs.openstack.org/59/454159/2/check/build-openstack-sphinx-docs/da47de4/html/ : SUCCESS in 3m 40s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/59/454159/2/check/legacy-tempest-dsvm-neutron-src/39ae26f/ : SUCCESS in 1h 18m 02s\n- openstack-tox-pylint http://logs.openstack.org/59/454159/2/check/openstack-tox-pylint/b5b7d82/ : SUCCESS in 5m 17s (non-voting)\n- cinderclient-dsvm-functional http://logs.openstack.org/59/454159/2/check/cinderclient-dsvm-functional/71ea893/ : SUCCESS in 27m 40s (non-voting)\n- cinderclient-dsvm-functional-identity-v3-only http://logs.openstack.org/59/454159/2/check/cinderclient-dsvm-functional-identity-v3-only/a209c73/ : SUCCESS in 27m 07s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"2fc06659840c559ce8b3c7958f8a928642448266","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2018-01-21 16:31:52.000000000","message":"Patch Set 2: Code-Review-1\n\nYou have pep8 failures.  Please make sure to run pep8 before uploading.  Thanks!","accounts_in_message":[],"_revision_number":2},{"id":"48d26865dba1e14664731504a61489bf3ce3b744","author":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"date":"2018-01-23 09:28:09.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"4cd9e2ddee00f14545f85d96415908298e9a270e","author":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"date":"2018-01-23 09:39:16.000000000","message":"Uploaded patch set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"5204991e48c69ea16bbd922ebb221623e4cf54c5","author":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"date":"2018-01-23 09:41:20.000000000","message":"Uploaded patch set 5: Commit message was updated.","accounts_in_message":[],"_revision_number":5},{"id":"9ee448260030ea4edf89ec9153d37884b0f01894","author":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"date":"2018-01-23 09:51:40.000000000","message":"Patch Set 5:\n\n\u003e You have pep8 failures.  Please make sure to run pep8 before\n \u003e uploading.  Thanks!\n\nSorry, I will make sure to run all the tests locally without fail next time. Thank you :). And please feel free to suggest any improvements. I am new to the community and would love to learn and contribute.  :)","accounts_in_message":[],"_revision_number":5},{"id":"6ddedb315759f662e0bc627eb0555376915be6cb","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-23 11:02:24.000000000","message":"Patch Set 5: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/59/454159/5/check/openstack-tox-pep8/6b91963/ : SUCCESS in 4m 56s\n- openstack-tox-py27 http://logs.openstack.org/59/454159/5/check/openstack-tox-py27/f5cd014/ : SUCCESS in 5m 20s\n- openstack-tox-py35 http://logs.openstack.org/59/454159/5/check/openstack-tox-py35/aab418c/ : SUCCESS in 4m 34s\n- build-openstack-sphinx-docs http://logs.openstack.org/59/454159/5/check/build-openstack-sphinx-docs/e5821fb/html/ : SUCCESS in 4m 13s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/59/454159/5/check/legacy-tempest-dsvm-neutron-src/1f2f5dc/ : SUCCESS in 1h 17m 58s\n- openstack-tox-pylint http://logs.openstack.org/59/454159/5/check/openstack-tox-pylint/84339ad/ : SUCCESS in 5m 24s (non-voting)\n- cinderclient-dsvm-functional http://logs.openstack.org/59/454159/5/check/cinderclient-dsvm-functional/38c9bd5/ : SUCCESS in 30m 01s (non-voting)\n- cinderclient-dsvm-functional-identity-v3-only http://logs.openstack.org/59/454159/5/check/cinderclient-dsvm-functional-identity-v3-only/0705b38/ : SUCCESS in 25m 51s (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"289e7cef0e25765c389144a06b8ac1556d1d2574","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2018-01-23 11:24:42.000000000","message":"Patch Set 5: Code-Review+2\n\n(2 comments)\n\nI think the commit message could use some addition information to explain the change, but it would appear from the utils class that setting the formatter will go through the capabilities dict and make sure all values are properly string formatted. So some things that could be improved if you do an update, but looks OK.","accounts_in_message":[],"_revision_number":5},{"id":"2f0bd05e8a2aa5591484aba267565decd888bdd4","author":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"date":"2018-01-23 13:06:14.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"9b3e4f69467e0053e87773ef0c3fcc0f67b67333","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2018-01-23 13:06:43.000000000","message":"Patch Set 6: Code-Review+2","accounts_in_message":[],"_revision_number":6},{"id":"302b5e2fee80a13ce82ee5d850fdb44bb0e7693c","author":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"date":"2018-01-23 13:17:09.000000000","message":"Patch Set 6:\n\n\u003e (2 comments)\n \u003e \n \u003e I think the commit message could use some addition information to\n \u003e explain the change, but it would appear from the utils class that\n \u003e setting the formatter will go through the capabilities dict and\n \u003e make sure all values are properly string formatted. So some things\n \u003e that could be improved if you do an update, but looks OK.\n\nSure :) Uploaded new patch with changes. Thanks for guiding me :). It was really helpful. :) \n\n \u003e (2 comments)\n \u003e \n \u003e I think the commit message could use some addition information to\n \u003e explain the change, but it would appear from the utils class that\n \u003e setting the formatter will go through the capabilities dict and\n \u003e make sure all values are properly string formatted. So some things\n \u003e that could be improved if you do an update, but looks OK.\n\nSure, uploaded a new patch with changes. Thanks for guiding me :) It was really helpful. :)","accounts_in_message":[],"_revision_number":6},{"id":"c9d24f1499f9cfd357ee4dfdd7f8d3de51c6b397","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-23 14:36:48.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/59/454159/6/check/openstack-tox-pep8/a084217/ : SUCCESS in 4m 41s\n- openstack-tox-py27 http://logs.openstack.org/59/454159/6/check/openstack-tox-py27/910ba4a/ : SUCCESS in 4m 26s\n- openstack-tox-py35 http://logs.openstack.org/59/454159/6/check/openstack-tox-py35/25db2b0/ : SUCCESS in 4m 30s\n- build-openstack-sphinx-docs http://logs.openstack.org/59/454159/6/check/build-openstack-sphinx-docs/4c7bf50/html/ : SUCCESS in 4m 30s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/59/454159/6/check/legacy-tempest-dsvm-neutron-src/c0b524b/ : SUCCESS in 1h 22m 15s\n- openstack-tox-pylint http://logs.openstack.org/59/454159/6/check/openstack-tox-pylint/3e3054e/ : SUCCESS in 4m 30s (non-voting)\n- cinderclient-dsvm-functional http://logs.openstack.org/59/454159/6/check/cinderclient-dsvm-functional/ae78ee0/ : SUCCESS in 27m 36s (non-voting)\n- cinderclient-dsvm-functional-identity-v3-only http://logs.openstack.org/59/454159/6/check/cinderclient-dsvm-functional-identity-v3-only/88ea95e/ : SUCCESS in 22m 40s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"ed6bccb0d372d6bd48f636a81c9ce9dc62f49a25","author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"date":"2018-01-23 16:06:56.000000000","message":"Patch Set 6: Workflow+1 Code-Review+2\n\nLooks ok now.","accounts_in_message":[],"_revision_number":6},{"id":"43b1e60b98614e736f20ba26faeb80916687e5b4","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-23 16:25:06.000000000","message":"Patch Set 6: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":6},{"id":"f4fa0de535ae97b26dc5ae37cb06039fde0e76c9","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-23 18:03:43.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":6},{"id":"6528336495a728bf2ce624e0a79caea58905a573","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2018-01-23 18:03:43.000000000","message":"Patch Set 6: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 http://logs.openstack.org/59/454159/6/gate/openstack-tox-pep8/9ca7a91/ : SUCCESS in 7m 27s\n- openstack-tox-py27 http://logs.openstack.org/59/454159/6/gate/openstack-tox-py27/f6a0cfa/ : SUCCESS in 8m 02s\n- openstack-tox-py35 http://logs.openstack.org/59/454159/6/gate/openstack-tox-py35/fd9e4d3/ : SUCCESS in 7m 58s\n- build-openstack-sphinx-docs http://logs.openstack.org/59/454159/6/gate/build-openstack-sphinx-docs/b031b1d/html/ : SUCCESS in 7m 41s\n- legacy-tempest-dsvm-neutron-src http://logs.openstack.org/59/454159/6/gate/legacy-tempest-dsvm-neutron-src/513e8d3/ : SUCCESS in 1h 16m 15s","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"c3a22519bcd91db227146a982bfb281092428f6c","revisions":{"b57e2c9987dbc9cbffd64d6665a31b85e5093c97":{"kind":"REWORK","_number":1,"created":"2017-04-06 12:39:23.000000000","uploader":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"ref":"refs/changes/59/454159/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/59/454159/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/1"}}},"commit":{"parents":[{"commit":"0cf8d9bf30eead4639b4c51924f7f2cec77213a1","subject":"Merge \"[BugFix] Add \u0027all_tenants\u0027, \u0027project_id\u0027 in attachment-list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0cf8d9bf30eead4639b4c51924f7f2cec77213a1"}]}],"author":{"name":"Huanxuan Ao","email":"huanxuan.ao@easystack.cn","date":"2017-04-06 12:30:09.000000000","tz":-420},"committer":{"name":"Huanxuan Ao","email":"huanxuan.ao@easystack.cn","date":"2017-04-06 12:38:30.000000000","tz":-420},"subject":"Remove useless \u0027u\u0027 in the output of \"cinder get-capabilities\"","message":"Remove useless \u0027u\u0027 in the output of \"cinder get-capabilities\"\n\nAfter this change, the output is fixed like this:\n+---------------------+---------------------------------------------------------------------------------------------+\n| Backend properties  | Value                                                                                       |\n+---------------------+---------------------------------------------------------------------------------------------+\n| compression         | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables compression.\u0027, \u0027title\u0027: \u0027Compression\u0027}          |\n| qos                 | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables QoS.\u0027, \u0027title\u0027: \u0027QoS\u0027}                          |\n| replication_enabled | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables replication.\u0027, \u0027title\u0027: \u0027Replication\u0027}          |\n| thin_provisioning   | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Sets thin provisioning.\u0027, \u0027title\u0027: \u0027Thin Provisioning\u0027} |\n+---------------------+---------------------------------------------------------------------------------------------+\n\nCloses-bug: #1680444\nChange-Id: Ie38236db364d59ddab42cb925d0435777b0ffe86\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b57e2c9987dbc9cbffd64d6665a31b85e5093c97"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b57e2c9987dbc9cbffd64d6665a31b85e5093c97"}]},"branch":"refs/heads/master"},"399f1e8b325c1e9c3bb2c6c562316ec6328c877f":{"kind":"REWORK","_number":2,"created":"2018-01-20 19:04:31.000000000","uploader":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"ref":"refs/changes/59/454159/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/59/454159/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/2"}}},"commit":{"parents":[{"commit":"0cf8d9bf30eead4639b4c51924f7f2cec77213a1","subject":"Merge \"[BugFix] Add \u0027all_tenants\u0027, \u0027project_id\u0027 in attachment-list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0cf8d9bf30eead4639b4c51924f7f2cec77213a1"}]}],"author":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2017-04-06 12:30:09.000000000","tz":-420},"committer":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2018-01-20 19:02:08.000000000","tz":330},"subject":"Remove useless \u0027u\u0027 in the output of \"cinder get-capabilities\"","message":"Remove useless \u0027u\u0027 in the output of \"cinder get-capabilities\"\n\nAfter this change, the output is fixed like this:\n+---------------------+---------------------------------------------------------------------------------------------+\n| Backend properties  | Value                                                                                       |\n+---------------------+---------------------------------------------------------------------------------------------+\n| compression         | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables compression.\u0027, \u0027title\u0027: \u0027Compression\u0027}          |\n| qos                 | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables QoS.\u0027, \u0027title\u0027: \u0027QoS\u0027}                          |\n| replication_enabled | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables replication.\u0027, \u0027title\u0027: \u0027Replication\u0027}          |\n| thin_provisioning   | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Sets thin provisioning.\u0027, \u0027title\u0027: \u0027Thin Provisioning\u0027} |\n+---------------------+---------------------------------------------------------------------------------------------+\n\nAdded formatters\u003dsorted(prop.keys()) as suggested by the reviewer, to avoid tying server\nknowledge to the client and also any update on the server side would easily reflect here.\nFixed Pep8 violations for /v2/fakes.py unit test file.\n\nCloses-bug: #1680444\nChange-Id: Ie38236db364d59ddab42cb925d0435777b0ffe86\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/399f1e8b325c1e9c3bb2c6c562316ec6328c877f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/399f1e8b325c1e9c3bb2c6c562316ec6328c877f"}]},"branch":"refs/heads/master"},"b711a22c51a99d3219553023a54bd55b3fd5f5fa":{"kind":"REWORK","_number":3,"created":"2018-01-23 09:28:09.000000000","uploader":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"ref":"refs/changes/59/454159/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/59/454159/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/3"}}},"commit":{"parents":[{"commit":"0a323eaf942869439edc05a89c6cc226033bbe6c","subject":"Merge \"Support for reporting backend state in service list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0a323eaf942869439edc05a89c6cc226033bbe6c"}]}],"author":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2017-04-06 12:30:09.000000000","tz":-420},"committer":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2018-01-23 09:26:56.000000000","tz":330},"subject":"Remove useless \u0027u\u0027 in the output of \"cinder get-capabilities\"","message":"Remove useless \u0027u\u0027 in the output of \"cinder get-capabilities\"\n\nAfter this change, the output is fixed like this:\n+---------------------+---------------------------------------------------------------------------------------------+\n| Backend properties  | Value                                                                                       |\n+---------------------+---------------------------------------------------------------------------------------------+\n| compression         | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables compression.\u0027, \u0027title\u0027: \u0027Compression\u0027}          |\n| qos                 | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables QoS.\u0027, \u0027title\u0027: \u0027QoS\u0027}                          |\n| replication_enabled | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Enables replication.\u0027, \u0027title\u0027: \u0027Replication\u0027}          |\n| thin_provisioning   | {\u0027type\u0027: \u0027boolean\u0027, \u0027description\u0027: \u0027Sets thin provisioning.\u0027, \u0027title\u0027: \u0027Thin Provisioning\u0027} |\n+---------------------+---------------------------------------------------------------------------------------------+\n\nAdded formatters\u003dsorted(prop.keys()) as suggested by the reviewer, to avoid tying server\nknowledge to the client and also any update on the server side would easily reflect here.\n*Fixed pep8 issues\n\nCloses-bug: #1680444\nChange-Id: Ie38236db364d59ddab42cb925d0435777b0ffe86\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b711a22c51a99d3219553023a54bd55b3fd5f5fa"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b711a22c51a99d3219553023a54bd55b3fd5f5fa"}]},"branch":"refs/heads/master"},"a7fac5abdfacde12a410fb4e41e7911a70e01d73":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2018-01-23 09:39:16.000000000","uploader":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"ref":"refs/changes/59/454159/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/59/454159/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/4"}}},"commit":{"parents":[{"commit":"0a323eaf942869439edc05a89c6cc226033bbe6c","subject":"Merge \"Support for reporting backend state in service list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0a323eaf942869439edc05a89c6cc226033bbe6c"}]}],"author":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2017-04-06 12:30:09.000000000","tz":-420},"committer":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2018-01-23 09:30:52.000000000","tz":330},"subject":"Remove useless \u0027u\u0027 in the output of \"cinder get-capabilities\" Removes unicode \u0027u\u0027 response for \"cinder get-capabilities\"","message":"Remove useless \u0027u\u0027 in the output of \"cinder get-capabilities\"\nRemoves unicode \u0027u\u0027 response for \"cinder get-capabilities\"\n\nThe output of \"cinder get-capabilities\" command returns unicoded response.\nAs a refactoring effort added formatters to return non-unicode response.\n\nAdded formatters\u003dsorted(prop.keys()) as suggested by the reviewer, to avoid tying server\nknowledge to the client and also any update on the server side would easily reflect here.\n*Fixed pep8 issues\n\nCloses-bug: #1680444\nChange-Id: Ie38236db364d59ddab42cb925d0435777b0ffe86\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a7fac5abdfacde12a410fb4e41e7911a70e01d73"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/a7fac5abdfacde12a410fb4e41e7911a70e01d73"}]},"branch":"refs/heads/master"},"8672eb66b96115efc3e794863b54ceeb0966708d":{"kind":"NO_CODE_CHANGE","_number":5,"created":"2018-01-23 09:41:20.000000000","uploader":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"ref":"refs/changes/59/454159/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/59/454159/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/5"}}},"commit":{"parents":[{"commit":"0a323eaf942869439edc05a89c6cc226033bbe6c","subject":"Merge \"Support for reporting backend state in service list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0a323eaf942869439edc05a89c6cc226033bbe6c"}]}],"author":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2017-04-06 12:30:09.000000000","tz":-420},"committer":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2018-01-23 09:40:16.000000000","tz":330},"subject":"Removes unicode \u0027u\u0027 response for \"cinder get-capabilities\"","message":"Removes unicode \u0027u\u0027 response for \"cinder get-capabilities\"\n\nThe output of \"cinder get-capabilities\" command returns unicoded response.\nAs a refactoring effort added formatters to return non-unicode response.\n\nAdded formatters\u003dsorted(prop.keys()) as suggested by the reviewer, to avoid tying server\nknowledge to the client and also any update on the server side would easily reflect here.\n*Fixed pep8 issues\n\nCloses-bug: #1680444\nChange-Id: Ie38236db364d59ddab42cb925d0435777b0ffe86\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8672eb66b96115efc3e794863b54ceeb0966708d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/8672eb66b96115efc3e794863b54ceeb0966708d"}]},"branch":"refs/heads/master"},"c3a22519bcd91db227146a982bfb281092428f6c":{"kind":"REWORK","_number":6,"created":"2018-01-23 13:06:14.000000000","uploader":{"_account_id":27311,"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","username":"abijithanadagouda"},"ref":"refs/changes/59/454159/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/59/454159/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/59/454159/6"}}},"commit":{"parents":[{"commit":"0a323eaf942869439edc05a89c6cc226033bbe6c","subject":"Merge \"Support for reporting backend state in service list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/0a323eaf942869439edc05a89c6cc226033bbe6c"}]}],"author":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2017-04-06 12:30:09.000000000","tz":-420},"committer":{"name":"Abijitha Nadagouda","email":"abijitha.nadagouda@tatacommunications.com","date":"2018-01-23 13:02:11.000000000","tz":330},"subject":"Removes unicode \u0027u\u0027 response for \"cinder get-capabilities\"","message":"Removes unicode \u0027u\u0027 response for \"cinder get-capabilities\"\n\nThe output of \"cinder get-capabilities\" command returns\nunicoded response. But it would appear from the utils\nclass that setting formatters will go through the\ncapabilities dict and make sure all values are properly\nstring formatted. Therefore added formatters to return\nstring formatted response.\n\nAdded formatters\u003dsorted(prop.keys()) line instead of\nstatic values as suggested by the reviewer, to avoid\ntying server knowledge to the client and also any\nupdate on the server side would easily reflect here.\n\nCloses-bug: #1680444\nChange-Id: Ie38236db364d59ddab42cb925d0435777b0ffe86\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c3a22519bcd91db227146a982bfb281092428f6c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/c3a22519bcd91db227146a982bfb281092428f6c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}