)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"f0a7adbdd0828f5628e7b6dc22446d3d7fdc4412","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1946509d_eaa2146c","updated":"2024-05-22 08:46:17.000000000","message":"see e.g. https://zuul.opendev.org/t/openstack/build/7e74783ab0bb4fa68b67d1c2a6b4e550 for the breakage.","commit_id":"b9361cef5ef2af589d655f68beed15604669dfbe"}],"docker/openstack-base/Dockerfile.j2":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6c91a9b0e5275bf720916b0e116daa4d69e02c58","unresolved":true,"context_lines":[{"line_number":164,"context_line":"        \u0027python-troveclient\u0027,"},{"line_number":165,"context_line":"        \u0027pytz\u0027,"},{"line_number":166,"context_line":"        \u0027repoze.lru\u0027,"},{"line_number":167,"context_line":"        \u0027requests\u003d\u003d2.31.0\u0027,"},{"line_number":168,"context_line":"        \u0027requestsexceptions\u0027,"},{"line_number":169,"context_line":"        \u0027retrying\u0027,"},{"line_number":170,"context_line":"        \u0027setproctitle\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"b069f983_b7c6e820","line":167,"updated":"2024-05-22 08:57:19.000000000","message":"You wan\u0027t to say we\u0027re not using upper-constraints here?","commit_id":"20cf8223562db2b798857bedd499fee4eb0f6343"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"8fd7353b64850062580726d6742b27153b02b407","unresolved":true,"context_lines":[{"line_number":164,"context_line":"        \u0027python-troveclient\u0027,"},{"line_number":165,"context_line":"        \u0027pytz\u0027,"},{"line_number":166,"context_line":"        \u0027repoze.lru\u0027,"},{"line_number":167,"context_line":"        \u0027requests\u003d\u003d2.31.0\u0027,"},{"line_number":168,"context_line":"        \u0027requestsexceptions\u0027,"},{"line_number":169,"context_line":"        \u0027retrying\u0027,"},{"line_number":170,"context_line":"        \u0027setproctitle\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3ee97aab_04210cae","line":167,"in_reply_to":"402785ed_96733508","updated":"2024-05-23 08:21:54.000000000","message":"I\u0027m not convinced we need to pin that in containers at all","commit_id":"20cf8223562db2b798857bedd499fee4eb0f6343"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"3e81ee198b7992d0bc0cd6b21809d13bab25e75d","unresolved":true,"context_lines":[{"line_number":164,"context_line":"        \u0027python-troveclient\u0027,"},{"line_number":165,"context_line":"        \u0027pytz\u0027,"},{"line_number":166,"context_line":"        \u0027repoze.lru\u0027,"},{"line_number":167,"context_line":"        \u0027requests\u003d\u003d2.31.0\u0027,"},{"line_number":168,"context_line":"        \u0027requestsexceptions\u0027,"},{"line_number":169,"context_line":"        \u0027retrying\u0027,"},{"line_number":170,"context_line":"        \u0027setproctitle\u0027,"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"402785ed_96733508","line":167,"in_reply_to":"b069f983_b7c6e820","updated":"2024-05-22 09:01:28.000000000","message":"No, I think I just pinned the package in the wrong place, didn\u0027t I?","commit_id":"20cf8223562db2b798857bedd499fee4eb0f6343"}],"tests/playbooks/pre.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"6c91a9b0e5275bf720916b0e116daa4d69e02c58","unresolved":true,"context_lines":[{"line_number":40,"context_line":"      command: python3 -m venv {{ virtualenv_path }}"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    - name: Install kolla"},{"line_number":43,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install requests\u003d\u003d2.31.0 {{ zuul.project.src_dir }}\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Install docker python library"},{"line_number":46,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install \u0027docker\u003c7\u0027\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9890dc12_65fb38b3","line":43,"updated":"2024-05-22 08:57:19.000000000","message":"Should we start using upper-constraints?","commit_id":"20cf8223562db2b798857bedd499fee4eb0f6343"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"98bb9fb4d94fb7e151a6a62f8453d38c6ffc0259","unresolved":true,"context_lines":[{"line_number":40,"context_line":"      command: python3 -m venv {{ virtualenv_path }}"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    - name: Install kolla"},{"line_number":43,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install requests\u003d\u003d2.31.0 {{ zuul.project.src_dir }}\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Install docker python library"},{"line_number":46,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install \u0027docker\u003c7\u0027\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"56b112c9_7d320134","line":43,"in_reply_to":"7dfad517_95cf3316","updated":"2024-05-23 08:24:17.000000000","message":"ah, we do, urllib3 - to fix that for users - shouldn\u0027t we pin in requirements.txt?","commit_id":"20cf8223562db2b798857bedd499fee4eb0f6343"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"1d7a718f2d1c18b44ef04d853424dea567a27439","unresolved":true,"context_lines":[{"line_number":40,"context_line":"      command: python3 -m venv {{ virtualenv_path }}"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    - name: Install kolla"},{"line_number":43,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install requests\u003d\u003d2.31.0 {{ zuul.project.src_dir }}\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Install docker python library"},{"line_number":46,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install \u0027docker\u003c7\u0027\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7dfad517_95cf3316","line":43,"in_reply_to":"821cedf7_9d0c7ddd","updated":"2024-05-23 08:22:27.000000000","message":"Ok, let\u0027s leave it here - but do we depend in requests in kolla code, or is that purely docker thing?","commit_id":"20cf8223562db2b798857bedd499fee4eb0f6343"},{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"3e81ee198b7992d0bc0cd6b21809d13bab25e75d","unresolved":true,"context_lines":[{"line_number":40,"context_line":"      command: python3 -m venv {{ virtualenv_path }}"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    - name: Install kolla"},{"line_number":43,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install requests\u003d\u003d2.31.0 {{ zuul.project.src_dir }}\""},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Install docker python library"},{"line_number":46,"context_line":"      command: \"{{ virtualenv_path }}/bin/python -m pip install \u0027docker\u003c7\u0027\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"821cedf7_9d0c7ddd","line":43,"in_reply_to":"9890dc12_65fb38b3","updated":"2024-05-22 09:01:28.000000000","message":"so, I\u0027m not sure what you are saying? should, or should I not implement upper constraints now?\n\nin the past such breakage was always fixed by temporarily pinning packages, not by introducing new features (using u-c here).\n\nbut it\u0027s okay to have higher standards for fixing bugs over time, I guess :)\n\nnot quite sure how I would reference u-c here though, let me check..","commit_id":"20cf8223562db2b798857bedd499fee4eb0f6343"}]}