)]}'
{"id":"openstack%2Fpython-cinderclient~436169","triplet_id":"openstack%2Fpython-cinderclient~master~I32693abe3e0aa56afdd945f782373b5c6fb5f8a4","project":"openstack/python-cinderclient","branch":"master","topic":"add_mode","hashtags":[],"change_id":"I32693abe3e0aa56afdd945f782373b5c6fb5f8a4","subject":"Add \"mode\" parameter mode for attachment action","status":"ABANDONED","created":"2017-02-20 19:58:06.000000000","updated":"2017-03-29 15:59:48.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"bbdfc9ab4dcbead60771e1551e32cae7790b3fb8","_number":436169,"virtual_id_number":436169,"owner":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"value":-1,"date":"2017-03-27 16:46:43.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"}],"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":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"disliked":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"all":[{"value":-1,"date":"2017-03-22 13:50:57.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2017-03-16 11:48:21.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},{"value":-1,"date":"2017-03-25 02:58:58.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"rejected":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"all":[{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"value":-1,"date":"2017-03-24 01:02:24.000000000","permitted_voting_range":{"min":-1,"max":0},"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"}],"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":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-03-16 11:48:21.000000000","updated_by":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"reviewer":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"state":"REVIEWER"},{"updated":"2017-03-22 13:50:57.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-03-25 02:58:58.000000000","updated_by":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"reviewer":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"state":"REVIEWER"},{"updated":"2017-03-27 16:46:43.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"7cfd6cb670652242ecc585aa38f28b391fb0c102","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-02-20 19:58:06.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"ae73736cc32b9266c0544dd953c3a597b88dc89a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-02-20 21:28:15.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/69/436169/1/check/gate-python-cinderclient-docs-ubuntu-xenial/e50a0d0//doc/build/html/ : SUCCESS in 2m 38s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/69/436169/1/check/gate-python-cinderclient-pep8-ubuntu-xenial/28b309b/ : SUCCESS in 5m 37s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/69/436169/1/check/gate-python-cinderclient-python27-ubuntu-xenial/9cf04cc/ : SUCCESS in 2m 17s\n- gate-python-cinderclient-python35 http://logs.openstack.org/69/436169/1/check/gate-python-cinderclient-python35/bf4313b/ : SUCCESS in 2m 35s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/69/436169/1/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/5621ff3/ : SUCCESS in 1h 28m 33s\n- gate-python-cinderclient-pypy http://logs.openstack.org/69/436169/1/check/gate-python-cinderclient-pypy/cb5f592/ : SUCCESS in 4m 39s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/69/436169/1/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/f9a44db/ : SUCCESS in 17m 11s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/69/436169/1/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/c938586/ : FAILURE in 18m 34s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/69/436169/1/check/gate-python-cinderclient-pylint-ubuntu-xenial/af1c126/ : SUCCESS in 4m 27s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"aaa3d000ebfee548460c6eb8dea2a1df469b0820","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-14 13:42:11.000000000","message":"Patch Set 1: 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":1},{"id":"0f71b76e868b4912b63e60d79286ba578fc0efa1","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-03-14 14:06:32.000000000","message":"Patch Set 2: Patch Set 1 was rebased","accounts_in_message":[],"_revision_number":1},{"id":"2e32267e11bf60a8c387d0aa924bda08f57a15fd","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-14 15:11:27.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/69/436169/2/check/gate-python-cinderclient-docs-ubuntu-xenial/a34ccaa//doc/build/html/ : SUCCESS in 1m 55s\n- gate-python-cinderclient-pep8-ubuntu-xenial http://logs.openstack.org/69/436169/2/check/gate-python-cinderclient-pep8-ubuntu-xenial/f32c7b7/ : SUCCESS in 4m 12s\n- gate-python-cinderclient-python27-ubuntu-xenial http://logs.openstack.org/69/436169/2/check/gate-python-cinderclient-python27-ubuntu-xenial/642220d/ : SUCCESS in 4m 08s\n- gate-python-cinderclient-python35 http://logs.openstack.org/69/436169/2/check/gate-python-cinderclient-python35/aa23894/ : SUCCESS in 3m 24s\n- gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial http://logs.openstack.org/69/436169/2/check/gate-tempest-dsvm-neutron-src-python-cinderclient-ubuntu-xenial/31bb06f/ : SUCCESS in 1h 03m 46s\n- gate-python-cinderclient-pypy http://logs.openstack.org/69/436169/2/check/gate-python-cinderclient-pypy/76002d8/ : SUCCESS in 2m 48s (non-voting)\n- gate-cinderclient-dsvm-functional-ubuntu-xenial-nv http://logs.openstack.org/69/436169/2/check/gate-cinderclient-dsvm-functional-ubuntu-xenial-nv/d384534/ : SUCCESS in 17m 14s (non-voting)\n- gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv http://logs.openstack.org/69/436169/2/check/gate-cinderclient-dsvm-functional-identity-v3-only-ubuntu-xenial-nv/ddb52bd/ : SUCCESS in 26m 23s (non-voting)\n- gate-python-cinderclient-pylint-ubuntu-xenial http://logs.openstack.org/69/436169/2/check/gate-python-cinderclient-pylint-ubuntu-xenial/32cf3ac/ : SUCCESS in 3m 55s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"730585ae92d79e0fe8b92fd0aaf5b20659146dfc","author":{"_account_id":14614,"name":"Kyrylo Romanenko","display_name":"K.Romanenko","email":"romanenko-kv@hotmail.com","username":"kromanenko"},"date":"2017-03-16 11:48:21.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"ecf6980a31331acf3ea1c789f338f6f7cb329309","author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"date":"2017-03-22 13:50:57.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)\n\nAdd please description why do we need this change.\nAlso from what api version we support this arg?\n\nAnd we need to add release note for it and unit tests.","accounts_in_message":[],"_revision_number":2},{"id":"f8e4bae76c8da05f7d0a5e875d8c88e1e5a1a73d","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-03-24 01:02:24.000000000","message":"Patch Set 2: Workflow-1\n\nupdate later","accounts_in_message":[],"_revision_number":2},{"id":"82eff445a48d9f086cf2dd8a1573b0a0cb070679","author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"date":"2017-03-25 02:58:58.000000000","message":"Patch Set 2: Code-Review-1\n\nHey wangxiyuan, the \u0027mode\u0027 parameter is designed not to be exposed by our CLI [1]. Also I think we really don\u0027t need the connect argument:)\n\n[1] http://eavesdrop.openstack.org/irclogs/%23openstack-cinder/%23openstack-cinder.2017-03-24.log.html#t2017-03-24T13:12:59","accounts_in_message":[],"_revision_number":2},{"id":"73e14f666b4476226550b77593a4e04ef8b83c26","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-03-25 03:13:15.000000000","message":"Patch Set 2:\n\nAccording the IRC log, i\u0027m still confuse about it. We expose every parameter, Why only \"mode\" is missing?\n\nOr  TBH, even this command should not be existed at CLI layer. Just leave it at lib layer so that it can be used by other projects like Nova but should not be used by users.","accounts_in_message":[],"_revision_number":2},{"id":"d2ab6429926bd53b7bddb8f54e4a92a1b41c428d","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-03-27 16:46:43.000000000","message":"Patch Set 2: 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":2},{"id":"109a455f97662c0423d6e662005055c0ae298fcd","author":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"date":"2017-03-29 15:59:48.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"bfde022f6c6939dad4964daa98e9a0025882f4eb","revisions":{"b8875534ac7921b18b68c8bc24afeab8a043f5af":{"kind":"REWORK","_number":1,"created":"2017-02-20 19:58:06.000000000","uploader":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"ref":"refs/changes/69/436169/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/69/436169/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/69/436169/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/69/436169/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/69/436169/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/69/436169/1"}}},"commit":{"parents":[{"commit":"427ec2b786d0d1c1fcc5cee83bdd42839785f80a","subject":"Merge \"Remove duplicate columns from list output\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/427ec2b786d0d1c1fcc5cee83bdd42839785f80a"}]}],"author":{"name":"wangxiyuan","email":"wangxiyuan@huawei.com","date":"2017-02-20 19:51:04.000000000","tz":-300},"committer":{"name":"wangxiyuan","email":"wangxiyuan@huawei.com","date":"2017-02-20 19:57:50.000000000","tz":-300},"subject":"Add \"mode\" parameter mode for attachment action","message":"Add \"mode\" parameter mode for attachment action\n\n1. Add the \"mode\" parameter for attachment create/update.\n\n2. Fix the \"multipath\"\u0027s help message.\n\n3. Updat the \"connect\"\u0027s help message.\n\nChange-Id: I32693abe3e0aa56afdd945f782373b5c6fb5f8a4\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b8875534ac7921b18b68c8bc24afeab8a043f5af"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/b8875534ac7921b18b68c8bc24afeab8a043f5af"}]},"branch":"refs/heads/master"},"bfde022f6c6939dad4964daa98e9a0025882f4eb":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2017-03-14 14:06:32.000000000","uploader":{"_account_id":15054,"name":"wangxiyuan","email":"wangxiyuan1007@gmail.com","username":"wangxiyuan"},"ref":"refs/changes/69/436169/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/69/436169/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/69/436169/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/69/436169/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/69/436169/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/69/436169/2"}}},"commit":{"parents":[{"commit":"3b60eba9aa022a5f5a7b84ce236626f31493485b","subject":"Fix all_tenants doesn\u0027t work for group list","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/3b60eba9aa022a5f5a7b84ce236626f31493485b"}]}],"author":{"name":"wangxiyuan","email":"wangxiyuan@huawei.com","date":"2017-02-20 19:51:04.000000000","tz":-300},"committer":{"name":"wangxiyuan","email":"wangxiyuan@huawei.com","date":"2017-03-14 14:06:32.000000000","tz":0},"subject":"Add \"mode\" parameter mode for attachment action","message":"Add \"mode\" parameter mode for attachment action\n\n1. Add the \"mode\" parameter for attachment create/update.\n\n2. Fix the \"multipath\"\u0027s help message.\n\n3. Updat the \"connect\"\u0027s help message.\n\nChange-Id: I32693abe3e0aa56afdd945f782373b5c6fb5f8a4\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/bfde022f6c6939dad4964daa98e9a0025882f4eb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/bfde022f6c6939dad4964daa98e9a0025882f4eb"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}