)]}'
{".zuul.yaml":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"311e6e0d9aa8ea8f8fb043f7470c5615852fc508","unresolved":true,"context_lines":[{"line_number":266,"context_line":"      This jobs configures Cinder with LVM, LIO, barbican and"},{"line_number":267,"context_line":"      runs tempest tests and cinderlib tests."},{"line_number":268,"context_line":"    parent: cinder-tempest-plugin-lvm-barbican-base"},{"line_number":269,"context_line":"    required-projects:"},{"line_number":270,"context_line":"      - name: opendev.org/openstack/cinder-tempest-plugin"},{"line_number":271,"context_line":"        override-checkout: 1.8.0"},{"line_number":272,"context_line":"    vars:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"323f1d61_d33e5de6","line":269,"updated":"2023-06-09 14:33:44.000000000","message":"you need to create a specialization of this job for the wallaby branch, basically another block like 151-174 but specialized for wallaby, and also exclude wallaby from the base definition of cinder-tempest-plugin-lvm-barbican-base on line 137","commit_id":"9d01eb173f85d5309ca5065218d8e44a7dda6a5c"},{"author":{"_account_id":35075,"name":"Alexander Deiter","email":"adeiter@infinidat.com","username":"adeiter"},"change_message_id":"0c38f201a350a2b7c45ab3877d37920b8d5fd77d","unresolved":false,"context_lines":[{"line_number":266,"context_line":"      This jobs configures Cinder with LVM, LIO, barbican and"},{"line_number":267,"context_line":"      runs tempest tests and cinderlib tests."},{"line_number":268,"context_line":"    parent: cinder-tempest-plugin-lvm-barbican-base"},{"line_number":269,"context_line":"    required-projects:"},{"line_number":270,"context_line":"      - name: opendev.org/openstack/cinder-tempest-plugin"},{"line_number":271,"context_line":"        override-checkout: 1.8.0"},{"line_number":272,"context_line":"    vars:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d7584c3f_271e463e","line":269,"in_reply_to":"323f1d61_d33e5de6","updated":"2023-06-09 14:52:55.000000000","message":"Hello Luigi,\n\nThank you very much for the review!\nFixed by patch set #2.\n\nThank you!","commit_id":"9d01eb173f85d5309ca5065218d8e44a7dda6a5c"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"61e1f44cd21628d5cae27c0ddf5a788a61031f9f","unresolved":true,"context_lines":[{"line_number":172,"context_line":"      fetch_subunit_output_additional_dirs:"},{"line_number":173,"context_line":"        - \"{{ ansible_user_dir }}/{{ zuul.projects[\u0027opendev.org/openstack/cinderlib\u0027].src_dir }}\""},{"line_number":174,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/tempest\"].src_dir }}/tools/tempest-integrated-gate-storage-exclude-list.txt\u0027"},{"line_number":175,"context_line":"- job:"},{"line_number":176,"context_line":"    name: cinder-tempest-plugin-lvm-barbican-base"},{"line_number":177,"context_line":"    description: |"},{"line_number":178,"context_line":"      This is a base job for lvm with lio \u0026 tgt targets"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"789d0d26_e43096ad","line":175,"updated":"2023-06-12 09:08:27.000000000","message":"nit: can you please leave a whitespace before the job here, just like it happens any two contiguous job definitions?","commit_id":"cfec6e591a9e67f294a8a08112381d33518711e0"},{"author":{"_account_id":35075,"name":"Alexander Deiter","email":"adeiter@infinidat.com","username":"adeiter"},"change_message_id":"60ab4602601a40cfb7047a8a843425ab9578ac5f","unresolved":false,"context_lines":[{"line_number":172,"context_line":"      fetch_subunit_output_additional_dirs:"},{"line_number":173,"context_line":"        - \"{{ ansible_user_dir }}/{{ zuul.projects[\u0027opendev.org/openstack/cinderlib\u0027].src_dir }}\""},{"line_number":174,"context_line":"      tempest_test_exclude_list: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/tempest\"].src_dir }}/tools/tempest-integrated-gate-storage-exclude-list.txt\u0027"},{"line_number":175,"context_line":"- job:"},{"line_number":176,"context_line":"    name: cinder-tempest-plugin-lvm-barbican-base"},{"line_number":177,"context_line":"    description: |"},{"line_number":178,"context_line":"      This is a base job for lvm with lio \u0026 tgt targets"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"c8bfd774_789e76a6","line":175,"in_reply_to":"789d0d26_e43096ad","updated":"2023-06-12 10:05:27.000000000","message":"Hello Luigi,\n\nThank you very much for the review!\nFixed in patch set #3.\n\nThank you!","commit_id":"cfec6e591a9e67f294a8a08112381d33518711e0"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":35075,"name":"Alexander Deiter","email":"adeiter@infinidat.com","username":"adeiter"},"change_message_id":"16f076de42e849e4672f56cbe25bc26b2acf8283","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"86c4f2b3_800b3a17","updated":"2023-06-09 19:16:25.000000000","message":"Please review the test results:\n\n885771: WIP: test for 871906 and 885770 | https://review.opendev.org/c/openstack/cinder/+/885771","commit_id":"cfec6e591a9e67f294a8a08112381d33518711e0"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"61bd899c0c1353c838689e39a6602006c3eefd1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ed495613_a043f0b2","updated":"2023-06-12 13:55:21.000000000","message":"LGTM!","commit_id":"9fd78d520dffc074b2f5e5327d1c6ecd1b9c17a8"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"801f0609e891c6cc7045d44b0a2d46ec6d9ad6ac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"09e040c2_b9058a6f","updated":"2023-06-12 11:23:06.000000000","message":"Thank you","commit_id":"9fd78d520dffc074b2f5e5327d1c6ecd1b9c17a8"}]}