)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"8cdcb7846d34b2d1dadf10c75b02165ae9cb9ceb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f7306630_7c11fc68","updated":"2026-09-23 02:17:41.000000000","message":"@kozhukalov@gmail.com I think I caught a truncation issue when reviewing and had it tighten up some checks.","commit_id":"37b0ac5047f8a95cb1444379838a4a9da8b12300"}],"playbooks/charts-publish.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"cb1f19e343ea59ecc132f9d32f993635522bb2f3","unresolved":true,"context_lines":[{"line_number":108,"context_line":"            {{ (\u0027--merge \u0027 + published_index) if not chart_index_bootstrap else \u0027\u0027 }}"},{"line_number":109,"context_line":"            --url https://tarballs.opendev.org/{{ zuul.project.name }}"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        - name: Ensure artifact directory exists"},{"line_number":112,"context_line":"          file:"},{"line_number":113,"context_line":"            path: \"{{ zuul.executor.work_root }}/artifacts/\""},{"line_number":114,"context_line":"            state: directory"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"237a6ba0_b301d251","line":111,"updated":"2026-09-21 18:05:56.000000000","message":"this must be unconditional because not all PRs update charts and this block does not always run. The publish parent job does not ensure this directory [1].\n\n[1] https://opendev.org/openstack/project-config/src/branch/master/playbooks/publish/openstack-artifacts.yaml","commit_id":"3355d56c6778b03217bc7a8da5e07cb0db2dd68c"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"47d2f91eaf79249afa66a806631e5f6eae8412d9","unresolved":false,"context_lines":[{"line_number":108,"context_line":"            {{ (\u0027--merge \u0027 + published_index) if not chart_index_bootstrap else \u0027\u0027 }}"},{"line_number":109,"context_line":"            --url https://tarballs.opendev.org/{{ zuul.project.name }}"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        - name: Ensure artifact directory exists"},{"line_number":112,"context_line":"          file:"},{"line_number":113,"context_line":"            path: \"{{ zuul.executor.work_root }}/artifacts/\""},{"line_number":114,"context_line":"            state: directory"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"02e17e71_4d07401a","line":111,"in_reply_to":"237a6ba0_b301d251","updated":"2026-09-23 00:34:13.000000000","message":"Done","commit_id":"3355d56c6778b03217bc7a8da5e07cb0db2dd68c"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"cb1f19e343ea59ecc132f9d32f993635522bb2f3","unresolved":true,"context_lines":[{"line_number":105,"context_line":"        - name: Index the built charts"},{"line_number":106,"context_line":"          command: \u003e-"},{"line_number":107,"context_line":"            helm repo index {{ package_dir }}"},{"line_number":108,"context_line":"            {{ (\u0027--merge \u0027 + published_index) if not chart_index_bootstrap else \u0027\u0027 }}"},{"line_number":109,"context_line":"            --url https://tarballs.opendev.org/{{ zuul.project.name }}"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        - name: Ensure artifact directory exists"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7150c8d6_2600dee3","line":108,"updated":"2026-09-21 18:05:56.000000000","message":"Here when the flag is set we replace the index with just few charts even if the index already exists. This could be destructive. Wouldn\u0027t it be more safe to check here that the index exists (status_code\u003d\u003d200)? The flag chart_index_bootstrap could still be used for relaxing the failed_when condition in the index fetch task to not fail the playbook if the index does not exist.","commit_id":"9436ed4ea363ecd4624acbd374ee7e2e3f2a47a7"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"47d2f91eaf79249afa66a806631e5f6eae8412d9","unresolved":false,"context_lines":[{"line_number":105,"context_line":"        - name: Index the built charts"},{"line_number":106,"context_line":"          command: \u003e-"},{"line_number":107,"context_line":"            helm repo index {{ package_dir }}"},{"line_number":108,"context_line":"            {{ (\u0027--merge \u0027 + published_index) if not chart_index_bootstrap else \u0027\u0027 }}"},{"line_number":109,"context_line":"            --url https://tarballs.opendev.org/{{ zuul.project.name }}"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        - name: Ensure artifact directory exists"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d2c94106_9ce12911","line":108,"in_reply_to":"7150c8d6_2600dee3","updated":"2026-09-23 00:34:13.000000000","message":"Done","commit_id":"9436ed4ea363ecd4624acbd374ee7e2e3f2a47a7"}]}