)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"8805f6752bffa9e817dbca0104a050263259a105","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"305e69da_05a0a553","updated":"2022-08-17 12:42:30.000000000","message":"LGTM. Thanks!","commit_id":"6df21bf91c6e9f8a4b6be36faa8853b400db406f"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"0df493ee784ee6f05925a4192f2d0c5c32af9468","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"03cdaa43_7e581e63","updated":"2022-08-17 12:47:15.000000000","message":"Thank for providing the logs, seems to work as expected. Thanks","commit_id":"6df21bf91c6e9f8a4b6be36faa8853b400db406f"},{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"6e07d9845b77b2b16d2e84891e27dc333af36100","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"36c4b43b_321a6866","updated":"2022-08-17 11:23:50.000000000","message":"Thanks marios, since the patch is already testes, IMHO, its ready to merge","commit_id":"6df21bf91c6e9f8a4b6be36faa8853b400db406f"},{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"7f973e1a97e06f5fbb7a86f094887688b28b5b94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ee17e119_a8738d6d","updated":"2022-08-18 05:08:59.000000000","message":"recheck","commit_id":"6df21bf91c6e9f8a4b6be36faa8853b400db406f"},{"author":{"_account_id":32458,"name":"Ananya","display_name":"Ananya Banerjee","email":"anbanerj@redhat.com","username":"frenzyfriday"},"change_message_id":"73d62bac12cc056bc56912b9fd9965091f4f2c85","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a152d295_0b3d8614","updated":"2022-08-17 11:28:29.000000000","message":"thanks for the short context on the review call, the logs look good!","commit_id":"6df21bf91c6e9f8a4b6be36faa8853b400db406f"}],"playbooks/multinode-overcloud-mixed-os-deploy-compute.yml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a93f62b9d06c4cffd7654d66913d285d018dd719","unresolved":true,"context_lines":[{"line_number":9,"context_line":"      when: containerized_overcloud|bool"},{"line_number":10,"context_line":"  tasks:"},{"line_number":11,"context_line":"    - name: Resolve dlrn_hash for periodic jobs (no content provider)"},{"line_number":12,"context_line":"      when: \"\u0027periodic\u0027 in zuul.job\""},{"line_number":13,"context_line":"      block:"},{"line_number":14,"context_line":"        - include_role:"},{"line_number":15,"context_line":"            name: repo-setup"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e271778b_931287d0","line":12,"range":{"start_line":12,"start_character":14,"end_line":12,"end_character":35},"updated":"2022-08-16 10:17:50.000000000","message":"this is OK but I think better conditional is job.consumer_job","commit_id":"9434ce49852270112a07bd57d1c7466b28ea098a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"14d768d085f214d6493173144d64e714451ae92e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"      when: containerized_overcloud|bool"},{"line_number":10,"context_line":"  tasks:"},{"line_number":11,"context_line":"    - name: Resolve dlrn_hash for periodic jobs (no content provider)"},{"line_number":12,"context_line":"      when: \"\u0027periodic\u0027 in zuul.job\""},{"line_number":13,"context_line":"      block:"},{"line_number":14,"context_line":"        - include_role:"},{"line_number":15,"context_line":"            name: repo-setup"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"827fd22f_3b63d495","line":12,"range":{"start_line":12,"start_character":14,"end_line":12,"end_character":35},"in_reply_to":"e271778b_931287d0","updated":"2022-08-16 10:28:48.000000000","message":"Done \n\nincoming update for this","commit_id":"9434ce49852270112a07bd57d1c7466b28ea098a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a93f62b9d06c4cffd7654d66913d285d018dd719","unresolved":true,"context_lines":[{"line_number":24,"context_line":"          set_fact:"},{"line_number":25,"context_line":"            container_build_id: \"{{ dlrn_hash }}\""},{"line_number":26,"context_line":"            cacheable: true"},{"line_number":27,"context_line":"    - name: set fact for container_build_id with dlrn_hash_tag {{ dlrn_hash_tag }} from content provider"},{"line_number":28,"context_line":"      when: \"\u0027periodic\u0027 not in zuul.job\""},{"line_number":29,"context_line":"      set_fact:"},{"line_number":30,"context_line":"        container_build_id: \"{{ dlrn_hash_tag }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"69402563_83bd045e","line":27,"updated":"2022-08-16 10:17:50.000000000","message":"so this seems to work for both cases OK:\n\nUPSTREAM TEST https://review.opendev.org/c/openstack/tripleo-heat-templates/+/852990/2#message-a3a52169a216290500c6b6929bec75fdfed5b6fa\n\nIt skips the tripleo-get-hash resolution and uses the content provider hash:\n\n        * https://478169e15542245e1fb2-1337eb6e44995ec1645c493c10c5a993.ssl.cf1.rackcdn.com/852990/2/check/tripleo-ci-centos-8-9-multinode-mixed-os/7fb84da/job-output.txt\n\n\n        * 2022-08-15 14:45:36.102426 | primary | TASK [set fact for container_build_id with resolved dlrn_hash 4e57f3a91e3779fe10fe43cdf0f4f8a2] ***\n2022-08-15 14:45:36.102485 | primary | Monday 15 August 2022  14:45:36 +0000 (0:00:00.031)       0:00:07.734 *********\n2022-08-15 14:45:36.116529 | primary | skipping: [undercloud]\n2022-08-15 14:45:36.141977 | primary |\n2022-08-15 14:45:36.142011 | primary | TASK [set fact for container_build_id with dlrn_hash_tag 9d4e525c6b6865e8d45d418921abee36 from content provider] ***\n2022-08-15 14:45:36.142044 | primary | Monday 15 August 2022  14:45:36 +0000 (0:00:00.039)       0:00:07.774 *********\n2022-08-15 14:45:36.191602 | primary | ok: [undercloud]\n\n\n\nPERIODIC TEST https://review.rdoproject.org/r/c/testproject/+/44234/7#message-00db167308fc1087e5ff6959008a633d5c2c56c2\n\nit uses the resolved tripleo-ci-testing:\n\n\n\n\n        * https://logserver.rdoproject.org/34/44234/7/check/periodic-tripleo-ci-centos-8-9-multinode-mixed-os/ad9ba43/job-output.txt\n        * 2022-08-15 10:48:20.510430 | primary | TASK [set fact for container_build_id with resolved dlrn_hash db222800051ca179d7d3195b6e732c93] ***\n2022-08-15 10:48:20.510443 | primary | Monday 15 August 2022  14:48:20 +0000 (0:00:00.049)       0:00:08.634 *********\n2022-08-15 10:48:20.558684 | primary | ok: [undercloud]\n2022-08-15 10:48:20.573558 | primary |\n2022-08-15 10:48:20.573597 | primary | TASK [set fact for container_build_id with dlrn_hash_tag tripleo-ci-testing from content provider] ***\n2022-08-15 10:48:20.573614 | primary | Monday 15 August 2022  14:48:20 +0000 (0:00:00.063)       0:00:08.697 *********\n2022-08-15 10:48:20.590000 | primary | skipping: [undercloud]","commit_id":"9434ce49852270112a07bd57d1c7466b28ea098a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"8d97aab801636c2cf581c4dddb3034303ee86ce4","unresolved":false,"context_lines":[{"line_number":24,"context_line":"          set_fact:"},{"line_number":25,"context_line":"            container_build_id: \"{{ dlrn_hash }}\""},{"line_number":26,"context_line":"            cacheable: true"},{"line_number":27,"context_line":"    - name: set fact for container_build_id with dlrn_hash_tag {{ dlrn_hash_tag }} from content provider"},{"line_number":28,"context_line":"      when: \"\u0027periodic\u0027 not in zuul.job\""},{"line_number":29,"context_line":"      set_fact:"},{"line_number":30,"context_line":"        container_build_id: \"{{ dlrn_hash_tag }}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bbb96703_2d630554","line":27,"in_reply_to":"69402563_83bd045e","updated":"2022-08-17 09:07:52.000000000","message":"Done","commit_id":"9434ce49852270112a07bd57d1c7466b28ea098a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"9c8832adf5bcaf36f532a8111b2b1fb69fd9bb96","unresolved":false,"context_lines":[{"line_number":25,"context_line":"            container_build_id: \"{{ dlrn_hash }}\""},{"line_number":26,"context_line":"            cacheable: true"},{"line_number":27,"context_line":"    - name: set fact for container_build_id with dlrn_hash_tag {{ dlrn_hash_tag }} from content provider"},{"line_number":28,"context_line":"      when: job.consumer_job | default(false) | bool"},{"line_number":29,"context_line":"      set_fact:"},{"line_number":30,"context_line":"        container_build_id: \"{{ dlrn_hash_tag }}\""},{"line_number":31,"context_line":"        cacheable: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"27c34ca0_f3285649","line":28,"updated":"2022-08-17 09:07:41.000000000","message":"k new conditionals looks OK in tests:\n\n\nPERIODIC:\n        \n        * https://logserver.rdoproject.org/34/44234/7/check/periodic-tripleo-ci-centos-8-9-multinode-mixed-os/1aeaabf/job-output.txt\n        * 2022-08-16 08:01:23.954189 | primary | TASK [set fact for container_build_id with resolved dlrn_hash db222800051ca179d7d3195b6e732c93] ***\n2022-08-16 08:01:23.954207 | primary | Tuesday 16 August 2022  12:01:23 +0000 (0:00:00.068)       0:00:08.760 ********\n2022-08-16 08:01:24.009531 | primary | ok: [undercloud]\n2022-08-16 08:01:24.026756 | primary |\n2022-08-16 08:01:24.026831 | primary | TASK [set fact for container_build_id with dlrn_hash_tag tripleo-ci-testing from content provider] ***\n2022-08-16 08:01:24.026846 | primary | Tuesday 16 August 2022  12:01:24 +0000 (0:00:00.072)       0:00:08.833 ********\n2022-08-16 08:01:24.060814 | primary | skipping: [undercloud]\n\nUPSTREAM:\n\n        * https://7dfec688604df0c6a301-3804bb164415e9aa6e4f241348a0a888.ssl.cf5.rackcdn.com/852990/2/check/tripleo-ci-centos-8-9-multinode-mixed-os/bc0b2ad/job-output.txt\n        * 2022-08-16 12:49:40.619045 | primary | TASK [set fact for container_build_id with resolved dlrn_hash 4e57f3a91e3779fe10fe43cdf0f4f8a2] ***\n2022-08-16 12:49:40.619091 | primary | Tuesday 16 August 2022  12:49:40 +0000 (0:00:00.046)       0:00:07.934 ********\n2022-08-16 12:49:40.647006 | primary | skipping: [undercloud]\n2022-08-16 12:49:40.675092 | primary |\n2022-08-16 12:49:40.675137 | primary | TASK [set fact for container_build_id with dlrn_hash_tag db222800051ca179d7d3195b6e732c93 from content provider] ***\n2022-08-16 12:49:40.675167 | primary | Tuesday 16 August 2022  12:49:40 +0000 (0:00:00.056)       0:00:07.990 ********\n2022-08-16 12:49:40.737280 | primary | ok: [undercloud]\n2022-08-16 12:49:40.751578 | primary |","commit_id":"6df21bf91c6e9f8a4b6be36faa8853b400db406f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"df8608dda227c3781a2c4f067a4fe46940a0c85b","unresolved":false,"context_lines":[{"line_number":25,"context_line":"            container_build_id: \"{{ dlrn_hash }}\""},{"line_number":26,"context_line":"            cacheable: true"},{"line_number":27,"context_line":"    - name: set fact for container_build_id with dlrn_hash_tag {{ dlrn_hash_tag }} from content provider"},{"line_number":28,"context_line":"      when: job.consumer_job | default(false) | bool"},{"line_number":29,"context_line":"      set_fact:"},{"line_number":30,"context_line":"        container_build_id: \"{{ dlrn_hash_tag }}\""},{"line_number":31,"context_line":"        cacheable: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"154dbb89_427b67a4","line":28,"in_reply_to":"27c34ca0_f3285649","updated":"2022-08-17 10:30:00.000000000","message":"Thank you for adding the logs. For periodic, it is resolving to tripleo-ci-testing and skipped for upstream.","commit_id":"6df21bf91c6e9f8a4b6be36faa8853b400db406f"}]}