)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"60a14d456723db72db42e9a7b1d3a2186ba2892c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3a478ea8_6f97043d","updated":"2022-09-12 11:33:54.000000000","message":"About activat","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"9dd59a2a08531de0b2021b55c0135f2c2584233a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"f96f2f68_0be28389","updated":"2022-09-12 11:20:55.000000000","message":"I will not give any vote because it\u0027s probably working. But I really don\u0027t like activate/deactivate (I have a feeling that deactivate is used from time to time just \"to be sure\" :D )","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"62536f0d_e289b324","updated":"2022-09-12 11:26:46.000000000","message":"I would not be receiving these comments if kolla-ansible was a python script.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"60a14d456723db72db42e9a7b1d3a2186ba2892c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"21b190cd_3de00a27","in_reply_to":"62536f0d_e289b324","updated":"2022-09-12 11:33:54.000000000","message":"Haha, can u rework it ? Because - as you know it will be helful for me in my another review 😄  I\u0027m justj joking.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"}],"tests/run.yml":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"17beb8012a9eedd7c3d71f7a22d8ff743ee0ad25","unresolved":true,"context_lines":[{"line_number":229,"context_line":"        cmd: \"python3 -m venv {{ kolla_ansible_venv_path }}\""},{"line_number":230,"context_line":"        creates: \"{{ kolla_ansible_venv_path }}\""},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"    - name: Ensure the latest pip"},{"line_number":233,"context_line":"      pip:"},{"line_number":234,"context_line":"        name: pip"},{"line_number":235,"context_line":"        version: latest"},{"line_number":236,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"    - name: Ensure the latest setuptools"},{"line_number":239,"context_line":"      pip:"},{"line_number":240,"context_line":"        name: setuptools"},{"line_number":241,"context_line":"        version: latest"},{"line_number":242,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    - name: install kolla-ansible and dependencies"},{"line_number":245,"context_line":"      vars:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"4fe45a83_3c45bea6","line":242,"range":{"start_line":232,"start_character":1,"end_line":242,"end_character":51},"updated":"2022-09-09 08:26:38.000000000","message":"pip\u003d\u003d22.* setuptools\u003d\u003d65.* so it will not hurt us during support months","commit_id":"c64f1a8ac86a869bb32692445fb4aa9483b66226"},{"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":"7bb367f904c268f9d7a0e7cfe89e7a009fde6369","unresolved":false,"context_lines":[{"line_number":229,"context_line":"        cmd: \"python3 -m venv {{ kolla_ansible_venv_path }}\""},{"line_number":230,"context_line":"        creates: \"{{ kolla_ansible_venv_path }}\""},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"    - name: Ensure the latest pip"},{"line_number":233,"context_line":"      pip:"},{"line_number":234,"context_line":"        name: pip"},{"line_number":235,"context_line":"        version: latest"},{"line_number":236,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":237,"context_line":""},{"line_number":238,"context_line":"    - name: Ensure the latest setuptools"},{"line_number":239,"context_line":"      pip:"},{"line_number":240,"context_line":"        name: setuptools"},{"line_number":241,"context_line":"        version: latest"},{"line_number":242,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    - name: install kolla-ansible and dependencies"},{"line_number":245,"context_line":"      vars:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"1fef6409_f81c1a7c","line":242,"range":{"start_line":232,"start_character":1,"end_line":242,"end_character":51},"in_reply_to":"4fe45a83_3c45bea6","updated":"2022-09-09 16:01:48.000000000","message":"Done","commit_id":"c64f1a8ac86a869bb32692445fb4aa9483b66226"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e62e50acd3dd5405d4c4c3447ae1330cbea0418e","unresolved":true,"context_lines":[{"line_number":256,"context_line":"          - \"ara\u003c1.0.0\""},{"line_number":257,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"    - name: install Ansible collections"},{"line_number":260,"context_line":"      shell: |"},{"line_number":261,"context_line":"        source {{ kolla_ansible_venv_path }}/bin/activate"},{"line_number":262,"context_line":"        kolla-ansible install-deps"},{"line_number":263,"context_line":"      args:"},{"line_number":264,"context_line":"        executable: /bin/bash"},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"    - name: get ARA callback plugin path"},{"line_number":267,"context_line":"      command: \"{{ kolla_ansible_venv_path }}/bin/python3 -m ara.setup.callback_plugins\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"ab95629d_fca0d84e","line":264,"range":{"start_line":259,"start_character":0,"end_line":264,"end_character":29},"updated":"2022-09-12 11:19:41.000000000","message":"I know that this is for setup gate and data are removed after tests so not need to be careful about garbage. But still, wouldn\u0027t be nice to use \"creates:\" as for example in  \" Create Kolla Ansible venv\" ?.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[{"line_number":256,"context_line":"          - \"ara\u003c1.0.0\""},{"line_number":257,"context_line":"        virtualenv: \"{{ kolla_ansible_venv_path }}\""},{"line_number":258,"context_line":""},{"line_number":259,"context_line":"    - name: install Ansible collections"},{"line_number":260,"context_line":"      shell: |"},{"line_number":261,"context_line":"        source {{ kolla_ansible_venv_path }}/bin/activate"},{"line_number":262,"context_line":"        kolla-ansible install-deps"},{"line_number":263,"context_line":"      args:"},{"line_number":264,"context_line":"        executable: /bin/bash"},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"    - name: get ARA callback plugin path"},{"line_number":267,"context_line":"      command: \"{{ kolla_ansible_venv_path }}/bin/python3 -m ara.setup.callback_plugins\""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"cfc4470f_4b1e446e","line":264,"range":{"start_line":259,"start_character":0,"end_line":264,"end_character":29},"in_reply_to":"ab95629d_fca0d84e","updated":"2022-09-12 11:26:46.000000000","message":"It is not changed. Also, this is CI, it is one-off. You can follow-up if you need.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"f6391ebbaffef083c0af8cb5ab9aeeb48dfea637","unresolved":true,"context_lines":[{"line_number":318,"context_line":"    # because the latter hijacks /etc/kolla permissions (due to same directory on the"},{"line_number":319,"context_line":"    # same host being used by both)"},{"line_number":320,"context_line":"    - name: create TLS certificates for octavia"},{"line_number":321,"context_line":"      shell: |"},{"line_number":322,"context_line":"        source {{ kolla_ansible_venv_path }}/bin/activate"},{"line_number":323,"context_line":"        kolla-ansible octavia-certificates"},{"line_number":324,"context_line":"      when: scenario in [\u0027octavia\u0027]"},{"line_number":325,"context_line":"      args:"},{"line_number":326,"context_line":"        executable: /bin/bash"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    # NOTE(mgoddard): We are using the script module here and later to ensure"},{"line_number":329,"context_line":"    # we use the local copy of these scripts, rather than the one on the remote"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3857b8f6_40e0cdfb","line":326,"range":{"start_line":321,"start_character":0,"end_line":326,"end_character":29},"updated":"2022-09-12 11:22:16.000000000","message":"also, wouldn\u0027t be nice to use \"creates\" ?","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[{"line_number":318,"context_line":"    # because the latter hijacks /etc/kolla permissions (due to same directory on the"},{"line_number":319,"context_line":"    # same host being used by both)"},{"line_number":320,"context_line":"    - name: create TLS certificates for octavia"},{"line_number":321,"context_line":"      shell: |"},{"line_number":322,"context_line":"        source {{ kolla_ansible_venv_path }}/bin/activate"},{"line_number":323,"context_line":"        kolla-ansible octavia-certificates"},{"line_number":324,"context_line":"      when: scenario in [\u0027octavia\u0027]"},{"line_number":325,"context_line":"      args:"},{"line_number":326,"context_line":"        executable: /bin/bash"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    # NOTE(mgoddard): We are using the script module here and later to ensure"},{"line_number":329,"context_line":"    # we use the local copy of these scripts, rather than the one on the remote"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"a863eaf4_76c28890","line":326,"range":{"start_line":321,"start_character":0,"end_line":326,"end_character":29},"in_reply_to":"3857b8f6_40e0cdfb","updated":"2022-09-12 11:26:46.000000000","message":"It is not changed. Also, this is CI, it is one-off. You can follow-up if you need.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e62e50acd3dd5405d4c4c3447ae1330cbea0418e","unresolved":true,"context_lines":[{"line_number":678,"context_line":"          command: \"{{ kolla_ansible_venv_path }}/bin/kolla-genpwd\""},{"line_number":679,"context_line":""},{"line_number":680,"context_line":"        - name: merge old and new passwords"},{"line_number":681,"context_line":"          command: \u003e-"},{"line_number":682,"context_line":"            {{ kolla_ansible_venv_path }}/bin/kolla-mergepwd"},{"line_number":683,"context_line":"            --old /etc/kolla/passwords.yml.old"},{"line_number":684,"context_line":"            --new /etc/kolla/passwords.yml"},{"line_number":685,"context_line":"            --final /etc/kolla/passwords.yml"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"660fa398_2abd23e9","line":682,"range":{"start_line":681,"start_character":0,"end_line":682,"end_character":60},"updated":"2022-09-12 11:19:41.000000000","message":"Why everything was reworked from command to shell (where sourcing venv activate) and not here ? I think this should be same as others.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[{"line_number":678,"context_line":"          command: \"{{ kolla_ansible_venv_path }}/bin/kolla-genpwd\""},{"line_number":679,"context_line":""},{"line_number":680,"context_line":"        - name: merge old and new passwords"},{"line_number":681,"context_line":"          command: \u003e-"},{"line_number":682,"context_line":"            {{ kolla_ansible_venv_path }}/bin/kolla-mergepwd"},{"line_number":683,"context_line":"            --old /etc/kolla/passwords.yml.old"},{"line_number":684,"context_line":"            --new /etc/kolla/passwords.yml"},{"line_number":685,"context_line":"            --final /etc/kolla/passwords.yml"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"dc158cb1_533d505f","line":682,"range":{"start_line":681,"start_character":0,"end_line":682,"end_character":60},"in_reply_to":"660fa398_2abd23e9","updated":"2022-09-12 11:26:46.000000000","message":"Because in here we don\u0027t rely on shell functionalities. I would prefer that we did not have to rely on them in others but it is quirky due to relying on the bash script. :/","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"60a14d456723db72db42e9a7b1d3a2186ba2892c","unresolved":false,"context_lines":[{"line_number":678,"context_line":"          command: \"{{ kolla_ansible_venv_path }}/bin/kolla-genpwd\""},{"line_number":679,"context_line":""},{"line_number":680,"context_line":"        - name: merge old and new passwords"},{"line_number":681,"context_line":"          command: \u003e-"},{"line_number":682,"context_line":"            {{ kolla_ansible_venv_path }}/bin/kolla-mergepwd"},{"line_number":683,"context_line":"            --old /etc/kolla/passwords.yml.old"},{"line_number":684,"context_line":"            --new /etc/kolla/passwords.yml"},{"line_number":685,"context_line":"            --final /etc/kolla/passwords.yml"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"f5ccf0b7_cf3ed656","line":682,"range":{"start_line":681,"start_character":0,"end_line":682,"end_character":60},"in_reply_to":"dc158cb1_533d505f","updated":"2022-09-12 11:33:54.000000000","message":"Hmm, now understand, but it\u0027s weird on first view that some is using shell and source and something is not. But now I see that something is python and something is shell. It would be really nice to have all in python.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"}],"tests/test-magnum.sh":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e62e50acd3dd5405d4c4c3447ae1330cbea0418e","unresolved":true,"context_lines":[{"line_number":38,"context_line":"EOF"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    RAW_INVENTORY\u003d/etc/kolla/inventory"},{"line_number":41,"context_line":"    deactivate"},{"line_number":42,"context_line":"    source $KOLLA_ANSIBLE_VENV_PATH/bin/activate"},{"line_number":43,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} --tags designate -vvv reconfigure \u0026\u003e /tmp/logs/ansible/reconfigure-designate"},{"line_number":44,"context_line":"    deactivate"}],"source_content_type":"text/x-sh","patch_set":9,"id":"30da5f94_592019b5","line":41,"range":{"start_line":41,"start_character":3,"end_line":41,"end_character":14},"updated":"2022-09-12 11:19:41.000000000","message":"why there is deactivate if not activated before ?","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[{"line_number":38,"context_line":"EOF"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    RAW_INVENTORY\u003d/etc/kolla/inventory"},{"line_number":41,"context_line":"    deactivate"},{"line_number":42,"context_line":"    source $KOLLA_ANSIBLE_VENV_PATH/bin/activate"},{"line_number":43,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} --tags designate -vvv reconfigure \u0026\u003e /tmp/logs/ansible/reconfigure-designate"},{"line_number":44,"context_line":"    deactivate"}],"source_content_type":"text/x-sh","patch_set":9,"id":"c77b4aa0_77e519e1","line":41,"range":{"start_line":41,"start_character":3,"end_line":41,"end_character":14},"in_reply_to":"30da5f94_592019b5","updated":"2022-09-12 11:26:46.000000000","message":"see below","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"function test_magnum_logged {"},{"line_number":66,"context_line":"    . /etc/kolla/admin-openrc.sh"},{"line_number":67,"context_line":"    . ~/openstackclient-venv/bin/activate"},{"line_number":68,"context_line":"    test_magnum_clusters"},{"line_number":69,"context_line":"    test_designate"},{"line_number":70,"context_line":"    test_trove"}],"source_content_type":"text/x-sh","patch_set":9,"id":"a50ae257_aae4e71b","line":67,"updated":"2022-09-12 11:26:46.000000000","message":"kevko, it is activated here; it is quirky, we can\u0027t really do better","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"}],"tests/test-mariadb.sh":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e62e50acd3dd5405d4c4c3447ae1330cbea0418e","unresolved":true,"context_lines":[{"line_number":33,"context_line":"function test_mariadb_logged {"},{"line_number":34,"context_line":"    RAW_INVENTORY\u003d/etc/kolla/inventory"},{"line_number":35,"context_line":"    source $KOLLA_ANSIBLE_VENV_PATH/bin/activate"},{"line_number":36,"context_line":"    test_recovery"},{"line_number":37,"context_line":"}"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"function test_mariadb {"}],"source_content_type":"text/x-sh","patch_set":9,"id":"fbb1c477_10cd5f9e","line":36,"range":{"start_line":36,"start_character":3,"end_line":36,"end_character":17},"updated":"2022-09-12 11:19:41.000000000","message":"deactivate after test_recovery ?","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[{"line_number":33,"context_line":"function test_mariadb_logged {"},{"line_number":34,"context_line":"    RAW_INVENTORY\u003d/etc/kolla/inventory"},{"line_number":35,"context_line":"    source $KOLLA_ANSIBLE_VENV_PATH/bin/activate"},{"line_number":36,"context_line":"    test_recovery"},{"line_number":37,"context_line":"}"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"function test_mariadb {"}],"source_content_type":"text/x-sh","patch_set":9,"id":"e7df1e1c_702b4e7a","line":36,"range":{"start_line":36,"start_character":3,"end_line":36,"end_character":17},"in_reply_to":"fbb1c477_10cd5f9e","updated":"2022-09-12 11:26:46.000000000","message":"No need, the execution ends.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"}],"tests/upgrade-bifrost.sh":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e62e50acd3dd5405d4c4c3447ae1330cbea0418e","unresolved":true,"context_lines":[{"line_number":17,"context_line":"    # CI."},{"line_number":18,"context_line":"    # TODO(mgoddard): make some configuration file changes and trigger a real"},{"line_number":19,"context_line":"    # upgrade."},{"line_number":20,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv deploy-bifrost \u0026\u003e /tmp/logs/ansible/upgrade-bifrost"},{"line_number":21,"context_line":"}"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"b10b6d9f_03f18e93","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":94},"updated":"2022-09-12 11:19:41.000000000","message":"deactivate after ?","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    # CI."},{"line_number":18,"context_line":"    # TODO(mgoddard): make some configuration file changes and trigger a real"},{"line_number":19,"context_line":"    # upgrade."},{"line_number":20,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv deploy-bifrost \u0026\u003e /tmp/logs/ansible/upgrade-bifrost"},{"line_number":21,"context_line":"}"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"6476397b_1bd8c3a9","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":94},"in_reply_to":"b10b6d9f_03f18e93","updated":"2022-09-12 11:26:46.000000000","message":"No need, the execution ends.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"}],"tests/upgrade.sh":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e62e50acd3dd5405d4c4c3447ae1330cbea0418e","unresolved":true,"context_lines":[{"line_number":19,"context_line":"    # NOTE(yoctozepto): These actions remove the leftovers of the admin port."},{"line_number":20,"context_line":"    # TODO(yoctozepto): Remove after Zed."},{"line_number":21,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv deploy --tags keystone \u0026\u003e /tmp/logs/ansible/upgrade-deploy"},{"line_number":22,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv post-deploy \u0026\u003e /tmp/logs/ansible/upgrade-post-deploy"},{"line_number":23,"context_line":"}"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"7681525a_e9b27a72","line":22,"range":{"start_line":22,"start_character":3,"end_line":22,"end_character":95},"updated":"2022-09-12 11:19:41.000000000","message":"deactivate after ?","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"},{"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":"2cb4394640038fc36cca167a70de8ace5b1eb567","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    # NOTE(yoctozepto): These actions remove the leftovers of the admin port."},{"line_number":20,"context_line":"    # TODO(yoctozepto): Remove after Zed."},{"line_number":21,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv deploy --tags keystone \u0026\u003e /tmp/logs/ansible/upgrade-deploy"},{"line_number":22,"context_line":"    kolla-ansible -i ${RAW_INVENTORY} -vvv post-deploy \u0026\u003e /tmp/logs/ansible/upgrade-post-deploy"},{"line_number":23,"context_line":"}"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"73abf105_4c3421b8","line":22,"range":{"start_line":22,"start_character":3,"end_line":22,"end_character":95},"in_reply_to":"7681525a_e9b27a72","updated":"2022-09-12 11:26:46.000000000","message":"No need, the execution ends.","commit_id":"2c7a8497fa681ad71032883fe1ec7c9645203826"}]}