)]}'
{"id":"openstack%2Fpython-openstackclient~331022","triplet_id":"openstack%2Fpython-openstackclient~master~Ief52766a5976ca06558bb5be0aa90fa91f034f7d","project":"openstack/python-openstackclient","branch":"master","topic":"bp/migrate-glanceclient-to-osc","hashtags":[],"change_id":"Ief52766a5976ca06558bb5be0aa90fa91f034f7d","subject":"Migrate \"image show\" command to osc from glanceclient.","status":"ABANDONED","created":"2016-06-17 10:36:35.000000000","updated":"2017-08-24 19:19:43.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f4fdf63c2c1aba8231887fba326f00a7acfc6b7a","_number":331022,"virtual_id_number":331022,"owner":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":-1,"date":"2016-06-20 17:27:55.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"date":"2016-06-19 12:17:20.000000000","_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"date":"2016-06-19 15:40:39.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"recommended":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"all":[{"value":1,"date":"2016-06-19 04:47:40.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"rejected":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"all":[{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2016-06-20 02:31:23.000000000","_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"value":-1,"date":"2016-06-20 04:36:22.000000000","permitted_voting_range":{"min":-1,"max":0},"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"}],"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":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-06-19 04:47:40.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-06-19 15:40:39.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-06-20 02:31:23.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-06-20 17:27:55.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"8dc0e99245d50b1873cc8c0ec15d4f59513ba7c0","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-06-17 10:36:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b9f465b8e37c7da266beb598f3f848981e231c5a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-17 11:55:56.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-openstackclient-docs http://docs-draft.openstack.org/22/331022/1/check/gate-python-openstackclient-docs/e4ad31a//doc/build/html/ : SUCCESS in 9m 17s\n- gate-python-openstackclient-pep8 http://logs.openstack.org/22/331022/1/check/gate-python-openstackclient-pep8/84c6239/ : SUCCESS in 7m 57s\n- gate-python-openstackclient-python27 http://logs.openstack.org/22/331022/1/check/gate-python-openstackclient-python27/e7bc9c3/ : SUCCESS in 9m 28s\n- gate-python-openstackclient-python34 http://logs.openstack.org/22/331022/1/check/gate-python-openstackclient-python34/5bd59e8/ : SUCCESS in 9m 45s\n- gate-python-openstackclient-releasenotes http://docs-draft.openstack.org/22/331022/1/check/gate-python-openstackclient-releasenotes/8d541c3//releasenotes/build/html/ : SUCCESS in 8m 20s\n- gate-tempest-dsvm-neutron-src-python-openstackclient http://logs.openstack.org/22/331022/1/check/gate-tempest-dsvm-neutron-src-python-openstackclient/c6cabb0/ : SUCCESS in 1h 14m 37s\n- check-osc-plugins http://logs.openstack.org/22/331022/1/check/check-osc-plugins/c9c7e41/ : SUCCESS in 9m 36s (non-voting)\n- gate-osc-dsvm-functional http://logs.openstack.org/22/331022/1/check/gate-osc-dsvm-functional/8adb8d1/ : SUCCESS in 28m 31s","accounts_in_message":[],"_revision_number":1},{"id":"c8f864a4e740a6bfbca36f93c5fddd9c11135649","author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"date":"2016-06-19 04:47:40.000000000","message":"Patch Set 1: Code-Review+1\n\nso this totally works, but i\u0027m wondering why not replace the current command with the SDK? i.e. no need for a beta-command option?","accounts_in_message":[],"_revision_number":1},{"id":"15bdf453d4cfe53d282f8edb1e951519bcf5f2b1","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-06-19 12:17:20.000000000","message":"Patch Set 1:\n\n\u003e so this totally works, but i\u0027m wondering why not replace the\n \u003e current command with the SDK? i.e. no need for a beta-command\n \u003e option?\n\nIf we repalce one command, we need change the API to the SDK. then the other image commands which are still using glanceclient can not be used. So I think we can using beta command temporary, after implementing all image commands with SDK, we just need to remove old command codes and \u0027beta-command\u0027 option, then we can replace all commands together at the same time.","accounts_in_message":[],"_revision_number":1},{"id":"e903f8d83083023fe6332e069808146ac19fbe73","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2016-06-19 15:40:39.000000000","message":"Patch Set 1:\n\nPlease check with briancurtin before using the SDK for image API, he is working on refactors and I do not believe he has completed Image yet.  We can not merge this until that fafactor has been released.","accounts_in_message":[],"_revision_number":1},{"id":"6dadb13a7a2c06c545266b60083eefe144268405","author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"date":"2016-06-20 02:31:23.000000000","message":"Patch Set 1:\n\nHi Steve,\n\nIt seems that image support in SDK has some bugs to fix. I\u0027m not sure yet. So I recommand to use beta-command option during the migration.","accounts_in_message":[],"_revision_number":1},{"id":"f33a3ab68d8c6257ef2cf14dc4cc0645bacddb00","author":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"date":"2016-06-20 04:36:22.000000000","message":"Patch Set 1: Workflow-1\n\n\u003e Please check with briancurtin before using the SDK for image API,\n \u003e he is working on refactors and I do not believe he has completed\n \u003e Image yet.  We can not merge this until that fafactor has been\n \u003e released.\n\nThank you, Dean, I will mark this work WIP until that refactors of image in SDK has been released","accounts_in_message":[],"_revision_number":1},{"id":"3caa171d70e6263caff8a2b2246755aa35b4758f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-06-20 17:27:55.000000000","message":"Patch Set 1: Verified-1\n\nMerge Failed.\n\nThis change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.","accounts_in_message":[],"_revision_number":1},{"id":"25cd926cf5e3ebc7b6d93f30046eb88379cbf290","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2017-08-24 19:19:43.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"d6adbb6e2b741d57a0353d2d55ebf8e6c46c8904","revisions":{"d6adbb6e2b741d57a0353d2d55ebf8e6c46c8904":{"kind":"REWORK","_number":1,"created":"2016-06-17 10:36:35.000000000","uploader":{"_account_id":21514,"name":"Huanxuan Ao","email":"aohuanxuan@163.com","username":"aohuanxuan"},"ref":"refs/changes/22/331022/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-openstackclient","ref":"refs/changes/22/331022/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/22/331022/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/22/331022/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-openstackclient refs/changes/22/331022/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-openstackclient refs/changes/22/331022/1"}}},"commit":{"parents":[{"commit":"2740291f491804be5f521770ee773fa62894e8a3","subject":"move release note to correct directory","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/2740291f491804be5f521770ee773fa62894e8a3"}]}],"author":{"name":"Huanxuan Ao","email":"huanxuan.ao@easystack.cn","date":"2016-06-17 08:40:45.000000000","tz":480},"committer":{"name":"Huanxuan Ao","email":"huanxuan.ao@easystack.cn","date":"2016-06-17 10:36:03.000000000","tz":480},"subject":"Migrate \"image show\" command to osc from glanceclient.","message":"Migrate \"image show\" command to osc from glanceclient.\n\nThis patch add a new \"image show\" command which is supported by SDK.\nThe old \"image show\" command still can be used because this new command\nis a beta command and needs to be used with \"--os-beta-command\" option.\n\nThe reasons of using \"--os-beta-command\" for this command are as belows:\n1.Using \"--os-beta-command\" option can retaining the old command, we\u0027d\nbetter retain the old command which is depend on glanceclient until the\nSDK support is good enough.\n2.When we migrate a image command, we should insure others image command\ncan be used and were not be influenced. Using \"--os-beta-command\" can\nmeeting this requirement.\n\nPartially-Implements: blueprint migrate-glanceclient-to-osc\nChange-Id: Ief52766a5976ca06558bb5be0aa90fa91f034f7d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d6adbb6e2b741d57a0353d2d55ebf8e6c46c8904"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-openstackclient/commit/d6adbb6e2b741d57a0353d2d55ebf8e6c46c8904"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}