)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3a018c1e597c6a4af09d58e6746160d95ab62fe5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4ac4bf6c_d39d30c5","updated":"2021-11-26 13:01:32.000000000","message":"14:59 \u003c ysandeep\u003e marios: where are we testing \nhttps://review.opendev.org/c/openstack/tripleo-quickstart/+/819100/5/roles/repo-setup/tasks/tripleo-get-hash-set-fact-no-collections.yml , Is it https://review.opendev.org/c/openstack/tripleo-ci/+/818222 \n14:59 \u003c ysandeep\u003e i couldn\u0027t find that playbook here: \nhttps://2ea249875dd23cd8a853-c94b7832c8857c8bbfdc284b48fd4077.ssl.cf1.rackcdn.com/818222/36/check/tripleo-buildimage-overcloud-full-centos-9/366bf80/job-output.txt \n15:00 \u003c marios\u003e ysandeep: that one is testing the non collection code \nhttps://review.rdoproject.org/r/c/testproject/+/36817/5#message-0aa79fffdccd82d156ddfed9e3b15be1ed785693 \n15:01 \u003c marios\u003e ysandeep: if you see any of those .. for example \nhttps://logserver.rdoproject.org/17/36817/5/check/periodic-tripleo-centos-9-buildimage-overcloud-hardened-uefi-full-master/ca6432a/job-output.txt \n15:01 \u003c marios\u003e ysandeep: grep \"repo-setup : find full hash line from tripleo-get-hash stdout_lines\" \n","commit_id":"6243c42243cb64d219a54deb3ee964b627400f9a"},{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"1c4b0375d9c5e0af25b7e1b7147e392df4217196","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c2fe57ff_f05ef36a","updated":"2021-11-26 12:32:13.000000000","message":"Good to know that tripleo_get_hash_result is success is a thing, i wasn\u0027t aware you can do that.","commit_id":"6243c42243cb64d219a54deb3ee964b627400f9a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"6ff60e03e5a69cb07b38ee9be92a8c5cc3ee6752","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b7744659_72651840","updated":"2021-11-26 12:42:03.000000000","message":"looks good, waiting more ci results only","commit_id":"6243c42243cb64d219a54deb3ee964b627400f9a"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"3507e82a8d44f118c0e57cd5b6a261af925122b9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"dee0580a_c49128b6","in_reply_to":"4ac4bf6c_d39d30c5","updated":"2021-11-26 13:24:08.000000000","message":"\u003e\u003e\u003e\u003e\u003e \u003c marios\u003e ysandeep: that one is testing the non collection code \nhttps://review.rdoproject.org/r/c/testproject/+/36817/5#message-0aa79fffdccd82d156ddfed9e3b15be1ed785693 \n\nI am a bit confused here because I don\u0027t see tripleo-get-hash-set-fact-no-collections.yml getting included in :-\n\nhttps://logserver.rdoproject.org/17/36817/5/check/periodic-tripleo-centos-9-buildimage-overcloud-hardened-uefi-full-master/ca6432a/job-output.txt\n\nInfact, the tripleo-get-hash-set-fact.yml is getting included:-\n\n~~~\n2021-11-26 09:56:59.678558 | primary | included: /var/lib/zuul/builds/ca6432a3da684e178343bdbadd938dd8/untrusted/project_5/opendev.org/openstack/tripleo-quickstart/roles/repo-setup/tasks/tripleo-get-hash-set-fact.yml\n2021-11-26 09:56:59.696056 | \n~~~\n\nIf we use non collection code i was expecting tripleo-get-hash-set-fact-no-collections.yml to be included.","commit_id":"6243c42243cb64d219a54deb3ee964b627400f9a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"6c7f04e620e88eb3fa5805a179f28c628f9e2280","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"92a25ede_d4742ecf","in_reply_to":"dee0580a_c49128b6","updated":"2021-11-26 13:25:39.000000000","message":"the logs are not pointing to this latest version... the latest version is me tidying it up. before v5 if you check there is no new \u0027no-collections\u0027 file i was editing it inline so it was icludeing tripleo-get-hash-set-fact\n\nhope it helps","commit_id":"6243c42243cb64d219a54deb3ee964b627400f9a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"624ac40a19a760512811c15df3395279bd66b93d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"609bd516_8d3c880d","updated":"2021-11-29 14:53:23.000000000","message":"Thanks!","commit_id":"81a8993f719dfbf44ac8bc823d8095d890d676eb"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"2110be4ec0729eda9da6c4cce0f3b0ee7878b37d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"179dc22e_d0d2762d","updated":"2021-11-29 21:22:51.000000000","message":"recheck","commit_id":"81a8993f719dfbf44ac8bc823d8095d890d676eb"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"054ccbe30ad37556b9e6cfcdccc4f4779991d27f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d980a75a_0f58015d","updated":"2021-11-30 14:19:43.000000000","message":"recheck","commit_id":"81a8993f719dfbf44ac8bc823d8095d890d676eb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"83ae141b125744b624235c0fd461f7daa2bbb3d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f1e574b0_55d9bab2","updated":"2021-11-30 09:55:07.000000000","message":"recheck ","commit_id":"81a8993f719dfbf44ac8bc823d8095d890d676eb"}],"roles/repo-setup/defaults/main.yml":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"6ff60e03e5a69cb07b38ee9be92a8c5cc3ee6752","unresolved":true,"context_lines":[{"line_number":13,"context_line":"# File to temporary store environment variables which will be loaded in repo_setup_newest.sh and repo_cmd_after_setup.sh"},{"line_number":14,"context_line":"repo_setup_env_filepath: \"/tmp/repo_setup_env\""},{"line_number":15,"context_line":"repo_run_live: true"},{"line_number":16,"context_line":"repo_setup_use_collections: true  # for zuul native cannot use this"},{"line_number":17,"context_line":"# Releases shortcuts"},{"line_number":18,"context_line":"stable:"},{"line_number":19,"context_line":"  - pike"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"016a640e_c65752e6","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":26},"updated":"2021-11-26 12:42:03.000000000","message":"great, we can reuse on yum-config too","commit_id":"6243c42243cb64d219a54deb3ee964b627400f9a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"14d7d5d81576c5bb4403e64cd5335ac960a65256","unresolved":true,"context_lines":[{"line_number":13,"context_line":"# File to temporary store environment variables which will be loaded in repo_setup_newest.sh and repo_cmd_after_setup.sh"},{"line_number":14,"context_line":"repo_setup_env_filepath: \"/tmp/repo_setup_env\""},{"line_number":15,"context_line":"repo_run_live: true"},{"line_number":16,"context_line":"repo_setup_use_collections: true  # for zuul native cannot use this"},{"line_number":17,"context_line":"# Releases shortcuts"},{"line_number":18,"context_line":"stable:"},{"line_number":19,"context_line":"  - pike"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ebd7a7d6_47f7a0e7","line":16,"range":{"start_line":16,"start_character":28,"end_line":16,"end_character":33},"updated":"2021-11-28 20:20:49.000000000","message":"do we want to calculate this automatically maybe? like \"if zuul is defined ...\"","commit_id":"81a8993f719dfbf44ac8bc823d8095d890d676eb"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"cc8a8372d2b5cabcbac33570440123010578537e","unresolved":true,"context_lines":[{"line_number":13,"context_line":"# File to temporary store environment variables which will be loaded in repo_setup_newest.sh and repo_cmd_after_setup.sh"},{"line_number":14,"context_line":"repo_setup_env_filepath: \"/tmp/repo_setup_env\""},{"line_number":15,"context_line":"repo_run_live: true"},{"line_number":16,"context_line":"repo_setup_use_collections: true  # for zuul native cannot use this"},{"line_number":17,"context_line":"# Releases shortcuts"},{"line_number":18,"context_line":"stable:"},{"line_number":19,"context_line":"  - pike"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"d442e6ce_0479a5d5","line":16,"range":{"start_line":16,"start_character":28,"end_line":16,"end_character":33},"in_reply_to":"ebd7a7d6_47f7a0e7","updated":"2021-12-01 08:59:17.000000000","message":"sorry didn\u0027t get to this before it merged.\nit\u0027s a decent suggestion ... maybe? for now I think it is OK as a role var and you can override this behaviour if you need to like we are doing for the build jobs.","commit_id":"81a8993f719dfbf44ac8bc823d8095d890d676eb"}],"roles/repo-setup/tasks/get-dlrn-hash-newest.yml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"aaf83f78e90633cf297e38690457fb12b3f72879","unresolved":true,"context_lines":[{"line_number":40,"context_line":"  when:"},{"line_number":41,"context_line":"    - dlrn_hash_tag_newest in known_hash_tags"},{"line_number":42,"context_line":"    - ansible_distribution_major_version is version(8, \u0027\u003e\u003d\u0027)"},{"line_number":43,"context_line":"    - when: not repo_setup_use_collections"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"- name: Set fact for dlrn_hash_path_newest"},{"line_number":46,"context_line":"  set_fact:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"e56a84f2_9eeb3d75","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":6},"updated":"2021-11-26 13:29:37.000000000","message":"oops","commit_id":"6243c42243cb64d219a54deb3ee964b627400f9a"}]}