)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"4fa3b47a4511872bc7a4985932762a4df1d3bd8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6c4b73f2_e7fefcce","updated":"2025-05-13 10:39:38.000000000","message":"recheck","commit_id":"5258e4dc126d39a0c601944c07b4657c4a725857"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"a4a0ce93a6909e67c91b2f2c55556d5b4b424bc4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"150fbad8_154a7a10","updated":"2025-06-06 14:36:27.000000000","message":"Something is definitely broken. \"baremetal: command not found\" errors","commit_id":"825772e0945f3867976292e91c903891902a0752"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"3eabec324fd3dc092767102fe02b85c3e49f264c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"bdc4b6f1_0d7229f3","updated":"2025-06-06 17:29:08.000000000","message":"The command not found problem is probably a side effect of keystone failing before ironic is installed.\n\nI have a question (inline) about consistency.","commit_id":"825772e0945f3867976292e91c903891902a0752"}],"playbooks/roles/bifrost-pip-install/tasks/main.yml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"5cd051b95e7955ac75310ec6d9a85730059d52e5","unresolved":true,"context_lines":[{"line_number":33,"context_line":"        - not source_install | bool"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    - name: \"Detect Python version in virtualenv\""},{"line_number":36,"context_line":"      command: \"{{ ansible_python_interpreter }} -c \u0027import sys; print(\\\".\\\".join(map(str, sys.version_info[:3])))\u0027\""},{"line_number":37,"context_line":"      register: ansible_python_interpreter_version"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    - name: \"Install {{ package }} package from pip using virtualenv\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"dba1aa88_cf29fd30","line":36,"updated":"2025-05-15 12:24:01.000000000","message":"nit: `{{ ansible_python_interpreter }} -V | awk \u0027{print $2;}\u0027`","commit_id":"825772e0945f3867976292e91c903891902a0752"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"605d2e6fa3f4f72edc158da2c6683396ebb52e74","unresolved":true,"context_lines":[{"line_number":33,"context_line":"        - not source_install | bool"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    - name: \"Detect Python version in virtualenv\""},{"line_number":36,"context_line":"      command: \"{{ ansible_python_interpreter }} -c \u0027import sys; print(\\\".\\\".join(map(str, sys.version_info[:3])))\u0027\""},{"line_number":37,"context_line":"      register: ansible_python_interpreter_version"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    - name: \"Install {{ package }} package from pip using virtualenv\""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"cbec6d10_0756e681","line":36,"in_reply_to":"dba1aa88_cf29fd30","updated":"2025-05-19 12:05:42.000000000","message":"yeah, I didn\u0027t want to rely on awk, I just wanted to use python here","commit_id":"825772e0945f3867976292e91c903891902a0752"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"3eabec324fd3dc092767102fe02b85c3e49f264c","unresolved":true,"context_lines":[{"line_number":60,"context_line":"      retries: 5"},{"line_number":61,"context_line":"      delay: 10"},{"line_number":62,"context_line":"      when: source_install | bool"},{"line_number":63,"context_line":"      environment: \"{{ (bifrost_venv_env | combine({\u0027SETUPTOOLS_USE_DISTUTILS\u0027: \u0027stdlib\u0027})) if ansible_facts.python_version is version(\u00273.12\u0027, \u0027\u003c\u0027) else bifrost_venv_env }}\""},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    # NOTE(dtantsur): do not use constraints here, it does not work when the"},{"line_number":66,"context_line":"    # package itself is constrained."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"6e7de173_0b6f3220","line":63,"updated":"2025-06-06 17:29:08.000000000","message":"Don\u0027t you want to change the lines 63 and 73 too?","commit_id":"825772e0945f3867976292e91c903891902a0752"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"1454a38e96f769dda2cd53a7fc5f50e99d86d597","unresolved":true,"context_lines":[{"line_number":60,"context_line":"      retries: 5"},{"line_number":61,"context_line":"      delay: 10"},{"line_number":62,"context_line":"      when: source_install | bool"},{"line_number":63,"context_line":"      environment: \"{{ (bifrost_venv_env | combine({\u0027SETUPTOOLS_USE_DISTUTILS\u0027: \u0027stdlib\u0027})) if ansible_facts.python_version is version(\u00273.12\u0027, \u0027\u003c\u0027) else bifrost_venv_env }}\""},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    # NOTE(dtantsur): do not use constraints here, it does not work when the"},{"line_number":66,"context_line":"    # package itself is constrained."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"96b26199_3408b5d4","line":63,"in_reply_to":"6e7de173_0b6f3220","updated":"2025-06-10 09:14:04.000000000","message":"mmm I got confused at some point and didn\u0027t change it back, but I think I noticed something odd here and in the next task\nI\u0027ll update both with the new check and see","commit_id":"825772e0945f3867976292e91c903891902a0752"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"215a3eed48a14273fc15370655e85fd519125886","unresolved":false,"context_lines":[{"line_number":60,"context_line":"      retries: 5"},{"line_number":61,"context_line":"      delay: 10"},{"line_number":62,"context_line":"      when: source_install | bool"},{"line_number":63,"context_line":"      environment: \"{{ (bifrost_venv_env | combine({\u0027SETUPTOOLS_USE_DISTUTILS\u0027: \u0027stdlib\u0027})) if ansible_facts.python_version is version(\u00273.12\u0027, \u0027\u003c\u0027) else bifrost_venv_env }}\""},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    # NOTE(dtantsur): do not use constraints here, it does not work when the"},{"line_number":66,"context_line":"    # package itself is constrained."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"f92be176_0f5e01fc","line":63,"in_reply_to":"96b26199_3408b5d4","updated":"2025-06-11 14:11:46.000000000","message":"Done","commit_id":"825772e0945f3867976292e91c903891902a0752"}]}