)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"20d997d9ac9c037987c5841286ffddd40db2e570","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9d46c408_857ab23d","updated":"2025-12-16 19:14:22.000000000","message":"I\u0027m giving this one a +2 since @kozhukalov@gmail.com re-implemented my original version and actually tested it.","commit_id":"dc1de268aafba335025abfb6806883cc64dbb5b1"}],"playbooks/build-chart.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"1ef3119e13fa76fcf7eba23f6bb43759a8cd4afe","unresolved":true,"context_lines":[{"line_number":14,"context_line":"- hosts: all"},{"line_number":15,"context_line":"  roles:"},{"line_number":16,"context_line":"    - ensure-helm"},{"line_number":17,"context_line":"    - ensure-chart-testing"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"  tasks:"},{"line_number":20,"context_line":"    - name: Get list of changed charts"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5b8da0b8_02bd6064","line":17,"updated":"2025-12-15 21:44:52.000000000","message":"This role is going to fail on Noble because it tries to install packages using pip system wide. We probably have to copy this role to the OSH repo and modify it to fit our case.","commit_id":"7cdaff0504bd94d3d93de06e79b53889c06b5540"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"1ef3119e13fa76fcf7eba23f6bb43759a8cd4afe","unresolved":true,"context_lines":[{"line_number":33,"context_line":"        msg: \"Changed charts: {{ changed_charts }}\""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    - name: Build each changed chart"},{"line_number":36,"context_line":"      include_role:"},{"line_number":37,"context_line":"        name: build-helm-chart"},{"line_number":38,"context_line":"      vars:"},{"line_number":39,"context_line":"        chart_name: \"{{ item }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"057e28eb_7410c5b5","line":36,"updated":"2025-12-15 21:44:52.000000000","message":"Why do we need a new role instead of just running \n```\nmake \u003cchart1\u003e \u003cchart2\u003e ...\n```\nAlso I see that the include_role ensure-python-command is removed and the role build-helm-chart skips building CHANGELOG.md","commit_id":"7cdaff0504bd94d3d93de06e79b53889c06b5540"}],"roles/build-helm-chart/tasks/main.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"1ef3119e13fa76fcf7eba23f6bb43759a8cd4afe","unresolved":true,"context_lines":[{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"---"},{"line_number":14,"context_line":"- name: Generate chart version using chart_version.sh"},{"line_number":15,"context_line":"  shell: \"./tools/chart_version.sh {{ chart_name }} {{ base_version }}\""},{"line_number":16,"context_line":"  args:"},{"line_number":17,"context_line":"    chdir: \"{{ chart_base_dir }}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f37fd66a_29206221","line":14,"updated":"2025-12-15 21:44:52.000000000","message":"As far as I can see the build-helm-chart role does not build CHANGELOG.md.","commit_id":"7cdaff0504bd94d3d93de06e79b53889c06b5540"}]}