)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"570bf1b09471f8cfc9b8529a61a8a071d54b0ce0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"080c61c0_43a4cb99","updated":"2022-06-09 21:13:40.000000000","message":"I have updated with subcharts/depedencies chart paths along with OSH-infra directory\n\n","commit_id":"eb304eb9444e37a8120442c977f50c342865d202"},{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"2f10a39bfb6ecc56853034d8a7861ef963ac868f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"405e4e2b_5da7e6dd","in_reply_to":"080c61c0_43a4cb99","updated":"2022-06-10 02:57:21.000000000","message":"Done","commit_id":"eb304eb9444e37a8120442c977f50c342865d202"},{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"68fe92f79717228cb07c3bef4bc10815d645cbd2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"98973b3f_acd2a10a","updated":"2022-06-13 15:46:09.000000000","message":"updated pattern as per comments","commit_id":"aa3cecfaf926b2b94ac1317fe556b581717477d0"}],"zuul.d/jobs-openstack-helm.yaml":[{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"9db7a622fb965d52ead31a94987ab7228ca99114","unresolved":true,"context_lines":[{"line_number":161,"context_line":"    name: openstack-helm-compute-kit-umbrella"},{"line_number":162,"context_line":"    parent: openstack-helm-chart-deploy"},{"line_number":163,"context_line":"    files:"},{"line_number":164,"context_line":"      - ^tools/deployment/component/common/."},{"line_number":165,"context_line":"      - ^tools/deployment/developer/common/."},{"line_number":166,"context_line":"      - ^tools/deployment/component/."},{"line_number":167,"context_line":"      - ^tools/gate/."},{"line_number":168,"context_line":"      - ^tools/deployment/common/."},{"line_number":169,"context_line":"    vars:"},{"line_number":170,"context_line":"      run_helm_tests: \"yes\""},{"line_number":171,"context_line":"      gate_scripts_relative_path: ../openstack-helm"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9aba8581_ce57af58","line":168,"range":{"start_line":164,"start_character":0,"end_line":168,"end_character":34},"updated":"2022-06-09 20:36:56.000000000","message":"We want the Helm chart patterns such as neutron, nova, etc. and openstack.\n\nFull list will be the paths in here: https://review.opendev.org/c/openstack/openstack-helm/+/837132/48/openstack/Chart.yaml\n\nand openstack directory too.","commit_id":"44aa98289448d26c3f28a18df4fc1d28e3b5b8b0"},{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"2f10a39bfb6ecc56853034d8a7861ef963ac868f","unresolved":false,"context_lines":[{"line_number":161,"context_line":"    name: openstack-helm-compute-kit-umbrella"},{"line_number":162,"context_line":"    parent: openstack-helm-chart-deploy"},{"line_number":163,"context_line":"    files:"},{"line_number":164,"context_line":"      - ^tools/deployment/component/common/."},{"line_number":165,"context_line":"      - ^tools/deployment/developer/common/."},{"line_number":166,"context_line":"      - ^tools/deployment/component/."},{"line_number":167,"context_line":"      - ^tools/gate/."},{"line_number":168,"context_line":"      - ^tools/deployment/common/."},{"line_number":169,"context_line":"    vars:"},{"line_number":170,"context_line":"      run_helm_tests: \"yes\""},{"line_number":171,"context_line":"      gate_scripts_relative_path: ../openstack-helm"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a93a7663_05e395e2","line":168,"range":{"start_line":164,"start_character":0,"end_line":168,"end_character":34},"in_reply_to":"9aba8581_ce57af58","updated":"2022-06-10 02:57:21.000000000","message":"Ack","commit_id":"44aa98289448d26c3f28a18df4fc1d28e3b5b8b0"},{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"1e34b552a169419434163157fbb2e989378ebb79","unresolved":true,"context_lines":[{"line_number":169,"context_line":"      - ^keystone/.*$"},{"line_number":170,"context_line":"      - ^heat/.*$"},{"line_number":171,"context_line":"      - ^glance/.*$"},{"line_number":172,"context_line":"      - ^nova/.*$"},{"line_number":173,"context_line":"      - ^placement/.*$"},{"line_number":174,"context_line":"      - ^neutron/.*$"},{"line_number":175,"context_line":"      - ^../openstack-helm-infra/libvirt/.*$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"457710e5_3d9b7f08","line":172,"updated":"2022-06-09 21:45:38.000000000","message":"please add one for openstack as well please","commit_id":"eb304eb9444e37a8120442c977f50c342865d202"},{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"2f10a39bfb6ecc56853034d8a7861ef963ac868f","unresolved":false,"context_lines":[{"line_number":169,"context_line":"      - ^keystone/.*$"},{"line_number":170,"context_line":"      - ^heat/.*$"},{"line_number":171,"context_line":"      - ^glance/.*$"},{"line_number":172,"context_line":"      - ^nova/.*$"},{"line_number":173,"context_line":"      - ^placement/.*$"},{"line_number":174,"context_line":"      - ^neutron/.*$"},{"line_number":175,"context_line":"      - ^../openstack-helm-infra/libvirt/.*$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ea5585fb_fe1983ff","line":172,"in_reply_to":"457710e5_3d9b7f08","updated":"2022-06-10 02:57:21.000000000","message":"Ack","commit_id":"eb304eb9444e37a8120442c977f50c342865d202"},{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"1e34b552a169419434163157fbb2e989378ebb79","unresolved":true,"context_lines":[{"line_number":172,"context_line":"      - ^nova/.*$"},{"line_number":173,"context_line":"      - ^placement/.*$"},{"line_number":174,"context_line":"      - ^neutron/.*$"},{"line_number":175,"context_line":"      - ^../openstack-helm-infra/libvirt/.*$"},{"line_number":176,"context_line":"      - ^../openstack-helm-infra/helm-toolkit/.*$"},{"line_number":177,"context_line":"      - ^../openstack-helm-infra/mariadb/.*$"},{"line_number":178,"context_line":"      - ^../openstack-helm-infra/rabbitmq/.*$"},{"line_number":179,"context_line":"      - ^../openstack-helm-infra/memcached/.*$"},{"line_number":180,"context_line":"      - ^../openstack-helm-infra/openvswitch/.*$"},{"line_number":181,"context_line":"    vars:"},{"line_number":182,"context_line":"      run_helm_tests: \"yes\""},{"line_number":183,"context_line":"      gate_scripts_relative_path: ../openstack-helm"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"a0207aab_6a8ce210","line":180,"range":{"start_line":175,"start_character":0,"end_line":180,"end_character":48},"updated":"2022-06-09 21:45:38.000000000","message":"You can remove openstack-helm-infra related directories.\n\nUnfortunately changes to openstack-helm-infra do not trigger changes in openstack-helm, so we can remove these.","commit_id":"eb304eb9444e37a8120442c977f50c342865d202"},{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"2f10a39bfb6ecc56853034d8a7861ef963ac868f","unresolved":false,"context_lines":[{"line_number":172,"context_line":"      - ^nova/.*$"},{"line_number":173,"context_line":"      - ^placement/.*$"},{"line_number":174,"context_line":"      - ^neutron/.*$"},{"line_number":175,"context_line":"      - ^../openstack-helm-infra/libvirt/.*$"},{"line_number":176,"context_line":"      - ^../openstack-helm-infra/helm-toolkit/.*$"},{"line_number":177,"context_line":"      - ^../openstack-helm-infra/mariadb/.*$"},{"line_number":178,"context_line":"      - ^../openstack-helm-infra/rabbitmq/.*$"},{"line_number":179,"context_line":"      - ^../openstack-helm-infra/memcached/.*$"},{"line_number":180,"context_line":"      - ^../openstack-helm-infra/openvswitch/.*$"},{"line_number":181,"context_line":"    vars:"},{"line_number":182,"context_line":"      run_helm_tests: \"yes\""},{"line_number":183,"context_line":"      gate_scripts_relative_path: ../openstack-helm"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"a54557a9_dd4b5075","line":180,"range":{"start_line":175,"start_character":0,"end_line":180,"end_character":48},"in_reply_to":"a0207aab_6a8ce210","updated":"2022-06-10 02:57:21.000000000","message":"changes have been done as per suggestion","commit_id":"eb304eb9444e37a8120442c977f50c342865d202"},{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"4a2855b8fd4363674a3e8c49f712c0fcd2490b23","unresolved":true,"context_lines":[{"line_number":161,"context_line":"    name: openstack-helm-compute-kit-umbrella"},{"line_number":162,"context_line":"    parent: openstack-helm-chart-deploy"},{"line_number":163,"context_line":"    files:"},{"line_number":164,"context_line":"      - ^tools/deployment/component/common/.*$"},{"line_number":165,"context_line":"      - ^tools/deployment/developer/common/.*$"},{"line_number":166,"context_line":"      - ^tools/deployment/component/.*$"},{"line_number":167,"context_line":"      - ^tools/gate/.*$"},{"line_number":168,"context_line":"      - ^tools/deployment/common/.*$"},{"line_number":169,"context_line":"      - ^keystone/.*$"},{"line_number":170,"context_line":"      - ^heat/.*$"},{"line_number":171,"context_line":"      - ^glance/.*$"},{"line_number":172,"context_line":"      - ^nova/.*$"},{"line_number":173,"context_line":"      - ^placement/.*$"},{"line_number":174,"context_line":"      - ^neutron/.*$"},{"line_number":175,"context_line":"      - ^openstack/.*$"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"    vars:"},{"line_number":178,"context_line":"      run_helm_tests: \"yes\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1326eef3_f3d67e4b","line":175,"range":{"start_line":164,"start_character":0,"end_line":175,"end_character":22},"updated":"2022-06-10 12:40:47.000000000","message":"nitpick: it would be nice if these were alphabetized","commit_id":"38b07cafe512f42bdd04371a71d7b5846d15b54e"},{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"d9f8469b133eca62f05f242c634ca5e2eaaae342","unresolved":false,"context_lines":[{"line_number":161,"context_line":"    name: openstack-helm-compute-kit-umbrella"},{"line_number":162,"context_line":"    parent: openstack-helm-chart-deploy"},{"line_number":163,"context_line":"    files:"},{"line_number":164,"context_line":"      - ^tools/deployment/component/common/.*$"},{"line_number":165,"context_line":"      - ^tools/deployment/developer/common/.*$"},{"line_number":166,"context_line":"      - ^tools/deployment/component/.*$"},{"line_number":167,"context_line":"      - ^tools/gate/.*$"},{"line_number":168,"context_line":"      - ^tools/deployment/common/.*$"},{"line_number":169,"context_line":"      - ^keystone/.*$"},{"line_number":170,"context_line":"      - ^heat/.*$"},{"line_number":171,"context_line":"      - ^glance/.*$"},{"line_number":172,"context_line":"      - ^nova/.*$"},{"line_number":173,"context_line":"      - ^placement/.*$"},{"line_number":174,"context_line":"      - ^neutron/.*$"},{"line_number":175,"context_line":"      - ^openstack/.*$"},{"line_number":176,"context_line":""},{"line_number":177,"context_line":"    vars:"},{"line_number":178,"context_line":"      run_helm_tests: \"yes\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1d0c95e5_dc6be14a","line":175,"range":{"start_line":164,"start_character":0,"end_line":175,"end_character":22},"in_reply_to":"1326eef3_f3d67e4b","updated":"2022-06-10 16:58:27.000000000","message":"updated","commit_id":"38b07cafe512f42bdd04371a71d7b5846d15b54e"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"04b9d72ff6ad50e00ce870d7107408ee16ce5deb","unresolved":true,"context_lines":[{"line_number":167,"context_line":"      - ^neutron/.*$"},{"line_number":168,"context_line":"      - ^nova/.*$"},{"line_number":169,"context_line":"      - ^openstack/.*$"},{"line_number":170,"context_line":"      - ^placement/.*$"},{"line_number":171,"context_line":"      - ^tools/deployment/component/common/.*$"},{"line_number":172,"context_line":"      - ^tools/deployment/developer/common/.*$"},{"line_number":173,"context_line":"      - ^tools/deployment/component/.*$"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"516f21d3_593b823b","line":170,"updated":"2022-06-10 21:17:10.000000000","message":"Should also put the zuul.d dir here too as is done in some of the other component jobs:\n\n^zuul\\.d/.*$","commit_id":"8da1995b3db53b336f89f55cb1f69bf661ce8857"},{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"68fe92f79717228cb07c3bef4bc10815d645cbd2","unresolved":false,"context_lines":[{"line_number":167,"context_line":"      - ^neutron/.*$"},{"line_number":168,"context_line":"      - ^nova/.*$"},{"line_number":169,"context_line":"      - ^openstack/.*$"},{"line_number":170,"context_line":"      - ^placement/.*$"},{"line_number":171,"context_line":"      - ^tools/deployment/component/common/.*$"},{"line_number":172,"context_line":"      - ^tools/deployment/developer/common/.*$"},{"line_number":173,"context_line":"      - ^tools/deployment/component/.*$"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"e10b9bf1_6c2e116f","line":170,"in_reply_to":"45e28cfa_3ed8474d","updated":"2022-06-13 15:46:09.000000000","message":"updated","commit_id":"8da1995b3db53b336f89f55cb1f69bf661ce8857"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"fc34b521ef54346a6dac7b00b2dfbc610ac8ac1f","unresolved":true,"context_lines":[{"line_number":167,"context_line":"      - ^neutron/.*$"},{"line_number":168,"context_line":"      - ^nova/.*$"},{"line_number":169,"context_line":"      - ^openstack/.*$"},{"line_number":170,"context_line":"      - ^placement/.*$"},{"line_number":171,"context_line":"      - ^tools/deployment/component/common/.*$"},{"line_number":172,"context_line":"      - ^tools/deployment/developer/common/.*$"},{"line_number":173,"context_line":"      - ^tools/deployment/component/.*$"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"45e28cfa_3ed8474d","line":170,"in_reply_to":"516f21d3_593b823b","updated":"2022-06-12 23:36:22.000000000","message":"Agreed","commit_id":"8da1995b3db53b336f89f55cb1f69bf661ce8857"},{"author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"change_message_id":"fc34b521ef54346a6dac7b00b2dfbc610ac8ac1f","unresolved":true,"context_lines":[{"line_number":168,"context_line":"      - ^nova/.*$"},{"line_number":169,"context_line":"      - ^openstack/.*$"},{"line_number":170,"context_line":"      - ^placement/.*$"},{"line_number":171,"context_line":"      - ^tools/deployment/component/common/.*$"},{"line_number":172,"context_line":"      - ^tools/deployment/developer/common/.*$"},{"line_number":173,"context_line":"      - ^tools/deployment/component/.*$"},{"line_number":174,"context_line":"      - ^tools/gate/.*$"},{"line_number":175,"context_line":"      - ^tools/deployment/common/.*$"},{"line_number":176,"context_line":"    vars:"},{"line_number":177,"context_line":"      run_helm_tests: \"yes\""},{"line_number":178,"context_line":"      gate_scripts_relative_path: ../openstack-helm"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"2a5fcf93_8e33bea7","line":175,"range":{"start_line":171,"start_character":0,"end_line":175,"end_character":36},"updated":"2022-06-12 23:36:22.000000000","message":"We shouldn\u0027t need all these, targeting the ./tools/deployment/component/common/openstack.sh script should be enough. Or expanding out to all the scripts defined below.","commit_id":"8da1995b3db53b336f89f55cb1f69bf661ce8857"},{"author":{"_account_id":34794,"name":"Syed Ahmed Hussain","email":"v-ahmesyed@microsoft.com","username":"ahmesyed"},"change_message_id":"68fe92f79717228cb07c3bef4bc10815d645cbd2","unresolved":false,"context_lines":[{"line_number":168,"context_line":"      - ^nova/.*$"},{"line_number":169,"context_line":"      - ^openstack/.*$"},{"line_number":170,"context_line":"      - ^placement/.*$"},{"line_number":171,"context_line":"      - ^tools/deployment/component/common/.*$"},{"line_number":172,"context_line":"      - ^tools/deployment/developer/common/.*$"},{"line_number":173,"context_line":"      - ^tools/deployment/component/.*$"},{"line_number":174,"context_line":"      - ^tools/gate/.*$"},{"line_number":175,"context_line":"      - ^tools/deployment/common/.*$"},{"line_number":176,"context_line":"    vars:"},{"line_number":177,"context_line":"      run_helm_tests: \"yes\""},{"line_number":178,"context_line":"      gate_scripts_relative_path: ../openstack-helm"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"23c669b3_37cded11","line":175,"range":{"start_line":171,"start_character":0,"end_line":175,"end_character":36},"in_reply_to":"2a5fcf93_8e33bea7","updated":"2022-06-13 15:46:09.000000000","message":"Agreed! will update as menitoned","commit_id":"8da1995b3db53b336f89f55cb1f69bf661ce8857"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5e881ef6ef6f10f510c020cadc2fa284184bf026","unresolved":true,"context_lines":[{"line_number":175,"context_line":"      - ^tools/deployment/component/common/openstack.sh$"},{"line_number":176,"context_line":"      - ^tools/deployment/developer/common/170-setup-gateway.sh$"},{"line_number":177,"context_line":"      - ^tools/deployment/developer/common/900-use-it.sh$"},{"line_number":178,"context_line":"      - ^tools/deployment/common/force-cronjob-run.sh$"},{"line_number":179,"context_line":"      - ^zuul\\.d/.*$"},{"line_number":180,"context_line":"    vars:"},{"line_number":181,"context_line":"      run_helm_tests: \"yes\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"6050b35d_3ad6dc4d","line":178,"updated":"2022-06-13 18:08:58.000000000","message":"nit: these tools* entries aren\u0027t alphabetical any more as Dustin noticed in an earlier PS, but just a nit","commit_id":"aa3cecfaf926b2b94ac1317fe556b581717477d0"},{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"87aeb51771440575fe74b0d64d4d3eaadcb2a302","unresolved":true,"context_lines":[{"line_number":175,"context_line":"      - ^tools/deployment/component/common/openstack.sh$"},{"line_number":176,"context_line":"      - ^tools/deployment/developer/common/170-setup-gateway.sh$"},{"line_number":177,"context_line":"      - ^tools/deployment/developer/common/900-use-it.sh$"},{"line_number":178,"context_line":"      - ^tools/deployment/common/force-cronjob-run.sh$"},{"line_number":179,"context_line":"      - ^zuul\\.d/.*$"},{"line_number":180,"context_line":"    vars:"},{"line_number":181,"context_line":"      run_helm_tests: \"yes\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"e17b8a34_cd26587e","line":178,"in_reply_to":"6050b35d_3ad6dc4d","updated":"2022-06-22 20:44:13.000000000","message":"I\u0027m cool with the tools file being the order they are used.","commit_id":"aa3cecfaf926b2b94ac1317fe556b581717477d0"},{"author":{"_account_id":21991,"name":"Kostyantyn Kalynovskyi","email":"kkalinovskiy@gmail.com","username":"kkalynovskyi"},"change_message_id":"630951fb4dc571d57cc5c8fb71d9761c0e0f9229","unresolved":false,"context_lines":[{"line_number":175,"context_line":"      - ^tools/deployment/component/common/openstack.sh$"},{"line_number":176,"context_line":"      - ^tools/deployment/developer/common/170-setup-gateway.sh$"},{"line_number":177,"context_line":"      - ^tools/deployment/developer/common/900-use-it.sh$"},{"line_number":178,"context_line":"      - ^tools/deployment/common/force-cronjob-run.sh$"},{"line_number":179,"context_line":"      - ^zuul\\.d/.*$"},{"line_number":180,"context_line":"    vars:"},{"line_number":181,"context_line":"      run_helm_tests: \"yes\""}],"source_content_type":"text/x-yaml","patch_set":6,"id":"8c8d2f76_eb2a0ae5","line":178,"in_reply_to":"e17b8a34_cd26587e","updated":"2022-06-24 23:05:09.000000000","message":"Done","commit_id":"aa3cecfaf926b2b94ac1317fe556b581717477d0"}]}