)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"8c4012067d73fa4444692ed21175399df74353c7","unresolved":true,"context_lines":[{"line_number":14,"context_line":"Fallback to using an existing sudoers file with sed, then followup with"},{"line_number":15,"context_line":"a patch to remove this logic once the package is updated."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On:  I039c49e6f5872b02c1fdc1690594e0d9bef303f5"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I2d06904cc89e398f82ffc8b4f2d862427507d0dc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"2dba7d17_5f2c5857","line":17,"range":{"start_line":17,"start_character":11,"end_line":17,"end_character":13},"updated":"2021-12-10 07:52:45.000000000","message":"the depends-on isn\u0027t working here its the same issue as you had last time Ollie looks like 2 spaces is that an editor thing? and/or the changeID vs gerrit uri  ;)\n\nit is not included here see the content provider\n\n        * https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c04/820408/5/check/tripleo-ci-centos-8-content-provider/c04695f/logs/undercloud/home/zuul/container_image_build.log\n...\nexcludes\u003d[\u0027neutron-mlnx-agent\u0027, \u0027tempest\u0027], extra_config\u003dNone, labels\u003d[], namespace\u003d\u0027tripleomaster\u0027, prefix\u003d\u0027openstack\u0027, push\u003dTrue, registry\u003d\u0027127.0.0.1:5001\u0027, rhel_modules\u003dNone, skip_build\u003dFalse, tag\u003d\u0027f989c9fd65e32486a970ce86a5e94e11\u0027, tcib_extras\u003d[\u0027tcib_release\u003d8\u0027, \u0027tcib_python_version\u003d3.6\u0027], volumes\u003d[\u0027/etc/yum.repos.d:/etc/distro.repos.d:z\u0027, \u0027/etc/pki/rpm-gpg:/etc/pki/rpm-gpg:z\u0027, \u0027/etc/yum.repos.d:/etc/distro.repos.d:z\u0027, \u0027/etc/pki/rpm-gpg:/etc/pki/rpm-gpg:z\u0027, \u0027/etc/dnf/vars:/etc/dnf/vars:z\u0027, \u0027/opt/gating_repo:/opt/gating_repo:z\u0027], work_dir\u003d\u0027/home/zuul/container-builds\u0027))\n\n\nand no tempest container built@        * https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c04/820408/5/check/tripleo-ci-centos-8-content-provider/c04695f/logs/container-builds/bcce9d17-8ce9-48f7-823e-7d89f762a131/base/os/index.html\n\ncan we get an update so we can see it run in check/gate ?\nthis is probably why it wasn\u0027t picking it up in https://review.rdoproject.org/r/c/testproject/+/37138/4#message-ca2800660564b1178a9e1e77ed1a519b081daeff","commit_id":"7b631f3fe5d492df021723d21c637a224df76f91"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"568adb3934f61061e3e5dc2013820d3ab180ae2f","unresolved":true,"context_lines":[{"line_number":14,"context_line":"Fallback to using an existing sudoers file with sed, then followup with"},{"line_number":15,"context_line":"a patch to remove this logic once the package is updated."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On:  I039c49e6f5872b02c1fdc1690594e0d9bef303f5"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I2d06904cc89e398f82ffc8b4f2d862427507d0dc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"b418ec78_0e741484","line":17,"range":{"start_line":17,"start_character":11,"end_line":17,"end_character":13},"in_reply_to":"2dba7d17_5f2c5857","updated":"2021-12-10 07:56:28.000000000","message":"Yes, Correct, Thanks Marios for highlighting, Now I got it why earlier the testproject was not picking the toci changes.","commit_id":"7b631f3fe5d492df021723d21c637a224df76f91"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"e9f6b0d623784f6da6bbd3aa44400b4bbbcd6f47","unresolved":true,"context_lines":[{"line_number":14,"context_line":"Fallback to using an existing sudoers file with sed, then followup with"},{"line_number":15,"context_line":"a patch to remove this logic once the package is updated."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On:  I039c49e6f5872b02c1fdc1690594e0d9bef303f5"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I2d06904cc89e398f82ffc8b4f2d862427507d0dc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"71dac485_5988ac0d","line":17,"range":{"start_line":17,"start_character":11,"end_line":17,"end_character":13},"in_reply_to":"b418ec78_0e741484","updated":"2021-12-10 10:25:06.000000000","message":"\u003e the depends-on isn\u0027t working here its the same issue as you had last time Ollie looks like 2 spaces is that an editor thing?\n\nIt\u0027s a me thing :-) still had the old commit msg in my local clone when I pushed a patch","commit_id":"7b631f3fe5d492df021723d21c637a224df76f91"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"6548d5a475bdc934b18b199f9bcb74a7344d780d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bb12c9b6_81cddc91","updated":"2021-12-09 11:25:03.000000000","message":"Will need to split into two patches. Cannot use the new tempest_sudoers file until it has been packaged.","commit_id":"b9647c8c1899e3828c77c043b07fbd3268e4f55f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"7a75c36759ce93be527676c4e2dcb2834c4652b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"50335c8b_45f042aa","updated":"2021-12-08 08:58:14.000000000","message":"looks like the depends-on worked this time @owalsh 127.0.0.1:5001/tripleomaster/openstack-tempest  f989c9fd65e32486a970ce86a5e94e11  504bca3fc14e  2 hours ago  1.84 GB https://044169259e11c7e10b08-c984d7d5f63b929ba64a3dbca97ee26a.ssl.cf5.rackcdn.com/820408/3/check/tripleo-ci-centos-8-content-provider/588054a/logs/report.html","commit_id":"b9647c8c1899e3828c77c043b07fbd3268e4f55f"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"219974f47c83a69a2e6d102e35ddc83ae867b18f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1380d491_224f4ac5","updated":"2021-12-07 12:06:20.000000000","message":"recheck","commit_id":"b9647c8c1899e3828c77c043b07fbd3268e4f55f"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"f8049a47a67ef2d78ec8a1bb550d29df3fb6b23d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"bf19473a_d6ae7035","updated":"2021-12-09 11:26:30.000000000","message":"this won\u0027t merge because of the depends-on however removing the votes for now as discussed\n\nsee https://review.opendev.org/c/openstack/tripleo-ci/+/820511/1#message-7155fbe4c2101c5fa08a17837acaaf795cf9cd10","commit_id":"b9647c8c1899e3828c77c043b07fbd3268e4f55f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"245896515752299da69151c06097eb0059029514","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d0ecb559_bd6c981d","updated":"2021-12-10 05:22:44.000000000","message":"Based on this https://review.rdoproject.org/r/c/testproject/+/37138\n\nperiodic-tripleo-ci-build-containers-ubi-8-push\tSUCCESS 1h 36m 06s\nperiodic-tripleo-ci-build-containers-centos-9-push-master\tSUCCESS 55m 11s\n\nand\n\ntrunk.registry.rdoproject.org/tripleomaster/openstack-tempest                      6688dae327caee4ff6075ef731fa5aa5_x86_64   1e769ce6069d   56 minutes ago      1.63 GB is there.","commit_id":"7b631f3fe5d492df021723d21c637a224df76f91"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"6cf0ea953166321969a2c1a6288bfc9553b8128d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"508d9d6e_e1e31a67","updated":"2021-12-09 17:36:26.000000000","message":"Good to merge, tested in https://review.rdoproject.org/r/c/testproject/+/37138 patchset 4","commit_id":"7b631f3fe5d492df021723d21c637a224df76f91"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"568adb3934f61061e3e5dc2013820d3ab180ae2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b8b5083e_4cf7c5fe","updated":"2021-12-10 07:56:28.000000000","message":"Waiting for adding marios comment","commit_id":"7b631f3fe5d492df021723d21c637a224df76f91"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"11cc6a571e776b4806aecc2449cbc033cfe0ad3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4f20112c_f2c3c6ad","updated":"2021-12-13 11:30:17.000000000","message":"We also need to backport this to wallaby as https://opendev.org/openstack/tripleo-common/src/branch/stable/wallaby/container-images/tcib/base/os/tempest tempest container is there.","commit_id":"b80b98ffe96c4a182efb264f8d476717220f8dca"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a45e27ba2451a8feb78e4486b21b5cecfa881c73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1a1afc98_7f554d63","updated":"2021-12-13 07:54:04.000000000","message":"thanks looks like it builds now \n\nhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_567/820408/7/check/tripleo-ci-centos-8-content-provider/5670646/logs/container-builds/3e633753-415c-4268-8cf7-0561e3650c49/base/os/tempest/tempest-build.log","commit_id":"b80b98ffe96c4a182efb264f8d476717220f8dca"}],"container-images/tcib/base/os/tempest/tempest.yaml":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"5fb2adeda01f16d1cdc9041e04b7bbd1041cf163","unresolved":true,"context_lines":[{"line_number":1,"context_line":"tcib_actions:"},{"line_number":2,"context_line":"- run: bash /usr/local/bin/uid_gid_manage {{ tcib_user }}"},{"line_number":3,"context_line":"- run: dnf -y install {{ tcib_packages[\u0027common\u0027] | join(\u0027 \u0027) }} \u0026\u0026 dnf clean all \u0026\u0026 rm -rf /var/cache/dnf"},{"line_number":4,"context_line":"- run: cp /usr/share/openstack-tripleo-common-containers/container-images/tcib/base/os/tempest/tempest_sudoers /etc/sudoers.d/tempest_sudoers"},{"line_number":5,"context_line":"- run: chmod 440 /etc/sudoers.d/tempest_sudoers"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"tcib_packages:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5abcd9b5_4654e97a","line":4,"range":{"start_line":4,"start_character":10,"end_line":4,"end_character":111},"updated":"2021-12-09 11:12:19.000000000","message":"https://logserver.rdoproject.org/38/37138/1/check/periodic-tripleo-ci-build-containers-centos-9-push-master/c7d5fdb/logs/container-builds/86dfefe7-085f-46ee-a7f1-67018f6f017b/base/os/tempest/tempest-build.log\n```\ncp: cannot stat \u0027/usr/share/openstack-tripleo-common-containers/container-images/tcib/base/os/tempest/tempest_sudoers\u0027: No such file or directory\n```\n\nThis path does not exists on the system during container-build,\n\nMay be we can add if condition to check for that path and then run the operation. then container build will pass","commit_id":"b9647c8c1899e3828c77c043b07fbd3268e4f55f"}]}