)]}'
{"id":"openstack%2Ftempest~141014","triplet_id":"openstack%2Ftempest~master~I35f55c895dafc4235c9c5a10b1ccf9ebfffd7265","project":"openstack/tempest","branch":"master","topic":"rest-client","hashtags":[],"change_id":"I35f55c895dafc4235c9c5a10b1ccf9ebfffd7265","subject":"Add BaseRestClient class to separate CONF values","status":"ABANDONED","created":"2014-12-11 11:10:32.000000000","updated":"2014-12-22 01:27:28.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"bedf23c334e03e164ce7a5b016a78e6027c1b030","_number":141014,"virtual_id_number":141014,"owner":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"disliked":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"date":"2014-12-15 22:14:21.000000000","_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"date":"2014-12-11 12:36:51.000000000","_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"value":1,"date":"2014-12-12 17:22:44.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"value":-1,"date":"2014-12-11 12:36:38.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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"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":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"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":"","default_value":0,"optional":true},"Workflow":{"rejected":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"all":[{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":-1,"date":"2014-12-11 14:10:27.000000000","permitted_voting_range":{"min":-1,"max":0},"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"_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":0,"max":2},"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"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":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"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":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-12-11 11:13:39.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":"2014-12-11 12:36:38.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2014-12-11 12:36:51.000000000","updated_by":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"reviewer":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"state":"REVIEWER"},{"updated":"2014-12-12 17:22:44.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":"2014-12-15 22:14:21.000000000","updated_by":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"reviewer":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"state":"REVIEWER"}],"messages":[{"id":"e4a2d7e9148774e70aa2400367e667fb224865a2","author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"date":"2014-12-11 11:10:32.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"e155f745c47f08311a1a67292299e6b12c057422","author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"date":"2014-12-11 11:43:32.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"69fe3e19ae2e8acb1995ac2a03b44d84c7edae80","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-12-11 12:36:38.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- check-tempest-dsvm-full http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-full/7304307 : SUCCESS in 31m 07s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-postgres-full/2319234 : SUCCESS in 29m 04s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-neutron-full/aae36b7 : SUCCESS in 46m 22s\n- check-grenade-dsvm http://logs.openstack.org/14/141014/2/check/check-grenade-dsvm/eeff621 : SUCCESS in 25m 31s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/14/141014/2/check/gate-tempest-dsvm-large-ops/a27486a : SUCCESS in 20m 25s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/14/141014/2/check/gate-tempest-dsvm-neutron-large-ops/60d01df : SUCCESS in 20m 12s\n- gate-tempest-pep8 http://logs.openstack.org/14/141014/2/check/gate-tempest-pep8/f17cf59 : SUCCESS in 2m 40s\n- gate-tempest-docs http://docs-draft.openstack.org/14/141014/2/check/gate-tempest-docs/4e0116b/doc/build/html/ : SUCCESS in 2m 42s\n- gate-tempest-python27 http://logs.openstack.org/14/141014/2/check/gate-tempest-python27/da240f4 : FAILURE in 3m 09s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/14/141014/2/check/check-grenade-dsvm-partial-ncpu/75f5f4c : SUCCESS in 25m 31s\n- check-tempest-dsvm-ironic-pxe_ssh http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-ironic-pxe_ssh/5e08dbd : SUCCESS in 32m 16s\n- check-tempest-dsvm-full-juno http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-full-juno/83e15a4 : SUCCESS in 31m 44s\n- check-tempest-dsvm-neutron-juno http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-neutron-juno/aab9dc0 : SUCCESS in 47m 23s\n- check-tempest-dsvm-postgres-full-juno http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-postgres-full-juno/700e53f : SUCCESS in 39m 05s\n- check-tempest-dsvm-neutron-full-juno http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-neutron-full-juno/53e7ecf : SUCCESS in 42m 52s\n- gate-tempest-dsvm-large-ops-juno http://logs.openstack.org/14/141014/2/check/gate-tempest-dsvm-large-ops-juno/2279e8e : SUCCESS in 34m 18s\n- gate-tempest-dsvm-neutron-large-ops-juno http://logs.openstack.org/14/141014/2/check/gate-tempest-dsvm-neutron-large-ops-juno/f0014ca : SUCCESS in 25m 19s\n- check-tempest-dsvm-full-icehouse http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-full-icehouse/5c5934e : SUCCESS in 38m 19s\n- check-tempest-dsvm-neutron-icehouse http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-neutron-icehouse/9a21704 : SUCCESS in 46m 49s\n- check-tempest-dsvm-postgres-full-icehouse http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-postgres-full-icehouse/0b156fc : FAILURE in 34m 22s\n- check-tempest-dsvm-neutron-full-icehouse http://logs.openstack.org/14/141014/2/check/check-tempest-dsvm-neutron-full-icehouse/ff76b0d : SUCCESS in 43m 53s (non-voting)\n- gate-tempest-dsvm-large-ops-icehouse http://logs.openstack.org/14/141014/2/check/gate-tempest-dsvm-large-ops-icehouse/ad405d3 : SUCCESS in 24m 11s\n- gate-tempest-dsvm-neutron-large-ops-icehouse http://logs.openstack.org/14/141014/2/check/gate-tempest-dsvm-neutron-large-ops-icehouse/2dadf23 : SUCCESS in 26m 44s","accounts_in_message":[],"_revision_number":2},{"id":"28f788915f80c0a59d01f72df8ac475b7edacbcb","author":{"_account_id":8871,"name":"Elastic Recheck","username":"elasticrecheck"},"date":"2014-12-11 12:36:51.000000000","message":"Patch Set 2:\n\nI noticed jenkins failed, I think you hit bug(s):\n\n- check-tempest-dsvm-postgres-full-icehouse: https://bugs.launchpad.net/bugs/1360415\n- gate-tempest-python27: unrecognized error\n\nSome of the tests failed in a way that we did not understand. Please help us classify these issues so that they can be part of Elastic Recheck http://status.openstack.org/elastic-recheck/\nFor more details on this and other bugs, please see http://status.openstack.org/elastic-recheck/","accounts_in_message":[],"_revision_number":2},{"id":"b5670e5a941fa8a55a1a1089d3bbd29da0087fa2","author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"date":"2014-12-11 14:10:27.000000000","message":"Patch Set 2: Workflow-1","accounts_in_message":[],"_revision_number":2},{"id":"6b0de4d286da753a952699ab8782e4d1f187e595","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2014-12-12 17:22:44.000000000","message":"Patch Set 2: Verified+1\n\nPassed using XenAPI driver with XenServer 6.2: Logs at http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/14/141014/2/2267/results.html\n\nUse \"xenserver: recheck\" to trigger only xenserver re-check.  XenServer CI contact: openstack@citrix.com.\n\nDebugging suggestions at https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures","accounts_in_message":[],"_revision_number":2},{"id":"dec76eb340cc8530ba178f7883b3df1e3ae6b8c2","author":{"_account_id":1192,"name":"David Kranz","email":"dkranz@redhat.com","username":"david-kranz"},"date":"2014-12-15 22:14:21.000000000","message":"Patch Set 2:\n\nKen\u0027ichi, how is this going? Is there some issue? These patches plus this one (ready for review) https://review.openstack.org/#/c/141152/ are needed to start the neutron in-project functional test work. Is there anything I can do to help with this?","accounts_in_message":[],"_revision_number":2},{"id":"e8ef4e1abdf3ecc3bacf822fe102cac5a95ec114","author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"date":"2014-12-22 01:27:16.000000000","message":"Patch Set 2:\n\n@David\n\nSorry for my late response here.\nNow this patch should be abandoned because each CONF values can be moved to original each project client like https://review.openstack.org/#/c/142384/ , https://review.openstack.org/#/c/142385/ , etc. So we will be able to remove CONF values from the existing RestClient and the class will be useful as a library function.","accounts_in_message":[],"_revision_number":2},{"id":"80a09a4f9ba71bec5fa7d9d442c3ff0734fe4950","author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"date":"2014-12-22 01:27:28.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"593cd2deb0c355c9cf4e8561b84e1e5f01945b5f","revisions":{"a6dc49f6de7cb4f05693b19db6372b5ce8a84cba":{"kind":"REWORK","_number":1,"created":"2014-12-11 11:10:32.000000000","uploader":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"ref":"refs/changes/14/141014/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/14/141014/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/141014/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/141014/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/141014/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/14/141014/1"}}},"commit":{"parents":[{"commit":"e36a328ce27ba590d0f5a0f02c793ce4c73d3ab3","subject":"Remove Nova v3 API code from RestClient","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/e36a328ce27ba590d0f5a0f02c793ce4c73d3ab3"}]}],"author":{"name":"Ken\u0027ichi Ohmichi","email":"oomichi@mxs.nes.nec.co.jp","date":"2014-12-11 10:23:37.000000000","tz":0},"committer":{"name":"Ken\u0027ichi Ohmichi","email":"oomichi@mxs.nes.nec.co.jp","date":"2014-12-11 10:39:23.000000000","tz":0},"subject":"Add BaseRestClient class to separate CONF values","message":"Add BaseRestClient class to separate CONF values\n\nTo use a rest client class at different projects, we need to separate\nTempest own config values from a base rest client class.\nThis patch adds a new BaseRestClient class for doing this.\nHowever, BaseRestClient class still contains a few Tempest own config\nvalues because _get_region() and _get_endpoint_type() might be necessary\nto be considered for each project setting and it is better to create\nanother patch for easy review.\n\nChange-Id: I35f55c895dafc4235c9c5a10b1ccf9ebfffd7265\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/a6dc49f6de7cb4f05693b19db6372b5ce8a84cba"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/a6dc49f6de7cb4f05693b19db6372b5ce8a84cba"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"e36a328ce27ba590d0f5a0f02c793ce4c73d3ab3","is_merged_in_target_branch":false,"change_id":"I46631092e1c851f4603035c3dcc11287b777323c","change_number":140995,"patch_set_number":1,"change_status":"MERGED"}],"branch":"refs/heads/master"},"593cd2deb0c355c9cf4e8561b84e1e5f01945b5f":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2014-12-11 11:43:32.000000000","uploader":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"ref":"refs/changes/14/141014/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tempest","ref":"refs/changes/14/141014/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/141014/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/141014/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tempest refs/changes/14/141014/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tempest refs/changes/14/141014/2"}}},"commit":{"parents":[{"commit":"e8dfd67c5564800cdd4630b73a59aaef11972148","subject":"Remove Nova v3 API code from RestClient","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/e8dfd67c5564800cdd4630b73a59aaef11972148"}]}],"author":{"name":"Ken\u0027ichi Ohmichi","email":"oomichi@mxs.nes.nec.co.jp","date":"2014-12-11 10:23:37.000000000","tz":0},"committer":{"name":"Ken\u0027ichi Ohmichi","email":"oomichi@mxs.nes.nec.co.jp","date":"2014-12-11 11:42:58.000000000","tz":0},"subject":"Add BaseRestClient class to separate CONF values","message":"Add BaseRestClient class to separate CONF values\n\nTo use a rest client class at different projects, we need to separate\nTempest own config values from a base rest client class.\nThis patch adds a new BaseRestClient class for doing this.\nHowever, BaseRestClient class still contains a few Tempest own config\nvalues because _get_region() and _get_endpoint_type() might be necessary\nto be considered for each project setting and it is better to create\nanother patch for easy review.\n\nChange-Id: I35f55c895dafc4235c9c5a10b1ccf9ebfffd7265\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/593cd2deb0c355c9cf4e8561b84e1e5f01945b5f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tempest/commit/593cd2deb0c355c9cf4e8561b84e1e5f01945b5f"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"e8dfd67c5564800cdd4630b73a59aaef11972148","is_merged_in_target_branch":true,"change_id":"I46631092e1c851f4603035c3dcc11287b777323c","change_number":140995,"patch_set_number":2,"change_status":"MERGED"}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}