)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"b95ed6c99fee9866f8a743641d12c5b49c435dd6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"31ddb609_57eba0fc","updated":"2022-12-20 16:52:01.000000000","message":"Thanks for providing the reasoning. python 3.10 is passing which is supported runtime so LGTM.\nit\u0027s good to see that python 3.11 (although non-voting) is also passing.","commit_id":"0c0879a6810273884556bbab6791e223e3768551"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"20597fbc67d837c0d56559eb2bdc9cbd0ada77e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f2aaef86_49dcea9c","updated":"2022-12-20 16:42:20.000000000","message":"one comment inline else LGTM","commit_id":"0c0879a6810273884556bbab6791e223e3768551"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"0c2cad7318e8bc382737f2ce1f3239fb869ccd6a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e7f3446c_b5e3d986","updated":"2022-12-16 15:16:24.000000000","message":"yay \\o/","commit_id":"0c0879a6810273884556bbab6791e223e3768551"}],"tox.ini":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"20597fbc67d837c0d56559eb2bdc9cbd0ada77e6","unresolved":true,"context_lines":[{"line_number":52,"context_line":"  find"},{"line_number":53,"context_line":"passenv \u003d *_proxy *_PROXY CINDERLIB_CONSTRAINTS_FILE"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"[testenv:py{3,38,39,310,311}]"},{"line_number":56,"context_line":"install_command \u003d {toxinidir}/tools/special_install.sh {env:PIP_OPTIONS:} {opts} {packages}"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"[testenv:functional]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"9868f029_629f9944","line":55,"range":{"start_line":55,"start_character":17,"end_line":55,"end_character":19},"updated":"2022-12-20 16:42:20.000000000","message":"should we remove 39 from here?","commit_id":"0c0879a6810273884556bbab6791e223e3768551"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"b95ed6c99fee9866f8a743641d12c5b49c435dd6","unresolved":false,"context_lines":[{"line_number":52,"context_line":"  find"},{"line_number":53,"context_line":"passenv \u003d *_proxy *_PROXY CINDERLIB_CONSTRAINTS_FILE"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"[testenv:py{3,38,39,310,311}]"},{"line_number":56,"context_line":"install_command \u003d {toxinidir}/tools/special_install.sh {env:PIP_OPTIONS:} {opts} {packages}"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"[testenv:functional]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"009850e2_62f29875","line":55,"range":{"start_line":55,"start_character":17,"end_line":55,"end_character":19},"in_reply_to":"3f1126c1_809662da","updated":"2022-12-20 16:52:01.000000000","message":"Yep, that checks out[1] and Debian 11 is also a supported runtime so i think it shouldn\u0027t be harm allowing it to be run locally.\n\n[1] https://wiki.debian.org/DebianBullseye","commit_id":"0c0879a6810273884556bbab6791e223e3768551"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3d65ff123b437d5f6058876c0dd76b88431bb5da","unresolved":true,"context_lines":[{"line_number":52,"context_line":"  find"},{"line_number":53,"context_line":"passenv \u003d *_proxy *_PROXY CINDERLIB_CONSTRAINTS_FILE"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"[testenv:py{3,38,39,310,311}]"},{"line_number":56,"context_line":"install_command \u003d {toxinidir}/tools/special_install.sh {env:PIP_OPTIONS:} {opts} {packages}"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"[testenv:functional]"}],"source_content_type":"text/x-properties","patch_set":3,"id":"3f1126c1_809662da","line":55,"range":{"start_line":55,"start_character":17,"end_line":55,"end_character":19},"in_reply_to":"9868f029_629f9944","updated":"2022-12-20 16:46:34.000000000","message":"The Tested Runtimes doc [0] says we don\u0027t have to run py39 tests in the gate, but adds that \"Debian 11 has Python 3.9 as default which we are not suggesting to run unit tests. We assume that anything that works on Python 3.8 and 3.10 will also work on 3.9.\".  So I think we want to be able to run tests in py39 locally.\n\n[0] https://governance.openstack.org/tc/reference/runtimes/2023.1.html","commit_id":"0c0879a6810273884556bbab6791e223e3768551"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"20597fbc67d837c0d56559eb2bdc9cbd0ada77e6","unresolved":true,"context_lines":[{"line_number":84,"context_line":"  {[testenv]allowlist_externals}"},{"line_number":85,"context_line":"  stestr"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"[testenv:functional-py{38,39,310,311}]"},{"line_number":88,"context_line":"install_command \u003d {[testenv:functional]install_command}"},{"line_number":89,"context_line":"usedevelop\u003dTrue"},{"line_number":90,"context_line":"passenv \u003d"}],"source_content_type":"text/x-properties","patch_set":3,"id":"244ee5e3_cda1a50f","line":87,"range":{"start_line":87,"start_character":26,"end_line":87,"end_character":28},"updated":"2022-12-20 16:42:20.000000000","message":"and from here as well?","commit_id":"0c0879a6810273884556bbab6791e223e3768551"}]}