)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"15c56039a10f1f59edf80b49f1179acf72bbd1b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ce563cb7_32abd51c","updated":"2026-07-10 07:34:32.000000000","message":"I\u0027m adding `verify` job to check that it does not break tempest verifier. It should land today. Meanwhile see my code comments","commit_id":"9c8239740ae527f652abde90150c597cf105da0a"}],"rally/verification/manager.py":[{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"15c56039a10f1f59edf80b49f1179acf72bbd1b8","unresolved":true,"context_lines":[{"line_number":21,"context_line":"import re"},{"line_number":22,"context_line":"import shutil"},{"line_number":23,"context_line":"import sys"},{"line_number":24,"context_line":"import venv"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"import packaging"},{"line_number":27,"context_line":"import packaging.requirements"}],"source_content_type":"text/x-python","patch_set":2,"id":"a7012b7b_29df3c3e","line":24,"updated":"2026-07-10 07:34:32.000000000","message":"In general, I\u0027m fine with switching to venv. But here is the problem - ubuntu\u0026debian do not ship venv package with python sys package which would result in crashing rally package. If we switch to venv, I prefer 1) moving import under `install_venv` function, so all cli, env, deployment, task continue working and only verification component is affeced 2) adding some validation step that prints user-friendly message 3) update CHANGELOG.rst with description of behaviour change","commit_id":"9c8239740ae527f652abde90150c597cf105da0a"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"15c56039a10f1f59edf80b49f1179acf72bbd1b8","unresolved":true,"context_lines":[{"line_number":253,"context_line":"            builder \u003d venv.EnvBuilder(clear\u003dTrue, with_pip\u003dTrue)"},{"line_number":254,"context_line":"            builder.create(self.venv_dir)"},{"line_number":255,"context_line":"        except Exception:"},{"line_number":256,"context_line":"            LOG.error(\"Failed to initialize virtual env in %s directory.\","},{"line_number":257,"context_line":"                      self.venv_dir)"},{"line_number":258,"context_line":"            raise"},{"line_number":259,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"c6acf5a7_5bb503f8","line":256,"updated":"2026-07-10 07:34:32.000000000","message":"use fstring instead. we do not allow \"extra\" args to be passed to logging functions as they are resolved at runtime, resulting in runtime problems that we prefer to catch earlier.","commit_id":"9c8239740ae527f652abde90150c597cf105da0a"}]}