)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"690305cfb13dbadd59e6f65fb21a7ed98b5fd2b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"76557202_36f1f01b","updated":"2025-05-06 06:04:02.000000000","message":"trivial, I assume the bug is only in master","commit_id":"e33902c30cb7e07d48621d3c574594b2084f1bdc"}],"docker/bifrost/bifrost-base/Dockerfile.j2":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"0fddf9d9aa717a0296ab1af3e17647773cef3563","unresolved":true,"context_lines":[{"line_number":48,"context_line":"    \u0026\u0026 python3 -m venv --system-site-packages {{ venv_path }} \\"},{"line_number":49,"context_line":"    \u0026\u0026 KOLLA_DISTRO_PYTHON_VERSION\u003d$(/usr/bin/python3 -c \"import sys; print(\u0027{}.{}\u0027.format(sys.version_info.major, sys.version_info.minor))\") \\"},{"line_number":50,"context_line":"    \u0026\u0026 cd {{ venv_path }}/lib \\"},{"line_number":51,"context_line":"    \u0026\u0026 rm -f python3 \\"},{"line_number":52,"context_line":"    \u0026\u0026 ln -s python${KOLLA_DISTRO_PYTHON_VERSION} python3"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"RUN ln -s bifrost-base-source/* bifrost \\"},{"line_number":55,"context_line":"    \u0026\u0026 {{ macros.install_pip([\u0027pip\u0027, \u0027wheel\u0027, \u0027setuptools\u0027]) }} \\"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"357b79b3_0acaca91","line":52,"range":{"start_line":51,"start_character":0,"end_line":52,"end_character":57},"updated":"2025-05-03 21:14:32.000000000","message":"replace with single\n`ln -sf python${KOLLA_DISTRO_PYTHON_VERSION} python3`\n?","commit_id":"e33902c30cb7e07d48621d3c574594b2084f1bdc"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"6f6bab6a9926893bf92f0a18af62c5e61ff784ae","unresolved":true,"context_lines":[{"line_number":48,"context_line":"    \u0026\u0026 python3 -m venv --system-site-packages {{ venv_path }} \\"},{"line_number":49,"context_line":"    \u0026\u0026 KOLLA_DISTRO_PYTHON_VERSION\u003d$(/usr/bin/python3 -c \"import sys; print(\u0027{}.{}\u0027.format(sys.version_info.major, sys.version_info.minor))\") \\"},{"line_number":50,"context_line":"    \u0026\u0026 cd {{ venv_path }}/lib \\"},{"line_number":51,"context_line":"    \u0026\u0026 rm -f python3 \\"},{"line_number":52,"context_line":"    \u0026\u0026 ln -s python${KOLLA_DISTRO_PYTHON_VERSION} python3"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"RUN ln -s bifrost-base-source/* bifrost \\"},{"line_number":55,"context_line":"    \u0026\u0026 {{ macros.install_pip([\u0027pip\u0027, \u0027wheel\u0027, \u0027setuptools\u0027]) }} \\"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"70624033_900f73bb","line":52,"range":{"start_line":51,"start_character":0,"end_line":52,"end_character":57},"in_reply_to":"357b79b3_0acaca91","updated":"2025-05-05 07:35:32.000000000","message":"I thought about the -f option too, but it didn\u0027t seem to work…","commit_id":"e33902c30cb7e07d48621d3c574594b2084f1bdc"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"37a883b94904b6cf2758bb8b0b147dc14b5e4803","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    \u0026\u0026 python3 -m venv --system-site-packages {{ venv_path }} \\"},{"line_number":49,"context_line":"    \u0026\u0026 KOLLA_DISTRO_PYTHON_VERSION\u003d$(/usr/bin/python3 -c \"import sys; print(\u0027{}.{}\u0027.format(sys.version_info.major, sys.version_info.minor))\") \\"},{"line_number":50,"context_line":"    \u0026\u0026 cd {{ venv_path }}/lib \\"},{"line_number":51,"context_line":"    \u0026\u0026 rm -f python3 \\"},{"line_number":52,"context_line":"    \u0026\u0026 ln -s python${KOLLA_DISTRO_PYTHON_VERSION} python3"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"RUN ln -s bifrost-base-source/* bifrost \\"},{"line_number":55,"context_line":"    \u0026\u0026 {{ macros.install_pip([\u0027pip\u0027, \u0027wheel\u0027, \u0027setuptools\u0027]) }} \\"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"a6291746_a12359d3","line":52,"range":{"start_line":51,"start_character":0,"end_line":52,"end_character":57},"in_reply_to":"70624033_900f73bb","updated":"2025-05-05 09:54:22.000000000","message":"It seems strange that it should overwrite a file with a new symlink. anyway LGTM as is.","commit_id":"e33902c30cb7e07d48621d3c574594b2084f1bdc"}]}
