)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"68bde95e18e8847c4a6577cda5ee50e0d30e7302","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"41b703b9_4e4d8220","updated":"2022-05-13 06:58:27.000000000","message":"Just as with Nova, we should not require user passing sasl credentials into this config, just rely on the libvirt client config. We can configure it in kolla and Mark\u0027s doing that in \n\nSee how it works in Nova: https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/host.py#L340\n\nThey simply configure the client to try auth. Also, remember to add the flag about read_only to keep the previous behaviour.","commit_id":"da87c620028ec7dd01b4a580b118aa6488e30325"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"8664c8b6d2475dd43ba0c5af8aee9fcf49a5743a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9324390a_283a7ec5","in_reply_to":"41b703b9_4e4d8220","updated":"2022-05-13 06:59:17.000000000","message":"(unresolved)","commit_id":"da87c620028ec7dd01b4a580b118aa6488e30325"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"412e5bca6235d0286c4c92830aff2ff121189843","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e4060969_5fca696e","in_reply_to":"9324390a_283a7ec5","updated":"2022-05-13 07:03:08.000000000","message":"I forgot to paste the link. Here it is:\nhttps://review.opendev.org/c/openstack/kolla-ansible/+/834459","commit_id":"da87c620028ec7dd01b4a580b118aa6488e30325"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"a15ccef569ce4e740cc273699bcb1a241d288aa3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e76c7e43_dea12690","in_reply_to":"e4060969_5fca696e","updated":"2022-05-13 12:19:08.000000000","message":"Done","commit_id":"da87c620028ec7dd01b4a580b118aa6488e30325"},{"author":{"_account_id":30623,"name":"suzhengwei","email":"suzhengwei@inspur.com","username":"SuZhengwei"},"change_message_id":"67f1cabcc0595b88491afd4d52196cbbc6511801","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"0499a89d_109a72c7","updated":"2022-05-18 09:09:55.000000000","message":"In many envs, it connects libvirt without authentication. It should select the correct function based on the configuration. only when connection with verification, it use the openAuth function.\nWould you please make it compatible with the original? ","commit_id":"7a44244f2534ee0ba17ab11fed81e525b2ddd14a"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"f1cfe1d137bf2df36d6ac340a5a08c3edebce5d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"613c61d8_bc4107c2","updated":"2022-05-13 18:43:49.000000000","message":"Radosław, I had to leave, thanks for help)\n","commit_id":"7a44244f2534ee0ba17ab11fed81e525b2ddd14a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"5ab0af0db2142570e67b92deb14c0c70166892d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2cc3ccbc_2435adfc","in_reply_to":"0499a89d_109a72c7","updated":"2022-05-18 09:18:20.000000000","message":"Hi suzhengwei, this is compatible with the original (and CI confirms that). We are now using the same code Nova uses so we ensure best compatibility.","commit_id":"7a44244f2534ee0ba17ab11fed81e525b2ddd14a"},{"author":{"_account_id":30623,"name":"suzhengwei","email":"suzhengwei@inspur.com","username":"SuZhengwei"},"change_message_id":"3ccfdb26ff11f188fb7fdcfe2f5f658a6c95d4ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"d74cc157_61aad15d","in_reply_to":"0499a89d_109a72c7","updated":"2022-05-18 09:17:47.000000000","message":"sorry, I notice Radoslaw Piliszek\u0027s comment after this comment.\nIt\u0027s OK.","commit_id":"7a44244f2534ee0ba17ab11fed81e525b2ddd14a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"1a92dde9df6201ed039530559e9405f3cbb1b6df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"a62ec7b7_71273b48","in_reply_to":"2cc3ccbc_2435adfc","updated":"2022-05-18 09:19:05.000000000","message":"Ack, let\u0027s merge.","commit_id":"7a44244f2534ee0ba17ab11fed81e525b2ddd14a"}],"test-requirements.txt":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"8664c8b6d2475dd43ba0c5af8aee9fcf49a5743a","unresolved":true,"context_lines":[{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"stestr\u003e\u003d1.0.0 # Apache-2.0"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"hacking\u003e\u003d3.0.1,\u003c3.1.0 # Apache-2.0"},{"line_number":8,"context_line":""}],"source_content_type":"text/plain","patch_set":4,"id":"11b03c86_9390fe76","line":5,"updated":"2022-05-13 06:59:17.000000000","message":"this should not be a part of this patch either, of course","commit_id":"da87c620028ec7dd01b4a580b118aa6488e30325"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"a15ccef569ce4e740cc273699bcb1a241d288aa3","unresolved":false,"context_lines":[{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"stestr\u003e\u003d1.0.0 # Apache-2.0"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"hacking\u003e\u003d3.0.1,\u003c3.1.0 # Apache-2.0"},{"line_number":8,"context_line":""}],"source_content_type":"text/plain","patch_set":4,"id":"e8a6b7bf_5b37e8cd","line":5,"in_reply_to":"11b03c86_9390fe76","updated":"2022-05-13 12:19:08.000000000","message":"Done","commit_id":"da87c620028ec7dd01b4a580b118aa6488e30325"}]}