)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"370caaa82b23412f6a3ac910873f04b168c76dff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1d136930_9f5b7a45","updated":"2021-11-16 14:24:45.000000000","message":"I had one concern but Kevin++ already addressed that:-\n\n~~~\n* ysandeep wondering do we expect customers to be able to install rpms in containers if they wish, I was wondering if removing ubi.repo conflict with that.\n\u003ccloudnull\u003e ysandeep we do have an interface for vendors to build with our containers, but when we did the tcib work we didn\u0027t have the expectation that users would ever install adhoc packages in containers ourside of the build system \n~~~","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"67bf0b46ea808d5a1a463c4410f38d74c1270bf4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e640e145_1140985f","updated":"2021-11-17 05:49:01.000000000","message":"I think it\u0027s better to reconsider https://review.opendev.org/c/openstack/python-tripleoclient/+/761702 and other series of patches instead of going with this patch for the specific issue mentioned in the bz. For upstream cases with CentOS stream in picture that no longer shouldn\u0027t be needed as stream will be ahead of ubi(which is based out of a rhel release).\n\nAlso can consider https://review.opendev.org/c/openstack/tripleo-common/+/763203 for rhel as well as those builds also mixing repos from ubi latest and rhel 8.4.\n\nFor downstream cases it may be good to not have ubi repos installed in containers(/me haven\u0027t checked if shipped containers have those installed) as osp releases may be bound to specific rhel releases and having repos pointing to latest rhel/ubi release doesn\u0027t sound good and other reason as Kevin pointed out don\u0027t expect customers to install packages.\nNo strong opinion for upstream where developers may want to install some packages for debugging, it may be better to have those repos already installed in containers, yes can always setup required repos manually whenever needed(haven\u0027t checked yet what we do for overcloud images, but if policy already defined for those images, same can be considered for container images). So if still want to remove repos from shipped containers good have a reasoning other than the issue mentioned in the bz.","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"4619f1f5c0cffa6a5fd5f323358466736a2978da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5e31fd45_1b6b648b","updated":"2021-11-16 17:29:11.000000000","message":"recheck","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"8e97295f2c231285eb9df721ec289d352daca096","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"783207be_e29567ce","updated":"2021-11-16 14:16:27.000000000","message":"testing with c9","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c54277ce5bd71c8b0d5145d0660675cafb03fe84","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"84a774fa_43d4408e","updated":"2021-11-16 14:19:23.000000000","message":"voting so it goes into my reviews and waiting on the testing","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"9b8c0e5c26c16038f7a7ed8e4213cf803010f9d0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2f5db9f8_9352d293","in_reply_to":"1e860851_0d46dc40","updated":"2021-11-17 13:32:29.000000000","message":"Release Team suggests we stop using ubi8 entirely and switch to registry.redhat.io/rhel8-4-els/rhel:latest","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"change_message_id":"8a7398e7bed8a1d452fa4f4f44f5a88de11f9976","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"87c6a82a_8c06b120","in_reply_to":"2f5db9f8_9352d293","updated":"2021-11-17 13:39:03.000000000","message":"NB ELS is not publicly available, so can\u0027t use it for upstream CI jobs","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"0db25ab9c54393a57364e668c2f5675a6a9e2401","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2bb7249b_efacd6e7","in_reply_to":"77b69aae_581ef07a","updated":"2021-11-17 14:12:12.000000000","message":"agree here - we stick with consistency","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"50d4c4d58f7de80f36fba1c7d40ff1b87a63b2ae","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c925ec25_3018bbfa","in_reply_to":"783207be_e29567ce","updated":"2021-11-17 03:06:58.000000000","message":"I was looking at c9 logs:-\n\nLooks like c9 containers already don\u0027t have ubi.repo\n\nhttps://logserver.rdoproject.org/53/18953/120/check/periodic-tripleo-ci-build-containers-centos-9-push-master/90a4a8d/logs/container-builds/92e98439-d2c7-4ad9-b40d-a24a9731dc9a/base/rsyslog/rsyslog-build.log\n\nStill I have reran a testproject with c9 container builds to confirm nothing breaks with this change.\n\nhttps://review.rdoproject.org/r/c/testproject/+/32054","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"e5a3fbb781e496818c534904748b45634b180d68","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"77b69aae_581ef07a","in_reply_to":"87c6a82a_8c06b120","updated":"2021-11-17 14:05:33.000000000","message":"if either brings back more of upstream vs downstream differences, please stick with ubi","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"528c57b19269ddb5bf762fc0309a46d6cd3411d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ad29e8dd_72df01e3","in_reply_to":"c925ec25_3018bbfa","updated":"2021-11-17 05:28:21.000000000","message":"testproject: https://review.rdoproject.org/r/c/testproject/+/32054 with depends-on this change passed.\n\nperiodic-tripleo-ci-build-containers-centos-9-push-master https://review.rdoproject.org/zuul/build/1e80eb2e6fbf425fa0f26041b374eb89 : SUCCESS in 40m 29s\n\n@Ronelle could you please remove -W.","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"0cab5506ad5bec2a756732445750f42a711413ad","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1e860851_0d46dc40","in_reply_to":"e640e145_1140985f","updated":"2021-11-17 08:51:49.000000000","message":"Thank you Yatin, \n\nI have tried both of the alternative approaches suggested by you and both of them works.\n\nAlternative approach 1:\n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nTCIB: Pick not best when building on rhel/ubi, similiar to what we are already doing for centos - https://review.opendev.org/c/openstack/tripleo-common/+/763203\n\n~~~\nDNM downstream patch: https://code.engineering.redhat.com/gerrit/c/openstack-tripleo-common/+/290958/8/container-images/tcib/base/base.yaml\n\ntestproject: https://code.engineering.redhat.com/gerrit/c/testproject/+/211643\n\nSuccessfull run: periodic-tripleo-build-containers-ubi-8-internal-rhel-8-build-push-upload-rhos-16.2 https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/build/bf585658df224885882c5445ab5fc1f7 : SUCCESS in 59m 06s\n~~~\n\nAlternative approach 2: \n\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\nRevert of https://review.opendev.org/c/openstack/python-tripleoclient/+/767278 that will hide container ubi.repo and will map hosts /etc/yum.repos.d in container.\n\n~~~\nDNM downstream patch: https://code.engineering.redhat.com/gerrit/c/python-tripleoclient/+/291150\n\ntestproject: https://code.engineering.redhat.com/gerrit/c/testproject/+/200295\n\nSuccessfull run: periodic-tripleo-build-containers-ubi-8-internal-rhel-8-build-push-upload-rhos-16.2 https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/build/e4c523cc06954bbabbbf72a800462781 : SUCCESS in 1h 15m 38s\n~~~\n\nI will let tripleo experts suggest the best approch here, whether the current patch or from one of the Alternative approaches suggested by you.","commit_id":"4fb684e62bd13fc2db4cc010f5e1a48549d8288d"}]}