)]}'
{"id":"openstack%2Fpython-openstackclient~360191","triplet_id":"openstack%2Fpython-openstackclient~master~Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6","project":"openstack/python-openstackclient","branch":"master","topic":"bug/1535704","hashtags":[],"change_id":"Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6","subject":"Support mark volume as bootable in volume set","status":"MERGED","created":"2016-08-25 02:54:46.000000000","updated":"2016-09-14 18:48:28.000000000","submitted":"2016-09-14 18:48:28.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":29,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c6f3d60e1f6305d70c24ae8cbcec180ecdeeb8e3","_number":360191,"virtual_id_number":360191,"owner":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":2,"date":"2016-09-14 18:48:28.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},{"value":0,"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"value":0,"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"value":0,"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":16443,"name":"Wenwu","email":"pengwenwu2008@163.com","username":"Wenwu"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},{"value":0,"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"value":0,"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"value":2,"date":"2016-09-14 16:37:30.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":16443,"name":"Wenwu","email":"pengwenwu2008@163.com","username":"Wenwu"}],"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":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"all":[{"value":0,"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},{"value":0,"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},{"value":0,"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"value":0,"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"value":1,"date":"2016-09-14 16:37:30.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"_account_id":16443,"name":"Wenwu","email":"pengwenwu2008@163.com","username":"Wenwu"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"_account_id":16443,"name":"Wenwu","email":"pengwenwu2008@163.com","username":"Wenwu"},{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-09-06 07:36:24.000000000","updated_by":{"_account_id":16443,"name":"Wenwu","email":"pengwenwu2008@163.com","username":"Wenwu"},"reviewer":{"_account_id":16443,"name":"Wenwu","email":"pengwenwu2008@163.com","username":"Wenwu"},"state":"REVIEWER"},{"updated":"2016-09-13 09:06:31.000000000","updated_by":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"reviewer":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"state":"REVIEWER"},{"updated":"2016-09-13 13:02:40.000000000","updated_by":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"reviewer":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"state":"REVIEWER"},{"updated":"2016-09-13 13:10:50.000000000","updated_by":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"reviewer":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"state":"REVIEWER"},{"updated":"2016-09-14 07:25:33.000000000","updated_by":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"reviewer":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"state":"REVIEWER"},{"updated":"2016-09-14 16:37:30.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2016-09-14 18:48:28.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"e0b057d602435973d2127392ce8d7e7745c20232","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-25 02:54:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a0f751e69bd07b1958ed1951f5fbef39370cc7d1","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-08-25 03:42:00.000000000","message":"Patch Set 1: Code-Review+2\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"d379766c7925825140292491f0cbf9fc59c71a14","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-25 04:10:28.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/1/check/gate-python-openstackclient-docs-ubuntu-xenial/96d70ff//doc/build/html/ : SUCCESS in 4m 18s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/1/check/gate-python-openstackclient-pep8-ubuntu-xenial/26a7488/ : SUCCESS in 2m 43s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/1/check/gate-python-openstackclient-python27-ubuntu-xenial/07c0cca/ : SUCCESS in 2m 43s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/1/check/gate-python-openstackclient-python34/4ddabbb/ : SUCCESS in 2m 59s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/1/check/gate-python-openstackclient-python35-nv/1fb8e46/ : SUCCESS in 4m 04s (non-voting)\n- gate-python-openstackclient-releasenotes http://logs.openstack.org/91/360191/1/check/gate-python-openstackclient-releasenotes/fb030d9/ : FAILURE in 6m 05s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/1/check/gate-tempest-dsvm-neutron-src-python-openstackclient/aff40c5/ : SUCCESS in 1h 13m 34s\n- check-osc-plugins http://logs.openstack.org/91/360191/1/check/check-osc-plugins/a4cda57/ : SUCCESS in 5m 19s\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/1/check/gate-osc-dsvm-functional/1c63e0e/ : FAILURE in 10m 15s","accounts_in_message":[],"_revision_number":1},{"id":"ffb9d5d3a6f6179780b022cac4fe98add31b7a94","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-25 05:42:30.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"3a93f5b352d7021034ed8e38373ef9b10933598f","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-25 05:43:19.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"0619cf6caa00f4ce2f74587b92a2019692c7dc1d","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-08-25 05:49:56.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"468d78c05b233537fe6b438bc1e170b3e9c31de4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-25 06:35:37.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/2/check/gate-python-openstackclient-docs-ubuntu-xenial/3fe792d//doc/build/html/ : SUCCESS in 2m 47s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/2/check/gate-python-openstackclient-pep8-ubuntu-xenial/79aed96/ : SUCCESS in 2m 35s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/2/check/gate-python-openstackclient-python27-ubuntu-xenial/8ac7f90/ : SUCCESS in 2m 42s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/2/check/gate-python-openstackclient-python34/f37c0b0/ : SUCCESS in 5m 20s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/2/check/gate-python-openstackclient-python35-nv/a43e338/ : SUCCESS in 3m 30s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/2/check/gate-python-openstackclient-releasenotes/77d2980//releasenotes/build/html/ : SUCCESS in 4m 44s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/2/check/gate-tempest-dsvm-neutron-src-python-openstackclient/7c107a9/ : FAILURE in 41m 23s\n- check-osc-plugins http://logs.openstack.org/91/360191/2/check/check-osc-plugins/d6ceb6b/ : SUCCESS in 3m 07s\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/2/check/gate-osc-dsvm-functional/3f77afa/ : SUCCESS in 50m 28s","accounts_in_message":[],"_revision_number":2},{"id":"8e4ce8316074f307d62c4e7f09a9625b3827491d","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-25 06:44:32.000000000","message":"Patch Set 2:\n\nConfused by the tempest result, who can help with some suggestions for this?","accounts_in_message":[],"_revision_number":2},{"id":"a36c1888a95579530d3828fab3733afe3f83391a","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-25 07:57:27.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"105642f93144d7cf928036e7d5a4177c5860e82e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-25 09:09:55.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/2/check/gate-python-openstackclient-docs-ubuntu-xenial/e8a9761//doc/build/html/ : SUCCESS in 6m 11s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/2/check/gate-python-openstackclient-pep8-ubuntu-xenial/28d9ff8/ : SUCCESS in 4m 26s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/2/check/gate-python-openstackclient-python27-ubuntu-xenial/080267a/ : SUCCESS in 4m 39s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/2/check/gate-python-openstackclient-python34/39dd8b6/ : SUCCESS in 3m 24s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/2/check/gate-python-openstackclient-python35-nv/7abc436/ : SUCCESS in 4m 53s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/2/check/gate-python-openstackclient-releasenotes/bc48816//releasenotes/build/html/ : SUCCESS in 2m 32s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/2/check/gate-tempest-dsvm-neutron-src-python-openstackclient/e61abf0/ : SUCCESS in 1h 08m 30s\n- check-osc-plugins http://logs.openstack.org/91/360191/2/check/check-osc-plugins/77eaa97/ : SUCCESS in 5m 41s\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/2/check/gate-osc-dsvm-functional/3fe0baa/ : SUCCESS in 35m 31s","accounts_in_message":[],"_revision_number":2},{"id":"1b8d2c61399c28da053a494e979141822303318d","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-25 12:42:19.000000000","message":"Patch Set 2: Code-Review-1\n\n(4 comments)\n\nThanks for adding this option for volume :)","accounts_in_message":[],"_revision_number":2},{"id":"9e677e75e2bf9645abf18c86b324e0881297062c","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-25 13:33:19.000000000","message":"Patch Set 2:\n\n(4 comments)","accounts_in_message":[],"_revision_number":2},{"id":"1e5d5c1ca833b93b61990fcc54847e7a59628cd8","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-25 13:35:43.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"7ae7a684328c8cdaef1205655aad688a1ac8a946","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-25 13:44:06.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"58cd3ac2f681f0fc1a8151e7b8516fee133d8e9e","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-25 14:04:35.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"5a613bdc7e271d0165b5be5643ea0e3fb1edaed5","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-26 02:54:30.000000000","message":"Patch Set 2:\n\n@Huanxuan \u0026 Steve\nFor the volume set bootable func in volume set, I think maybe we can \njust make it easy as the one in the cinder manage.\n\nWhat\u0027s your opinion for this? thx~","accounts_in_message":[],"_revision_number":2},{"id":"f55a4386f1b49469baa80dd62751205034020809","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-08-26 03:00:34.000000000","message":"Patch Set 2:\n\nwhoa, what exactly does \"cinder manage\" do? how is it different than cinder update?","accounts_in_message":[],"_revision_number":2},{"id":"79816964540153026e4148c8a4a3acd1e6d2fbab","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-08-26 03:00:37.000000000","message":"Patch Set 2: -Code-Review","accounts_in_message":[],"_revision_number":2},{"id":"11915432a602c9114c4714ede0fc572393c0ad2b","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-26 03:16:37.000000000","message":"Patch Set 2:\n\n\u003e whoa, what exactly does \"cinder manage\" do? how is it different\n \u003e than cinder update?\n\nHi Steve, I failed to find the cinder update command in cinder help.\nI think what you mean is just the cinder manage.\n\nroot@controller01:/home/viouser# cinder help | grep update\n    backup-reset-state  Explicitly updates the backup state.\n    consisgroup-update  Updates a consistencygroup.\n    encryption-type-update\n    metadata-update-all\n    quota-class-update  Updates quotas for a quota class.\n    quota-update        Updates quotas for a tenant.\n    readonly-mode-update\n    reset-state         Explicitly updates the volume state in the Cinder\n    snapshot-metadata-update-all\n                        Explicitly updates the snapshot state.\n    type-update         Updates volume type name, description, and/or\nroot@controller01:/home/viouser# cinder help | grep manage\n    manage              Manage an existing volume.\n    snapshot-manage     Manage an existing snapshot.\n    snapshot-unmanage   Stop managing a snapshot.\n    unmanage            Stop managing a volume.","accounts_in_message":[],"_revision_number":2},{"id":"90583aae145b14b40ea6d9a8f09b1e081d7868e9","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-26 03:34:42.000000000","message":"Patch Set 2:\n\n\u003e \u003e whoa, what exactly does \"cinder manage\" do? how is it different\n \u003e \u003e than cinder update?\n \u003e \n \u003e Hi Steve, I failed to find the cinder update command in cinder\n \u003e help.\n \u003e I think what you mean is just the cinder manage.\n \u003e \n \u003e root@controller01:/home/viouser# cinder help | grep update\n \u003e backup-reset-state  Explicitly updates the backup state.\n \u003e consisgroup-update  Updates a consistencygroup.\n \u003e encryption-type-update\n \u003e metadata-update-all\n \u003e quota-class-update  Updates quotas for a quota class.\n \u003e quota-update        Updates quotas for a tenant.\n \u003e readonly-mode-update\n \u003e reset-state         Explicitly updates the volume state in the\n \u003e Cinder\n \u003e snapshot-metadata-update-all\n \u003e Explicitly updates the snapshot state.\n \u003e type-update         Updates volume type name, description, and/or\n \u003e root@controller01:/home/viouser# cinder help | grep manage\n \u003e manage              Manage an existing volume.\n \u003e snapshot-manage     Manage an existing snapshot.\n \u003e snapshot-unmanage   Stop managing a snapshot.\n \u003e unmanage            Stop managing a volume.\n\nHi, Qing Tang, cinderclient has not a update command for volume, but it has many subcommands whitch are the part of setting.\nFor example:\ncinder reset-state \u003d\u003d\u003e openstack volume set --state\nSo I think:\ncinder set-bootable \u003d\u003d\u003e openstack volume set --bootable","accounts_in_message":[],"_revision_number":2},{"id":"e7fda04db04037a860f201f923c04909e6d7283a","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-08-26 04:18:39.000000000","message":"Patch Set 2:\n\nwow, i wonder why they did it like that? do you guys see a reason to implement the \"cinder manage\" command? or just incorporate it all into \"volume set\" ?","accounts_in_message":[],"_revision_number":2},{"id":"094afca4738d8a91a12dd16f4a6bfe2d1ca90b18","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-26 05:04:52.000000000","message":"Patch Set 2:\n\n\u003e wow, i wonder why they did it like that? do you guys see a reason\n \u003e to implement the \"cinder manage\" command? or just incorporate it\n \u003e all into \"volume set\" ?\n\nList the help details of cinder manage and openstack volume set below:\n\nroot@controller01:/home/viouser# cinder help manage\nusage: cinder manage [--id-type \u003cid-type\u003e] [--name \u003cname\u003e]\n                     [--description \u003cdescription\u003e]\n                     [--volume-type \u003cvolume-type\u003e]\n                     [--availability-zone \u003cavailability-zone\u003e]\n                     [--metadata [\u003ckey\u003dvalue\u003e [\u003ckey\u003dvalue\u003e ...]]] [--bootable]\n                     \u003chost\u003e \u003cidentifier\u003e\n\nManage an existing volume.\n\nPositional arguments:\n  \u003chost\u003e                Cinder host on which the existing volume resides;\n                        takes the form: host@backend-name#pool\n  \u003cidentifier\u003e          Name or other Identifier for existing volume\n\nOptional arguments:\n  --id-type \u003cid-type\u003e   Type of backend device identifier provided, typically\n                        source-name or source-id (Default\u003dsource-name)\n  --name \u003cname\u003e         Volume name (Default\u003dNone)\n  --description \u003cdescription\u003e\n                        Volume description (Default\u003dNone)\n  --volume-type \u003cvolume-type\u003e\n                        Volume type (Default\u003dNone)\n  --availability-zone \u003cavailability-zone\u003e\n                        Availability zone for volume (Default\u003dNone)\n  --metadata [\u003ckey\u003dvalue\u003e [\u003ckey\u003dvalue\u003e ...]]\n                        Metadata key\u003dvalue pairs (Default\u003dNone)\n  --bootable            Specifies that the newly created volume should be\n                        marked as bootable\n\nroot@controller01:/home/viouser# openstack --insecure help volume set\nusage: openstack volume set [-h] [--name \u003cname\u003e] [--description \u003cdescription\u003e]\n                            [--size \u003csize\u003e] [--property \u003ckey\u003dvalue\u003e]\n                            \u003cvolume\u003e\n\nSet volume properties\n\npositional arguments:\n  \u003cvolume\u003e              Volume to change (name or ID)\n\noptional arguments:\n  -h, --help            show this help message and exit\n  --name \u003cname\u003e         New volume name\n  --description \u003cdescription\u003e\n                        New volume description\n  --size \u003csize\u003e         Extend volume size in GB\n  --property \u003ckey\u003dvalue\u003e\n                        Property to add or modify for this volume (repeat\n                        option to set multiple properties)\n\nAs you see, there are some similar func set in these two commands.\n\nI think make the volume set as a centralised place for volume instance setting maybe a good choice, thus will make the openstack volume part more simple and clear.","accounts_in_message":[],"_revision_number":2},{"id":"71e22c559ea68933fae41c64431d4548ab91c836","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-26 05:24:46.000000000","message":"Patch Set 2:\n\nNow we have 2 choices here:\n\n1 As the one in cinder manage, which I think is straightforward for using but will\nset the bootable to false when executing without the bootable option. (cinder manage also doing like this.)\n\n2 As cinder set-bootable, which is precise but complex for user input from the choice list.\n   The default value is None.\n\nSo which style you like more?","accounts_in_message":[],"_revision_number":2},{"id":"25d905ea0d847e509fea71b560f79ac563679608","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-26 07:20:23.000000000","message":"Patch Set 2:\n\n\u003e Now we have 2 choices here:\n \u003e \n \u003e 1 As the one in cinder manage, which I think is straightforward for\n \u003e using but will\n \u003e set the bootable to false when executing without the bootable\n \u003e option. (cinder manage also doing like this.)\n \u003e \n \u003e 2 As cinder set-bootable, which is precise but complex for user\n \u003e input from the choice list.\n \u003e The default value is None.\n \u003e \n \u003e So which style you like more?\n\nmanage and set_bootable call different API，you can find the APIs in\nhttps://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volumes.py\nWhat does manage command aim to do? whether can we replace the manage command by using set command? please make sure about it.","accounts_in_message":[],"_revision_number":2},{"id":"2d120dba58638f1b43fff3f0d06c22fc1937026b","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-26 07:42:48.000000000","message":"Patch Set 2:\n\n\u003e \u003e Now we have 2 choices here:\n \u003e \u003e\n \u003e \u003e 1 As the one in cinder manage, which I think is straightforward\n \u003e for\n \u003e \u003e using but will\n \u003e \u003e set the bootable to false when executing without the bootable\n \u003e \u003e option. (cinder manage also doing like this.)\n \u003e \u003e\n \u003e \u003e 2 As cinder set-bootable, which is precise but complex for user\n \u003e \u003e input from the choice list.\n \u003e \u003e The default value is None.\n \u003e \u003e\n \u003e \u003e So which style you like more?\n \u003e \n \u003e manage and set_bootable call different API，you can find the APIs in\n \u003e https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volumes.py\n \u003e What does manage command aim to do? whether can we replace the\n \u003e manage command by using set command? please make sure about it.\n\nYes, agree with you.That\u0027s depending on how we define the property of volume.\n\nI think the meaning of property here besides the key\u003dvalue property, but also including other attributes such as name, size.  It is just like what to do in the cinder manage.\n\nRef the openstack horizon page, we can see the spec list of volume:\nSpecs\n\nSize   1 GiB\nBootable  No\nEncrypted  No\nCreated  Aug. 25, 2016, 6:42 a.m.\n\nSo bootable is in the same spec table as size, encrypted.\n\nIf the bootable attribute also can be traded as one property of volume disk, then I think we can set it in volume set. If so, we don\u0027t need an external cmd to set the bootable attribute.\n\nOtherwise, we need consider another way for this.","accounts_in_message":[],"_revision_number":2},{"id":"8702e84eea13f8f5943a955177c094abae4a2d00","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-26 09:34:14.000000000","message":"Patch Set 2:\n\n\u003e \u003e \u003e Now we have 2 choices here:\n \u003e \u003e \u003e\n \u003e \u003e \u003e 1 As the one in cinder manage, which I think is straightforward\n \u003e \u003e for\n \u003e \u003e \u003e using but will\n \u003e \u003e \u003e set the bootable to false when executing without the bootable\n \u003e \u003e \u003e option. (cinder manage also doing like this.)\n \u003e \u003e \u003e\n \u003e \u003e \u003e 2 As cinder set-bootable, which is precise but complex for user\n \u003e \u003e \u003e input from the choice list.\n \u003e \u003e \u003e The default value is None.\n \u003e \u003e \u003e\n \u003e \u003e \u003e So which style you like more?\n \u003e \u003e\n \u003e \u003e manage and set_bootable call different API，you can find the APIs\n \u003e in\n \u003e \u003e https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volumes.py\n \u003e \u003e What does manage command aim to do? whether can we replace the\n \u003e \u003e manage command by using set command? please make sure about it.\n \u003e \n \u003e Yes, agree with you.That\u0027s depending on how we define the property\n \u003e of volume.\n \u003e \n \u003e I think the meaning of property here besides the key\u003dvalue\n \u003e property, but also including other attributes such as name, size. \n \u003e It is just like what to do in the cinder manage.\n \u003e \n \u003e Ref the openstack horizon page, we can see the spec list of volume:\n \u003e Specs\n \u003e \n \u003e Size   1 GiB\n \u003e Bootable  No\n \u003e Encrypted  No\n \u003e Created  Aug. 25, 2016, 6:42 a.m.\n \u003e \n \u003e So bootable is in the same spec table as size, encrypted.\n \u003e \n \u003e If the bootable attribute also can be traded as one property of\n \u003e volume disk, then I think we can set it in volume set. If so, we\n \u003e don\u0027t need an external cmd to set the bootable attribute.\n \u003e \n \u003e Otherwise, we need consider another way for this.\n\nI see the help message of option \"--bootable\" in cinder manage command:\n  --bootable    Specifies that the newly created volume should be marked as bootable\nDose this option works for a newly created volume?","accounts_in_message":[],"_revision_number":2},{"id":"992844f1dc64c6db53b312df49f2778e0e71b45b","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-26 13:37:03.000000000","message":"Patch Set 2:\n\n\u003e \u003e \u003e \u003e Now we have 2 choices here:\n \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e 1 As the one in cinder manage, which I think is\n \u003e straightforward\n \u003e \u003e \u003e for\n \u003e \u003e \u003e \u003e using but will\n \u003e \u003e \u003e \u003e set the bootable to false when executing without the bootable\n \u003e \u003e \u003e \u003e option. (cinder manage also doing like this.)\n \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e 2 As cinder set-bootable, which is precise but complex for\n \u003e user\n \u003e \u003e \u003e \u003e input from the choice list.\n \u003e \u003e \u003e \u003e The default value is None.\n \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e So which style you like more?\n \u003e \u003e \u003e\n \u003e \u003e \u003e manage and set_bootable call different API，you can find the\n \u003e APIs\n \u003e \u003e in\n \u003e \u003e \u003e https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volumes.py\n \u003e \u003e \u003e What does manage command aim to do? whether can we replace the\n \u003e \u003e \u003e manage command by using set command? please make sure about it.\n \u003e \u003e\n \u003e \u003e Yes, agree with you.That\u0027s depending on how we define the\n \u003e property\n \u003e \u003e of volume.\n \u003e \u003e\n \u003e \u003e I think the meaning of property here besides the key\u003dvalue\n \u003e \u003e property, but also including other attributes such as name, size.\n \u003e \u003e It is just like what to do in the cinder manage.\n \u003e \u003e\n \u003e \u003e Ref the openstack horizon page, we can see the spec list of\n \u003e volume:\n \u003e \u003e Specs\n \u003e \u003e\n \u003e \u003e Size   1 GiB\n \u003e \u003e Bootable  No\n \u003e \u003e Encrypted  No\n \u003e \u003e Created  Aug. 25, 2016, 6:42 a.m.\n \u003e \u003e\n \u003e \u003e So bootable is in the same spec table as size, encrypted.\n \u003e \u003e\n \u003e \u003e If the bootable attribute also can be traded as one property of\n \u003e \u003e volume disk, then I think we can set it in volume set. If so, we\n \u003e \u003e don\u0027t need an external cmd to set the bootable attribute.\n \u003e \u003e\n \u003e \u003e Otherwise, we need consider another way for this.\n \u003e \n \u003e I see the help message of option \"--bootable\" in cinder manage\n \u003e command:\n \u003e --bootable    Specifies that the newly created volume should be\n \u003e marked as bootable\n \u003e Dose this option works for a newly created volume?\n\nFrom the help guide, it is yes.\nBut since the newly created volume is also the normal volume as others,so this cmd also can be used for changing the bootable attribute of volume.\n\nAlso from the code level, cinder set-volume also\ncalled the set_bootable func in volumes.\nSo I think they are the same one from the func level.\n\ndef set_bootable(self, volume, flag):\n    return self._action(\u0027os-set_bootable\u0027,\n                        base.getid(volume),\n                        {\u0027bootable\u0027: flag})\n\n@utils.arg(\u0027volume\u0027, metavar\u003d\u0027\u003cvolume\u003e\u0027, help\u003d\u0027ID of the volume to update.\u0027)\n@utils.arg(\u0027bootable\u0027,\n           metavar\u003d\u0027\u003cTrue|true|False|false\u003e\u0027,\n           choices\u003d[\u0027True\u0027, \u0027true\u0027, \u0027False\u0027, \u0027false\u0027],\n           help\u003d\u0027Flag to indicate whether volume is bootable.\u0027)\n@utils.service_type(\u0027volumev3\u0027)\ndef do_set_bootable(cs, args):\n    \"\"\"Update bootable status of a volume.\"\"\"\n    volume \u003d utils.find_volume(cs, args.volume)\n    cs.volumes.set_bootable(volume,\n                            strutils.bool_from_string(args.bootable,\n                                                      strict\u003dTrue))","accounts_in_message":[],"_revision_number":2},{"id":"c2fea6da058d02732364e194a0a7defc82e10763","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-26 13:38:42.000000000","message":"Patch Set 2:\n\n\u003e \u003e \u003e \u003e \u003e Now we have 2 choices here:\n \u003e \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e \u003e 1 As the one in cinder manage, which I think is\n \u003e \u003e straightforward\n \u003e \u003e \u003e \u003e for\n \u003e \u003e \u003e \u003e \u003e using but will\n \u003e \u003e \u003e \u003e \u003e set the bootable to false when executing without the\n \u003e bootable\n \u003e \u003e \u003e \u003e \u003e option. (cinder manage also doing like this.)\n \u003e \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e \u003e 2 As cinder set-bootable, which is precise but complex for\n \u003e \u003e user\n \u003e \u003e \u003e \u003e \u003e input from the choice list.\n \u003e \u003e \u003e \u003e \u003e The default value is None.\n \u003e \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e \u003e So which style you like more?\n \u003e \u003e \u003e \u003e\n \u003e \u003e \u003e \u003e manage and set_bootable call different API，you can find the\n \u003e \u003e APIs\n \u003e \u003e \u003e in\n \u003e \u003e \u003e \u003e https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/volumes.py\n \u003e \u003e \u003e \u003e What does manage command aim to do? whether can we replace\n \u003e the\n \u003e \u003e \u003e \u003e manage command by using set command? please make sure about\n \u003e it.\n \u003e \u003e \u003e\n \u003e \u003e \u003e Yes, agree with you.That\u0027s depending on how we define the\n \u003e \u003e property\n \u003e \u003e \u003e of volume.\n \u003e \u003e \u003e\n \u003e \u003e \u003e I think the meaning of property here besides the key\u003dvalue\n \u003e \u003e \u003e property, but also including other attributes such as name,\n \u003e size.\n \u003e \u003e \u003e It is just like what to do in the cinder manage.\n \u003e \u003e \u003e\n \u003e \u003e \u003e Ref the openstack horizon page, we can see the spec list of\n \u003e \u003e volume:\n \u003e \u003e \u003e Specs\n \u003e \u003e \u003e\n \u003e \u003e \u003e Size   1 GiB\n \u003e \u003e \u003e Bootable  No\n \u003e \u003e \u003e Encrypted  No\n \u003e \u003e \u003e Created  Aug. 25, 2016, 6:42 a.m.\n \u003e \u003e \u003e\n \u003e \u003e \u003e So bootable is in the same spec table as size, encrypted.\n \u003e \u003e \u003e\n \u003e \u003e \u003e If the bootable attribute also can be traded as one property of\n \u003e \u003e \u003e volume disk, then I think we can set it in volume set. If so,\n \u003e we\n \u003e \u003e \u003e don\u0027t need an external cmd to set the bootable attribute.\n \u003e \u003e \u003e\n \u003e \u003e \u003e Otherwise, we need consider another way for this.\n \u003e \u003e\n \u003e \u003e I see the help message of option \"--bootable\" in cinder manage\n \u003e \u003e command:\n \u003e \u003e --bootable    Specifies that the newly created volume should be\n \u003e \u003e marked as bootable\n \u003e \u003e Dose this option works for a newly created volume?\n \u003e \n \u003e From the help guide, it is yes.\n \u003e But since the newly created volume is also the normal volume as\n \u003e others,so this cmd also can be used for changing the bootable\n \u003e attribute of volume.\n \u003e \n \u003e Also from the code level, cinder set-volume also\n \u003e called the set_bootable func in volumes.\n \u003e So I think they are the same one from the func level.\n \u003e \n \u003e def set_bootable(self, volume, flag):\n \u003e return self._action(\u0027os-set_bootable\u0027,\n \u003e base.getid(volume),\n \u003e {\u0027bootable\u0027: flag})\n \u003e \n \u003e @utils.arg(\u0027volume\u0027, metavar\u003d\u0027\u003cvolume\u003e\u0027, help\u003d\u0027ID of the volume to\n \u003e update.\u0027)\n \u003e @utils.arg(\u0027bootable\u0027,\n \u003e metavar\u003d\u0027\u003cTrue|true|False|false\u003e\u0027,\n \u003e choices\u003d[\u0027True\u0027, \u0027true\u0027, \u0027False\u0027, \u0027false\u0027],\n \u003e help\u003d\u0027Flag to indicate whether volume is bootable.\u0027)\n \u003e @utils.service_type(\u0027volumev3\u0027)\n \u003e def do_set_bootable(cs, args):\n \u003e \"\"\"Update bootable status of a volume.\"\"\"\n \u003e volume \u003d utils.find_volume(cs, args.volume)\n \u003e cs.volumes.set_bootable(volume,\n \u003e strutils.bool_from_string(args.bootable,\n \u003e strict\u003dTrue))\n\nI mean \u0027cinder set-bootable\u0027 in \"Also from the code level, cinder set-volume...\". ~~","accounts_in_message":[],"_revision_number":2},{"id":"6c3c7200a78f58b482d115e9b3ab1601503fb87b","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-27 01:16:07.000000000","message":"Patch Set 2:\n\nWell, seems the --bootable in cinder manage not working as expected.\nMaybe there is some issue for that cmd func.\n\nBut anyway, in the current fix, the same func called as the one\nin cinder set-bootable when we try to set the bootable property\nin volume set. So it works fine.","accounts_in_message":[],"_revision_number":2},{"id":"882b13c5e51023656413585de318230c56eb1966","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-28 12:04:19.000000000","message":"Patch Set 2:\n\nLet me sort out my thoughts:\n1). I think manage command is similar to create command not set command, it will create a new volume and set the properties for the newly created volume.\n2). As you said, we can also use a set command to set properties for a new volume, but it is easier to use only a option to set them when new volume created. In OSC you will find many options in create commands are the same set commands, this is good thing.\n3). The code logic of options in create commands are different to that in set commands. In create command, we usually specify a default value for the option, so that if the option is not specified, then will work with the default value, but in set command we cannot do it, the vaule of the option should be specified only by users, if not, then nothing happen.\n4). Totally, we cannot use volume set command to replace manage command, these two command are all needed. And we cannot implement this \"--bootable\" option follow the option in manage command, it is different thing, we should implement this match the \"cinder set-bootable\" command.\nHope you can understand my thoughts","accounts_in_message":[],"_revision_number":2},{"id":"39fac779977533b3e2ba9504af1ca977456af9ec","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-28 12:14:40.000000000","message":"Patch Set 2:\n\nIf you agree my opinion, I have two suggestions, you can choose one way to implement this option:\n1)Match the \"cinder set-bootable\":\n    volume set \u003cvolume\u003e [--bootable \u003ctrue|false|True|False\u003e]\n2)Add two mutually exclusive options, such as \"--bootable\" and \"--no-bootable\":\n    volume set \u003cvolume\u003e [--bootable|--no-bootable]\nWhat do you think about?","accounts_in_message":[],"_revision_number":2},{"id":"5f2b781ed55e7f7dcfdf7ef127e9cff082e2f8bc","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-29 02:06:51.000000000","message":"Patch Set 2:\n\n\u003e If you agree my opinion, I have two suggestions, you can choose one\n \u003e way to implement this option:\n \u003e 1)Match the \"cinder set-bootable\":\n \u003e volume set \u003cvolume\u003e [--bootable \u003ctrue|false|True|False\u003e]\n \u003e 2)Add two mutually exclusive options, such as \"--bootable\" and\n \u003e \"--no-bootable\":\n \u003e volume set \u003cvolume\u003e [--bootable|--no-bootable]\n \u003e What do you think about?\n\nThanks for your suggestions.\nFrom my openstack client UE, I prefer the second one.\nActually in my first design, it is --bootable | --non-bootable.But I drop it\nwhen I compare the cinder manage --bootable.\nTo avoid the side effect, the default value here will set be None in volume set.\nWill send out my set 3 later.~","accounts_in_message":[],"_revision_number":2},{"id":"15e08bddc717036198ce8f556e76a20b7cfeaec2","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-29 05:19:54.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"cbc9b005d0201acab1dbced589476333254c2b89","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-29 06:26:28.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/3/check/gate-python-openstackclient-docs-ubuntu-xenial/3ea5b48//doc/build/html/ : SUCCESS in 6m 13s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/3/check/gate-python-openstackclient-pep8-ubuntu-xenial/428d8ce/ : SUCCESS in 2m 32s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/3/check/gate-python-openstackclient-python27-ubuntu-xenial/ed8e731/ : SUCCESS in 2m 35s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/3/check/gate-python-openstackclient-python34/ec2f568/ : SUCCESS in 4m 15s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/3/check/gate-python-openstackclient-python35-nv/c07a67d/ : SUCCESS in 3m 00s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/3/check/gate-python-openstackclient-releasenotes/bff7a13//releasenotes/build/html/ : SUCCESS in 2m 33s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/3/check/gate-tempest-dsvm-neutron-src-python-openstackclient/8465b0b/ : SUCCESS in 1h 06m 21s\n- check-osc-plugins http://logs.openstack.org/91/360191/3/check/check-osc-plugins/2120c6d/ : SUCCESS in 3m 29s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/3/check/gate-osc-dsvm-functional/be6aaf0/ : SUCCESS in 36m 49s","accounts_in_message":[],"_revision_number":3},{"id":"cebb56c8a7216e47113f28c4971791559663265b","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-29 10:19:42.000000000","message":"Patch Set 3: Code-Review-1\n\n(4 comments)","accounts_in_message":[],"_revision_number":3},{"id":"50d195182268736fe10ba637d098930ed0c85fb4","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-29 10:35:42.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"0c5d141ce239d8e982d05b51f7a9cd434758e0b2","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-30 03:44:06.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"17d4bad8a9b1cc55e9c5d50c85e63127f51b4764","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-30 03:46:08.000000000","message":"Patch Set 3:\n\n(5 comments)\n\n\u003e (1 comment)\n\nPlease help to review the set 4.~  Thanks.","accounts_in_message":[],"_revision_number":3},{"id":"e0d1194e8612a58d793ffc7c5c78e1cd88508bce","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-30 04:52:26.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/4/check/gate-python-openstackclient-docs-ubuntu-xenial/96226ac//doc/build/html/ : SUCCESS in 2m 42s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/4/check/gate-python-openstackclient-pep8-ubuntu-xenial/743a623/ : SUCCESS in 2m 45s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/4/check/gate-python-openstackclient-python27-ubuntu-xenial/299fd84/ : SUCCESS in 2m 36s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/4/check/gate-python-openstackclient-python34/344cee7/ : SUCCESS in 4m 04s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/4/check/gate-python-openstackclient-python35-nv/a106513/ : SUCCESS in 3m 06s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/4/check/gate-python-openstackclient-releasenotes/33cce7e//releasenotes/build/html/ : SUCCESS in 2m 25s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/4/check/gate-tempest-dsvm-neutron-src-python-openstackclient/a3ff4e9/ : SUCCESS in 1h 07m 49s\n- check-osc-plugins http://logs.openstack.org/91/360191/4/check/check-osc-plugins/bd82370/ : SUCCESS in 3m 32s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/4/check/gate-osc-dsvm-functional/c4491f0/ : SUCCESS in 26m 36s","accounts_in_message":[],"_revision_number":4},{"id":"64ec4a7da8aeacec0af0fb25d5cc329d4d545104","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-30 08:33:02.000000000","message":"Patch Set 4: Code-Review+1\n\nThe code looks good, but it is better to have a functional test for these options","accounts_in_message":[],"_revision_number":4},{"id":"c9116b8d2637f1a67c966dce167d94e10a201014","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-31 03:54:21.000000000","message":"Patch Set 4:\n\n\u003e The code looks good, but it is better to have a functional test for\n \u003e these options\n\nHi Huanxuan, could you help to supply more details about the func test?\n\nIn current UT, we confirm to pass the correct value to the set_bootable func in cinderclient.\nDo you mean also add the test for the cindercilent set_bootable to make sure it works fine?\n\nBTW, I have verified my fix on my local site and it works.\n\nroot@controller01:~# openstack --insecure volume set f718cdc8-b66b-4ffe-bd16-ce0c51e663fe --bootable\nroot@controller01:~# openstack --insecure volume show f718cdc8-b66b-4ffe-bd16-ce0c51e663fe | grep bootable\n| bootable                       | true \nroot@controller01:~# openstack --insecure volume set f718cdc8-b66b-4ffe-bd16-ce0c51e663fe --non-bootable\nroot@controller01:~# openstack --insecure volume show f718cdc8-b66b-4ffe-bd16-ce0c51e663fe | grep bootable\n| bootable                       | false  \nroot@controller01:~# openstack --insecure volume set f718cdc8-b66b-4ffe-bd16-ce0c51e663fe --property k\u003d2\nroot@controller01:~# openstack --insecure volume show f718cdc8-b66b-4ffe-bd16-ce0c51e663fe | grep bootable\n| bootable                       | false","accounts_in_message":[],"_revision_number":4},{"id":"00bac118b21f69efcdd5e7163e3ba1d4778cd260","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-31 04:03:43.000000000","message":"Patch Set 4:\n\n\u003e \u003e The code looks good, but it is better to have a functional test\n \u003e for\n \u003e \u003e these options\n \u003e \n \u003e Hi Huanxuan, could you help to supply more details about the func\n \u003e test?\n \u003e \n \u003e In current UT, we confirm to pass the correct value to the\n \u003e set_bootable func in cinderclient.\n \u003e Do you mean also add the test for the cindercilent set_bootable to\n \u003e make sure it works fine?\n \u003e \n \u003e BTW, I have verified my fix on my local site and it works.\n \u003e \n \u003e root@controller01:~# openstack --insecure volume set\n \u003e f718cdc8-b66b-4ffe-bd16-ce0c51e663fe --bootable\n \u003e root@controller01:~# openstack --insecure volume show\n \u003e f718cdc8-b66b-4ffe-bd16-ce0c51e663fe | grep bootable\n \u003e | bootable                       | true\n \u003e root@controller01:~# openstack --insecure volume set\n \u003e f718cdc8-b66b-4ffe-bd16-ce0c51e663fe --non-bootable\n \u003e root@controller01:~# openstack --insecure volume show\n \u003e f718cdc8-b66b-4ffe-bd16-ce0c51e663fe | grep bootable\n \u003e | bootable                       | false\n \u003e root@controller01:~# openstack --insecure volume set\n \u003e f718cdc8-b66b-4ffe-bd16-ce0c51e663fe --property k\u003d2\n \u003e root@controller01:~# openstack --insecure volume show\n \u003e f718cdc8-b66b-4ffe-bd16-ce0c51e663fe | grep bootable\n \u003e | bootable                       | false\n\nYes, I know it works. OSC has functional test for commands in functional/test directory, you can find some volume functional tests in functional/test/volume/v2/test_volume.py and functional/test/volume/v1/test_volume.py, you can see the existing tests for example, functional test is also a import part in OSC :)","accounts_in_message":[],"_revision_number":4},{"id":"12e3c6e7b905cdcb56519a88bef97962116ac977","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-31 05:15:12.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"d4bab4f9e2b3b5826956d1f2dbc916cc723dbbae","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-31 05:29:00.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"609782706b01013475d75e53464a0ce1d546accb","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-31 05:30:13.000000000","message":"Patch Set 6:\n\nAdded func test for both v1 \u0026 v2.","accounts_in_message":[],"_revision_number":6},{"id":"8349f353d9d889bda9f4041d640804e428a9d4e8","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-31 08:06:35.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/6/check/gate-python-openstackclient-docs-ubuntu-xenial/982461c//doc/build/html/ : SUCCESS in 3m 11s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/6/check/gate-python-openstackclient-pep8-ubuntu-xenial/0b07953/ : FAILURE in 4m 16s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/6/check/gate-python-openstackclient-python27-ubuntu-xenial/7b9f382/ : SUCCESS in 2m 52s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/6/check/gate-python-openstackclient-python34/c7a353b/ : SUCCESS in 7m 43s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/6/check/gate-python-openstackclient-python35-nv/6ee71a4/ : SUCCESS in 3m 19s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/6/check/gate-python-openstackclient-releasenotes/cad7e3c//releasenotes/build/html/ : SUCCESS in 3m 18s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/6/check/gate-tempest-dsvm-neutron-src-python-openstackclient/621ddde/ : SUCCESS in 1h 04m 11s\n- check-osc-plugins http://logs.openstack.org/91/360191/6/check/check-osc-plugins/ae66d13/ : SUCCESS in 3m 27s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/6/check/gate-osc-dsvm-functional/51030f7/ : SUCCESS in 34m 16s","accounts_in_message":[],"_revision_number":6},{"id":"b37129ebeda4a7bb3a876835f7bf1974997d6268","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-08-31 08:27:23.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"0b169144f56c01dcf2c384474a31cfddc313ce48","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-08-31 09:30:17.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/7/check/gate-python-openstackclient-docs-ubuntu-xenial/141b85d//doc/build/html/ : SUCCESS in 3m 07s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/7/check/gate-python-openstackclient-pep8-ubuntu-xenial/7900657/ : SUCCESS in 2m 54s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/7/check/gate-python-openstackclient-python27-ubuntu-xenial/7ab8806/ : SUCCESS in 4m 11s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/7/check/gate-python-openstackclient-python34/f8a5f16/ : SUCCESS in 4m 17s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/7/check/gate-python-openstackclient-python35-nv/9e1e6f3/ : SUCCESS in 3m 42s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/7/check/gate-python-openstackclient-releasenotes/4603db6//releasenotes/build/html/ : SUCCESS in 3m 06s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/7/check/gate-tempest-dsvm-neutron-src-python-openstackclient/b0e5374/ : SUCCESS in 1h 01m 33s\n- check-osc-plugins http://logs.openstack.org/91/360191/7/check/check-osc-plugins/755c18e/ : SUCCESS in 3m 26s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/7/check/gate-osc-dsvm-functional/2a050cd/ : SUCCESS in 27m 55s","accounts_in_message":[],"_revision_number":7},{"id":"e2c0523f3d5e2f9d5b5f0245c406eeb3042a5521","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-08-31 11:26:25.000000000","message":"Patch Set 7: Code-Review+1\n\nlgtm, thanks.\nFYI: The OSC core team is holding new feature merges before OSC 3.0 stabilized, so it will take a while for new patches to get reviewed and merged.","accounts_in_message":[],"_revision_number":7},{"id":"c5457c5100e61c176c58fc810ddd19b9411fb997","author":{"_account_id":16443,"name":"Wenwu","email":"pengwenwu2008@163.com","username":"Wenwu"},"date":"2016-09-06 07:36:24.000000000","message":"Patch Set 7: Code-Review+1","accounts_in_message":[],"_revision_number":7},{"id":"e05be87159e96d4993c341d06051b63b4696b06e","author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"date":"2016-09-06 11:34:33.000000000","message":"Patch Set 7: Code-Review+1\n\nLGTM, thank you adding the feature.","accounts_in_message":[],"_revision_number":7},{"id":"bf4bc2abc13c0061b2d83143b6e8d0bfe72885b9","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-09-08 08:32:46.000000000","message":"Patch Set 7:\n\nAnymore comments for this? Thanks.~","accounts_in_message":[],"_revision_number":7},{"id":"78cbbf3091395a2275d66a33162abdd34094d2b2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-09 15:54:08.000000000","message":"Patch Set 7: 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":7},{"id":"bbd7ccd7f22c7c955482a1ec30530c6391485311","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-09-12 02:28:00.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"1e95bfc72a9c77705510605a0f8358a2c13a81f2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-12 03:34:01.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/8/check/gate-python-openstackclient-docs-ubuntu-xenial/f3a6fff//doc/build/html/ : SUCCESS in 3m 46s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/8/check/gate-python-openstackclient-pep8-ubuntu-xenial/f7dc789/ : SUCCESS in 2m 50s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/8/check/gate-python-openstackclient-python27-ubuntu-xenial/cbb7d48/ : SUCCESS in 2m 35s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/8/check/gate-python-openstackclient-python34/2832ede/ : SUCCESS in 4m 15s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/8/check/gate-python-openstackclient-python35-nv/4d00b88/ : SUCCESS in 6m 24s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/8/check/gate-python-openstackclient-releasenotes/7ff1440//releasenotes/build/html/ : SUCCESS in 3m 46s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/8/check/gate-tempest-dsvm-neutron-src-python-openstackclient/ac0564a/ : SUCCESS in 1h 05m 50s\n- check-osc-plugins http://logs.openstack.org/91/360191/8/check/check-osc-plugins/ea815bd/ : SUCCESS in 3m 28s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/8/check/gate-osc-dsvm-functional/b7dd080/ : SUCCESS in 42m 43s","accounts_in_message":[],"_revision_number":8},{"id":"175e4ae1b1f765aaf58837d643369d251d525c0a","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-09-12 21:23:12.000000000","message":"Patch Set 8: Code-Review+2\n\n(1 comment)\n\nawesome","accounts_in_message":[],"_revision_number":8},{"id":"6a970cf5bc8b33b4383d1e1e83c7ae4b4415914f","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-09-13 03:32:25.000000000","message":"Patch Set 8:\n\n(1 comment)","accounts_in_message":[],"_revision_number":8},{"id":"67183914615fbc508abb165bec84ae5e04fa5c88","author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"date":"2016-09-13 09:06:31.000000000","message":"Patch Set 8: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":8},{"id":"e62d8755f1f30530c6fe16a3ef7304c1464e03d6","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-09-13 09:43:12.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"8e0a16cfcc23b3f5c7f1edacc77477935b5d0307","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-09-13 09:44:09.000000000","message":"Patch Set 8:\n\n(2 comments)","accounts_in_message":[],"_revision_number":8},{"id":"be4c056f2c40dc678e3d2f08d74f8403072ab7f1","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-09-13 10:33:06.000000000","message":"Patch Set 9: Code-Review+1","accounts_in_message":[],"_revision_number":9},{"id":"f926909b01a737db0a301a69d02afd1d7b857a56","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-13 10:50:45.000000000","message":"Patch Set 9: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/9/check/gate-python-openstackclient-docs-ubuntu-xenial/f4e1c23//doc/build/html/ : SUCCESS in 4m 35s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/9/check/gate-python-openstackclient-pep8-ubuntu-xenial/c43734a/ : FAILURE in 3m 47s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/9/check/gate-python-openstackclient-python27-ubuntu-xenial/f94e7ff/ : SUCCESS in 3m 54s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/9/check/gate-python-openstackclient-python34/8e595ab/ : FAILURE in 2m 35s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/9/check/gate-python-openstackclient-python35-nv/1e19f5a/ : FAILURE in 4m 19s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/9/check/gate-python-openstackclient-releasenotes/aa09d27//releasenotes/build/html/ : SUCCESS in 3m 38s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/9/check/gate-tempest-dsvm-neutron-src-python-openstackclient/14cb7a0/ : SUCCESS in 1h 04m 38s\n- check-osc-plugins http://logs.openstack.org/91/360191/9/check/check-osc-plugins/118e04b/ : SUCCESS in 3m 26s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/9/check/gate-osc-dsvm-functional/026489f/ : SUCCESS in 15m 12s","accounts_in_message":[],"_revision_number":9},{"id":"5a61f58a9b812adf247e762eecbf6f0a4779fcfd","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-09-13 10:59:40.000000000","message":"Patch Set 9: Code-Review-1\n\nLooks like there are some pep8 errors need to fix.","accounts_in_message":[],"_revision_number":9},{"id":"2c1c1937a968e04e70963a04989cf8ec4f7f9177","author":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"date":"2016-09-13 11:39:44.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"0a1953e13bd89d05375fcb3d341f2391fc4a71a7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-13 12:48:51.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/10/check/gate-python-openstackclient-docs-ubuntu-xenial/8b0bb93//doc/build/html/ : SUCCESS in 3m 55s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/10/check/gate-python-openstackclient-pep8-ubuntu-xenial/ce01ed0/ : SUCCESS in 2m 52s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/10/check/gate-python-openstackclient-python27-ubuntu-xenial/a9d16a3/ : SUCCESS in 2m 49s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/10/check/gate-python-openstackclient-python34/6d474d4/ : SUCCESS in 3m 38s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/10/check/gate-python-openstackclient-python35-nv/a24c808/ : SUCCESS in 3m 28s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/10/check/gate-python-openstackclient-releasenotes/30a23a4//releasenotes/build/html/ : SUCCESS in 2m 41s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/10/check/gate-tempest-dsvm-neutron-src-python-openstackclient/7ac0143/ : SUCCESS in 1h 05m 36s\n- check-osc-plugins http://logs.openstack.org/91/360191/10/check/check-osc-plugins/ba2d2cf/ : SUCCESS in 7m 37s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/10/check/gate-osc-dsvm-functional/351a302/ : SUCCESS in 42m 29s","accounts_in_message":[],"_revision_number":10},{"id":"09165ad262b99d9f6dd5a8c7ae050a22f0dcf8d1","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-09-13 13:02:40.000000000","message":"Patch Set 10: Code-Review+1","accounts_in_message":[],"_revision_number":10},{"id":"27b98c726a81a21405eeb473a7f42e6e595a2963","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-09-13 13:10:50.000000000","message":"Patch Set 10: Code-Review+2\n\nlgtm!","accounts_in_message":[],"_revision_number":10},{"id":"0b3613c5779d5249eb32fc08c16108ea17b33b17","author":{"_account_id":8276,"name":"kiwik","email":"chenrui.momo@gmail.com","username":"kiwik"},"date":"2016-09-14 07:25:33.000000000","message":"Patch Set 10: Code-Review+1\n\nLGTM","accounts_in_message":[],"_revision_number":10},{"id":"323dbc656ed24159ff332f54793e74248ffafa40","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2016-09-14 16:37:09.000000000","message":"Patch Set 11: Published edit on patch set 10","accounts_in_message":[],"_revision_number":11},{"id":"78fc45983e5deb434e2e9803a45ffc2e40938874","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2016-09-14 16:37:30.000000000","message":"Patch Set 11: Code-Review+2 Workflow+1\n\nI tweaked the release not slightly to explicitly include the new options.","accounts_in_message":[],"_revision_number":11},{"id":"023a6a61913b9d5aa83d228246ff9e8ae3ae48c2","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-14 17:45:04.000000000","message":"Patch Set 11: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/11/check/gate-python-openstackclient-docs-ubuntu-xenial/400ba48//doc/build/html/ : SUCCESS in 3m 08s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/11/check/gate-python-openstackclient-pep8-ubuntu-xenial/ec397c1/ : SUCCESS in 2m 48s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/11/check/gate-python-openstackclient-python27-ubuntu-xenial/c8fc9d2/ : SUCCESS in 2m 45s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/11/check/gate-python-openstackclient-python34/c073099/ : SUCCESS in 3m 57s\n- gate-python-openstackclient-python35-nv http://logs.openstack.org/91/360191/11/check/gate-python-openstackclient-python35-nv/be3e3ce/ : SUCCESS in 4m 01s (non-voting)\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/11/check/gate-python-openstackclient-releasenotes/048fe50//releasenotes/build/html/ : SUCCESS in 2m 35s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/11/check/gate-tempest-dsvm-neutron-src-python-openstackclient/93eeda8/ : SUCCESS in 54m 42s\n- check-osc-plugins http://logs.openstack.org/91/360191/11/check/check-osc-plugins/55c69c8/ : SUCCESS in 4m 32s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/11/check/gate-osc-dsvm-functional/4f83730/ : SUCCESS in 25m 10s","accounts_in_message":[],"_revision_number":11},{"id":"0f17d66875703fa3525043c72d63f8083b2a4e18","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-14 17:45:15.000000000","message":"Patch Set 11: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":11},{"id":"428577ad82ec345db35e39e9521418b058c3453d","date":"2016-09-14 18:48:28.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":11},{"id":"acff6b5fb9f2f17c9c5974a5758df0d5db6edde6","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-14 18:48:28.000000000","message":"Patch Set 11: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-openstackclient-docs-ubuntu-xenial http://docs-draft.openstack.org/91/360191/11/gate/gate-python-openstackclient-docs-ubuntu-xenial/c22ed7c//doc/build/html/ : SUCCESS in 3m 40s\n- gate-python-openstackclient-pep8-ubuntu-xenial http://logs.openstack.org/91/360191/11/gate/gate-python-openstackclient-pep8-ubuntu-xenial/ec7770b/ : SUCCESS in 3m 42s\n- gate-python-openstackclient-python27-ubuntu-xenial http://logs.openstack.org/91/360191/11/gate/gate-python-openstackclient-python27-ubuntu-xenial/79bebe0/ : SUCCESS in 4m 45s\n- gate-python-openstackclient-python34 http://logs.openstack.org/91/360191/11/gate/gate-python-openstackclient-python34/a792e4e/ : SUCCESS in 3m 40s\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/91/360191/11/gate/gate-python-openstackclient-releasenotes/59bf96d//releasenotes/build/html/ : SUCCESS in 3m 54s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/91/360191/11/gate/gate-tempest-dsvm-neutron-src-python-openstackclient/e6b3dd2/ : SUCCESS in 1h 02m 05s\n- gate-osc-dsvm-functional http://logs.openstack.org/91/360191/11/gate/gate-osc-dsvm-functional/d9bbece/ : SUCCESS in 21m 34s","accounts_in_message":[],"_revision_number":11}],"current_revision_number":11,"current_revision":"6a914d0056e810e1ef37eaf4f01cd5c85217aba6","revisions":{"757e7382fb3ac85891c1871a41dfb0f466686167":{"kind":"REWORK","_number":1,"created":"2016-08-25 02:54:46.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/1"}}},"commit":{"parents":[{"commit":"351bfb79154777dca1bc117e9383606f16b02353","subject":"Merge \"Fix post and periodic jobs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/351bfb79154777dca1bc117e9383606f16b02353"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:59:27.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable option in volume set to mark or unmark\nvolume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/757e7382fb3ac85891c1871a41dfb0f466686167"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/757e7382fb3ac85891c1871a41dfb0f466686167"}]},"branch":"refs/heads/master"},"f4293a21709dff1cd1068339e62525aa06946392":{"kind":"REWORK","_number":2,"created":"2016-08-25 05:42:30.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/2"}}},"commit":{"parents":[{"commit":"afda6f846fc3abea47c83e6384974cb9bdb8db3b","subject":"Updated from global requirements","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/afda6f846fc3abea47c83e6384974cb9bdb8db3b"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 05:47:04.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable option in volume set to mark or unmark\nvolume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/f4293a21709dff1cd1068339e62525aa06946392"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/f4293a21709dff1cd1068339e62525aa06946392"}]},"branch":"refs/heads/master"},"011172f00ce3ad22611651a28c5cf6ca0f9b7b4b":{"kind":"REWORK","_number":3,"created":"2016-08-29 05:19:54.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/3"}}},"commit":{"parents":[{"commit":"a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e","subject":"Merge \"Restore default auth-type for token/endpoint\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-29 05:24:03.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/011172f00ce3ad22611651a28c5cf6ca0f9b7b4b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/011172f00ce3ad22611651a28c5cf6ca0f9b7b4b"}]},"branch":"refs/heads/master"},"16df3060d746ea0f53811526a25947c3a64fad05":{"kind":"REWORK","_number":4,"created":"2016-08-30 03:44:06.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/4"}}},"commit":{"parents":[{"commit":"a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e","subject":"Merge \"Restore default auth-type for token/endpoint\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-30 03:49:02.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/16df3060d746ea0f53811526a25947c3a64fad05"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/16df3060d746ea0f53811526a25947c3a64fad05"}]},"branch":"refs/heads/master"},"50c06fba8269dab2c6e00850482ffec103b11749":{"kind":"REWORK","_number":5,"created":"2016-08-31 05:15:12.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/5"}}},"commit":{"parents":[{"commit":"a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e","subject":"Merge \"Restore default auth-type for token/endpoint\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-31 05:20:12.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/50c06fba8269dab2c6e00850482ffec103b11749"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/50c06fba8269dab2c6e00850482ffec103b11749"}]},"branch":"refs/heads/master"},"c6f6defa3173f31d55f708f707069fd3505861c0":{"kind":"REWORK","_number":6,"created":"2016-08-31 05:29:00.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/6"}}},"commit":{"parents":[{"commit":"a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e","subject":"Merge \"Restore default auth-type for token/endpoint\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-31 05:34:00.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/c6f6defa3173f31d55f708f707069fd3505861c0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/c6f6defa3173f31d55f708f707069fd3505861c0"}]},"branch":"refs/heads/master"},"67a365ec5db0056babd58ea3a417de87f962cc1f":{"kind":"REWORK","_number":7,"created":"2016-08-31 08:27:23.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/7"}}},"commit":{"parents":[{"commit":"a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e","subject":"Merge \"Restore default auth-type for token/endpoint\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/a08b62523fa634d5a61d85d1e9f3b89ab2d4a14e"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-31 08:32:17.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/67a365ec5db0056babd58ea3a417de87f962cc1f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/67a365ec5db0056babd58ea3a417de87f962cc1f"}]},"branch":"refs/heads/master"},"e6a28ba4649e222695a032a94028f27780fb1362":{"kind":"REWORK","_number":8,"created":"2016-09-12 02:28:00.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/8"}}},"commit":{"parents":[{"commit":"d2273ecea5d540f4dacc89772870722355f2492f","subject":"Implement \"volume transfer request delete\" command","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d2273ecea5d540f4dacc89772870722355f2492f"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-09-12 02:33:39.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/e6a28ba4649e222695a032a94028f27780fb1362"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/e6a28ba4649e222695a032a94028f27780fb1362"}]},"branch":"refs/heads/master"},"584723266941a74437025a1e2d4ebb69b8499b78":{"kind":"REWORK","_number":9,"created":"2016-09-13 09:43:12.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/9"}}},"commit":{"parents":[{"commit":"d2273ecea5d540f4dacc89772870722355f2492f","subject":"Implement \"volume transfer request delete\" command","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d2273ecea5d540f4dacc89772870722355f2492f"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-09-13 09:48:50.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/584723266941a74437025a1e2d4ebb69b8499b78"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/584723266941a74437025a1e2d4ebb69b8499b78"}]},"branch":"refs/heads/master"},"952740781c9bf8568ba4c2892c249d0a4f41083b":{"kind":"REWORK","_number":10,"created":"2016-09-13 11:39:44.000000000","uploader":{"_account_id":22754,"name":"Qiang Tang","email":"qtang@vmware.com","username":"qtang"},"ref":"refs/changes/91/360191/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/10"}}},"commit":{"parents":[{"commit":"d2273ecea5d540f4dacc89772870722355f2492f","subject":"Implement \"volume transfer request delete\" command","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d2273ecea5d540f4dacc89772870722355f2492f"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"qtang","email":"qtang@vmware.com","date":"2016-09-13 11:45:30.000000000","tz":480},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/952740781c9bf8568ba4c2892c249d0a4f41083b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/952740781c9bf8568ba4c2892c249d0a4f41083b"}]},"branch":"refs/heads/master"},"6a914d0056e810e1ef37eaf4f01cd5c85217aba6":{"kind":"REWORK","_number":11,"created":"2016-09-14 16:37:09.000000000","uploader":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"ref":"refs/changes/91/360191/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/91/360191/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/91/360191/11"}}},"commit":{"parents":[{"commit":"d2273ecea5d540f4dacc89772870722355f2492f","subject":"Implement \"volume transfer request delete\" command","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d2273ecea5d540f4dacc89772870722355f2492f"}]}],"author":{"name":"qtang","email":"qtang@vmware.com","date":"2016-08-25 02:56:58.000000000","tz":480},"committer":{"name":"Dean Troyer","email":"dtroyer@gmail.com","date":"2016-09-14 16:35:49.000000000","tz":0},"subject":"Support mark volume as bootable in volume set","message":"Support mark volume as bootable in volume set\n\nAdd --bootable | --non-bootable option in volume set\nto mark or unmark volume as bootable.\n\nChange-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6\nCloses-Bug:#1535704\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/6a914d0056e810e1ef37eaf4f01cd5c85217aba6"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/6a914d0056e810e1ef37eaf4f01cd5c85217aba6"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}