)]}'
{"id":"openstack%2Ftempest~409575","triplet_id":"openstack%2Ftempest~master~I9cefdbffe346dc3c551a9b418c634f7ed28b5e19","project":"openstack/tempest","branch":"master","topic":"remove_os_in_client","hashtags":[],"change_id":"I9cefdbffe346dc3c551a9b418c634f7ed28b5e19","subject":"Use self.xx_client instead of self.os.xx_client in testcases","status":"ABANDONED","created":"2016-12-12 02:18:03.000000000","updated":"2016-12-20 07:00:58.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"68128994d0961b76b18e5f3a0babc711c1d84b53","_number":409575,"virtual_id_number":409575,"owner":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"date":"2016-12-16 00:49:20.000000000","_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"date":"2016-12-12 05:43:12.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":1,"date":"2016-12-12 03:30:44.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":{"approved":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"disliked":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"all":[{"value":2,"date":"2016-12-15 02:31:51.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},{"value":-1,"date":"2016-12-15 14:53:10.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":-1,"date":"2016-12-15 09:27:08.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-1":"Release Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-12-12 03:30:44.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2016-12-12 05:43:12.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2016-12-15 02:31:51.000000000","updated_by":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"reviewer":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"state":"REVIEWER"},{"updated":"2016-12-15 09:27:08.000000000","updated_by":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"reviewer":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"state":"REVIEWER"},{"updated":"2016-12-15 14:53:10.000000000","updated_by":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"reviewer":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"state":"REVIEWER"}],"messages":[{"id":"64b1ee4d3bf72fa1aeaf0dd6b73b085a15215712","author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"date":"2016-12-12 02:18:03.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e38e39fb57b9f3989fede12c7ab5934a945061f7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-12-12 03:30:44.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/3a97cee/ : SUCCESS in 52m 42s\n- gate-grenade-dsvm-neutron-ubuntu-xenial http://logs.openstack.org/75/409575/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/9392f96/ : SUCCESS in 34m 08s\n- gate-tempest-docs-ubuntu-xenial http://docs-draft.openstack.org/75/409575/1/check/gate-tempest-docs-ubuntu-xenial/9d814fb//doc/build/html/ : SUCCESS in 2m 48s\n- gate-tempest-pep8-ubuntu-xenial http://logs.openstack.org/75/409575/1/check/gate-tempest-pep8-ubuntu-xenial/bd7e9ff/ : SUCCESS in 1m 35s\n- gate-tempest-python27-ubuntu-xenial http://logs.openstack.org/75/409575/1/check/gate-tempest-python27-ubuntu-xenial/df2ab4c/ : SUCCESS in 1m 59s\n- gate-tempest-python34 http://logs.openstack.org/75/409575/1/check/gate-tempest-python34/7cbd63d/ : SUCCESS in 2m 12s\n- gate-tempest-python35 http://logs.openstack.org/75/409575/1/check/gate-tempest-python35/c78e2ae/ : SUCCESS in 2m 34s\n- gate-grenade-dsvm-neutron-multinode-ubuntu-xenial http://logs.openstack.org/75/409575/1/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/9e601a5/ : SUCCESS in 37m 05s\n- gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial-nv/13222ba/ : SUCCESS in 26m 27s (non-voting)\n- gate-tempest-dsvm-neutron-full-ssh http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-neutron-full-ssh/d730f3b/ : SUCCESS in 51m 37s (non-voting)\n- gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-neutron-identity-v3-only-full-ubuntu-xenial-nv/8d73961/ : SUCCESS in 50m 16s (non-voting)\n- gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/394b640/ : SUCCESS in 1h 05m 59s\n- gate-puppet-openstack-integration-4-scenario001-tempest-centos-7-nv http://logs.openstack.org/75/409575/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7-nv/31a28ec/ : SUCCESS in 34m 50s (non-voting)\n- gate-puppet-openstack-integration-4-scenario002-tempest-centos-7-nv http://logs.openstack.org/75/409575/1/check/gate-puppet-openstack-integration-4-scenario002-tempest-centos-7-nv/8253056/ : SUCCESS in 46m 38s (non-voting)\n- gate-puppet-openstack-integration-4-scenario003-tempest-centos-7-nv http://logs.openstack.org/75/409575/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7-nv/1f5e484/ : SUCCESS in 44m 34s (non-voting)\n- gate-tempest-dsvm-layer4-ubuntu-xenial http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-layer4-ubuntu-xenial/1208cbc/ : SUCCESS in 23m 51s\n- gate-tempest-dsvm-neutron-dvr-ubuntu-xenial http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-neutron-dvr-ubuntu-xenial/8cb91c8/ : SUCCESS in 51m 57s\n- gate-tempest-dsvm-full-ubuntu-xenial-newton http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-full-ubuntu-xenial-newton/24119e8/ : SUCCESS in 1h 12m 14s\n- gate-tempest-dsvm-neutron-full-ubuntu-xenial-newton http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial-newton/ce7bef1/ : SUCCESS in 53m 17s\n- gate-tempest-dsvm-full-ubuntu-trusty-mitaka http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-full-ubuntu-trusty-mitaka/feee69f/ : SUCCESS in 42m 30s\n- gate-tempest-dsvm-neutron-full-ubuntu-trusty-mitaka http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-neutron-full-ubuntu-trusty-mitaka/276388c/ : SUCCESS in 47m 16s\n- gate-tempest-dsvm-full-ubuntu-trusty-liberty http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-full-ubuntu-trusty-liberty/f671963/ : SUCCESS in 36m 40s\n- gate-tempest-dsvm-neutron-full-ubuntu-trusty-liberty http://logs.openstack.org/75/409575/1/check/gate-tempest-dsvm-neutron-full-ubuntu-trusty-liberty/577ada9/ : SUCCESS in 55m 47s","accounts_in_message":[],"_revision_number":1},{"id":"60ccd7a88798b34417f19c5d68d1845df0b92c3b","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2016-12-12 05:43:12.000000000","message":"Patch Set 1:\n\nBuild succeeded.\n\n- dsvm-tempest-neutron-network http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/75/409575/1/check/dsvm-tempest-neutron-network/21e6d0b : SUCCESS in 2h 08m 09s","accounts_in_message":[],"_revision_number":1},{"id":"cc6580aeebb4fccdcba4b82595458c685b68c297","author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"date":"2016-12-15 02:31:51.000000000","message":"Patch Set 1: Code-Review+2\n\nI thought we did that in some recent patch.","accounts_in_message":[],"_revision_number":1},{"id":"0b30da245ff247aa6b5e169dd7d00066bf6f883e","author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"date":"2016-12-15 09:27:08.000000000","message":"Patch Set 1: Code-Review-1\n\nI disagree with this, I think this creates just another layer of indirection, make the code harder to read and debug. If something, I would go the other way around and remove a lot of this aliasing and re-aliasing.","accounts_in_message":[],"_revision_number":1},{"id":"03139e914f99589ea812ccf61a70ef9c35472295","author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"date":"2016-12-15 14:53:10.000000000","message":"Patch Set 1: Code-Review-1\n\nIt\u0027s not worth it. We need to come with a broader plan related to client aliasing. Renaming stuff around is not the best we can do now.","accounts_in_message":[],"_revision_number":1},{"id":"002a484a162494bd2f55398ade422a871b0093b8","author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"date":"2016-12-16 00:49:20.000000000","message":"Patch Set 1:\n\nThanks, it is true that we need broader plan(such as Andrea Frittoli\u0027s solution, I think so too). This patch is just to get the clients naming consistency. Would you please give some advice on what should I do now?","accounts_in_message":[],"_revision_number":1},{"id":"baa6d4136bd241e423460a5500d566eb97dff1a2","author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"date":"2016-12-20 07:00:58.000000000","message":"Abandoned\n\nWait for better completed solution","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"8ccf3db142ebbd0bf77690f8ab3cfe70db90f4bb","revisions":{"8ccf3db142ebbd0bf77690f8ab3cfe70db90f4bb":{"kind":"REWORK","_number":1,"created":"2016-12-12 02:18:03.000000000","uploader":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"ref":"refs/changes/75/409575/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/75/409575/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/75/409575/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/75/409575/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/75/409575/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/75/409575/1"}}},"commit":{"parents":[{"commit":"63746daa93694dc6a11666d7b0539fe9c3422981","subject":"Docs: Add the remote tag to the badge image","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/63746daa93694dc6a11666d7b0539fe9c3422981"}]}],"author":{"name":"zhufl","email":"zhu.fanglei@zte.com.cn","date":"2016-12-12 02:36:30.000000000","tz":480},"committer":{"name":"zhufl","email":"zhu.fanglei@zte.com.cn","date":"2016-12-12 02:36:30.000000000","tz":480},"subject":"Use self.xx_client instead of self.os.xx_client in testcases","message":"Use self.xx_client instead of self.os.xx_client in testcases\n\nSince there ready exists self.xx_client definitions in base.py,\nso in testcases self.xx_client should be used instead of\nself.os.xx_client\n\nChange-Id: I9cefdbffe346dc3c551a9b418c634f7ed28b5e19\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/8ccf3db142ebbd0bf77690f8ab3cfe70db90f4bb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/8ccf3db142ebbd0bf77690f8ab3cfe70db90f4bb"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"63746daa93694dc6a11666d7b0539fe9c3422981","is_merged_in_target_branch":true,"change_id":"Ic305813473eabbe1ab43a78fc767e0d80b097a07","change_number":409215,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}