)]}'
{"/COMMIT_MSG":[{"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":"b18bcaa39c4db13dbee1cec8ced9601e3a99c80a","unresolved":true,"context_lines":[{"line_number":16,"context_line":"- fixed source type builds."},{"line_number":17,"context_line":"- upgrade pip to \u003c21 before using it (21.0 is Py3 only)"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Coauthored-by: Marcin Juszkiewicz \u003cmarcin.juszkiewicz@linaro.org\u003e"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Depends-on: https://review.opendev.org/c/openstack/kolla-ansible/+/772198"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9f68c42a_c70ef1a8","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":10},"updated":"2021-01-24 14:15:10.000000000","message":"Co-authored","commit_id":"c166e3472f2ed52bafa09e22f8a811f4076bd88f"}],"docker/macros.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":"b4c20c7bc2815ab5905ccb666ac1fb7eb75d0b5e","unresolved":true,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"{% macro install_pip(packages, constraints \u003d true) %}"},{"line_number":40,"context_line":"{%- if packages is sequence and packages|length \u003e 0 -%}"},{"line_number":41,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade \"pip\u003c21\" \u0026\u0026 \\"},{"line_number":42,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade{{ \u0027 \u0027 }}"},{"line_number":43,"context_line":"    {%- if constraints %}-c /requirements/upper-constraints.txt {% endif -%}"},{"line_number":44,"context_line":"    {{ packages | join(\u0027 \u0027) }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"ab4f4a9a_869d587a","line":41,"range":{"start_line":41,"start_character":4,"end_line":41,"end_character":91},"updated":"2021-01-24 14:13:18.000000000","message":"Do we have to?","commit_id":"c166e3472f2ed52bafa09e22f8a811f4076bd88f"},{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"1b14de2f59c42d65f7cb72a6c597eb5fa68f5f80","unresolved":true,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"{% macro install_pip(packages, constraints \u003d true) %}"},{"line_number":40,"context_line":"{%- if packages is sequence and packages|length \u003e 0 -%}"},{"line_number":41,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade \"pip\u003c21\" \u0026\u0026 \\"},{"line_number":42,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade{{ \u0027 \u0027 }}"},{"line_number":43,"context_line":"    {%- if constraints %}-c /requirements/upper-constraints.txt {% endif -%}"},{"line_number":44,"context_line":"    {{ packages | join(\u0027 \u0027) }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"ceffb675_e7d17e9d","line":41,"range":{"start_line":41,"start_character":4,"end_line":41,"end_character":91},"in_reply_to":"ab4f4a9a_869d587a","updated":"2021-01-24 16:15:18.000000000","message":"There were some issues in past with pip 8/9 compared to 18-20 one. So instead of relying on whatever pip is in distro we update to latest good one as we did in past with get_pip() macro.\n\nJust the question is where to add that upgrade.","commit_id":"c166e3472f2ed52bafa09e22f8a811f4076bd88f"},{"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":"270ed1dc0e8f16df38c86d87e7bc53d11d5d61e4","unresolved":true,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"{% macro install_pip(packages, constraints \u003d true) %}"},{"line_number":40,"context_line":"{%- if packages is sequence and packages|length \u003e 0 -%}"},{"line_number":41,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade \"pip\u003c21\" \u0026\u0026 \\"},{"line_number":42,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade{{ \u0027 \u0027 }}"},{"line_number":43,"context_line":"    {%- if constraints %}-c /requirements/upper-constraints.txt {% endif -%}"},{"line_number":44,"context_line":"    {{ packages | join(\u0027 \u0027) }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"ae324395_431ab4cc","line":41,"range":{"start_line":41,"start_character":4,"end_line":41,"end_character":91},"in_reply_to":"ceffb675_e7d17e9d","updated":"2021-01-24 16:19:01.000000000","message":"Mhm, trying to get close to Ussuri behaviour. Perhaps u-c can save us. We will see soon.","commit_id":"c166e3472f2ed52bafa09e22f8a811f4076bd88f"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"f8a2a09f2e3493b8dd5175c408b4122de0223c87","unresolved":true,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"{% macro install_pip(packages, constraints \u003d true) %}"},{"line_number":40,"context_line":"{%- if packages is sequence and packages|length \u003e 0 -%}"},{"line_number":41,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade{{ \u0027 \u0027 }}"},{"line_number":42,"context_line":"    {%- if constraints %}-c /requirements/upper-constraints.txt {% endif -%}"},{"line_number":43,"context_line":"    {{ packages | join(\u0027 \u0027) }}"},{"line_number":44,"context_line":"{%- else -%}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"f1e76676_173c7173","line":41,"updated":"2021-01-25 11:09:10.000000000","message":"this should not work with python2.","commit_id":"278893ef1d9bc8f7d013685d0676ce2458fcaaaa"},{"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":"a181e15dc5cfe35e633c4e7e61f3e195a5ab938e","unresolved":true,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"{% macro install_pip(packages, constraints \u003d true) %}"},{"line_number":40,"context_line":"{%- if packages is sequence and packages|length \u003e 0 -%}"},{"line_number":41,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade{{ \u0027 \u0027 }}"},{"line_number":42,"context_line":"    {%- if constraints %}-c /requirements/upper-constraints.txt {% endif -%}"},{"line_number":43,"context_line":"    {{ packages | join(\u0027 \u0027) }}"},{"line_number":44,"context_line":"{%- else -%}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"d56b2981_06a424bf","line":41,"in_reply_to":"70a8a7bc_f436f815","updated":"2021-01-25 14:03:35.000000000","message":"Ack, no problem.","commit_id":"278893ef1d9bc8f7d013685d0676ce2458fcaaaa"},{"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":"1a7d1d5b43f33261ef108c09fc9b93ad3f93f874","unresolved":true,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"{% macro install_pip(packages, constraints \u003d true) %}"},{"line_number":40,"context_line":"{%- if packages is sequence and packages|length \u003e 0 -%}"},{"line_number":41,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade{{ \u0027 \u0027 }}"},{"line_number":42,"context_line":"    {%- if constraints %}-c /requirements/upper-constraints.txt {% endif -%}"},{"line_number":43,"context_line":"    {{ packages | join(\u0027 \u0027) }}"},{"line_number":44,"context_line":"{%- else -%}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"ffdcb37f_6913e1d0","line":41,"in_reply_to":"f1e76676_173c7173","updated":"2021-01-25 11:49:43.000000000","message":"Why? It works.","commit_id":"278893ef1d9bc8f7d013685d0676ce2458fcaaaa"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"c56a6e92d887b4444dcd0843c73dfc58b9eba161","unresolved":true,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"{% macro install_pip(packages, constraints \u003d true) %}"},{"line_number":40,"context_line":"{%- if packages is sequence and packages|length \u003e 0 -%}"},{"line_number":41,"context_line":"    python{{ distro_python_version }} -m pip --no-cache-dir install --upgrade{{ \u0027 \u0027 }}"},{"line_number":42,"context_line":"    {%- if constraints %}-c /requirements/upper-constraints.txt {% endif -%}"},{"line_number":43,"context_line":"    {{ packages | join(\u0027 \u0027) }}"},{"line_number":44,"context_line":"{%- else -%}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"70a8a7bc_f436f815","line":41,"in_reply_to":"ffdcb37f_6913e1d0","updated":"2021-01-25 12:04:29.000000000","message":"sorry , i misunderstand the docs. i tested it with python2 it works.","commit_id":"278893ef1d9bc8f7d013685d0676ce2458fcaaaa"}]}