)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"201f01b62a55aa05ac6d506cef820e70850753ad","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[Redhat8] Add PythonInterpreter for standalone role"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Set pythoninterpreter to python3 for rhel8 system while"},{"line_number":10,"context_line":"creating standalone parameters."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I8f1c883c23dd5b2c93f1087434ba7af64fba0a18"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"7faddb67_c9f1df97","line":9,"range":{"start_line":9,"start_character":37,"end_line":9,"end_character":42},"updated":"2019-07-23 06:31:14.000000000","message":"rhel or centos 8","commit_id":"941dd2d7f1a09fea5a676075dc3de43cc5c9a733"}],"roles/standalone/templates/standalone_config.yaml.j2":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"b4b1c335cdbde5bf873c525777f9b1328a6942e6","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  DockerInsecureRegistryAddress:"},{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"{% if ansible_distribution in [\u0027RedHat\u0027] %}"},{"line_number":15,"context_line":"  PythonInterpreter: /usr/bin/python3"},{"line_number":16,"context_line":"{% endif %}"},{"line_number":17,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"7faddb67_7ae0c1cd","line":14,"range":{"start_line":14,"start_character":32,"end_line":14,"end_character":38},"updated":"2019-07-22 06:55:40.000000000","message":"sanity checking is it really camel case?","commit_id":"d6bcddc8bb66d5ba9c3b4b6ceacb3398ebb0fb7d"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"19d0d75ad69170cff01c1c480a684e537bc21d91","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  DockerInsecureRegistryAddress:"},{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"{% if ansible_distribution in [\u0027RedHat\u0027] %}"},{"line_number":15,"context_line":"  PythonInterpreter: /usr/bin/python3"},{"line_number":16,"context_line":"{% endif %}"},{"line_number":17,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"7faddb67_60eece8c","line":14,"range":{"start_line":14,"start_character":32,"end_line":14,"end_character":38},"in_reply_to":"7faddb67_7ae0c1cd","updated":"2019-07-22 09:38:46.000000000","message":"Yes, for example here: https://logs.rdoproject.org/70/21270/43/check/periodic-tripleo-ci-rhel-8-standalone-master/d7c88c3/zuul-info/host-info.primary.yaml\n  ansible_distribution: RedHat","commit_id":"d6bcddc8bb66d5ba9c3b4b6ceacb3398ebb0fb7d"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"f228e4d711020dab636126ece3342b91b8749d57","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  DockerInsecureRegistryAddress:"},{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"{% if ansible_distribution in [\u0027RedHat\u0027] %}"},{"line_number":15,"context_line":"  PythonInterpreter: /usr/bin/python3"},{"line_number":16,"context_line":"{% endif %}"},{"line_number":17,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"},{"line_number":18,"context_line":"  DockerRegistryMirror: {{undercloud_docker_registry_mirror}}"},{"line_number":19,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"7faddb67_005b5a54","line":16,"range":{"start_line":14,"start_character":0,"end_line":16,"end_character":11},"updated":"2019-07-22 09:49:20.000000000","message":"ansible_distribution is RedHat even on centos7, so this. line is clearly not currect.","commit_id":"2a80fb64ca55a749d2dea4e0fd5141ad1f6aebef"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"c3f65bbf3f8288fd20e55f4c297202c997c221c9","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  DockerInsecureRegistryAddress:"},{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"{% if ansible_distribution in [\u0027RedHat\u0027] and ansible_distribution_major_version|int() \u003e\u003d 8 %}"},{"line_number":15,"context_line":"  PythonInterpreter: /usr/bin/python3"},{"line_number":16,"context_line":"{% endif %}"},{"line_number":17,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"7faddb67_be8db82b","line":14,"range":{"start_line":14,"start_character":83,"end_line":14,"end_character":85},"updated":"2019-07-23 08:57:10.000000000","message":"AFAIK filters are not called with parantheses. It may work with them but better use the standard syntax, the same we are using everywhere.","commit_id":"941dd2d7f1a09fea5a676075dc3de43cc5c9a733"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"c3f65bbf3f8288fd20e55f4c297202c997c221c9","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"{% if ansible_distribution in [\u0027RedHat\u0027] and ansible_distribution_major_version|int() \u003e\u003d 8 %}"},{"line_number":15,"context_line":"  PythonInterpreter: /usr/bin/python3"},{"line_number":16,"context_line":"{% endif %}"},{"line_number":17,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"},{"line_number":18,"context_line":"  DockerRegistryMirror: {{undercloud_docker_registry_mirror}}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"7faddb67_1ea88c75","line":15,"range":{"start_line":15,"start_character":21,"end_line":15,"end_character":37},"updated":"2019-07-23 08:57:10.000000000","message":"Why to stick with a hardcoded path instead of using ansible facts which return path to python?","commit_id":"941dd2d7f1a09fea5a676075dc3de43cc5c9a733"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"201f01b62a55aa05ac6d506cef820e70850753ad","unresolved":false,"context_lines":[{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"{% if ansible_distribution in [\u0027RedHat\u0027] and ansible_distribution_major_version|int() \u003e\u003d 8 %}"},{"line_number":15,"context_line":"  PythonInterpreter: /usr/bin/python3"},{"line_number":16,"context_line":"{% endif %}"},{"line_number":17,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"},{"line_number":18,"context_line":"  DockerRegistryMirror: {{undercloud_docker_registry_mirror}}"},{"line_number":19,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"7faddb67_89dbe71f","line":16,"updated":"2019-07-23 06:31:14.000000000","message":"again, anyone know why we didn\u0027t need this for fedora standalone?","commit_id":"941dd2d7f1a09fea5a676075dc3de43cc5c9a733"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"c3f65bbf3f8288fd20e55f4c297202c997c221c9","unresolved":false,"context_lines":[{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"{% if ansible_distribution in [\u0027RedHat\u0027] and ansible_distribution_major_version|int() \u003e\u003d 8 %}"},{"line_number":15,"context_line":"  PythonInterpreter: /usr/bin/python3"},{"line_number":16,"context_line":"{% endif %}"},{"line_number":17,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"},{"line_number":18,"context_line":"  DockerRegistryMirror: {{undercloud_docker_registry_mirror}}"},{"line_number":19,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":8,"id":"7faddb67_beb618dc","line":16,"in_reply_to":"7faddb67_89dbe71f","updated":"2019-07-23 08:57:10.000000000","message":"Probably because we installed both pythons on fedora.","commit_id":"941dd2d7f1a09fea5a676075dc3de43cc5c9a733"},{"author":{"_account_id":8175,"name":"Rafael Folco","email":"rfolco@redhat.com","username":"rfolco"},"change_message_id":"1432e8cb1c91e8b242ab3d268f4270aa1309e668","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  DockerInsecureRegistryAddress:"},{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"  PythonInterpreter: {{ ansible_python.executable }}"},{"line_number":15,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"},{"line_number":16,"context_line":"  DockerRegistryMirror: {{undercloud_docker_registry_mirror}}"},{"line_number":17,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"7faddb67_f02cb44b","line":14,"range":{"start_line":14,"start_character":21,"end_line":14,"end_character":52},"updated":"2019-07-23 16:32:29.000000000","message":"what is used when both py2/py3 are available?","commit_id":"700d742a25b210080369b7cc1ab1f1ac4ddbbb54"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"9f782a257147a73470028d1e53788962f50828af","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  DockerInsecureRegistryAddress:"},{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"  PythonInterpreter: {{ ansible_python.executable }}"},{"line_number":15,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"},{"line_number":16,"context_line":"  DockerRegistryMirror: {{undercloud_docker_registry_mirror}}"},{"line_number":17,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"7faddb67_f0db541e","line":14,"range":{"start_line":14,"start_character":21,"end_line":14,"end_character":52},"in_reply_to":"7faddb67_7093e45d","updated":"2019-07-23 16:42:01.000000000","message":"This would use whatever python the ansible was invoked as.  However as of 7.6, /usr/libexec/platform-python can be used which will be wither py2 or py3 depending on the system. I\u0027m unsure if this is available in fedora however","commit_id":"700d742a25b210080369b7cc1ab1f1ac4ddbbb54"},{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"8398d494f21e08b378b06ad9f5e26898c86b24c0","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  DockerInsecureRegistryAddress:"},{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"  PythonInterpreter: {{ ansible_python.executable }}"},{"line_number":15,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"},{"line_number":16,"context_line":"  DockerRegistryMirror: {{undercloud_docker_registry_mirror}}"},{"line_number":17,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"7faddb67_7093e45d","line":14,"range":{"start_line":14,"start_character":21,"end_line":14,"end_character":52},"in_reply_to":"7faddb67_f02cb44b","updated":"2019-07-23 16:37:35.000000000","message":"Folco, what is used is not my decision, user can always *** the config and override it at runtime. What I do know is that we officially support only py2 on c7 and py3 on rhel8 and ansible default behavior respects this in current version.\n\nSomeone with more experience should be consulted here but AFAIK, we don\u0027t support mixes.","commit_id":"700d742a25b210080369b7cc1ab1f1ac4ddbbb54"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"ddc0eb4828d1e6ff52ae94753bc6b2886f46633f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  DockerInsecureRegistryAddress:"},{"line_number":12,"context_line":"  - {{ standalone_ip }}:8787"},{"line_number":13,"context_line":"{% endif %}"},{"line_number":14,"context_line":"  PythonInterpreter: {{ ansible_python.executable }}"},{"line_number":15,"context_line":"{% if undercloud_docker_registry_mirror is defined and undercloud_docker_registry_mirror %}"},{"line_number":16,"context_line":"  DockerRegistryMirror: {{undercloud_docker_registry_mirror}}"},{"line_number":17,"context_line":"{% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"7faddb67_f004b480","line":14,"range":{"start_line":14,"start_character":21,"end_line":14,"end_character":52},"in_reply_to":"7faddb67_f0db541e","updated":"2019-07-23 16:42:52.000000000","message":"Also this is supposed to be autofilled in based on the version of python used to run python tripleoclient i believe. If not, that\u0027s likely the bug","commit_id":"700d742a25b210080369b7cc1ab1f1ac4ddbbb54"}]}