)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"change_message_id":"cdcb83e6b346bb1f469c8982008a9bac02e267ce","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Dr. Jens Harbott \u003cfrickler@offenerstapel.de\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-06-09 07:22:14 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Upgrade from Yoga"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I89dd99b5f25502f5ebe15a9e90ccecc4bd6feafa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"0d26b55b_94c8b598","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":1},"updated":"2022-06-09 07:52:19.000000000","message":"I would love to have commit messages which do not require reading whole patch.\n------------------------------------------------------------------------------\n\nSwitch to master branch\n\nAs we are at R-x week it is time to move from Xena/Yoga (previous/current) \nto Yoga/master code.","commit_id":"f4c780314112b33851463d111b8c9edbed485f46"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"5481273b556c4151aa3719fbecb55642142090c6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Dr. Jens Harbott \u003cfrickler@offenerstapel.de\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-06-09 07:22:14 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Upgrade from Yoga"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I89dd99b5f25502f5ebe15a9e90ccecc4bd6feafa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"2b249d72_bb6f6bfb","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":1},"in_reply_to":"0d26b55b_94c8b598","updated":"2022-06-09 21:18:25.000000000","message":"Done","commit_id":"f4c780314112b33851463d111b8c9edbed485f46"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d240c8b3994f691709e4816bd488d9c0ba52c0ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a3d6183a_2bc2af01","updated":"2022-06-13 14:53:14.000000000","message":"Don\u0027t want to self-approve my own changes, so this is +2 for the original parts.","commit_id":"ebe570bb840d79f78816d13c72729a5c80f159f0"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"2f67a197de17e6e8e788387b8582d18d44021618","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0806ae08_d3e748ec","updated":"2022-06-15 05:33:58.000000000","message":"Giving +2 to frickler\u0027s parts, so that gives a joint +2 from me and frickler ;-)","commit_id":"ebe570bb840d79f78816d13c72729a5c80f159f0"}],"tests/run.yml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"921731f81684853756acf8480f5b7a260dc5664a","unresolved":true,"context_lines":[{"line_number":211,"context_line":"        - name: write requirements.yml"},{"line_number":212,"context_line":"          copy:"},{"line_number":213,"context_line":"            content: \"{{ new_requirements | to_nice_yaml }}\""},{"line_number":214,"context_line":"            dest: \"{{ kolla_ansible_src_dir }}/requirements.yml\""},{"line_number":215,"context_line":"          vars:"},{"line_number":216,"context_line":"            old_requirements: \"{{ requirements_yml.content | b64decode | from_yaml }}\""},{"line_number":217,"context_line":"            new_requirement:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"114608c6_13b72493","line":214,"updated":"2022-06-10 09:39:56.000000000","message":"Or add \"backup: yes\" here, and restore the backup before we switch to the current release?","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d240c8b3994f691709e4816bd488d9c0ba52c0ae","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        - name: write requirements.yml"},{"line_number":212,"context_line":"          copy:"},{"line_number":213,"context_line":"            content: \"{{ new_requirements | to_nice_yaml }}\""},{"line_number":214,"context_line":"            dest: \"{{ kolla_ansible_src_dir }}/requirements.yml\""},{"line_number":215,"context_line":"          vars:"},{"line_number":216,"context_line":"            old_requirements: \"{{ requirements_yml.content | b64decode | from_yaml }}\""},{"line_number":217,"context_line":"            new_requirement:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5038f765_509f1e5c","line":214,"in_reply_to":"114608c6_13b72493","updated":"2022-06-13 14:53:14.000000000","message":"Done","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"ec37c94b6565643d9002d89f98aab0f381cf53d3","unresolved":true,"context_lines":[{"line_number":570,"context_line":"              git log --pretty\u003doneline  -1"},{"line_number":571,"context_line":"            chdir: \"{{ ansible_collection_kolla_src_dir }}\""},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"        # NOTE(frickler): We modified the old checkout, so we need to stash local"},{"line_number":574,"context_line":"        # changes in order for the checkout to succeed"},{"line_number":575,"context_line":"        - name: checkout the current kolla-ansible branch"},{"line_number":576,"context_line":"          shell:"},{"line_number":577,"context_line":"            cmd: |"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5966e045_5ea16faf","line":574,"range":{"start_line":573,"start_character":0,"end_line":574,"end_character":54},"updated":"2022-06-10 09:01:34.000000000","message":"What gets modified?","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"2fa52b5d7c8525d1240152a4233b7ba734a8c2d0","unresolved":true,"context_lines":[{"line_number":570,"context_line":"              git log --pretty\u003doneline  -1"},{"line_number":571,"context_line":"            chdir: \"{{ ansible_collection_kolla_src_dir }}\""},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"        # NOTE(frickler): We modified the old checkout, so we need to stash local"},{"line_number":574,"context_line":"        # changes in order for the checkout to succeed"},{"line_number":575,"context_line":"        - name: checkout the current kolla-ansible branch"},{"line_number":576,"context_line":"          shell:"},{"line_number":577,"context_line":"            cmd: |"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"741eac0c_faba1607","line":574,"range":{"start_line":573,"start_character":0,"end_line":574,"end_character":54},"in_reply_to":"149ec524_fd484787","updated":"2022-06-13 08:21:44.000000000","message":"Not that weird - if I\u0027d known the consequences as the time I\u0027d have made this change myself.\n\nAgree that \"git checkout requirements.yml\" would be better, to avoid inadvertent changes to the repo as part of normal operation.","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a7b0ba93ef2a1fe0b17751a1facc729aff1d65d9","unresolved":true,"context_lines":[{"line_number":570,"context_line":"              git log --pretty\u003doneline  -1"},{"line_number":571,"context_line":"            chdir: \"{{ ansible_collection_kolla_src_dir }}\""},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"        # NOTE(frickler): We modified the old checkout, so we need to stash local"},{"line_number":574,"context_line":"        # changes in order for the checkout to succeed"},{"line_number":575,"context_line":"        - name: checkout the current kolla-ansible branch"},{"line_number":576,"context_line":"          shell:"},{"line_number":577,"context_line":"            cmd: |"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"149ec524_fd484787","line":574,"range":{"start_line":573,"start_character":0,"end_line":574,"end_character":54},"in_reply_to":"5966e045_5ea16faf","updated":"2022-06-10 09:34:41.000000000","message":"Weird that you have to ask when you introduced the code yourself. ;)\n\nrequirements.yml, see L211 ff. above\n\nWe might consider doing \"git checkout requirements.yml\" instead in order to be more explicit.","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"34232860e51cdea0f6c62796a501a9f4ff0f5b2e","unresolved":false,"context_lines":[{"line_number":570,"context_line":"              git log --pretty\u003doneline  -1"},{"line_number":571,"context_line":"            chdir: \"{{ ansible_collection_kolla_src_dir }}\""},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"        # NOTE(frickler): We modified the old checkout, so we need to stash local"},{"line_number":574,"context_line":"        # changes in order for the checkout to succeed"},{"line_number":575,"context_line":"        - name: checkout the current kolla-ansible branch"},{"line_number":576,"context_line":"          shell:"},{"line_number":577,"context_line":"            cmd: |"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"07f6d6d2_7b4ae752","line":574,"range":{"start_line":573,"start_character":0,"end_line":574,"end_character":54},"in_reply_to":"741eac0c_faba1607","updated":"2022-06-13 13:05:42.000000000","message":"Done","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"}],"zuul.d/base.yaml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"726091300743ae38692876d6270bd2b423786bde","unresolved":true,"context_lines":[{"line_number":10,"context_line":"      - openstack/ansible-collection-kolla"},{"line_number":11,"context_line":"      - openstack/kolla"},{"line_number":12,"context_line":"      - openstack/kolla-ansible"},{"line_number":13,"context_line":"      - name: openstack/requirements"},{"line_number":14,"context_line":"    irrelevant-files:"},{"line_number":15,"context_line":"      - ^.*\\.rst$"},{"line_number":16,"context_line":"      - ^doc/.*"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"174fd7f1_a3830b6f","line":13,"range":{"start_line":13,"start_character":8,"end_line":13,"end_character":14},"updated":"2022-06-07 08:37:26.000000000","message":"remove?","commit_id":"cb96ca50207dbbece450b919f932f42a83a7a870"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"17c5167ede23b3b34afa888e0de7f48203611060","unresolved":false,"context_lines":[{"line_number":10,"context_line":"      - openstack/ansible-collection-kolla"},{"line_number":11,"context_line":"      - openstack/kolla"},{"line_number":12,"context_line":"      - openstack/kolla-ansible"},{"line_number":13,"context_line":"      - name: openstack/requirements"},{"line_number":14,"context_line":"    irrelevant-files:"},{"line_number":15,"context_line":"      - ^.*\\.rst$"},{"line_number":16,"context_line":"      - ^doc/.*"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b5b690d9_b1bc9d03","line":13,"range":{"start_line":13,"start_character":8,"end_line":13,"end_character":14},"in_reply_to":"174fd7f1_a3830b6f","updated":"2022-06-09 07:22:30.000000000","message":"Done","commit_id":"cb96ca50207dbbece450b919f932f42a83a7a870"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"726091300743ae38692876d6270bd2b423786bde","unresolved":true,"context_lines":[{"line_number":247,"context_line":"    run: tests/run-hashi-vault.yml"},{"line_number":248,"context_line":"    required-projects:"},{"line_number":249,"context_line":"      - openstack/kolla-ansible"},{"line_number":250,"context_line":"      - name: openstack/requirements"},{"line_number":251,"context_line":"    voting: false"},{"line_number":252,"context_line":"    irrelevant-files:"},{"line_number":253,"context_line":"      - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"cd971bff_cf04ba83","line":250,"range":{"start_line":250,"start_character":8,"end_line":250,"end_character":14},"updated":"2022-06-07 08:37:26.000000000","message":"ditto?","commit_id":"cb96ca50207dbbece450b919f932f42a83a7a870"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"17c5167ede23b3b34afa888e0de7f48203611060","unresolved":false,"context_lines":[{"line_number":247,"context_line":"    run: tests/run-hashi-vault.yml"},{"line_number":248,"context_line":"    required-projects:"},{"line_number":249,"context_line":"      - openstack/kolla-ansible"},{"line_number":250,"context_line":"      - name: openstack/requirements"},{"line_number":251,"context_line":"    voting: false"},{"line_number":252,"context_line":"    irrelevant-files:"},{"line_number":253,"context_line":"      - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a7cf9ffa_8e59831f","line":250,"range":{"start_line":250,"start_character":8,"end_line":250,"end_character":14},"in_reply_to":"cd971bff_cf04ba83","updated":"2022-06-09 07:22:30.000000000","message":"Done","commit_id":"cb96ca50207dbbece450b919f932f42a83a7a870"}],"zuul.d/project.yaml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"ae45832cec168ba120a8cff65a8f2f0630c6556f","unresolved":true,"context_lines":[{"line_number":11,"context_line":"      - periodic-stable-jobs"},{"line_number":12,"context_line":"    check:"},{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        # - kolla-ansible-centos8s-source"},{"line_number":15,"context_line":"        - kolla-ansible-debian-source"},{"line_number":16,"context_line":"        # - kolla-ansible-openeuler-source"},{"line_number":17,"context_line":"        # - kolla-ansible-rocky8-source"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"e6ec7cd4_a18e579c","line":14,"range":{"start_line":14,"start_character":8,"end_line":14,"end_character":41},"updated":"2022-06-10 09:01:16.000000000","message":"Is it possible to do this in a separate patch?","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"34232860e51cdea0f6c62796a501a9f4ff0f5b2e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"      - periodic-stable-jobs"},{"line_number":12,"context_line":"    check:"},{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        # - kolla-ansible-centos8s-source"},{"line_number":15,"context_line":"        - kolla-ansible-debian-source"},{"line_number":16,"context_line":"        # - kolla-ansible-openeuler-source"},{"line_number":17,"context_line":"        # - kolla-ansible-rocky8-source"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"22bafd01_303e7e0a","line":14,"range":{"start_line":14,"start_character":8,"end_line":14,"end_character":41},"in_reply_to":"17f8793d_2d400f66","updated":"2022-06-13 13:05:42.000000000","message":"Done","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"2fa52b5d7c8525d1240152a4233b7ba734a8c2d0","unresolved":true,"context_lines":[{"line_number":11,"context_line":"      - periodic-stable-jobs"},{"line_number":12,"context_line":"    check:"},{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        # - kolla-ansible-centos8s-source"},{"line_number":15,"context_line":"        - kolla-ansible-debian-source"},{"line_number":16,"context_line":"        # - kolla-ansible-openeuler-source"},{"line_number":17,"context_line":"        # - kolla-ansible-rocky8-source"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"17f8793d_2d400f66","line":14,"range":{"start_line":14,"start_character":8,"end_line":14,"end_character":41},"in_reply_to":"daa1063e_c84bd0dc","updated":"2022-06-13 08:21:44.000000000","message":"Are they tied to this patch due to the change in branch of the requirements repo? An explanatory comment in the commit message would be appreciated.","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a7b0ba93ef2a1fe0b17751a1facc729aff1d65d9","unresolved":true,"context_lines":[{"line_number":11,"context_line":"      - periodic-stable-jobs"},{"line_number":12,"context_line":"    check:"},{"line_number":13,"context_line":"      jobs:"},{"line_number":14,"context_line":"        # - kolla-ansible-centos8s-source"},{"line_number":15,"context_line":"        - kolla-ansible-debian-source"},{"line_number":16,"context_line":"        # - kolla-ansible-openeuler-source"},{"line_number":17,"context_line":"        # - kolla-ansible-rocky8-source"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"daa1063e_c84bd0dc","line":14,"range":{"start_line":14,"start_character":8,"end_line":14,"end_character":41},"in_reply_to":"e6ec7cd4_a18e579c","updated":"2022-06-10 09:34:41.000000000","message":"The more important change is excluding the broken openeuler and rocky8 jobs below, are you referring to them, too?\n\nBecause those are actually tied to this patch which makes them no longer pass, so I\u0027d prefer to keep them in here.\n\nAnd then this cosmetic change IMO doesn\u0027t really matter, if you really don\u0027t like it, we can just omit it, but it\u0027s not worth a separate patch.","commit_id":"913f20c2ca2984405a36dcb2fe31facb1c429c02"}]}