)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"4fbdac14498eccce7e81618d0cf752a818a6f652","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"04ea1aca_db575e0b","updated":"2022-12-19 11:09:18.000000000","message":"Thank you marios for adding the logs, can we also sync it in collect-logs also as bogdan suggested?","commit_id":"818212fdbdcb7f739b4e2297f0e63443d6694cbb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"57de43591506e4062135668592efae654786b501","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fcf27d64_c697697f","in_reply_to":"04ea1aca_db575e0b","updated":"2022-12-20 15:24:45.000000000","message":"ack","commit_id":"818212fdbdcb7f739b4e2297f0e63443d6694cbb"}],"toci-quickstart/config/collect-logs.yml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"e8de9a589904b0963db3a083a5495444cef78105","unresolved":true,"context_lines":[{"line_number":296,"context_line":"  - /var/log/swift/"},{"line_number":297,"context_line":"  - /var/log/tacker/"},{"line_number":298,"context_line":"  - /var/log/tempest/"},{"line_number":299,"context_line":"  - /var/log/tripleo-container-image-prepare.*.log"},{"line_number":300,"context_line":"  - /var/log/trove/"},{"line_number":301,"context_line":"  - /var/log/vitrage/"},{"line_number":302,"context_line":"  - /var/log/validations/"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"2ec44a1f_b6ce347b","line":299,"updated":"2022-12-16 15:29:53.000000000","message":"we prolly need to mirror this to the https://opendev.org/openstack/ansible-role-collect-logs repo as well","commit_id":"818212fdbdcb7f739b4e2297f0e63443d6694cbb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e2b45d4c56c2fee4b9acec2059da17814990b23f","unresolved":true,"context_lines":[{"line_number":296,"context_line":"  - /var/log/swift/"},{"line_number":297,"context_line":"  - /var/log/tacker/"},{"line_number":298,"context_line":"  - /var/log/tempest/"},{"line_number":299,"context_line":"  - /var/log/tripleo-container-image-prepare.*.log"},{"line_number":300,"context_line":"  - /var/log/trove/"},{"line_number":301,"context_line":"  - /var/log/vitrage/"},{"line_number":302,"context_line":"  - /var/log/validations/"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"79ab9b78_d68964ec","line":299,"updated":"2022-12-16 15:29:58.000000000","message":"works for us - see result @ https://sf.hosted.upshift.rdu2.redhat.com/logs/04/437804/3/check/periodic-tripleo-rhel-9-8-rhos-17.1-mixed-os-compute-internal/5483684/logs/undercloud/var/log/\n\nwe have all three now:\n\n \ttripleo-container-image-prepare-ansible.log\t\n \ttripleo-container-image-prepare-compute.log\t\n \ttripleo-container-image-prepare.log","commit_id":"818212fdbdcb7f739b4e2297f0e63443d6694cbb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"57de43591506e4062135668592efae654786b501","unresolved":false,"context_lines":[{"line_number":296,"context_line":"  - /var/log/swift/"},{"line_number":297,"context_line":"  - /var/log/tacker/"},{"line_number":298,"context_line":"  - /var/log/tempest/"},{"line_number":299,"context_line":"  - /var/log/tripleo-container-image-prepare.*.log"},{"line_number":300,"context_line":"  - /var/log/trove/"},{"line_number":301,"context_line":"  - /var/log/vitrage/"},{"line_number":302,"context_line":"  - /var/log/validations/"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"0efa3c8c_d637b2a5","line":299,"in_reply_to":"2ec44a1f_b6ce347b","updated":"2022-12-20 15:24:45.000000000","message":"The idea is we have things in [1] which are more generally applicable and then we override the artcl_collect_list with the tripleo specific things here. \n\nI think this new tripleo-container-image-prepare-compute.log being collected here is really tripleo job specific. Having said that however, i noticed we *do* have  \n\n - /var/log/tripleo-container-image-prepare.log\n \n \nthere already [2] so i guess it is OK to update that... posted @ https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/868219 \n\n\n[1] https://opendev.org/openstack/ansible-role-collect-logs/src/commit/b44c7cc7b418b4aa4dad05439e8fdda6823b78cc/roles/collect_logs/defaults/main.yml#L14\n[2] https://opendev.org/openstack/ansible-role-collect-logs/src/commit/b44c7cc7b418b4aa4dad05439e8fdda6823b78cc/roles/collect_logs/defaults/main.yml#L83","commit_id":"818212fdbdcb7f739b4e2297f0e63443d6694cbb"}]}
