)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"e0fbed13e17ff63fadc886deaaf47a74b40e59fd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"67fe78c3_d954228e","updated":"2024-11-12 10:33:46.000000000","message":"open a bug in launchpad - unless you don\u0027t want that backported","commit_id":"08da2f89f0ced63e20667fec43815968f174c8cb"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"2f449e99575d8249ff044bb2f05d4085e40b62c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"585c33ae_a7923eb3","in_reply_to":"67fe78c3_d954228e","updated":"2024-11-12 13:25:05.000000000","message":"Acknowledged","commit_id":"08da2f89f0ced63e20667fec43815968f174c8cb"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"527ffcd5896eeec05ba92030925e9b8706804349","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"56174d08_314f49e0","updated":"2024-11-14 20:15:16.000000000","message":"this should be merged with https://review.opendev.org/c/openstack/kolla/+/935143","commit_id":"bf590a8a0b68b2b2e846d5c10ae6c684a8275e58"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"839648096d3f9c2029c346ab7b0ee3028d616ced","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"63987d5b_8c56de50","updated":"2024-11-15 17:08:36.000000000","message":"at the last we should have an upgrade section in the reno with mention of UPPER_CONSTRAINTS_FILE (re)moved.","commit_id":"96f5ec23d4a5f06f695ac9b6fb148dffb50b95b1"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"d9d6221183e57915f7e4da9063fc91b85150578f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1459ed68_7441edf4","in_reply_to":"63987d5b_8c56de50","updated":"2024-11-18 08:25:02.000000000","message":"Acknowledged","commit_id":"96f5ec23d4a5f06f695ac9b6fb148dffb50b95b1"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"89f93e0db0792270daa378de4305cbe9393a2f97","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"4a3fb7a3_6f81e812","updated":"2024-11-19 18:38:43.000000000","message":"Can you split out the things that can not be backported to a separate patch (that obviously won\u0027t be backported)?","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"b57bd36c5058d749c977d852937d4056e5b28ab3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"ffb99f50_b19abe4c","updated":"2024-11-18 11:02:06.000000000","message":"perfect, thanks","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"297f33fc905285667c9b0d1ed420aea404b1b0b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"e7fcd0d7_79316079","updated":"2024-11-19 13:41:53.000000000","message":"recheck","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"b4d7b225df7a99e508cbe29534b1c0d2642a8ad9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"83ee129a_262fc6da","in_reply_to":"4a3fb7a3_6f81e812","updated":"2024-12-05 11:23:09.000000000","message":"What about not backporting this patch? If we can agree that Bifrost upper-constraints using different source compared to other services is a feature until Epoxy, we can say this is a feature change from Epoxy onwards.","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"d2e48a72cef7ff962576b3d0364a4ca8a8320416","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"a7a63699_ff9158ba","in_reply_to":"83ee129a_262fc6da","updated":"2025-01-16 13:31:44.000000000","message":"Done","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"7474735530bc97457fc82b518ebc0e31f92dd29d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"330220cc_d944e2b4","updated":"2024-12-19 11:46:07.000000000","message":"recheck","commit_id":"e7a9653a213ae3eeca476b94e7784e270a8cf279"},{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"b19df629d1b58bff8494f52012aa174fb9b779a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"b81ccb12_f9c512ab","updated":"2025-02-05 15:37:59.000000000","message":"Looks sensible to me","commit_id":"4cab35b1a81055c3696e5fa42d794d6fc96076e6"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"7d129d61e5c55995e6b58b23063473a33eeedc7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"a8322cf6_50eb7490","updated":"2025-03-27 11:25:21.000000000","message":"So many +1s, treating as second +2 ;-)","commit_id":"4cab35b1a81055c3696e5fa42d794d6fc96076e6"}],"doc/source/admin/image-building.rst":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"b2eeaa9aac053bb3e38630a13d6f0b5bdc64a841","unresolved":true,"context_lines":[{"line_number":368,"context_line":""},{"line_number":369,"context_line":"``kolla-toolbox`` image needs different approach as it does not uses"},{"line_number":370,"context_line":"``openstack-base`` as a base image."},{"line_number":371,"context_line":"An environment variable ``UPPER_CONSTRAINTS_FILE`` can be set for the"},{"line_number":372,"context_line":"Dockerfile of ``kolla-toolbox``."},{"line_number":373,"context_line":"To set the environment variable, add the following contents to the"},{"line_number":374,"context_line":"``kolla_toolbox_pip_conf`` block in your template file, for example,"},{"line_number":375,"context_line":"``template-overrides.j2``:"},{"line_number":376,"context_line":""},{"line_number":377,"context_line":".. code-block:: jinja"},{"line_number":378,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"d02deea3_5d99fe3e","line":375,"range":{"start_line":371,"start_character":0,"end_line":375,"end_character":26},"updated":"2025-02-05 15:36:22.000000000","message":"```suggestion\nA variable ``UPPER_CONSTRAINTS_FILE`` is set in the\nDockerfile of ``kolla-toolbox``.\nTo change variable, add the following contents to the\n``kolla_toolbox_pip_conf`` block in your template file, for example,\n``template-overrides.j2``:\n```\nJust changing the wording a bit here to make it clearer","commit_id":"e7a9653a213ae3eeca476b94e7784e270a8cf279"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"6b3658249e8476f003317b941903243cfd0cf5e9","unresolved":false,"context_lines":[{"line_number":368,"context_line":""},{"line_number":369,"context_line":"``kolla-toolbox`` image needs different approach as it does not uses"},{"line_number":370,"context_line":"``openstack-base`` as a base image."},{"line_number":371,"context_line":"An environment variable ``UPPER_CONSTRAINTS_FILE`` can be set for the"},{"line_number":372,"context_line":"Dockerfile of ``kolla-toolbox``."},{"line_number":373,"context_line":"To set the environment variable, add the following contents to the"},{"line_number":374,"context_line":"``kolla_toolbox_pip_conf`` block in your template file, for example,"},{"line_number":375,"context_line":"``template-overrides.j2``:"},{"line_number":376,"context_line":""},{"line_number":377,"context_line":".. code-block:: jinja"},{"line_number":378,"context_line":""}],"source_content_type":"text/x-rst","patch_set":11,"id":"0453b5f0_d96b4f41","line":375,"range":{"start_line":371,"start_character":0,"end_line":375,"end_character":26},"in_reply_to":"d02deea3_5d99fe3e","updated":"2025-02-05 15:37:00.000000000","message":"Fix applied.","commit_id":"e7a9653a213ae3eeca476b94e7784e270a8cf279"}],"docker/kolla-toolbox/Dockerfile.j2":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"88437f73cd4b6f75c977af8767af136a618476ea","unresolved":true,"context_lines":[{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block kolla_toolbox_header %}"},{"line_number":7,"context_line":"# ENV UPPER_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/{{ openstack_release }}"},{"line_number":8,"context_line":"{% endblock %}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% set venv_path\u003d\u0027/opt/ansible\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"88ddb4be_9fc433c2","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"updated":"2024-11-15 12:55:01.000000000","message":"remove comment? imho we should use this by default?","commit_id":"9d0705b8d645643a545f6c35e526004e17a213b7"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"c64697d78eb076b5db1bfb163351fd7ac6e811e5","unresolved":true,"context_lines":[{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block kolla_toolbox_header %}"},{"line_number":7,"context_line":"# ENV UPPER_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/{{ openstack_release }}"},{"line_number":8,"context_line":"{% endblock %}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% set venv_path\u003d\u0027/opt/ansible\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"ffe8fad6_1ca2dcef","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"3b664961_a1f8afca","updated":"2024-11-15 14:22:51.000000000","message":"sure. it should be the separate named block. header block should be empty for other user customisations.","commit_id":"9d0705b8d645643a545f6c35e526004e17a213b7"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"0749922e40c625dd119f85947ac978cf6cfbf510","unresolved":true,"context_lines":[{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block kolla_toolbox_header %}"},{"line_number":7,"context_line":"# ENV UPPER_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/{{ openstack_release }}"},{"line_number":8,"context_line":"{% endblock %}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% set venv_path\u003d\u0027/opt/ansible\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"d0b9023a_c592bbbf","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"88ddb4be_9fc433c2","updated":"2024-11-15 13:46:28.000000000","message":"It is used by default because of https://opendev.org/openstack/kolla/src/branch/master/docker/kolla-toolbox/Dockerfile.j2#L83\nThis line sets the value of UPPER_CONSTRAINTS_FILE to where default upper-constraints is if it was not defined before.","commit_id":"9d0705b8d645643a545f6c35e526004e17a213b7"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"b93ea713722fc3ba81db52f5e0f0d0c8e0d24da8","unresolved":true,"context_lines":[{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block kolla_toolbox_header %}"},{"line_number":7,"context_line":"# ENV UPPER_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/{{ openstack_release }}"},{"line_number":8,"context_line":"{% endblock %}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% set venv_path\u003d\u0027/opt/ansible\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"3b664961_a1f8afca","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"9ada3fce_7771ff34","updated":"2024-11-15 14:15:11.000000000","message":"I like the idea. But rather than do that in header block, what about a new block named something like kolla_toolbox_pip_conf for example?","commit_id":"9d0705b8d645643a545f6c35e526004e17a213b7"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"16d74b67f6633da8d2d039b7e6f499ad3ca9224d","unresolved":true,"context_lines":[{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block kolla_toolbox_header %}"},{"line_number":7,"context_line":"# ENV UPPER_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/{{ openstack_release }}"},{"line_number":8,"context_line":"{% endblock %}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% set venv_path\u003d\u0027/opt/ansible\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"9ada3fce_7771ff34","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"d0b9023a_c592bbbf","updated":"2024-11-15 13:59:24.000000000","message":"sure. This is was my change: https://opendev.org/openstack/kolla/commit/83efe325462e683e2c99d088012cd611d7a0b138\n\nlets say another way: may be we need to move defaults to this block and remove bash variable override mechanism?","commit_id":"9d0705b8d645643a545f6c35e526004e17a213b7"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"4a379bec16f35c6e5cb9f840a4665d0555092ff8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block kolla_toolbox_header %}"},{"line_number":7,"context_line":"# ENV UPPER_CONSTRAINTS_FILE\u003dhttps://releases.openstack.org/constraints/upper/{{ openstack_release }}"},{"line_number":8,"context_line":"{% endblock %}"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"{% set venv_path\u003d\u0027/opt/ansible\u0027 %}"}],"source_content_type":"text/x-jinja2","patch_set":7,"id":"bf889ffa_fe7cd750","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"in_reply_to":"ffe8fad6_1ca2dcef","updated":"2024-11-15 14:28:19.000000000","message":"Acknowledged","commit_id":"9d0705b8d645643a545f6c35e526004e17a213b7"}],"releasenotes/notes/fix-bifrost-using-wrong-upper-constraints-886dc7b51bd4ad98.yaml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"89f93e0db0792270daa378de4305cbe9393a2f97","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    Fixed Bifrost install playbook ignoring upper-constraints of openstack-base"},{"line_number":5,"context_line":"    and using upstream upper-constraints even if it was overridden from"},{"line_number":6,"context_line":"    kolla-build.conf `LP#2087960 \u003chttps://launchpad.net/bugs/2087960\u003e`__"},{"line_number":7,"context_line":"features:"},{"line_number":8,"context_line":"  - |"},{"line_number":9,"context_line":"    Updated the contents of Python package options in documentation to reflect"},{"line_number":10,"context_line":"    latest customisation methods."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"b672ed30_29654a22","line":7,"updated":"2024-11-19 18:38:43.000000000","message":"We do not backport features","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"d2e48a72cef7ff962576b3d0364a4ca8a8320416","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    Fixed Bifrost install playbook ignoring upper-constraints of openstack-base"},{"line_number":5,"context_line":"    and using upstream upper-constraints even if it was overridden from"},{"line_number":6,"context_line":"    kolla-build.conf `LP#2087960 \u003chttps://launchpad.net/bugs/2087960\u003e`__"},{"line_number":7,"context_line":"features:"},{"line_number":8,"context_line":"  - |"},{"line_number":9,"context_line":"    Updated the contents of Python package options in documentation to reflect"},{"line_number":10,"context_line":"    latest customisation methods."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"aab95bbb_44365d11","line":7,"in_reply_to":"b672ed30_29654a22","updated":"2025-01-16 13:31:44.000000000","message":"Done","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"89f93e0db0792270daa378de4305cbe9393a2f97","unresolved":true,"context_lines":[{"line_number":8,"context_line":"  - |"},{"line_number":9,"context_line":"    Updated the contents of Python package options in documentation to reflect"},{"line_number":10,"context_line":"    latest customisation methods."},{"line_number":11,"context_line":"upgrade:"},{"line_number":12,"context_line":"  - |"},{"line_number":13,"context_line":"    The use of UPPER_CONSTRAINTS_FILE environment variable was moved from"},{"line_number":14,"context_line":"    ``base_pip_conf`` block in base Dockerfile to ``kolla_toolbox_pip_conf``"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"727902aa_d11ca68f","line":11,"updated":"2024-11-19 18:38:43.000000000","message":"This is not backwards compatible, so can\u0027t be backported","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"d2e48a72cef7ff962576b3d0364a4ca8a8320416","unresolved":false,"context_lines":[{"line_number":8,"context_line":"  - |"},{"line_number":9,"context_line":"    Updated the contents of Python package options in documentation to reflect"},{"line_number":10,"context_line":"    latest customisation methods."},{"line_number":11,"context_line":"upgrade:"},{"line_number":12,"context_line":"  - |"},{"line_number":13,"context_line":"    The use of UPPER_CONSTRAINTS_FILE environment variable was moved from"},{"line_number":14,"context_line":"    ``base_pip_conf`` block in base Dockerfile to ``kolla_toolbox_pip_conf``"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"8281eb36_711a77c0","line":11,"in_reply_to":"727902aa_d11ca68f","updated":"2025-01-16 13:31:44.000000000","message":"Done","commit_id":"b0381c69991c92ac4a1d147fd504a530f8d08dac"}]}