)]}'
{".zuul.yaml":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"e44628fb63e9500165fb34a2586f1ec6afe52ede","unresolved":true,"context_lines":[{"line_number":55,"context_line":"      Runs tempest tests on os-brick source against ceph."},{"line_number":56,"context_line":"      This voting job runs on changes that could affect rbd."},{"line_number":57,"context_line":"    parent: os-brick-src-devstack-plugin-ceph"},{"line_number":58,"context_line":"    files:"},{"line_number":59,"context_line":"      - ^os_brick/initiator/connector.py$"},{"line_number":60,"context_line":"      - ^os_brick/initiator/connectors/rbd.py$"},{"line_number":61,"context_line":"      - ^os_brick/initiator/linuxrbd.py$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3a8de8b0_713a1044","line":58,"updated":"2022-07-14 14:07:32.000000000","message":"I think you can do\n    files: \u0026rbd_files\n    \nhere to create the \"rbd_files\" variable, and then in irrelevant_files below for the nv job, use\n\n    irrelevant_files: *rbd_files\n      - other/file/a\n      - other/file/b","commit_id":"9b855d979e44774f39410413c35bfdf3a54ddc44"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cadbf9e7a6319f39aba361183033fcedb40888ee","unresolved":true,"context_lines":[{"line_number":55,"context_line":"      Runs tempest tests on os-brick source against ceph."},{"line_number":56,"context_line":"      This voting job runs on changes that could affect rbd."},{"line_number":57,"context_line":"    parent: os-brick-src-devstack-plugin-ceph"},{"line_number":58,"context_line":"    files:"},{"line_number":59,"context_line":"      - ^os_brick/initiator/connector.py$"},{"line_number":60,"context_line":"      - ^os_brick/initiator/connectors/rbd.py$"},{"line_number":61,"context_line":"      - ^os_brick/initiator/linuxrbd.py$"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3ca1b9dd_bf974fcd","line":58,"in_reply_to":"3a8de8b0_713a1044","updated":"2022-07-14 15:59:23.000000000","message":"I thought about that, but it\u0027s illegal YAML (at least according to http://www.yamllint.com/ ).  And if you try\n\n    irrelevant_files: \n      - *rbd_files\n      - other/file/a\n      - other/file/b\n \nYAML sees that as a 3 item list, the first element of which is itself a list.  I don\u0027t know if zuul has implemented list flattening.  YAML errors get picked up immediately by zuul, so it will be quick to find out if this works.","commit_id":"9b855d979e44774f39410413c35bfdf3a54ddc44"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"36dafd864618ce2418f35096d0a7f285a6eadcb3","unresolved":true,"context_lines":[{"line_number":16,"context_line":"            voting: false"},{"line_number":17,"context_line":"    gate:"},{"line_number":18,"context_line":"      jobs:"},{"line_number":19,"context_line":"        - os-brick-src-tempest-lvm-lio-barbican"},{"line_number":20,"context_line":"    experimental:"},{"line_number":21,"context_line":"      jobs:"},{"line_number":22,"context_line":"        - openstack-tox-pylint"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"530f96bb_eb033918","line":19,"updated":"2022-07-27 12:24:05.000000000","message":"?: Should we have it as well for the gate jobs?  Or we can do it later.","commit_id":"c1ea65e80e18948c65805df4873ab61eaa083740"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"9ceba9f3c535018a02d9841713b537b37541d427","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"faf0fa33_8fa111b1","updated":"2022-07-14 16:01:00.000000000","message":"For the record, https://review.opendev.org/c/openstack/os-brick/+/849858/ and https://review.opendev.org/c/openstack/os-brick/+/849859/ show that this patch is working as expected.","commit_id":"9b855d979e44774f39410413c35bfdf3a54ddc44"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"4d08a83293c3eff937ed7088ce9857d4c3b808c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b40fef24_791a7312","updated":"2022-07-14 16:03:42.000000000","message":"I\u0027ll try the list that needs to be flattened, and let\u0027s see what happens.","commit_id":"e7a19331ba8359ca92a6a4da7e96cbaa6a839e74"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"76cd09f745866b5d8d2d58740422bb8b80bdb9cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a625f858_4a100b50","updated":"2022-07-14 16:10:12.000000000","message":"@Eric: I tried, but no dice.","commit_id":"c1ea65e80e18948c65805df4873ab61eaa083740"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"c7604cec80da6bbf625890d9685991d8206612f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"455b1f2f_6cd61468","updated":"2022-07-15 15:37:51.000000000","message":"Don\u0027t have a lot of idea about YAML syntax but generic logic looks good.","commit_id":"c1ea65e80e18948c65805df4873ab61eaa083740"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ef1b3f3924ed8590d37323bb46720967951b761f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0eb749be_ad6628f1","updated":"2022-07-27 12:33:23.000000000","message":"I think enough cores have taken a look at this and the job can be modified later if found any issues. Thanks Brian.","commit_id":"c1ea65e80e18948c65805df4873ab61eaa083740"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"36dafd864618ce2418f35096d0a7f285a6eadcb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e7e31f1a_94457e8c","updated":"2022-07-27 12:24:05.000000000","message":"LGTM, thanks for adding this","commit_id":"c1ea65e80e18948c65805df4873ab61eaa083740"}]}
