)]}'
{"id":"openstack%2Fcyborg~682565","triplet_id":"openstack%2Fcyborg~master~I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc","project":"openstack/cyborg","branch":"master","topic":"modify_get_nova_and_placement_client","hashtags":[],"change_id":"I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc","subject":"Fix the hardcoding of user role using sdk_adapter approach","status":"MERGED","created":"2019-09-17 07:43:10.000000000","updated":"2019-09-29 04:21:36.000000000","submitted":"2019-09-29 04:20:37.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":64,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"682565-1569730837867-654589fa","meta_rev_id":"c8690fa231abfcbc8b6ed721eccd03eff4f3fd57","_number":682565,"virtual_id_number":682565,"owner":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},{"value":0,"_account_id":14131,"name":"shaohef","email":"shaohe.feng@intel.com","username":"shaohefeng"},{"value":0,"_account_id":22899,"name":"coco-Gao","email":"419546439@qq.com","username":"Coco"},{"value":0,"_account_id":17813,"name":"wangzhh","email":"wzh_1993@126.com","username":"wangzhh"},{"value":0,"date":"2019-09-27 12:20:15.000000000","_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":30759,"name":"Shogo Saito","email":"shogo.saito.ac@hco.ntt.co.jp","username":"s.shogo"},{"value":0,"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},{"value":0,"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"value":0,"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},{"value":0,"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2019-09-29 04:20:37.000000000","post_submit":true,"permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2019-09-29 01:19:15.000000000","_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"}],"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":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},"all":[{"value":0,"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},{"value":0,"_account_id":14131,"name":"shaohef","email":"shaohe.feng@intel.com","username":"shaohefeng"},{"value":0,"_account_id":22899,"name":"coco-Gao","email":"419546439@qq.com","username":"Coco"},{"value":0,"_account_id":17813,"name":"wangzhh","email":"wzh_1993@126.com","username":"wangzhh"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":30759,"name":"Shogo Saito","email":"shogo.saito.ac@hco.ntt.co.jp","username":"s.shogo"},{"value":2,"date":"2019-09-27 07:13:57.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},{"value":1,"date":"2019-09-26 06:29:46.000000000","permitted_voting_range":{"min":1,"max":2},"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"value":2,"date":"2019-09-26 07:00:58.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},{"value":1,"date":"2019-09-26 06:31:38.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"}],"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":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},"all":[{"value":0,"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},{"value":0,"_account_id":14131,"name":"shaohef","email":"shaohe.feng@intel.com","username":"shaohefeng"},{"value":0,"_account_id":22899,"name":"coco-Gao","email":"419546439@qq.com","username":"Coco"},{"value":0,"_account_id":17813,"name":"wangzhh","email":"wzh_1993@126.com","username":"wangzhh"},{"value":0,"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"value":0,"_account_id":30759,"name":"Shogo Saito","email":"shogo.saito.ac@hco.ntt.co.jp","username":"s.shogo"},{"value":1,"date":"2019-09-29 03:50:50.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},{"value":0,"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"value":0,"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},{"value":0,"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},{"_account_id":14131,"name":"shaohef","email":"shaohe.feng@intel.com","username":"shaohefeng"},{"_account_id":17813,"name":"wangzhh","email":"wzh_1993@126.com","username":"wangzhh"},{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":22899,"name":"coco-Gao","email":"419546439@qq.com","username":"Coco"},{"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},{"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},{"_account_id":30759,"name":"Shogo Saito","email":"shogo.saito.ac@hco.ntt.co.jp","username":"s.shogo"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-09-18 09:28:36.000000000","updated_by":{"_account_id":14131,"name":"shaohef","email":"shaohe.feng@intel.com","username":"shaohefeng"},"reviewer":{"_account_id":14131,"name":"shaohef","email":"shaohe.feng@intel.com","username":"shaohefeng"},"state":"REVIEWER"},{"updated":"2019-09-18 09:28:40.000000000","updated_by":{"_account_id":22899,"name":"coco-Gao","email":"419546439@qq.com","username":"Coco"},"reviewer":{"_account_id":22899,"name":"coco-Gao","email":"419546439@qq.com","username":"Coco"},"state":"REVIEWER"},{"updated":"2019-09-18 09:28:53.000000000","updated_by":{"_account_id":17813,"name":"wangzhh","email":"wzh_1993@126.com","username":"wangzhh"},"reviewer":{"_account_id":17813,"name":"wangzhh","email":"wzh_1993@126.com","username":"wangzhh"},"state":"REVIEWER"},{"updated":"2019-09-18 09:28:59.000000000","updated_by":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"reviewer":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"state":"REVIEWER"},{"updated":"2019-09-19 01:24:49.000000000","updated_by":{"_account_id":30759,"name":"Shogo Saito","email":"shogo.saito.ac@hco.ntt.co.jp","username":"s.shogo"},"reviewer":{"_account_id":30759,"name":"Shogo Saito","email":"shogo.saito.ac@hco.ntt.co.jp","username":"s.shogo"},"state":"REVIEWER"},{"updated":"2019-09-26 06:29:46.000000000","updated_by":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"reviewer":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"state":"REVIEWER"},{"updated":"2019-09-26 06:31:38.000000000","updated_by":{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},"reviewer":{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},"state":"REVIEWER"},{"updated":"2019-09-26 07:00:58.000000000","updated_by":{"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},"reviewer":{"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},"state":"REVIEWER"},{"updated":"2019-09-27 12:20:15.000000000","updated_by":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"reviewer":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"state":"REVIEWER"},{"updated":"2019-09-29 03:50:50.000000000","updated_by":{"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},"reviewer":{"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},"state":"REVIEWER"},{"updated":"2019-09-29 04:20:37.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"c78e10295b954c60f515fbfc5346228dbac6729c","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-17 07:43:10.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b28216c84b8657df38aa9c1171d341f07f3ea21d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-17 08:19:40.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2f002b75d8a646f6953da4fa43f74a33 : FAILURE in 3m 08s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/63485b0985204268916432e946e043d3 : FAILURE in 3m 40s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/abc60bcef2774583839ae3cafc52f64e : FAILURE in 2m 53s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/16579d915a2842bcbe6af51537f721d3 : FAILURE in 7m 37s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/bb2c51abf1ec44ab8c5d0eb35096a18b : SUCCESS in 4m 27s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/1174f314b92f48e0a4dfff3f21677a83 : FAILURE in 32m 53s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"e475dff8eef350b827ae91a0ad52d87ee0e1ae71","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-17 09:28:22.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"3dcc30be7d070c0974a1be52ad0ee87cd6d1b213","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-17 10:11:15.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c6e8cc0fe7b44d139c2a4b4887fff7c4 : SUCCESS in 4m 14s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/a1c6dcdd527f400a8104612b441f7b55 : SUCCESS in 3m 50s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/6142b9e35261458795c97db85cd3dcab : SUCCESS in 3m 33s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/454ccbc9c0af4dde95ba6f283d30a1ff : RETRY_LIMIT in 2m 03s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/ed4860413aef4fcbba20c037164da959 : SUCCESS in 5m 19s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/248396981b6c44ec984d3ea3d9481bfd : SUCCESS in 40m 28s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"5a3abac8e0430c4ce69eed5d0295a11ec11328eb","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-17 10:20:56.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"9cc010f0f60fbcc7d43f47e35c4c09f4437b7df6","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-17 10:59:25.000000000","message":"Patch Set 2:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/826a863c0b494c619d2bab4cdfcf7a3c : SUCCESS in 4m 15s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/f53269e13fbb4cadb8383c692448b9f6 : RETRY_LIMIT in 2m 37s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/444f02a0455446ba896617c7a047d16f : SUCCESS in 4m 47s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/f430adad0c244d8486fb0306414549f2 : SUCCESS in 4m 03s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/079379de89b64a2bad6fcab6373a43fd : SUCCESS in 4m 38s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/42302dc4d0af45d5a51829aabc774596 : SUCCESS in 34m 37s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"5f5e472ca9dad24edc43ccdfec163428d9e2a445","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-17 11:00:30.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"7c969d87160de03f0c455b319f33bd470cccad36","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-17 11:35:51.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/8878033b40124ac79d538e3dc639f055 : SUCCESS in 4m 26s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/d463b4e3a7ae4b03ab8feb045ff5e1db : SUCCESS in 3m 44s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/dbfe3fb3734041d59343ab801c21b329 : SUCCESS in 4m 40s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/803abb2d4e154f389af7269bb5937542 : SUCCESS in 4m 17s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b5ad558fcb1c472fa9ac4207a2988e60 : SUCCESS in 4m 07s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/04fad0e1b27c48b8be874dd3eb68fe56 : SUCCESS in 33m 33s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"9d0c8f8cabf634297c4e8a22849c9341d6de85b7","author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"date":"2019-09-17 12:07:26.000000000","message":"Patch Set 2:\n\n(1 comment)\n\nThanks for the quick turnaround, chenke!","accounts_in_message":[],"_revision_number":2},{"id":"558f93691f1b18b1abfd3ee83fcdc7550fba07dc","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-17 12:40:18.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"796949bea606e6c05f7d9e594ffac17b997f5d59","author":{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},"date":"2019-09-18 01:39:12.000000000","message":"Patch Set 2: Code-Review-1\n\nAlmost nearly, I checked the code, now the placement conf code was in conf/default.py, I would to see you seperate the code to conf/placement.py in this patch. That will make code more clearly. thanks","accounts_in_message":[],"_revision_number":2},{"id":"ef46e2bd016188a59758a50b150787a034672c5b","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-18 01:47:08.000000000","message":"Patch Set 2: Workflow-1\n\n\u003e Almost nearly, I checked the code, now the placement conf code was\n \u003e in conf/default.py, I would to see you seperate the code to\n \u003e conf/placement.py in this patch. That will make code more clearly.\n \u003e thanks\n\nThanks. will seperate it in a indepent placement.py file.","accounts_in_message":[],"_revision_number":2},{"id":"87167a93d9ebf84eeb8e05952e9995cef418c0ef","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-18 04:01:24.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"f805ed18798e90e4a6660d8ae1fe5c32ea66a460","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-18 04:25:36.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"5c7ce65663e3ae8901612134a1c1f7f8bdd49d5f","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-18 04:47:46.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/626552b60edf4ca09d96412f48aa69ac : FAILURE in 4m 37s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/c2c20a17333d4a78b2c0535e448f8921 : FAILURE in 4m 33s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/636425fe26ea47fba46f71feca780006 : FAILURE in 4m 25s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/e118af3931b84fb98b46593facaed5d0 : FAILURE in 4m 29s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b93a168eb6724a12be78b25d52bf850e : FAILURE in 4m 36s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/9343e6bb64b04693bc01505e24d80ce3 : FAILURE in 35m 07s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"455dce20127ce75374a428b0f96b7820aaa7a1d0","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-18 06:03:38.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"7a8a0cc1124b4d3eba1956ec728ba24fc00435f2","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-18 07:06:02.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5c9dba3700804d9ca09f77341f30aab4 : SUCCESS in 3m 54s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/24fcfb158ab74c93803bf53c473adadb : SUCCESS in 3m 59s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/3507101eb4dd4ce69872bc44edf15a8a : SUCCESS in 3m 39s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/36f78761d4b3482ea6c632c52eb47352 : SUCCESS in 3m 53s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/52b08d1505324b1c856ae7b17caa8c03 : SUCCESS in 4m 29s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/9e6c82960f3e49a39a682c518ecd8d88 : SUCCESS in 38m 57s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"61dbb1b761a5c686f26d20398851503002ce7a90","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-18 09:09:55.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"807bb1afdc4db06235273a0749a70b9a9b45043a","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-18 09:11:24.000000000","message":"Uploaded patch set 6: Commit message was updated.","accounts_in_message":[],"_revision_number":6},{"id":"eee1f61e563dc8a01798bd2a18824c2f9113f8a2","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-18 09:49:14.000000000","message":"Patch Set 6: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/dd2b12b43cf541ed85630023541da409 : SUCCESS in 4m 18s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/0be2178cf6e0453d8b1d9a9ccc80a38f : SUCCESS in 4m 51s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/e89ea4622e8049568024a7f499c7a626 : SUCCESS in 3m 05s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/51c28434fbb34eb28035d0b20e704fb5 : SUCCESS in 5m 16s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3691acf51ad5480c8139de1a62070988 : SUCCESS in 4m 36s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/084669a3d32e498896b23d06cebbc3e8 : SUCCESS in 29m 39s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"262cf5333f8b78c45bc713f3f4c4e5a93a14b9df","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-18 10:32:09.000000000","message":"Patch Set 6:\n\n@Eric Fried Hello. I hope you can review this patch when you have free time. Thanks you!","accounts_in_message":[],"_revision_number":6},{"id":"dd357476390523c1684b36d7ab7ecce2722165c8","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-18 14:26:05.000000000","message":"Patch Set 6: Code-Review-1\n\n(17 comments)\n\nI like the direction this is going.","accounts_in_message":[],"_revision_number":6},{"id":"04fa0942425775b3ed158b550b5ea1f98985fdb0","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-18 14:33:49.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"32ff335c7119fc1111a07b45255893b88e576c74","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-18 15:09:45.000000000","message":"Patch Set 6:\n\n(2 comments)","accounts_in_message":[],"_revision_number":6},{"id":"a6bb5cebcebfdf3352206eb862df14c3c7c5baab","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-19 08:52:50.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"a8cded4b3fe8156f7533e6ee029c5eda903a8070","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-19 09:26:54.000000000","message":"Patch Set 7: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/8fe6e1c9a4b04d1ea44651119770a3f7 : SUCCESS in 3m 06s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/665e37990b3747f7856929ac6f9353bd : SUCCESS in 3m 58s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/51b625e10bd54915b59daa7561337ea7 : SUCCESS in 3m 13s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/25af26c2d48c4fbfb35d4f4705618d6d : SUCCESS in 4m 01s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/3ca285785abb41bfa248cf5f15b5dc29 : SUCCESS in 5m 17s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/9741ba041f4b4cc0bb9ed25c0947a23f : FAILURE in 27m 58s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"cd9441f7bc0d6ca9a146d64d4b8d20550ce451ec","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-19 12:08:51.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"44e2b970ef8d64d8dd61ecafbf6749e7569f9e1f","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-19 12:19:31.000000000","message":"Patch Set 6:\n\n(14 comments)","accounts_in_message":[],"_revision_number":6},{"id":"8d95c14d1098d111b987c16014204accf4118339","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-19 12:53:47.000000000","message":"Patch Set 8: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ab3acd29c1a14aafbbb267a2a3570de6 : SUCCESS in 2m 59s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/7693db80c2974976a52584449dfe2a0e : SUCCESS in 4m 35s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/f81ec677076b48999b45e5c5a0fe846e : SUCCESS in 3m 24s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/7c137f68975a473fbe59ce6b80332593 : SUCCESS in 5m 53s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/7cc6e2516b834fb5ba26d91dff839292 : SUCCESS in 5m 55s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/d501f69ad48144e59911574a904547f2 : SUCCESS in 38m 35s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"638869ccca73834d6cac854ca97fdf76c7b1abc2","author":{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},"date":"2019-09-20 06:23:42.000000000","message":"Patch Set 8: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":8},{"id":"77d9574b7ae41c15a59f383459aec2cb40a642ec","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-20 09:32:31.000000000","message":"Uploaded patch set 9.","accounts_in_message":[],"_revision_number":9},{"id":"2e420b7396f0801e402499a67c0ed21aebd6554a","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-20 09:34:35.000000000","message":"Patch Set 9:\n\n(1 comment)","accounts_in_message":[],"_revision_number":9},{"id":"c3a64bcfff15d7ad8f21827af64141454519b851","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-20 09:44:13.000000000","message":"Patch Set 9:\n\n(1 comment)","accounts_in_message":[],"_revision_number":9},{"id":"a995847fcefa370f5195dc6eb1f3f0980a71be3a","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-20 10:14:14.000000000","message":"Patch Set 9: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/c377909583fd4de28aa1b2f194e17eda : SUCCESS in 3m 28s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/0260c045265642018767ebd64d87229b : SUCCESS in 5m 19s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/129da3ff70044fa7af01bfecf8314c94 : SUCCESS in 3m 24s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/37b854a055a746e99a51f9f381488543 : SUCCESS in 4m 02s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4e6a995840c54baeb02b5b010ac0bf51 : SUCCESS in 4m 07s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/db70790d9f62465491ca70e20c054865 : SUCCESS in 34m 55s (non-voting)","accounts_in_message":[],"_revision_number":9},{"id":"7e1db50eb48f776a704211e1d665916353c4c9ee","author":{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},"date":"2019-09-23 01:20:37.000000000","message":"Patch Set 9: Code-Review+1\n\nLGTM, next follow-up patch should change the glance client.","accounts_in_message":[],"_revision_number":9},{"id":"528c19141bc9e0f987c0d90b8d08daaf1d74aed1","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-23 02:00:07.000000000","message":"Patch Set 9:\n\n\u003e LGTM, next follow-up patch should change the glance client.\n\nYep. will fix glanceclient in the next patch.","accounts_in_message":[],"_revision_number":9},{"id":"8aed41f23e75d1827b827c52ac9ac8d8d82c9db6","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-23 14:20:48.000000000","message":"Patch Set 9: Code-Review+1\n\n(7 comments)\n\nI\u0027d like to downvote for the missing requirements, but somehow this is working. It should be addressed in a subsequent patch, though, for sure.\n\nI have a pretty serious problem with the fact that there\u0027s zero unit test for any of this (cyborg/tests/unit/services/_test_placement_client.py doesn\u0027t run because its name starts with an underscore -- I tried renaming it and it doesn\u0027t even get past imports -- and I don\u0027t even see any attempt at UT for the nova interaction).\n\nBut I appreciate the need to get this code into Train, if only to avoid a messy conf support matrix moving forward, and the code itself looks good to me, so I suppose we should land it.","accounts_in_message":[],"_revision_number":9},{"id":"32e02986fc8aa54d9f8ae3e40484cf09ebbf18fa","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-24 01:48:42.000000000","message":"Patch Set 9:\n\n(2 comments)\n\n\u003e (7 comments)\n \u003e \n \u003e I\u0027d like to downvote for the missing requirements, but somehow this\n \u003e is working. It should be addressed in a subsequent patch, though,\n \u003e for sure.\n Will update the requirement in the subsequent patch. Thanks.","accounts_in_message":[],"_revision_number":9},{"id":"b098cbbb982c137228611e87a60fc002b53350d7","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-24 01:56:02.000000000","message":"Patch Set 10: Published edit on patch set 9.","accounts_in_message":[],"_revision_number":10},{"id":"c73e52250c0cbe40a57475f2773a80456cef577e","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-24 01:57:01.000000000","message":"Patch Set 10:\n\n(3 comments)","accounts_in_message":[],"_revision_number":10},{"id":"060c756f1e4c6a70c1c5c8e2cbb24b60b6f69958","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-24 02:32:18.000000000","message":"Patch Set 10: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/363a4953ecc8435e8f962a8fc6d196cc : SUCCESS in 7m 04s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/d6045640d7fe44bcb75f1ca3de5cf92b : SUCCESS in 3m 23s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/bc38f75788574c69ad8dd26a8fce6ca3 : SUCCESS in 3m 36s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/5198822780ac44579f24d659d776f3ad : SUCCESS in 5m 39s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/934dd33f561c45389951ba34036fa4a2 : SUCCESS in 3m 24s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/9965f8f829e84d69b050aa56f03d1286 : SUCCESS in 31m 06s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"3a8ff08be6d1cae5728732dcc64c4691465efd0a","author":{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},"date":"2019-09-24 03:31:24.000000000","message":"Patch Set 10: Code-Review+1","accounts_in_message":[],"_revision_number":10},{"id":"68ae646dc29b46c4f537fffe5fc6b2951a0d2fbf","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-24 18:10:17.000000000","message":"Patch Set 10: Code-Review-1\n\nSeveral things have been bugging me about this. Adding to the list, I looked a little deeper into the cyborg-tempest job, and (as I should have realized if I had thought it through) it\u0027s not hitting any code paths that would cause cyborg to need to talk to nova. Combined with the lack of unit/functional testing, that means there\u0027s nothing actually exercising the code in this patch. I\u0027ve heard rumors that it has been tested in various local manual devstack type environments, with varying results; but even if all of those reports were positive, IMO it is important that we demonstrate this working in upstream CI.\n\nWhat this means is that we need a patch somewhere that pulls in this one *and* the nova side [1] and passes test_server_basic_ops such that we can inspect the various logs and prove that cyborg talked to both placement and nova correctly. I see [2], which is a good start; it may be adequate to make it Depends-On this patch and add cyborg to required_projects.\n\n[1] https://review.opendev.org/#/q/topic:bp/nova-cyborg-interaction+status:open\n[2] https://review.opendev.org/#/c/670999/","accounts_in_message":[],"_revision_number":10},{"id":"3e59313db4139e3e01df0fad9963990871aba545","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-24 19:00:09.000000000","message":"Patch Set 10:\n\n(3 comments)","accounts_in_message":[],"_revision_number":10},{"id":"4fb7c1b9d46f31f501997005bf9e5101b4513b91","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-25 05:58:00.000000000","message":"Patch Set 11: Commit message was updated.","accounts_in_message":[],"_revision_number":11},{"id":"e70972a16f285ea3e6742c6d17c8a8a29d14417e","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-25 05:58:52.000000000","message":"Patch Set 11: 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":11},{"id":"6855c625193752c5ca0c703a04ffe3ba47282dc0","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-25 06:00:13.000000000","message":"Patch Set 12: Patch Set 11 was rebased","accounts_in_message":[],"_revision_number":12},{"id":"5cabf44ac8d76b6ce125d9d30e1a420bab9aa109","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-25 06:01:07.000000000","message":"Patch Set 12: 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":12},{"id":"61f4847bf1210f8183ee60cd67cd4201a9c60de0","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-25 06:19:01.000000000","message":"Uploaded patch set 13: Patch Set 12 was rebased.","accounts_in_message":[],"_revision_number":13},{"id":"9d06ec2845d8eb6a2231fdc824f18d275a0ad722","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-25 06:19:20.000000000","message":"Patch Set 13:\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":13},{"id":"1392675444e070207023d8a1ab1c4c1838e393cf","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-25 07:31:41.000000000","message":"Uploaded patch set 14: Commit message was updated.","accounts_in_message":[],"_revision_number":14},{"id":"927b19edea3a15ccd00d6fb7766fdefd1b72a2f5","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-25 08:22:58.000000000","message":"Patch Set 14: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/07a19b528c6a4927a34260a870cd62cd : SUCCESS in 4m 02s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/6d6fd16a4ad54940993d29871ac7f85c : SUCCESS in 3m 43s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/f35867328d2d429b91a9ed240bd93a91 : SUCCESS in 4m 27s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/378c16b79f8d41ff86439b4d7a279746 : SUCCESS in 4m 01s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/7a3134a5f940473f9789f79bde2aa376 : SUCCESS in 5m 20s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/15fe4f940b854a0294e42fcb9e5c8cc0 : SUCCESS in 38m 36s (non-voting)\n- cyborg-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/999940d9034b4003b426f78859a5f8fe : SUCCESS in 42m 32s (non-voting)","accounts_in_message":[],"_revision_number":14},{"id":"d76a2ad9079d4c47fef7811061bc4727eea87b44","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-25 21:24:50.000000000","message":"Patch Set 14:\n\nPS14 seems to be back to PS10: https://review.opendev.org/#/c/682565/10..14","accounts_in_message":[],"_revision_number":14},{"id":"3a14e66c72bfb46bbcf5296515db35592cf05afb","author":{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},"date":"2019-09-26 02:15:56.000000000","message":"Patch Set 14: Code-Review-1\n\nThis is indeed need more tests for this. in the tempest tests gives the error: ServiceDiscoveryException: Failed to create a working proxy for service compute: No valid endpoint was discoverable.\nreflog: https://openstack.fortnebula.com:13808/v1/AUTH_e8fd161dc34c421a979a9e6421f823e9/zuul_opendev_logs_7d4/670999/12/experimental/cyborg-tempest/7d4c0f4/controller/logs/screen-cyborg-api.txt.gz","accounts_in_message":[],"_revision_number":14},{"id":"237073ac897105118c7edda786fd7ee742c805a2","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 02:34:19.000000000","message":"Patch Set 14:\n\n\u003e PS14 seems to be back to PS10: https://review.opendev.org/#/c/682565/10..14\n\nYes. Thanks eric. This patch has been added in nova\u0027s patch about cyborg-tempest[1].\n[1]. https://review.opendev.org/#/c/670999/","accounts_in_message":[],"_revision_number":14},{"id":"d2f0ef24c2848f4a7415f7d74669147dcd029225","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 02:35:01.000000000","message":"Patch Set 14:\n\n\u003e This is indeed need more tests for this. in the tempest tests gives\n \u003e the error: ServiceDiscoveryException: Failed to create a working\n \u003e proxy for service compute: No valid endpoint was discoverable.\n \u003e reflog: https://openstack.fortnebula.com:13808/v1/AUTH_e8fd161dc34c421a979a9e6421f823e9/zuul_opendev_logs_7d4/670999/12/experimental/cyborg-tempest/7d4c0f4/controller/logs/screen-cyborg-api.txt.gz\n\nThanks zhurong and Sundar. I will update it according this error.","accounts_in_message":[],"_revision_number":14},{"id":"a7690f2346f08b5efa09a703c503941a64d6edbb","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-26 02:44:33.000000000","message":"Patch Set 14:\n\nIt\u0027s a bug in the sdk. I opened a story for it today, will get the link when I\u0027m back at my desk. In the meantime, you can use strict_proxies\u003dFalse as a workaround.","accounts_in_message":[],"_revision_number":14},{"id":"ec1ab0f38c4484defa9f4d2b67ec6f5ee7e4c136","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 02:48:46.000000000","message":"Patch Set 14:\n\n\u003e It\u0027s a bug in the sdk. I opened a story for it today, will get the\n \u003e link when I\u0027m back at my desk. In the meantime, you can use\n \u003e strict_proxies\u003dFalse as a workaround.\n\nOk. Thanks.  If the tempest in nova patch passed after the update, do you recommend that this commit be merged into this Stein version?","accounts_in_message":[],"_revision_number":14},{"id":"cc6057b21eca79cd6b7bd6d1c56e8326a17effa2","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 02:50:42.000000000","message":"Uploaded patch set 15.","accounts_in_message":[],"_revision_number":15},{"id":"93a820929b5f81f0a3125bbebbe63d059744934f","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 02:50:56.000000000","message":"Patch Set 14:\n\n\u003e \u003e It\u0027s a bug in the sdk. I opened a story for it today, will get\n \u003e the\n \u003e \u003e link when I\u0027m back at my desk. In the meantime, you can use\n \u003e \u003e strict_proxies\u003dFalse as a workaround.\n \u003e \n \u003e Ok. Thanks.  If the tempest in nova patch passed after the update,\n \u003e do you recommend that this commit be merged into this Stein\n \u003e version?\n\nSorry. Train version.","accounts_in_message":[],"_revision_number":14},{"id":"b5d9e1ad4f887acc7b28d9ef5f3056266e4525ed","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 02:59:10.000000000","message":"Patch Set 10:\n\n(1 comment)","accounts_in_message":[],"_revision_number":10},{"id":"980e1e4b2a26bb5415f4d28d79a5a2d312e831af","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-26 03:21:29.000000000","message":"Patch Set 15:\n\n(1 comment)\n\nI think merging this in Train is unnecessarily risky, but if you can show the CI working, it\u0027s at least somewhat justifiable.\n\nNot sure there\u0027s a reason to hurry, though. There\u0027s no code in Train that\u0027s capable of using this, so why not just slow down, wait for ussuri, and do it right?","accounts_in_message":[],"_revision_number":15},{"id":"5de0123dbab299f0ac276be57b5386b4cf9faa32","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 03:28:31.000000000","message":"Uploaded patch set 16.","accounts_in_message":[],"_revision_number":16},{"id":"cb7b51a382aa7269b7fda9448efdad146288080e","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 03:30:44.000000000","message":"Patch Set 16:\n\n(3 comments)\n\nThanks eric.","accounts_in_message":[],"_revision_number":16},{"id":"685d45bbb61d4f3c1972f4265ce3dd999f63ba8d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-26 04:05:36.000000000","message":"Patch Set 16: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9b54a9248a534ed389b8dd0c8de9a59d : SUCCESS in 3m 49s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/50bc2b8a3a7c4c56bb1f8e16900c9a88 : SUCCESS in 5m 17s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/765cd742f30045de99753cafadc75407 : SUCCESS in 4m 53s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/7b8dba12907b487180dd84d39ce81b87 : SUCCESS in 3m 59s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/91ef9bea991a4fa3b61a83f682b5dbd3 : SUCCESS in 3m 15s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/445f26ae3d9b46bc9553487e7d368887 : SUCCESS in 31m 11s (non-voting)\n- cyborg-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9dafe80a89ca4d3fb0ac7ddf642f388d : SUCCESS in 29m 55s (non-voting)","accounts_in_message":[],"_revision_number":16},{"id":"4d62833b8d24c09fab1ac457a44c21ea8d705272","author":{"_account_id":21672,"name":"Sundar Nadathur","email":"sundar.nadathur@intel.com","username":"nsundar"},"date":"2019-09-26 06:29:46.000000000","message":"Patch Set 16: Code-Review+1\n\nIn the latest cyborg-tempest run from https://review.opendev.org/#/c/670999/:\n* Cyborg notified Nova successfully after bind [1].\n* Instance spawned successfully [2]\n* Test failed because device profile could not be deleted, which is because ARQ was not deleted. Those seem to be different.\n\n[1] https://storage.gra1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_fa3/670999/12/experimental/cyborg-tempest/fa3f4fa/controller/logs/screen-cyborg-api.txt.gz Sep 26 04:05:02.898768\n[2] https://storage.gra1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_fa3/670999/12/experimental/cyborg-tempest/fa3f4fa/controller/logs/screen-n-cpu.txt.gz Sep 26 04:05:09.903259","accounts_in_message":[],"_revision_number":16},{"id":"79865ab777e424ab1b8365ef89e5d44b9d851765","author":{"_account_id":14107,"name":"zhurong","email":"aaronzhu1121@gmail.com","username":"zhurong"},"date":"2019-09-26 06:31:38.000000000","message":"Patch Set 16: Code-Review+1","accounts_in_message":[],"_revision_number":16},{"id":"1efaf91cbbe715bbae50dd3b48fee530933fbc8c","author":{"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},"date":"2019-09-26 06:37:30.000000000","message":"Patch Set 16:\n\nDon\u0027t we need nova patches as a dependency here? cause it changes nova-cyborg integration part.","accounts_in_message":[],"_revision_number":16},{"id":"fb06f893008871a84e87f88f228ab34ecb2217c0","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 06:43:17.000000000","message":"Patch Set 16:\n\n\u003e Don\u0027t we need nova patches as a dependency here? cause it changes\n \u003e nova-cyborg integration part.\n\nThis patch doesn\u0027t needs. Because yumeng\u0027s patch depends on both nova\u0027s patch and this. pls see https://review.opendev.org/#/c/670999/","accounts_in_message":[],"_revision_number":16},{"id":"0b63af28193227b49fd52fbe8e3a94da28d494ee","author":{"_account_id":25738,"name":"Xinran WANG","email":"xin-ran.wang@intel.com","username":"Xinran"},"date":"2019-09-26 07:00:58.000000000","message":"Patch Set 16: Code-Review+2\n\n\u003e \u003e Don\u0027t we need nova patches as a dependency here? cause it changes\n \u003e \u003e nova-cyborg integration part.\n \u003e \n \u003e This patch doesn\u0027t needs. Because yumeng\u0027s patch depends on both\n \u003e nova\u0027s patch and this. pls see https://review.opendev.org/#/c/670999/\n\nOk, I see it. Thanks Chenke.","accounts_in_message":[],"_revision_number":16},{"id":"496d0994564421d181566eb7081db7050a2db8b5","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-26 07:28:27.000000000","message":"Patch Set 16:\n\n\u003e (1 comment)\n \u003e \n \u003e I think merging this in Train is unnecessarily risky, but if you\n \u003e can show the CI working, it\u0027s at least somewhat justifiable.\n \u003e \n \u003e Not sure there\u0027s a reason to hurry, though. There\u0027s no code in\n \u003e Train that\u0027s capable of using this, so why not just slow down, wait\n \u003e for ussuri, and do it right?\n\nHi Eric. Now the tempest result is good. pls see this:\nhttps://review.opendev.org/#/c/670999/","accounts_in_message":[],"_revision_number":16},{"id":"37ca705befc1821d372e2fbf9f6af411c53f33f8","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-26 14:40:01.000000000","message":"Patch Set 16:\n\n(4 comments)\n\nThe CI is working \\o/, but I\u0027m actually not sure how -- see my commentary in that patch [1].\n\nThis question is still worth contemplating though:\n\n\u003e Not sure there\u0027s a reason to hurry, though. There\u0027s no code in\nTrain that\u0027s capable of using this, so why not just slow down, wait\nfor ussuri, and do it right?\n\n[1] https://review.opendev.org/#/c/670999/14","accounts_in_message":[],"_revision_number":16},{"id":"0bddf283ca5e269563977392954cbe38d8c6a434","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-26 22:02:17.000000000","message":"Patch Set 16:\n\n\u003e It\u0027s a bug in the sdk. I opened a story for it today, will get the link when I\u0027m back at my desk.\n\nBetter late than never.\n- Story against sdk: https://storyboard.openstack.org/#!/story/2006614\n- WIP patch in sdk for it: https://review.opendev.org/#/c/685043/\n- ...which depends on enablement in ksa (also WIP): https://review.opendev.org/#/c/685042/\n- But the real problem is that compute shouldn\u0027t be requiring auth for version discovery: https://bugs.launchpad.net/nova/+bug/1845530\n- Functional repro for that: https://review.opendev.org/685180\n- And the seed of a fix, for which I need to phone a friend: https://review.opendev.org/685181\n\nIncidentally, in looking for clues for the nova fix, I noticed that several other services (at least cinder and neutron) appear to have the same problem :(","accounts_in_message":[],"_revision_number":16},{"id":"a46f41cb903e3dfe44ccf08100202850b6423651","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-27 01:54:15.000000000","message":"Patch Set 16:\n\n\u003e (4 comments)\n \u003e \n \u003e The CI is working \\o/, but I\u0027m actually not sure how -- see my\n \u003e commentary in that patch [1].\n \u003e \n \u003e This question is still worth contemplating though:\n \u003e \n \u003e \u003e Not sure there\u0027s a reason to hurry, though. There\u0027s no code in\n \u003e Train that\u0027s capable of using this, so why not just slow down, wait\n \u003e for ussuri, and do it right?\n \u003e \n \u003e [1] https://review.opendev.org/#/c/670999/14\n\nYes, the nova-related code has not been merged, and it is more reliable to put it in the next version. But there is the content of the placement client, which is needed for this version.","accounts_in_message":[],"_revision_number":16},{"id":"8097c5aec0c8569c8049809a1c8c541fcfb4ab11","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-27 01:58:47.000000000","message":"Patch Set 16:\n\n\u003e and then configure_cyborg_conductor can say\n\n\u003e configure_cyborg_for_service placement\n\u003e configure_cyborg_for_service glance\n\u003e configure_cyborg_for_service nova\nGood Catch. But glance and nova are a bit different because it has a dictionary api-server[1]. I am not sure if this item is necessary.","accounts_in_message":[],"_revision_number":16},{"id":"497535f815a7298594a73f5fb84951fe0f9cf826","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-27 02:00:58.000000000","message":"Patch Set 16:\n\n\u003e \u003e and then configure_cyborg_conductor can say\n \u003e \n \u003e \u003e configure_cyborg_for_service placement\n \u003e \u003e configure_cyborg_for_service glance\n \u003e \u003e configure_cyborg_for_service nova\n \n Good Catch. But glance and nova are a bit different because it has\n a config for api-server[1]. I am not sure if this item is\n necessary.\n[1]. https://github.com/openstack/cyborg/blob/master/devstack/lib/cyborg#L235","accounts_in_message":[],"_revision_number":16},{"id":"077ffb43f87ff973d31dc3f14f4c382239c75c01","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-27 02:05:01.000000000","message":"Patch Set 16:\n\n\u003e Incidentally, in looking for clues for the nova fix, I noticed that\n \u003e several other services (at least cinder and neutron) appear to have\n \u003e the same problem :(\n\nWow. This is a good tip.","accounts_in_message":[],"_revision_number":16},{"id":"982ef31f7dfb80ad35a4cb61b6f6184871f2be61","author":{"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},"date":"2019-09-27 07:13:57.000000000","message":"Patch Set 16: Code-Review+2\n\nThanks chenker for the patch! Since we need this patch for placement_client and CI shows no distinct harm to other project, I will give my +2. But maybe two things should keep in mind:\n1) After https://review.opendev.org/#/c/685043/ merged in the future, we may need back here and check again.\n2) glance_client part update","accounts_in_message":[],"_revision_number":16},{"id":"3ee471fb8e0a5087f31fcc4d3b5af1fe32bf4cb5","author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"date":"2019-09-27 12:20:15.000000000","message":"Patch Set 16:\n\n\u003e glance and nova are a bit different because it has a dictionary api-server[1]. I am not sure if this item is necessary.\n\nYou should get rid of the api_servers conf opt immediately. In fact, that *is* something I would encourage getting into Train before it is released, so you don\u0027t have to continue to support it and deprecate it and so forth. Here\u0027s the story:\n\nI\u0027m assuming you picked up api_servers when you copied in glanceclient wrappers from nova. I *thought* I had deprecated it in nova when I did [1], but now I see I didn\u0027t :(\n\nThe point of api_servers was to provide a rotating list of endpoints. When using ksa config (which is what you should be doing - whether actual ksa or sdk - and getting rid of glanceclient) it is superseded by [a load balancer plus] either the service catalog (preferred) or the `endpoint_override` conf option.\n\nTL;DR: It\u0027ll go away when you cut glance over to SDK regardless, but you should get rid of it now and just this gorp [2] until that happens.\n\n...which I see you already have in place [3] -- so you should simply get rid of the api_servers conf option entirely and remove this condition [4].\n\nAnother example of cyborg inheriting bad debt from nova.\n\n[1] https://review.opendev.org/#/c/488137/\n[2] https://opendev.org/openstack/nova/src/branch/master/nova/image/glance.py#L122-L134\n[3] https://opendev.org/openstack/cyborg/src/branch/master/cyborg/image/glance.py#L120-L132\n[4] https://opendev.org/openstack/cyborg/src/branch/master/cyborg/image/glance.py#L113-L118","accounts_in_message":[],"_revision_number":16},{"id":"2b801e51e1650c758fbe75299ebecd41cb29d36c","author":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"date":"2019-09-29 01:19:15.000000000","message":"Patch Set 16:\n\n\u003e \u003e glance and nova are a bit different because it has a dictionary\n \u003e api-server[1]. I am not sure if this item is necessary.\n \u003e \n \u003e You should get rid of the api_servers conf opt immediately. In\n \u003e fact, that *is* something I would encourage getting into Train\n \u003e before it is released, so you don\u0027t have to continue to support it\n \u003e and deprecate it and so forth. Here\u0027s the story:\n \u003e \n \u003e I\u0027m assuming you picked up api_servers when you copied in\n \u003e glanceclient wrappers from nova. I *thought* I had deprecated it in\n \u003e nova when I did [1], but now I see I didn\u0027t :(\n \u003e \n \u003e The point of api_servers was to provide a rotating list of\n \u003e endpoints. When using ksa config (which is what you should be doing\n \u003e - whether actual ksa or sdk - and getting rid of glanceclient) it\n \u003e is superseded by [a load balancer plus] either the service catalog\n \u003e (preferred) or the `endpoint_override` conf option.\n \u003e \n \u003e TL;DR: It\u0027ll go away when you cut glance over to SDK regardless,\n \u003e but you should get rid of it now and just this gorp [2] until that\n \u003e happens.\n \u003e \n \u003e ...which I see you already have in place [3] -- so you should\n \u003e simply get rid of the api_servers conf option entirely and remove\n \u003e this condition [4].\n \u003e \n \u003e Another example of cyborg inheriting bad debt from nova.\n \u003e \n \u003e [1] https://review.opendev.org/#/c/488137/\n \u003e [2] https://opendev.org/openstack/nova/src/branch/master/nova/image/glance.py#L122-L134\n \u003e [3] https://opendev.org/openstack/cyborg/src/branch/master/cyborg/image/glance.py#L120-L132\n \u003e [4] https://opendev.org/openstack/cyborg/src/branch/master/cyborg/image/glance.py#L113-L118\n\nThank Eric very much. I will use a new patch for this. Pls review it when you have free time.","accounts_in_message":[],"_revision_number":16},{"id":"85c226b4ddefe0d0593767ffb4d08ed77e76d5f2","author":{"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},"date":"2019-09-29 03:50:50.000000000","message":"Patch Set 16: Workflow+1","accounts_in_message":[],"_revision_number":16},{"id":"b5d155ee0aea317e7c89e2d4694cf53aba0e3dfb","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-29 03:51:01.000000000","message":"Patch Set 16: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":16},{"id":"ba6f18db6ce6bf1fb277a3f1703046accb3aad4b","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-29 04:20:37.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":16},{"id":"6b72f49e72d4a0f3b7f59437f5f89d4ac23c7e6f","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-29 04:20:37.000000000","message":"Patch Set 16: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/b93cd8ed69954a5585fbf34bdc22419c : SUCCESS in 3m 00s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/4725509cab4d4ee8943e2b1ba18f51b9 : SUCCESS in 3m 17s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/acc2471dcd4f419085a57fc13b49b005 : SUCCESS in 3m 23s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/f0c53176fecb4403a72a56b0e251d480 : SUCCESS in 3m 40s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2ec87794f15e4208a1ee25e17e1ee542 : SUCCESS in 3m 43s\n- cyborg-tempest https://zuul.opendev.org/t/openstack/build/3188f420962747f1b8677fb5ae769ef6 : SUCCESS in 27m 56s (non-voting)","accounts_in_message":[],"_revision_number":16},{"id":"cf3f8bc4d4dba6e538ecdcafe54a54842b4ad056","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-09-29 04:21:36.000000000","message":"Patch Set 16:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/aa734c53364d49edbf7e3aef9a260506 : SUCCESS in 43s","accounts_in_message":[],"_revision_number":16}],"current_revision_number":16,"current_revision":"71df0a289d78019dc96168da719c940c9d62e735","revisions":{"8aea35a82db4cc8192c352fa0eae13966a799b87":{"kind":"REWORK","_number":1,"created":"2019-09-17 07:43:10.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/1"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"subject":"(WIP)Mofity the style of nova_client create","message":"(WIP)Mofity the style of nova_client create\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/8aea35a82db4cc8192c352fa0eae13966a799b87"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/8aea35a82db4cc8192c352fa0eae13966a799b87"}]},"branch":"refs/heads/master"},"6d4f250babda2f63c698298d1810174a2ec0cf56":{"kind":"REWORK","_number":2,"created":"2019-09-17 09:28:22.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/2"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 09:27:46.000000000","tz":480},"subject":"Mofity the create style for nova_client and placement_client","message":"Mofity the create style for nova_client and placement_client\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\n2. UnitTest will be updated.\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/6d4f250babda2f63c698298d1810174a2ec0cf56"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/6d4f250babda2f63c698298d1810174a2ec0cf56"}]},"branch":"refs/heads/master"},"18f5a79c09f58fb89f3c4e476df9a7d07df1404b":{"kind":"REWORK","_number":3,"created":"2019-09-18 04:01:24.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/3"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-18 03:59:49.000000000","tz":480},"subject":"(WIP)Fix the hardcoding of user role using sdk_adapter approach","message":"(WIP)Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. UnitTest will be updated.\n\n3. Move placement_client define from default to seperate placement file.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/18f5a79c09f58fb89f3c4e476df9a7d07df1404b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/18f5a79c09f58fb89f3c4e476df9a7d07df1404b"}]},"branch":"refs/heads/master"},"276c2a4bf112e5fd57965a7418ff59b7f194e791":{"kind":"REWORK","_number":4,"created":"2019-09-18 06:03:38.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/4"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-18 06:03:13.000000000","tz":480},"subject":"(WIP)Fix the hardcoding of user role using sdk_adapter approach","message":"(WIP)Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. UnitTest will be updated.\n\n3. Move placement_client define from default to seperate placement file.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/276c2a4bf112e5fd57965a7418ff59b7f194e791"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/276c2a4bf112e5fd57965a7418ff59b7f194e791"}]},"branch":"refs/heads/master"},"b580f16d75f748e8b453191b3f646bb7da8314ef":{"kind":"REWORK","_number":5,"created":"2019-09-18 09:09:55.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/5"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-18 09:09:41.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in laterst\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/b580f16d75f748e8b453191b3f646bb7da8314ef"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/b580f16d75f748e8b453191b3f646bb7da8314ef"}]},"branch":"refs/heads/master"},"74876cd4127bfecc6fa66791f6e41768d558f5a2":{"kind":"NO_CODE_CHANGE","_number":6,"created":"2019-09-18 09:11:24.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/6"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-18 09:10:37.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/74876cd4127bfecc6fa66791f6e41768d558f5a2"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/74876cd4127bfecc6fa66791f6e41768d558f5a2"}]},"branch":"refs/heads/master"},"89f931eb6f15763098c618d94f7da65b42f4d476":{"kind":"REWORK","_number":7,"created":"2019-09-19 08:52:50.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/7"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-19 06:52:17.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/89f931eb6f15763098c618d94f7da65b42f4d476"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/89f931eb6f15763098c618d94f7da65b42f4d476"}]},"branch":"refs/heads/master"},"8dc13647e2ad43b1ed735c9b5bfeed5333f20ea3":{"kind":"REWORK","_number":8,"created":"2019-09-19 12:08:51.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/8"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-19 11:29:45.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/8dc13647e2ad43b1ed735c9b5bfeed5333f20ea3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/8dc13647e2ad43b1ed735c9b5bfeed5333f20ea3"}]},"branch":"refs/heads/master"},"6f48e4643fe5cadb6fa488800ad661c0bfa01aee":{"kind":"REWORK","_number":9,"created":"2019-09-20 09:32:31.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/9"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-20 09:32:01.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/6f48e4643fe5cadb6fa488800ad661c0bfa01aee"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/6f48e4643fe5cadb6fa488800ad661c0bfa01aee"}]},"branch":"refs/heads/master"},"ee80ff410278a4de8084700f0afe7566695d3171":{"kind":"REWORK","_number":10,"created":"2019-09-24 01:56:02.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/10"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenker","email":"chen.ke14@zte.com.cn","date":"2019-09-24 01:55:54.000000000","tz":0},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/ee80ff410278a4de8084700f0afe7566695d3171"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/ee80ff410278a4de8084700f0afe7566695d3171"}]},"branch":"refs/heads/master"},"60a231ae1ef819a971d99713380730695a2b3d11":{"kind":"NO_CODE_CHANGE","_number":11,"created":"2019-09-25 05:58:00.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/11"}}},"commit":{"parents":[{"commit":"68dc701a4f7e6fda130348c0ff0f416dc5bcc62d","subject":"Merge \"Make ResourceNotFound as NotFound exception\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/68dc701a4f7e6fda130348c0ff0f416dc5bcc62d"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenker","email":"chen.ke14@zte.com.cn","date":"2019-09-25 05:57:57.000000000","tz":0},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nDepends-On: https://review.opendev.org/#/c/673735/13\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/60a231ae1ef819a971d99713380730695a2b3d11"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/60a231ae1ef819a971d99713380730695a2b3d11"}]},"branch":"refs/heads/master"},"e0178e6602e1598ead721f73347bf776ce48bbee":{"kind":"TRIVIAL_REBASE","_number":12,"created":"2019-09-25 06:00:13.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/12","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/12","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/12 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/12 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/12 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/12"}}},"commit":{"parents":[{"commit":"a7f1df159607c17403eea7b05f2fcdcb69406f94","subject":"Merge \"[train][goal] Run \u0027cyborg-tempest-ipv6-only\u0027 job in gate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/a7f1df159607c17403eea7b05f2fcdcb69406f94"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenker","email":"chen.ke14@zte.com.cn","date":"2019-09-25 06:00:13.000000000","tz":0},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nDepends-On: https://review.opendev.org/#/c/673735/13\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/e0178e6602e1598ead721f73347bf776ce48bbee"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/e0178e6602e1598ead721f73347bf776ce48bbee"}]},"branch":"refs/heads/master"},"4c5901030b189af949231beb14075a1f4df47683":{"kind":"NO_CHANGE","_number":13,"created":"2019-09-25 06:19:01.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/13","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/13","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/13 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/13 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/13 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/13"}}},"commit":{"parents":[{"commit":"a7f1df159607c17403eea7b05f2fcdcb69406f94","subject":"Merge \"[train][goal] Run \u0027cyborg-tempest-ipv6-only\u0027 job in gate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/a7f1df159607c17403eea7b05f2fcdcb69406f94"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-25 06:18:30.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nDepends-On: https://review.opendev.org/#/c/673735/13\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/4c5901030b189af949231beb14075a1f4df47683"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/4c5901030b189af949231beb14075a1f4df47683"}]},"branch":"refs/heads/master"},"05e0cf2bb74160c4a5ae114c6e9d9bc48889b984":{"kind":"NO_CODE_CHANGE","_number":14,"created":"2019-09-25 07:31:41.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/14","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/14","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/14 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/14 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/14 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/14"}}},"commit":{"parents":[{"commit":"a7f1df159607c17403eea7b05f2fcdcb69406f94","subject":"Merge \"[train][goal] Run \u0027cyborg-tempest-ipv6-only\u0027 job in gate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/a7f1df159607c17403eea7b05f2fcdcb69406f94"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-25 07:31:00.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/05e0cf2bb74160c4a5ae114c6e9d9bc48889b984"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/05e0cf2bb74160c4a5ae114c6e9d9bc48889b984"}]},"branch":"refs/heads/master"},"179a4af7381ef3d1ad4f9de51454cab9db22a822":{"kind":"REWORK","_number":15,"created":"2019-09-26 02:50:42.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/15","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/15","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/15 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/15 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/15 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/15"}}},"commit":{"parents":[{"commit":"a7f1df159607c17403eea7b05f2fcdcb69406f94","subject":"Merge \"[train][goal] Run \u0027cyborg-tempest-ipv6-only\u0027 job in gate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/a7f1df159607c17403eea7b05f2fcdcb69406f94"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-26 02:50:11.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/179a4af7381ef3d1ad4f9de51454cab9db22a822"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/179a4af7381ef3d1ad4f9de51454cab9db22a822"}]},"branch":"refs/heads/master"},"71df0a289d78019dc96168da719c940c9d62e735":{"kind":"REWORK","_number":16,"created":"2019-09-26 03:28:31.000000000","uploader":{"_account_id":28748,"name":"chenker","email":"chen.ke14@zte.com.cn","username":"chenke"},"ref":"refs/changes/65/682565/16","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/cyborg","ref":"refs/changes/65/682565/16","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/16 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/16 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/cyborg refs/changes/65/682565/16 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/cyborg refs/changes/65/682565/16"}}},"commit":{"parents":[{"commit":"a7f1df159607c17403eea7b05f2fcdcb69406f94","subject":"Merge \"[train][goal] Run \u0027cyborg-tempest-ipv6-only\u0027 job in gate\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/a7f1df159607c17403eea7b05f2fcdcb69406f94"}]}],"author":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-17 07:42:12.000000000","tz":480},"committer":{"name":"chenke","email":"chen.ke14@zte.com.cn","date":"2019-09-26 03:28:04.000000000","tz":480},"subject":"Fix the hardcoding of user role using sdk_adapter approach","message":"Fix the hardcoding of user role using sdk_adapter approach\n\n1. Refer to nova\u0027s latest code, use get_sdk to create the client[1].\n\n2. Move placement_client define from default to seperate placement file.\n\nNote:\nThe new method for get nova and placement client has been tested in latest\ncyborg devstack environment. And the test results are good.\n\nBecause now cyborg does not have unit tests for the common and conf directory,\nunit tests related to this patch will be added in the future, but not in this patch.\n\nReference:\n[1]. https://github.com/openstack/nova/blob/3f019f2db373d82cbc93f0a30de0059d3d07432b/nova/utils.py#L1013\n\nChange-Id: I2a335d47ee3cc24f78b4d49942f1cccd586a0dcc\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/71df0a289d78019dc96168da719c940c9d62e735"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/cyborg/commit/71df0a289d78019dc96168da719c940c9d62e735"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}