)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"a88e2a9e0ce02654b537a24d80dbddf72e2a02d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7309a339_8022e51f","updated":"2023-06-01 17:01:01.000000000","message":"Also Helm version is defined here https://opendev.org/openstack/openstack-helm-infra/src/branch/master/playbooks/lint.yml#L19 This playbook is used by the openstack-helm-lint job.","commit_id":"bf95238dc134d6e7eaa4c703cf78ae5841bb21d2"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"e885ad3dcb9f397566691835193ab5d8d625ccfe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b7944be1_a6b65ca5","updated":"2023-06-01 16:57:23.000000000","message":"As far as I know we have been using Helm v3 for quite a while and the Helm version is defined here https://opendev.org/openstack/openstack-helm-infra/src/branch/master/tools/gate/deploy-k8s.sh#L16 Also the very last line in this script builds all chart packages. And I can not see the place where we use this role build-helm-packages. I think we can get rid of this role.","commit_id":"bf95238dc134d6e7eaa4c703cf78ae5841bb21d2"},{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"9377f699f46fa73132138bb9708cb87a06e0a455","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3146d560_b03baff1","in_reply_to":"0bf6d7c6_5a1eb817","updated":"2023-06-01 18:17:27.000000000","message":"Yep. You are right. openstack-helm-lint job runs playbooks/lint.yml playbook which calls tox lint endpoint which runs lint.sh script which runs build-helm-packages role.","commit_id":"bf95238dc134d6e7eaa4c703cf78ae5841bb21d2"},{"author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"change_message_id":"863cc12137bbf35d92f3ce90dbda7bbfba3d867d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0bf6d7c6_5a1eb817","in_reply_to":"b7944be1_a6b65ca5","updated":"2023-06-01 17:03:19.000000000","message":"This is used by lint. https://review.opendev.org/c/openstack/openstack-helm-infra/+/884297 introduces Rook using copies of the official rook-helm charts, which are dependent on Helm v3. This change is required for the lint check to pass.","commit_id":"bf95238dc134d6e7eaa4c703cf78ae5841bb21d2"}]}