)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"179db5009a39b42539484fc23d97e2b01ad40cdd","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the centos-compose-repos pre playbook into"},{"line_number":10,"context_line":"upstream base jobs to enable compose pinning feature available"},{"line_number":11,"context_line":"for jobs that parent them. This is a follow up patch after changes "},{"line_number":12,"context_line":"on re-parenting work[1][2]."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.rdoproject.org/r/q/topic:cs-parenting-build-containers"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ee394b09_e50d0d54","line":11,"updated":"2022-03-31 06:42:43.000000000","message":"whitespace","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"d490d9b89294c4fd374a36e3634aef3fd2790c9e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds the centos-compose-repos pre playbook into"},{"line_number":10,"context_line":"upstream base jobs to enable compose pinning feature available"},{"line_number":11,"context_line":"for jobs that parent them. This is a follow up patch after changes "},{"line_number":12,"context_line":"on re-parenting work[1][2]."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[1] https://review.rdoproject.org/r/q/topic:cs-parenting-build-containers"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ea51805a_2764118e","line":11,"in_reply_to":"ee394b09_e50d0d54","updated":"2022-03-31 12:04:35.000000000","message":"Done","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"7f79be5b4da0c4f3898099947ae66fecec7cea4a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"99bd832a_66426382","updated":"2022-03-31 00:17:29.000000000","message":"Test project results:\nhttps://review.rdoproject.org/r/c/testproject/+/38635/19#message-c3fbafae96e2007525fff944af3405b998585709\n\nhttps://logserver.rdoproject.org/35/38635/19/check/periodic-tripleo-ci-centos-9-standalone-master-compose-pinning/cb376f5/logs/undercloud/etc/yum.repos.d/\nhttps://logserver.rdoproject.org/35/38635/19/check/periodic-tripleo-ci-build-containers-centos-9-push-master-compose-pinning/554c8df/etc/yum.repos.d/","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"067a67f27689c35fc5d3ba5641795ba80598803d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"68540d31_7e5e90cb","updated":"2022-03-31 11:58:32.000000000","message":"Yes, you guys are correct, we can see the same playbook running twice here:\nhttps://logserver.rdoproject.org/35/38635/19/check/periodic-tripleo-ci-build-containers-centos-9-push-master-compose-pinning/554c8df/job-output.txt","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"55f7e4e7b64e73b7b22cc806690173cc801ff3f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"15cd0e28_8e2fa045","updated":"2022-03-31 14:37:00.000000000","message":"Test results:\nhttps://review.rdoproject.org/r/c/testproject/+/38635/20#message-c17cd7b35ae66464daf84143d7e224de255566f1\n\nhttps://logserver.rdoproject.org/35/38635/20/check/periodic-tripleo-ci-centos-9-standalone-master-compose-pinning/2e95ec3/logs/undercloud/etc/yum.repos.d/\nhttps://logserver.rdoproject.org/35/38635/20/check/periodic-tripleo-ci-build-containers-centos-9-push-master-compose-pinning/b07b027/etc/yum.repos.d/","commit_id":"21cdfd038576ef8ff9721b5aa96dbbefd009dcfc"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"d490d9b89294c4fd374a36e3634aef3fd2790c9e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"334aafa1_a9b6744b","updated":"2022-03-31 12:04:35.000000000","message":"Thanks for your reviews.\n\nTesting again here: https://review.rdoproject.org/r/c/testproject/+/38635\n","commit_id":"21cdfd038576ef8ff9721b5aa96dbbefd009dcfc"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"4995aca86ae5da398bc6b0c35992547a562cc82c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"58002375_4e4e7eab","updated":"2022-03-31 17:20:46.000000000","message":"recheck","commit_id":"21cdfd038576ef8ff9721b5aa96dbbefd009dcfc"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"7bce1d6473d69f480db466663ba182c8bf090fb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d00d156e_40155925","updated":"2022-04-01 11:43:44.000000000","message":"abandon and restore to try the tox molecule hopefully transient? ","commit_id":"82e8826200a398560b4ad02e7fa554eeeeb09e64"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"59e883157d55ba9c5971af8082d1a76d5eb418a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6fe8685e_dcc99475","updated":"2022-04-04 10:33:10.000000000","message":"recheck","commit_id":"82e8826200a398560b4ad02e7fa554eeeeb09e64"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a47c52747cd403b2eebaa05b9de1bd8a446592f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7f92ce63_bc262546","updated":"2022-04-07 14:54:07.000000000","message":"recheck","commit_id":"82e8826200a398560b4ad02e7fa554eeeeb09e64"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"4b833882e747340025c2253df401b7af9178a9d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c473a7d1_25a653d6","updated":"2022-04-03 14:58:12.000000000","message":"recheck","commit_id":"82e8826200a398560b4ad02e7fa554eeeeb09e64"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"4707ad98e5b6b850efd481319e7c885a9570b173","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fa7da99c_466f06f3","updated":"2022-04-01 05:25:03.000000000","message":"recheck","commit_id":"82e8826200a398560b4ad02e7fa554eeeeb09e64"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"e643e30119db48bb8433e714070668cd4b409cbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"fed0c7cb_aecbba33","updated":"2022-04-08 17:43:19.000000000","message":"recheck\n","commit_id":"82e8826200a398560b4ad02e7fa554eeeeb09e64"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"3ec7dece7bf5f4ad3482fda9eb30ebe04ec13511","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"282b78d4_c3fa501d","updated":"2022-04-01 08:57:57.000000000","message":"recheck \n\nshould be fixed now ? ( https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/836091 )","commit_id":"82e8826200a398560b4ad02e7fa554eeeeb09e64"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"82140b2862c1230292d8110052e8ce76faa53469","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f9a8a786_2a0359d7","in_reply_to":"282b78d4_c3fa501d","updated":"2022-04-01 11:43:30.000000000","message":"yes it was fixed by tox molecule fails now for https://zuul.opendev.org/t/openstack/build/80feebf4238d44ebb408897b9cc14665\n\n\nFailed to download metadata for repo \u0027centos-nfv-ovs\u0027: Yum repo downloading error: Downloading error(s): repodata/d87549d0a181888b3d6acedb913a702fea695cf9e9f555c25d6b1f7e97318894-primary.xml.gz - Cannot download, all mirrors were already tried without success; repodata/ba39a52f4878defb26bff81dc1a8a6bb0dc846caf722e7d2b1ad98f43b337095-filelists.xml.gz - Cannot download, all mirrors were already tried without success","commit_id":"82e8826200a398560b4ad02e7fa554eeeeb09e64"}],"zuul.d/base-upstream.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"179db5009a39b42539484fc23d97e2b01ad40cdd","unresolved":true,"context_lines":[{"line_number":73,"context_line":"    pre-run:"},{"line_number":74,"context_line":"      - playbooks/nodepool-provider/pre.yaml"},{"line_number":75,"context_line":"      - playbooks/openstack-zuul-jobs/legacy/pre.yaml"},{"line_number":76,"context_line":"      - playbooks/tripleo-ci/centos-compose-repos.yml"},{"line_number":77,"context_line":"      - playbooks/tripleo-ci/ceph.yaml"},{"line_number":78,"context_line":"      - playbooks/tripleo-ci/pre.yaml"},{"line_number":79,"context_line":"    run: playbooks/tripleo-ci/run-v3.yaml"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5f4e8b1f_9c2a2512","line":76,"updated":"2022-03-31 06:42:43.000000000","message":"i think sandeep is correct... this one should be enough?\nthe other two places you added it are children of this definition\n\neg go to https://zuul.openstack.org/jobs\nadd \"tripleo-ci-containers-rdo-upstream-pre\" into the search box\n\nsame for \"tripleo-ci-build-containers-required-projects-upstream\"","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"d490d9b89294c4fd374a36e3634aef3fd2790c9e","unresolved":false,"context_lines":[{"line_number":73,"context_line":"    pre-run:"},{"line_number":74,"context_line":"      - playbooks/nodepool-provider/pre.yaml"},{"line_number":75,"context_line":"      - playbooks/openstack-zuul-jobs/legacy/pre.yaml"},{"line_number":76,"context_line":"      - playbooks/tripleo-ci/centos-compose-repos.yml"},{"line_number":77,"context_line":"      - playbooks/tripleo-ci/ceph.yaml"},{"line_number":78,"context_line":"      - playbooks/tripleo-ci/pre.yaml"},{"line_number":79,"context_line":"    run: playbooks/tripleo-ci/run-v3.yaml"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ba3c134a_8c18a96f","line":76,"in_reply_to":"5f4e8b1f_9c2a2512","updated":"2022-03-31 12:04:35.000000000","message":"Yes, this should be enough for now, thanks","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"d107153f3e2bf3202ef86f9920c49a28389d3628","unresolved":true,"context_lines":[{"line_number":252,"context_line":"    timeout: 7200"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"- job:"},{"line_number":255,"context_line":"    name: tripleo-ci-containers-rdo-upstream-pre"},{"line_number":256,"context_line":"    abstract: true"},{"line_number":257,"context_line":"    description: |"},{"line_number":258,"context_line":"                Base abstract job for container build TripleO CI zuulv3 jobs"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1f7a2678_425307f8","line":255,"range":{"start_line":255,"start_character":10,"end_line":255,"end_character":48},"updated":"2022-03-31 06:16:06.000000000","message":"Hey douglas,\n\nLooking at parent-child hierarchy[1]\n\nAFAIK, first parent\u0027s pre playbook executes and then child pre-playbooks.\n\ntripleo-ci-base-standard is super parent of tripleo-ci-containers-rdo-upstream-pre already, So tripleo-ci-containers-rdo-upstream-pre job should get that pre already. Do we need to add this pre(playbooks/tripleo-ci/centos-compose-repos.yml) again here?\n\n\n[1]\n~~~\nbase The base job for OpenDev\u0027s installation of Zuul.\n\nmultinode Do additional setup needed for multi-node jobs such as setting up\n\ntripleo-ci-base-common-required-projects Base abstract job to add common required-projects for tripleo-ci-base\n\ntripleo-ci-base-standard Base abstract job for all TripleO CI zuulv3 jobs  \u003e\u003e\u003e\u003e\u003e\u003e\u003e\u003e\u003e\u003e\n\ntripleo-ci-base-singlenode-standard Base abstract job for singlenode TripleO CI zuulv3 jobs\n\ntripleo-ci-base-containers-standard Base abstract job for container build TripleO CI zuulv3 jobs\n\ntripleo-ci-containers-rdo-upstream-pre Base abstract job for container build TripleO CI zuulv3 jobs\n~~~","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"179db5009a39b42539484fc23d97e2b01ad40cdd","unresolved":true,"context_lines":[{"line_number":252,"context_line":"    timeout: 7200"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"- job:"},{"line_number":255,"context_line":"    name: tripleo-ci-containers-rdo-upstream-pre"},{"line_number":256,"context_line":"    abstract: true"},{"line_number":257,"context_line":"    description: |"},{"line_number":258,"context_line":"                Base abstract job for container build TripleO CI zuulv3 jobs"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"371a3a0a_2e680b44","line":255,"range":{"start_line":255,"start_character":10,"end_line":255,"end_character":48},"in_reply_to":"1f7a2678_425307f8","updated":"2022-03-31 06:42:43.000000000","message":"+1","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"d490d9b89294c4fd374a36e3634aef3fd2790c9e","unresolved":false,"context_lines":[{"line_number":252,"context_line":"    timeout: 7200"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":"- job:"},{"line_number":255,"context_line":"    name: tripleo-ci-containers-rdo-upstream-pre"},{"line_number":256,"context_line":"    abstract: true"},{"line_number":257,"context_line":"    description: |"},{"line_number":258,"context_line":"                Base abstract job for container build TripleO CI zuulv3 jobs"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"b105c394_b96a65b3","line":255,"range":{"start_line":255,"start_character":10,"end_line":255,"end_character":48},"in_reply_to":"371a3a0a_2e680b44","updated":"2022-03-31 12:04:35.000000000","message":"Yes, you are correct, thanks for noticing","commit_id":"133e50534aec3d5d73e1c95480c33220b90b7433"}]}