)]}'
{"tests/run.yml":[{"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":"5c333e3902579f185582a754096f0300a2d2b34b","unresolved":false,"context_lines":[{"line_number":326,"context_line":"            remote_src: true"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"        - name: generate new passwords"},{"line_number":329,"context_line":"          command: kolla-genpwd"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"        - name: merge old and new passwords"},{"line_number":332,"context_line":"          command: \u003e-"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3fa7e38b_0b34dede","line":329,"updated":"2019-11-26 16:22:34.000000000","message":"good catch","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"},{"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":"1b77e1746849a601106b97c61becf5f93ccfb61c","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        openstack_core_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":23,"context_line":"        openstack_core_tested: \"{{ scenario in [\u0027core\u0027, \u0027ceph\u0027, \u0027cinder-lvm\u0027, \u0027cells\u0027] }}\""},{"line_number":24,"context_line":"        dashboard_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":25,"context_line":"        # TODO(mgoddard): Remove when previous_release is ussuri."},{"line_number":26,"context_line":"        playbook_python_version: \"{{ \u00272\u0027 if is_upgrade and previous_release \u003d\u003d \u0027train\u0027 else \u00273\u0027 }}\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - name: Prepare disks for Ceph or LVM"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_53e8a779","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":65},"updated":"2019-12-09 12:35:13.000000000","message":"not only, we will have to also upgrade train-\u003eussuri on py3","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"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":"ee02c22ad6e70d4e1e56f922135563d98de8ed39","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        openstack_core_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":23,"context_line":"        openstack_core_tested: \"{{ scenario in [\u0027core\u0027, \u0027ceph\u0027, \u0027cinder-lvm\u0027, \u0027cells\u0027] }}\""},{"line_number":24,"context_line":"        dashboard_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":25,"context_line":"        # TODO(mgoddard): Remove when previous_release is ussuri."},{"line_number":26,"context_line":"        playbook_python_version: \"{{ \u00272\u0027 if is_upgrade and previous_release \u003d\u003d \u0027train\u0027 else \u00273\u0027 }}\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - name: Prepare disks for Ceph or LVM"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_53018038","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":65},"in_reply_to":"3fa7e38b_134228ad","updated":"2019-12-09 18:05:36.000000000","message":"Nah, that would be wasted time. Just remember it\u0027s probably going to need to be changed sooner than V.","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"902cf1fc4a95e358a24e7d3afefe9892abdcced7","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        openstack_core_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":23,"context_line":"        openstack_core_tested: \"{{ scenario in [\u0027core\u0027, \u0027ceph\u0027, \u0027cinder-lvm\u0027, \u0027cells\u0027] }}\""},{"line_number":24,"context_line":"        dashboard_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":25,"context_line":"        # TODO(mgoddard): Remove when previous_release is ussuri."},{"line_number":26,"context_line":"        playbook_python_version: \"{{ \u00272\u0027 if is_upgrade and previous_release \u003d\u003d \u0027train\u0027 else \u00273\u0027 }}\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - name: Prepare disks for Ceph or LVM"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_cee950ab","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":65},"in_reply_to":"3fa7e38b_53e8a779","updated":"2019-12-09 13:33:20.000000000","message":"Not necessarily for the local python interpreter. That could stay on py2.","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"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":"39ddf32b82cd0df4690f4526d4ba6f6f87577f2e","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        openstack_core_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":23,"context_line":"        openstack_core_tested: \"{{ scenario in [\u0027core\u0027, \u0027ceph\u0027, \u0027cinder-lvm\u0027, \u0027cells\u0027] }}\""},{"line_number":24,"context_line":"        dashboard_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":25,"context_line":"        # TODO(mgoddard): Remove when previous_release is ussuri."},{"line_number":26,"context_line":"        playbook_python_version: \"{{ \u00272\u0027 if is_upgrade and previous_release \u003d\u003d \u0027train\u0027 else \u00273\u0027 }}\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - name: Prepare disks for Ceph or LVM"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_9377f88e","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":65},"in_reply_to":"3fa7e38b_89d5921d","updated":"2019-12-09 17:38:21.000000000","message":"Hmm, I always forget we are throwing out C7, yet not U18. You might be right but it will need fixing (making realistic) anyway for C8 upgrades.","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"2f5ccc647ae24e1f45d0a41ae12f6cbdb6751660","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        openstack_core_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":23,"context_line":"        openstack_core_tested: \"{{ scenario in [\u0027core\u0027, \u0027ceph\u0027, \u0027cinder-lvm\u0027, \u0027cells\u0027] }}\""},{"line_number":24,"context_line":"        dashboard_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":25,"context_line":"        # TODO(mgoddard): Remove when previous_release is ussuri."},{"line_number":26,"context_line":"        playbook_python_version: \"{{ \u00272\u0027 if is_upgrade and previous_release \u003d\u003d \u0027train\u0027 else \u00273\u0027 }}\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - name: Prepare disks for Ceph or LVM"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_134228ad","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":65},"in_reply_to":"3fa7e38b_9377f88e","updated":"2019-12-09 17:52:29.000000000","message":"Is there something you\u0027d like me to change?","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"9fe92423c4a2bbfe608d874c8f0a1aa117eed7b3","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        openstack_core_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":23,"context_line":"        openstack_core_tested: \"{{ scenario in [\u0027core\u0027, \u0027ceph\u0027, \u0027cinder-lvm\u0027, \u0027cells\u0027] }}\""},{"line_number":24,"context_line":"        dashboard_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":25,"context_line":"        # TODO(mgoddard): Remove when previous_release is ussuri."},{"line_number":26,"context_line":"        playbook_python_version: \"{{ \u00272\u0027 if is_upgrade and previous_release \u003d\u003d \u0027train\u0027 else \u00273\u0027 }}\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - name: Prepare disks for Ceph or LVM"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_89d5921d","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":65},"in_reply_to":"3fa7e38b_aea4144d","updated":"2019-12-09 14:19:31.000000000","message":"You mean that because we\u0027ll test upgrades on CentOS 8, we\u0027ll need to switch to py3 before V? I see your point. Although potentially we could test py2 train -\u003e py3 ussuri upgrades using ubuntu.","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"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":"608667b48752fce334ac5e630b18b4464837eb5a","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        openstack_core_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":23,"context_line":"        openstack_core_tested: \"{{ scenario in [\u0027core\u0027, \u0027ceph\u0027, \u0027cinder-lvm\u0027, \u0027cells\u0027] }}\""},{"line_number":24,"context_line":"        dashboard_enabled: \"{{ openstack_core_enabled }}\""},{"line_number":25,"context_line":"        # TODO(mgoddard): Remove when previous_release is ussuri."},{"line_number":26,"context_line":"        playbook_python_version: \"{{ \u00272\u0027 if is_upgrade and previous_release \u003d\u003d \u0027train\u0027 else \u00273\u0027 }}\""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    - name: Prepare disks for Ceph or LVM"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_aea4144d","line":25,"range":{"start_line":25,"start_character":32,"end_line":25,"end_character":65},"in_reply_to":"3fa7e38b_cee950ab","updated":"2019-12-09 13:46:05.000000000","message":"This is CI - you want C8 jobs with C7 controller? :-)\nMy comment is mostly about the comment in code itself.","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"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":"1b77e1746849a601106b97c61becf5f93ccfb61c","unresolved":false,"context_lines":[{"line_number":154,"context_line":"          - \"{{ kolla_ansible_src_dir }}\""},{"line_number":155,"context_line":"          - \"ansible{{ ansible_version_constraint }}\""},{"line_number":156,"context_line":"          - \"ara\u003c1.0.0\""},{"line_number":157,"context_line":"        # TODO(mgoddard): Always use pip3 when previous_release is ussuri."},{"line_number":158,"context_line":"        executable: \"pip{{ playbook_python_version }}\""},{"line_number":159,"context_line":"      become: true"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_13feaf37","line":157,"updated":"2019-12-09 12:35:13.000000000","message":"ditto","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"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":"1b77e1746849a601106b97c61becf5f93ccfb61c","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        executable: \"pip{{ playbook_python_version }}\""},{"line_number":159,"context_line":"      become: true"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"    # TODO(mgoddard): Always use python3 when previous_release is ussuri."},{"line_number":162,"context_line":"    - name: get ARA callback plugin path"},{"line_number":163,"context_line":"      command: \"python{{ playbook_python_version }} -m ara.setup.callback_plugins\""},{"line_number":164,"context_line":"      changed_when: false"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3fa7e38b_b3f6bb5d","line":161,"updated":"2019-12-09 12:35:13.000000000","message":"ditto","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"}],"tests/templates/globals-default.j2":[{"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":"5c333e3902579f185582a754096f0300a2d2b34b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# Force the use of python2 on remote hosts. This is necessary for delegate_to:"},{"line_number":3,"context_line":"# localhost, which will otherwise use the local python interpreter (python3)."},{"line_number":4,"context_line":"# On CentOS 7, that fails due to a lack of python3 bindings for SELinux."},{"line_number":5,"context_line":"# https://bugs.centos.org/view.php?id\u003d16389"},{"line_number":6,"context_line":"ansible_python_interpreter: /usr/bin/python"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"3fa7e38b_2b52fa13","line":3,"range":{"start_line":2,"start_character":66,"end_line":3,"end_character":11},"updated":"2019-11-26 16:22:34.000000000","message":"only delegates? I thought it tries the same interpreter always","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"c17f798394450c75e93a047c4f5ecbc6907ca98f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# Force the use of python2 on remote hosts. This is necessary for delegate_to:"},{"line_number":3,"context_line":"# localhost, which will otherwise use the local python interpreter (python3)."},{"line_number":4,"context_line":"# On CentOS 7, that fails due to a lack of python3 bindings for SELinux."},{"line_number":5,"context_line":"# https://bugs.centos.org/view.php?id\u003d16389"},{"line_number":6,"context_line":"ansible_python_interpreter: /usr/bin/python"}],"source_content_type":"text/x-jinja2","patch_set":3,"id":"3fa7e38b_6e2c80b2","line":3,"range":{"start_line":2,"start_character":66,"end_line":3,"end_character":11},"in_reply_to":"3fa7e38b_2b52fa13","updated":"2019-11-26 17:49:00.000000000","message":"Nope, seems to default to /usr/bin/python for other hosts. Tested 2.6.","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"9577b02bae24d97d6e593d8d6dcb5ef667eba974","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# localhost, which will otherwise use the local python interpreter (python3)."},{"line_number":4,"context_line":"# On CentOS 7, that fails due to a lack of python3 bindings for SELinux."},{"line_number":5,"context_line":"# https://bugs.centos.org/view.php?id\u003d16389"},{"line_number":6,"context_line":"ansible_python_interpreter: /usr/bin/python"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"kolla_base_distro: \"{{ base_distro }}\""},{"line_number":9,"context_line":"kolla_install_type: \"{{ install_type }}\""}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"3fa7e38b_d4be36ff","line":6,"range":{"start_line":6,"start_character":0,"end_line":6,"end_character":43},"updated":"2019-11-29 12:47:08.000000000","message":"/usr/bin/python2 please\n\n13:46 (4s) hrw@puchatek:~$ /usr/bin/python --version\nPython 3.7.5","commit_id":"e880b079408ed54a27739d561b0292c3a17ed0a4"},{"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":"1b77e1746849a601106b97c61becf5f93ccfb61c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# Force the use of python2 on remote hosts. This is necessary for delegate_to:"},{"line_number":3,"context_line":"# localhost, which will otherwise use the local python interpreter (python3)."},{"line_number":4,"context_line":"# On CentOS 7, that fails due to a lack of python3 bindings for SELinux."},{"line_number":5,"context_line":"# https://bugs.centos.org/view.php?id\u003d16389"},{"line_number":6,"context_line":"ansible_python_interpreter: /usr/bin/python2"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"kolla_base_distro: \"{{ base_distro }}\""},{"line_number":9,"context_line":"kolla_install_type: \"{{ install_type }}\""}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"3fa7e38b_539d0707","line":6,"range":{"start_line":2,"start_character":0,"end_line":6,"end_character":44},"updated":"2019-12-09 12:35:13.000000000","message":"might not be relevant when we get to c8 (and we must)","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"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":"608667b48752fce334ac5e630b18b4464837eb5a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# Force the use of python2 on remote hosts. This is necessary for delegate_to:"},{"line_number":3,"context_line":"# localhost, which will otherwise use the local python interpreter (python3)."},{"line_number":4,"context_line":"# On CentOS 7, that fails due to a lack of python3 bindings for SELinux."},{"line_number":5,"context_line":"# https://bugs.centos.org/view.php?id\u003d16389"},{"line_number":6,"context_line":"ansible_python_interpreter: /usr/bin/python2"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"kolla_base_distro: \"{{ base_distro }}\""},{"line_number":9,"context_line":"kolla_install_type: \"{{ install_type }}\""}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"3fa7e38b_cea1503c","line":6,"range":{"start_line":2,"start_character":0,"end_line":6,"end_character":44},"in_reply_to":"3fa7e38b_0e39a83d","updated":"2019-12-09 13:46:05.000000000","message":"ack,","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"902cf1fc4a95e358a24e7d3afefe9892abdcced7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# Force the use of python2 on remote hosts. This is necessary for delegate_to:"},{"line_number":3,"context_line":"# localhost, which will otherwise use the local python interpreter (python3)."},{"line_number":4,"context_line":"# On CentOS 7, that fails due to a lack of python3 bindings for SELinux."},{"line_number":5,"context_line":"# https://bugs.centos.org/view.php?id\u003d16389"},{"line_number":6,"context_line":"ansible_python_interpreter: /usr/bin/python2"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"kolla_base_distro: \"{{ base_distro }}\""},{"line_number":9,"context_line":"kolla_install_type: \"{{ install_type }}\""}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"3fa7e38b_0e39a83d","line":6,"range":{"start_line":2,"start_character":0,"end_line":6,"end_character":44},"in_reply_to":"3fa7e38b_539d0707","updated":"2019-12-09 13:33:20.000000000","message":"Will be addressed in a later patch.","commit_id":"be59439fba03ae385aa1e7348f461b242b4a096d"}],"tools/kolla-ansible":[{"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":"5c333e3902579f185582a754096f0300a2d2b34b","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# This script can be used to interact with kolla via ansible."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"function find_base_dir {"},{"line_number":6,"context_line":"    # FIXME: should support python3."},{"line_number":7,"context_line":"    local real_path\u003d$(python -c \"import os;print(os.path.realpath(\u0027$0\u0027))\")"},{"line_number":8,"context_line":"    local dir_name\u003d\"$(dirname \"$real_path\")\""},{"line_number":9,"context_line":"    if [ -z \"$SNAP\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"3fa7e38b_0b393ece","line":6,"updated":"2019-11-26 16:22:34.000000000","message":"anonymous are bad, I have to scan git log to blame :-)","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"c17f798394450c75e93a047c4f5ecbc6907ca98f","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# This script can be used to interact with kolla via ansible."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"function find_base_dir {"},{"line_number":6,"context_line":"    # FIXME: should support python3."},{"line_number":7,"context_line":"    local real_path\u003d$(python -c \"import os;print(os.path.realpath(\u0027$0\u0027))\")"},{"line_number":8,"context_line":"    local dir_name\u003d\"$(dirname \"$real_path\")\""},{"line_number":9,"context_line":"    if [ -z \"$SNAP\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"3fa7e38b_4e3bc4e9","line":6,"in_reply_to":"3fa7e38b_0b393ece","updated":"2019-11-26 17:49:00.000000000","message":"just temporary, will insert a fix before this patch.","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"},{"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":"5c333e3902579f185582a754096f0300a2d2b34b","unresolved":false,"context_lines":[{"line_number":121,"context_line":"RAW_ARGS\u003d\"$*\""},{"line_number":122,"context_line":"ARGS\u003d$(getopt -o \"${SHORT_OPTS}\" -l \"${LONG_OPTS}\" --name \"$0\" -- \"$@\") || { usage \u003e\u00262; exit 2; }"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"# FIXME: fails check due to not using python3."},{"line_number":125,"context_line":"python -c \u0027import kolla_ansible\u0027 \u0026\u003e/dev/null || ("},{"line_number":126,"context_line":"    echo \"ERROR: kolla_ansible has to be available in the PYTHONPATH (e.g. installed)\" \u003e\u00262"},{"line_number":127,"context_line":"    exit 1"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"3fa7e38b_cb42463c","line":124,"updated":"2019-11-26 16:22:34.000000000","message":"ditto","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"}],"tools/setup_gate.sh":[{"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":"5c333e3902579f185582a754096f0300a2d2b34b","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    fi"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # TODO(SamYaple): Move to virtualenv"},{"line_number":97,"context_line":"    sudo pip3 install -U \"ansible${ANSIBLE_VERSION}\" \"ara\u003c1.0.0\""},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    sudo mkdir /etc/ansible"},{"line_number":100,"context_line":"    ara_location\u003d$(python3 -m ara.setup.callback_plugins)"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3fa7e38b_6b96b2a3","line":97,"range":{"start_line":97,"start_character":53,"end_line":97,"end_character":64},"updated":"2019-11-26 16:22:34.000000000","message":"not sure about this pin being still valid","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"c17f798394450c75e93a047c4f5ecbc6907ca98f","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    fi"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"    # TODO(SamYaple): Move to virtualenv"},{"line_number":97,"context_line":"    sudo pip3 install -U \"ansible${ANSIBLE_VERSION}\" \"ara\u003c1.0.0\""},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    sudo mkdir /etc/ansible"},{"line_number":100,"context_line":"    ara_location\u003d$(python3 -m ara.setup.callback_plugins)"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3fa7e38b_ee1f106f","line":97,"range":{"start_line":97,"start_character":53,"end_line":97,"end_character":64},"in_reply_to":"3fa7e38b_6b96b2a3","updated":"2019-11-26 17:49:00.000000000","message":"who knows...","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"},{"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":"5c333e3902579f185582a754096f0300a2d2b34b","unresolved":false,"context_lines":[{"line_number":97,"context_line":"    sudo pip3 install -U \"ansible${ANSIBLE_VERSION}\" \"ara\u003c1.0.0\""},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    sudo mkdir /etc/ansible"},{"line_number":100,"context_line":"    ara_location\u003d$(python3 -m ara.setup.callback_plugins)"},{"line_number":101,"context_line":"    sudo tee /etc/ansible/ansible.cfg\u003c\u003cEOF"},{"line_number":102,"context_line":"[defaults]"},{"line_number":103,"context_line":"callback_plugins \u003d ${ara_location}"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3fa7e38b_8b768ea3","line":100,"updated":"2019-11-26 16:22:34.000000000","message":"which reminds me we have to finally fix ara :-)","commit_id":"13dbe052dc59d297f488161f5d048eab59b8fba9"}]}