)]}'
{".zuul.d/jobs.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e729c88f195eae269b090832866b1f86abe01555","unresolved":true,"context_lines":[{"line_number":191,"context_line":"    vars:"},{"line_number":192,"context_line":"      nodepool_diskimage:"},{"line_number":193,"context_line":"        base_element: centos-minimal"},{"line_number":194,"context_line":"        release: \u00279-stream\u0027"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"- job:"},{"line_number":197,"context_line":"    name: dib-nodepool-functional-openstack-fedora-34-containerfile-src"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"87f51fd0_0ee1505c","line":194,"updated":"2021-10-13 05:04:00.000000000","message":"This will try to build centos-minimal, which builds images from scratch in a chroot.  This change is not updating that element.  This is a full end-to-end test that builds nodes, uploads them to a devstack and boots them.\n\nThe change you have proposed is updating the \"centos\" element, which builds images based on the upstream .qcow2 image.  We do not have end-to-end testing for this type of build, only a functional test that ensure we build and get a valid .qcow2 out (we don\u0027t try to boot it, though).\n\nI have uploaded a new patchset with this testing enabled\n\nThe job to watch is \"dib-functests-bionic-python3-image\".  This is testing the upstream-image based builds running the builds on a bionic host.  in the logs directory (e.g. [1]) you will see the results we are looking for.\n\n[1] https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_282/806819/11/check/dib-functests-bionic-python3-image/2822d4c/logs/","commit_id":"048344b101305a0128f06c4fa589b9571b4288e2"}],"/COMMIT_MSG":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"40c3a943c7f0e1d9b15b23b29a2367d660112e1a","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Shnaidman Sagi (Sergey) \u003csshnaidm@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-10-07 08:30:18 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Correct path for CentOS 9 stream base image"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"See https://review.opendev.org/q/topic:cs9 for related patches."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"6d1a6c01_332e3784","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":43},"updated":"2021-10-11 21:39:47.000000000","message":"This is doing more than this.  Please update this to describe the intent of the change","commit_id":"c07ccb5f061709bed74a6b83e1ee96bbbebba008"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"acde3914f68c87dd6a56ed52ba66e1ed4002a368","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Shnaidman Sagi (Sergey) \u003csshnaidm@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-10-07 08:30:18 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Correct path for CentOS 9 stream base image"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"See https://review.opendev.org/q/topic:cs9 for related patches."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"eaf6fe72_6a973a40","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":43},"in_reply_to":"6d1a6c01_332e3784","updated":"2021-10-11 21:58:22.000000000","message":"Done","commit_id":"c07ccb5f061709bed74a6b83e1ee96bbbebba008"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"40c3a943c7f0e1d9b15b23b29a2367d660112e1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"05b88e64_3edf4368","updated":"2021-10-11 21:39:47.000000000","message":"https://review.opendev.org/c/openstack/diskimage-builder/+/811392 seems to be adding 9-stream but with CI testing.  What is the relationship between these two?","commit_id":"c07ccb5f061709bed74a6b83e1ee96bbbebba008"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"06f2db2d5fd176903c6d3c1d50e7f5f0a23fb63e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"365fdbd9_de3236d6","in_reply_to":"05b88e64_3edf4368","updated":"2021-10-11 22:00:09.000000000","message":"I thinks it\u0027s building c9 images for nodepool, but I\u0027ll let @Alfredo to explain.","commit_id":"c07ccb5f061709bed74a6b83e1ee96bbbebba008"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"9676ef9aaa46b7ff3216296cb4cadc0ea11cc670","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"95fef940_1a88f201","updated":"2021-10-12 05:51:58.000000000","message":"I don\u0027t like putting this in untested.  I would like to see this combined with the other change to make sure we have some sort of coverage.  otherwise we\u0027ll have tripleo depending on these bits and the dib gate doesn\u0027t follow these paths at all","commit_id":"f8568e7389c37f7dc6515f009f2e3a58ee416d6a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"ad2c97f979238e19d0497d150defc3c50abb3a22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"dd10c09a_63376be0","in_reply_to":"95fef940_1a88f201","updated":"2021-10-12 11:34:23.000000000","message":"fwiw we have this in our testing chain at https://review.rdoproject.org/r/c/testproject/+/35465/10#message-b90fbe5330a14c6d4507916864917620a5858b60 \nlatest run is good with this\n\nfor testing coverage - we will be adding check/gate jobs to build our images for the tripleo repos (there https://opendev.org/openstack/tripleo-ci/src/commit/f3cd56a7e8a01fbb51f31157a0dfdb67abee26a1/zuul.d/build-image.yaml) \n\nwe\u0027re at the start of that process trying to get the image build to work first (with tests like https://review.rdoproject.org/r/c/testproject/+/35465 ) and once we have all the bits in play like this patch we\u0027ll be adding the tripleo check/gate jobs.\n\nbut i am not sure if you meant testing in *this* repo? I just checked and I don\u0027t think we have any tripleo specific jobs in here https://opendev.org/openstack/diskimage-builder/src/commit/6e83df4d407fa00237871f6ef500eb8ef036d08d/.zuul.d/jobs.yaml","commit_id":"f8568e7389c37f7dc6515f009f2e3a58ee416d6a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"5f559648e070777daec8c30a1c0bcc8cfc625e73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"94bb8190_a314bdbb","in_reply_to":"dd10c09a_63376be0","updated":"2021-10-12 12:19:28.000000000","message":"actually we do have jobs tripleo jobs here via the template https://opendev.org/openstack/diskimage-builder/src/commit/6e83df4d407fa00237871f6ef500eb8ef036d08d/.zuul.d/project.yaml#L9\n\nI am working on adding those c9 jobs and will post an update (the template lives there https://opendev.org/openstack/tripleo-ci/src/commit/f3cd56a7e8a01fbb51f31157a0dfdb67abee26a1/zuul.d/build-image.yaml#L125 \nbut we don\u0027t have the upstream nodeset yet so until we do we can\u0027t have those jobs running","commit_id":"f8568e7389c37f7dc6515f009f2e3a58ee416d6a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"e96e6b66adb4d0527baa05947b505d1ee2cf3c1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"86de6301_7bfd55f6","updated":"2021-10-12 18:13:05.000000000","message":"@Ian, please take a look at failure, it has nothing to do with my patch","commit_id":"048344b101305a0128f06c4fa589b9571b4288e2"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"e39d634fbbcfdf2bef44e25a05ade86d3f8eebf2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"e60561be_db0e5fcb","updated":"2021-10-13 09:22:54.000000000","message":"recheck","commit_id":"cb0a113c9d828ad1426d3a25edf84fe29ea04ad8"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"0fad1e0c5fde158fa21b4af6da79fb59ea4d9cc6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"b1c6591a_9e95ef5d","updated":"2021-10-14 04:37:01.000000000","message":"So \n\n1) the gate is broken due to unrelated registry failures from updating zuul to new containers.  See https://review.opendev.org/c/zuul/zuul-jobs/+/813924/\n\n2) https://review.opendev.org/c/openstack/diskimage-builder/+/813922 fixes the centos-8 testing, which was broken and stopping us getting to centos-9-stream testing.\n\n2) after testing in https://review.opendev.org/c/openstack/diskimage-builder/+/813912 we can not mount the XFS volume in the centos-9 .qcow2 on bionic, focal or centos-8.  I have filed https://bugzilla.redhat.com/show_bug.cgi?id\u003d2013894 to see if this image can use a more compatible format for it\u0027s main partition.\n\nThe other options are centos-minimal, which builds a chroot using RPM on the host, or perhaps the containerfile element, which gets an upstream container image, extracts it to a tar and then uses that as a chroot.  \n\nIt\u0027s not impossible I\u0027ve missed something of course.  But at this point, I can\u0027t see how we can get this to work as-is.\n","commit_id":"c396a1d3dca8a720a2ed0d85b71b6d88fb5be14a"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"1a1a4efc432bc3c79bbb72723d0cf23dbd1ca0fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"0042779c_d5f5dce4","updated":"2021-10-14 11:58:29.000000000","message":"was","commit_id":"c396a1d3dca8a720a2ed0d85b71b6d88fb5be14a"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"d36e076c00c6799ad73c613290aaaef233206774","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":14,"id":"9298be4b_ade2df9e","in_reply_to":"274777c8_2a6db462","updated":"2021-10-15 07:03:56.000000000","message":"https://review.opendev.org/c/openstack/diskimage-builder/+/814081\n\nis the top of a stack that takes an audit of the current functional tests and updates things from building on bionic to building on bullseye.\n\nI\u0027m out of time, but my hope is that this change can stack ontop of that and will work to build there.  the other option seems to be build on centos 8-stream -- I would prefer to not introduce more platforms into the dib gate if we can avoid it, but it remains an option.","commit_id":"c396a1d3dca8a720a2ed0d85b71b6d88fb5be14a"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"3be2ca97213fb6ff7e93d93763b19901eca8e8b4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":14,"id":"d5b51f94_978f86b8","in_reply_to":"a8713e65_721477ca","updated":"2021-10-14 09:04:43.000000000","message":"I guess you must be building on a CentOS 9 host?  \n\nI understand your point, but you should also gauge it from the POV of the independent upstream project that we are putting in untestable paths; it will not build on any current OpenDev supported platform.  triple-o will not be dealing with the people complaining that the centos element with 9-stream doesn\u0027t actually work except for one specific exception.\n\nFrom my investigation today, I think the only realistic thing to say is that the centos element can only build centos-9-stream images on a centos-9-stream host.  This is definitely a departure from the history of the element which has basically been buildable with any LTS distro as a host (the way it is used in other projects).  We should clearly document this constraint in the centos element, at least, and probably even put in a code check that fails on unsupported platforms to avoid confusion.\n\nWe might be able to test on fedora-34.  That is *very* unappealing; the long term implications of keeping that up-to-date is maintenance nobody wants to do.  Fedora 34\u0027s networking bring-up is also unstable in OpenDev as is.\n\nI will try to prioritise looking at getting OpenDev centos-9-stream images via the containerfile element, similar to how we build fedora 34.  we could run this build testing on these nodes.  This may not be immediate, but if we have some preliminary evidence this path will work, it would certainly help the argument that it may not be testable now, but there is a path forward.","commit_id":"c396a1d3dca8a720a2ed0d85b71b6d88fb5be14a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"ed65a7d3dfa11e8c1975cdee3902b6734acd735f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"a8713e65_721477ca","in_reply_to":"b1c6591a_9e95ef5d","updated":"2021-10-14 08:19:50.000000000","message":"@Ian, sorry, I don\u0027t see how all this is blocking the patch. That\u0027s all legit issues, but they have nothing to do with topic of the patch.\nThe patch doesn\u0027t break anything because it adds a new functionality which is not used, only in TripleO. And in TripleO it definitely works, you can see logs from jobs brought by Marios in previous comments.\nEven if it\u0027s not so correct about C9 specific bits or something else doesn\u0027t work for CentOS9 - you can always fix it later, it\u0027s not used anyway now.\nWaiting until we solve all problems of Openstack will block and hurt a lot of work related to new releases and C9 adoption, it doesn\u0027t contribute to anyone. Currently TripleO work is blocked because of this patch, let\u0027s start step by step approach instead of blocking everyone and waiting for months.","commit_id":"c396a1d3dca8a720a2ed0d85b71b6d88fb5be14a"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"1a1a4efc432bc3c79bbb72723d0cf23dbd1ca0fb","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":14,"id":"274777c8_2a6db462","in_reply_to":"d5b51f94_978f86b8","updated":"2021-10-14 11:58:29.000000000","message":"appareny the xfs feature is \"big-time\" which uses 2038 compliant timestamps.  it seems to be in 5.10, which means bullsye should work ...\n\nthis maps to the nodepool-builder container environment, so it makes sense to use it.  I think this is worth investigating","commit_id":"c396a1d3dca8a720a2ed0d85b71b6d88fb5be14a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"137083c577f1ecdb4bf7118b880143d1ae6bb87f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"e9764e32_2ea22fe9","updated":"2021-10-20 08:27:17.000000000","message":"https://dcdef67f370bf151c86e-9e1e29a469f8f669289667ecce245a85.ssl.cf1.rackcdn.com/806819/17/gate/tripleo-buildimage-overcloud-full-centos-8/67c6763/job-output.txt\n```\nTASK [oooci-build-images : Run sanity command on overcloud images]\n2021-10-20 07:53:45.307348 | primary | libguestfs: trace: set_verbose true\n2021-10-20 07:53:45.307555 | primary | libguestfs: trace: set_verbose \u003d 0\n2021-10-20 07:53:45.307594 | primary | libguestfs: trace: set_memsize 2048\n2021-10-20 07:53:45.307624 | primary | libguestfs: trace: set_memsize \u003d 0\n2021-10-20 07:53:45.307652 | primary | libguestfs: trace: set_smp 2\n2021-10-20 07:53:45.307684 | primary | libguestfs: trace: set_smp \u003d 0\n2021-10-20 07:53:45.307713 | primary | libguestfs: trace: set_network true\n2021-10-20 07:53:45.307741 | primary | libguestfs: trace: set_network \u003d 0\n2021-10-20 07:53:45.307785 | primary | libguestfs: trace: add_drive \"overcloud-full.qcow2\" \"readonly:false\" \"protocol:file\" \"discard:besteffort\"\n2021-10-20 07:53:45.307819 | primary | libguestfs: trace: add_drive \u003d 0\n2021-10-20 07:53:45.307845 | primary | libguestfs: trace: launch\n```\nis causing timeout, may be we can disable it https://opendev.org/openstack/tripleo-ci/src/branch/master/roles/oooci-build-images/defaults/main.yaml#L52 from here currently.","commit_id":"d5a01519c6cba0857c746727be805a1b5e0a5af0"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"cb7dcbe82cc3b4b6058ca6bc859026eb1ee2bd6c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"0bd7223d_ecb1eb0b","updated":"2021-10-20 04:22:52.000000000","message":"recheck","commit_id":"d5a01519c6cba0857c746727be805a1b5e0a5af0"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"6dbb26999f6d535bd1eb580b8e5540d0a6e54c07","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"7bf24d7a_3c2c80b8","updated":"2021-10-20 12:42:19.000000000","message":"recheck","commit_id":"d5a01519c6cba0857c746727be805a1b5e0a5af0"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"99ddb952c7854ec5771e3a9866ac90a086021d38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"c2e1687e_7dd9f6d1","updated":"2021-10-20 11:23:36.000000000","message":"recheck","commit_id":"d5a01519c6cba0857c746727be805a1b5e0a5af0"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"16e149b204d2f103ebd5d7e7b170572e937dbc9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"6c3d8b16_bec76bed","in_reply_to":"e9764e32_2ea22fe9","updated":"2021-10-20 08:41:46.000000000","message":"11:40 \u003c marios\u003e chandankumar: ack yeah i have it disabled still on the testproject patch \nhttps://review.rdoproject.org/r/c/testproject/+/35465/14/.zuul.yaml","commit_id":"d5a01519c6cba0857c746727be805a1b5e0a5af0"}],"diskimage_builder/elements/centos/root.d/10-centos-cloud-image":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"f6b08e2e0effef2f9d10800a5e09f6a2aad8ffe9","unresolved":true,"context_lines":[{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"},{"line_number":36,"context_line":"        fi"},{"line_number":37,"context_line":"        DIB_CLOUD_IMAGES\u003d${DIB_CLOUD_IMAGES:-http://cloud.centos.org/centos/${dib_release_path}/${ARCH}/images}"},{"line_number":38,"context_line":"    else"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"28e287c8_fd0879f2","line":35,"updated":"2021-09-02 15:14:33.000000000","message":"How was 8-stream working? (or is it not?)","commit_id":"b4f26d45e907bcde568f4d747cfed58bb3618624"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"13dfae1b690f1f047b4fd0b37e7624373364d841","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        elif [[ \"arm64\" \u003d~ \"$ARCH\" ]]; then"},{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"d4f484f1_738c3e34","line":32,"updated":"2021-09-10 12:40:00.000000000","message":"It\u0027s convenient to support just \"9\", but the canonical way is \"9-stream\" (or at least that\u0027s how it works for 8-stream).","commit_id":"13cfa06a96deaf154a61cd4083a3218f6b5624da"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"ba8f3ccdd4fb90aee0196141cea8d383198d8b19","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        elif [[ \"arm64\" \u003d~ \"$ARCH\" ]]; then"},{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"bae408ef_874ebc68","line":32,"in_reply_to":"27e22e86_f2b8941d","updated":"2021-10-01 08:51:42.000000000","message":"so we can\u0027t pass 9-stream because all our elements have pkg-map with keys that are explicitly \"8\" or \"9\" and not \"9-stream\" e.g. there\n\n        * https://opendev.org/openstack/tripleo-puppet-elements/src/commit/9753de458a3922ad765ceb8d41db6d76dbbad0b0/elements/overcloud-base/pkg-map#L9-L17\n\nso for example we hit https://logserver.rdoproject.org/65/35465/7/check/tripleo-stream9-development-buildimage-overcloud-full-master/342ee3c/build.log\n\n        * 2021-09-30 11:51:49.904 | Error: Unable to find a match: python-ipaddr\n\ndib_release will have to be passed as \u00279\u0027 for now anyhow.\n\nWRT supporting \u00279-stream\u0027 i think it still does... the if clause only applies when we pass in \u00279\u0027 for dib_release and we need to make it 9-stream ... i.e. precisely because there is no centos/9 only 9-stream @ https://cloud.centos.org/centos/","commit_id":"13cfa06a96deaf154a61cd4083a3218f6b5624da"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"07783f2ba3cf0ee4aa8ba29316c2f92522c9b47b","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        elif [[ \"arm64\" \u003d~ \"$ARCH\" ]]; then"},{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"27e22e86_f2b8941d","line":32,"in_reply_to":"c2366592_372cf632","updated":"2021-09-30 08:53:34.000000000","message":"if we pass \u00279-stream\u0027 in https://review.opendev.org/c/openstack/tripleo-ci/+/808177/4/roles/oooci-build-images/vars/centos-9.yaml then we can just use the original URL here i.e. no change needed\n\n        DIB_CLOUD_IMAGES\u003d${DIB_CLOUD_IMAGES:-http://cloud.centos.org/centos/${DIB_RELEASE}/${ARCH}/images}\n\nI will update","commit_id":"13cfa06a96deaf154a61cd4083a3218f6b5624da"},{"author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"change_message_id":"91c26550eff122079d3674942388bdf125fb6980","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        elif [[ \"arm64\" \u003d~ \"$ARCH\" ]]; then"},{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"c2366592_372cf632","line":32,"in_reply_to":"d4f484f1_738c3e34","updated":"2021-09-21 15:02:05.000000000","message":"Agreed, and it matches 8-stream.","commit_id":"13cfa06a96deaf154a61cd4083a3218f6b5624da"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"4016ac76cfd402f299dfa6155b8a325249c4c81b","unresolved":true,"context_lines":[{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"},{"line_number":36,"context_line":"        fi"}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"d69fdbd2_853ff3e5","line":33,"range":{"start_line":33,"start_character":29,"end_line":33,"end_character":37},"updated":"2021-10-07 15:40:27.000000000","message":"shouldn\u0027t we quote that string? Just to be clean.","commit_id":"c07ccb5f061709bed74a6b83e1ee96bbbebba008"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"c0c4a016c29ebf7ef2b95643f4f8e46d13d43383","unresolved":true,"context_lines":[{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"},{"line_number":36,"context_line":"        fi"}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"afd52576_5f01f57a","line":33,"range":{"start_line":33,"start_character":29,"end_line":33,"end_character":37},"in_reply_to":"d69fdbd2_853ff3e5","updated":"2021-10-11 12:10:44.000000000","message":"no spaces here, I think it\u0027s fine w/o quotes.","commit_id":"c07ccb5f061709bed74a6b83e1ee96bbbebba008"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"4016ac76cfd402f299dfa6155b8a325249c4c81b","unresolved":true,"context_lines":[{"line_number":57,"context_line":"        BASE_IMAGE_FILE\u003d${BASE_IMAGE_FILE:-CentOS-${DIB_RELEASE}-${ARCH}-${DIB_FLAVOR}.qcow2.xz}"},{"line_number":58,"context_line":"    else"},{"line_number":59,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":60,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":61,"context_line":"        else"},{"line_number":62,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"},{"line_number":63,"context_line":"        fi"}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"afee5605_59296037","line":60,"range":{"start_line":60,"start_character":29,"end_line":60,"end_character":37},"updated":"2021-10-07 15:40:27.000000000","message":"ditto","commit_id":"c07ccb5f061709bed74a6b83e1ee96bbbebba008"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"c0c4a016c29ebf7ef2b95643f4f8e46d13d43383","unresolved":true,"context_lines":[{"line_number":57,"context_line":"        BASE_IMAGE_FILE\u003d${BASE_IMAGE_FILE:-CentOS-${DIB_RELEASE}-${ARCH}-${DIB_FLAVOR}.qcow2.xz}"},{"line_number":58,"context_line":"    else"},{"line_number":59,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":60,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":61,"context_line":"        else"},{"line_number":62,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"},{"line_number":63,"context_line":"        fi"}],"source_content_type":"application/x-shellscript","patch_set":7,"id":"2bd6abc4_4cb5933f","line":60,"range":{"start_line":60,"start_character":29,"end_line":60,"end_character":37},"in_reply_to":"afee5605_59296037","updated":"2021-10-11 12:10:44.000000000","message":"same","commit_id":"c07ccb5f061709bed74a6b83e1ee96bbbebba008"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"63d85d86eb3f9ec45ea8323272666d66ff9b27f9","unresolved":true,"context_lines":[{"line_number":29,"context_line":"        elif [[ \"arm64\" \u003d~ \"$ARCH\" ]]; then"},{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"0f850dd9_d74291d8","line":32,"updated":"2021-10-12 06:25:01.000000000","message":"can\u0027t we define this at some upper level, as a global, instead of copy-pasting it multiple times? That would be better for the future imho.","commit_id":"f8568e7389c37f7dc6515f009f2e3a58ee416d6a"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"e7b0c521c04f27f24fe57a3f41f60e5ecb9b68b8","unresolved":true,"context_lines":[{"line_number":29,"context_line":"        elif [[ \"arm64\" \u003d~ \"$ARCH\" ]]; then"},{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"95f53f64_a88bbc56","line":32,"range":{"start_line":32,"start_character":17,"end_line":32,"end_character":28},"updated":"2021-10-12 12:48:41.000000000","message":"considering there will be no centos9, just centos9-stream, i think users should set DIB_RELEASE to 9-stream explicitly instead of handling it here.\n\nIF DIB_RELEASE is 9 can instead error out as not supported.","commit_id":"f8568e7389c37f7dc6515f009f2e3a58ee416d6a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"50865be223dc72d0af140f1675921f47cb5b74c9","unresolved":true,"context_lines":[{"line_number":29,"context_line":"        elif [[ \"arm64\" \u003d~ \"$ARCH\" ]]; then"},{"line_number":30,"context_line":"            ARCH\u003d\"aarch64\""},{"line_number":31,"context_line":"        fi"},{"line_number":32,"context_line":"        if [[ \"${DIB_RELEASE}\" \u003d\u003d \"9\" ]]; then"},{"line_number":33,"context_line":"            dib_release_path\u003d9-stream"},{"line_number":34,"context_line":"        else"},{"line_number":35,"context_line":"            dib_release_path\u003d${DIB_RELEASE}"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"61267626_525359c3","line":32,"range":{"start_line":32,"start_character":17,"end_line":32,"end_character":28},"in_reply_to":"95f53f64_a88bbc56","updated":"2021-10-12 15:12:46.000000000","message":"If it will be just 9-stream, so there\u0027s no need to identify and differentiate between them, I think both should be fine then? Because they will mean same","commit_id":"f8568e7389c37f7dc6515f009f2e3a58ee416d6a"}],"diskimage_builder/elements/centos/test-elements/9-strem-build-succeeds/element-deps":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a9abb637159c232bd5d942d28ec785b87d593e7c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":12,"id":"d74aa806_d82102de","updated":"2021-10-13 06:43:19.000000000","message":"typo on the directory 9-strem-build-succeeds \ni guess it must be causing the \u0027not a known test\u0027 error \n\n2021-10-13 05:01:48.699024 | ubuntu-bionic | centos/9-stream-build-succeeds : not a known test (see -l)\nhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f4b/806819/12/check/dib-functests-bionic-python3-image/f4bb55a/job-output.txt","commit_id":"f52c0a19813cee43345d62a94baf0d22ce1a4e3d"}],"diskimage_builder/elements/epel/pre-install.d/05-rpm-epel-release":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"13dfae1b690f1f047b4fd0b37e7624373364d841","unresolved":false,"context_lines":[{"line_number":6,"context_line":"set -eu"},{"line_number":7,"context_line":"set -o pipefail"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if [[ ${DISTRO_NAME} \u003d~ \"centos\" \u0026\u0026 \"${DIB_RELEASE}\" \u003c 9 ]]; then"},{"line_number":10,"context_line":"    # Centos has \"epel-release\" in extras, which is default enabled."},{"line_number":11,"context_line":"    ${YUM} install -y epel-release"},{"line_number":12,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"ec270412_15503bb0","line":9,"updated":"2021-09-10 12:40:00.000000000","message":"How will this work with \"9-stream\", which is the canonical DIB_RELEASE for it?","commit_id":"13cfa06a96deaf154a61cd4083a3218f6b5624da"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"ec00c554795b5bb1a86f97315448ca46c8d0a3f1","unresolved":false,"context_lines":[{"line_number":6,"context_line":"set -eu"},{"line_number":7,"context_line":"set -o pipefail"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if [[ ${DISTRO_NAME} \u003d~ \"centos\" \u0026\u0026 \"${DIB_RELEASE}\" \u003c 9 ]]; then"},{"line_number":10,"context_line":"    # Centos has \"epel-release\" in extras, which is default enabled."},{"line_number":11,"context_line":"    ${YUM} install -y epel-release"},{"line_number":12,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"d3bdc148_7e36dcd4","line":9,"in_reply_to":"66104ddd_c726a132","updated":"2021-09-29 11:02:57.000000000","message":"we can instead use \n\n${DIB_RELEASE} -lt \u00279\u0027\n\n\nthis will also catch 9 and 9-stream\n\n\n$ DIB_RELEASE\u003d8\n$ if [[ ${DIB_RELEASE} -lt \u00279\u0027 ]]; then echo \"wew\"; fi\nwew\n\n$ DIB_RELEASE\u003d\u00278\u0027\n$ if [[ ${DIB_RELEASE} -lt \u00279\u0027 ]]; then echo \"wew\"; fi\nwew\n\n$ DIB_RELEASE\u003d\u00279\u0027\n$ if [[ ${DIB_RELEASE} -lt \u00279\u0027 ]]; then echo \"wew\"; fi\n\n$ DIB_RELEASE\u003d\u00279-stream\u0027\n$ if [[ ${DIB_RELEASE} -lt \u00279\u0027 ]]; then echo \"wew\"; fi\n\n$ DIB_RELEASE\u003d\u00278-stream\u0027\n$ if [[ ${DIB_RELEASE} -lt \u00279\u0027 ]]; then echo \"wew\"; fi\nwew","commit_id":"13cfa06a96deaf154a61cd4083a3218f6b5624da"},{"author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"change_message_id":"91c26550eff122079d3674942388bdf125fb6980","unresolved":false,"context_lines":[{"line_number":6,"context_line":"set -eu"},{"line_number":7,"context_line":"set -o pipefail"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if [[ ${DISTRO_NAME} \u003d~ \"centos\" \u0026\u0026 \"${DIB_RELEASE}\" \u003c 9 ]]; then"},{"line_number":10,"context_line":"    # Centos has \"epel-release\" in extras, which is default enabled."},{"line_number":11,"context_line":"    ${YUM} install -y epel-release"},{"line_number":12,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"66104ddd_c726a132","line":9,"in_reply_to":"ec270412_15503bb0","updated":"2021-09-21 15:02:05.000000000","message":"This is going to fail for 9-stream, since it will move to the else clause, and try to install the epel-release package from dl.fedoraproject.org","commit_id":"13cfa06a96deaf154a61cd4083a3218f6b5624da"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"8bf59527e00215e1b57a9b40afa6c60f2201d9e4","unresolved":true,"context_lines":[{"line_number":6,"context_line":"set -eu"},{"line_number":7,"context_line":"set -o pipefail"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if [[ ${DISTRO_NAME} \u003d~ \"centos\" \u0026\u0026 \"${DIB_RELEASE}\" -lt \u00279\u0027 ]]; then"},{"line_number":10,"context_line":"    # Centos has \"epel-release\" in extras, which is default enabled."},{"line_number":11,"context_line":"    ${YUM} install -y epel-release"},{"line_number":12,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"fa5f0540_f9923451","line":9,"updated":"2021-10-06 22:02:27.000000000","message":"https://zuul.opendev.org/t/openstack/build/9b64574a6c474dc292305c64faa60857/log/job-output.txt#1854\n\nIt appears to be treating \"8-stream\" as 8 minus stream","commit_id":"013f7062213ad7aa327fae6e277cd4f18e062d2a"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"6be3d6d43194d5eadc78f698f7165bfdad8f37fd","unresolved":false,"context_lines":[{"line_number":6,"context_line":"set -eu"},{"line_number":7,"context_line":"set -o pipefail"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if [[ ${DISTRO_NAME} \u003d~ \"centos\" \u0026\u0026 \"${DIB_RELEASE}\" -lt \u00279\u0027 ]]; then"},{"line_number":10,"context_line":"    # Centos has \"epel-release\" in extras, which is default enabled."},{"line_number":11,"context_line":"    ${YUM} install -y epel-release"},{"line_number":12,"context_line":"else"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"eed6a7d5_35f586f0","line":9,"in_reply_to":"fa5f0540_f9923451","updated":"2021-10-07 08:30:30.000000000","message":"Done","commit_id":"013f7062213ad7aa327fae6e277cd4f18e062d2a"}],"diskimage_builder/elements/redhat-common/pkg-map":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"7d9fc1b1e8f83d0ac7a80443b234a6e2fc40d49a","unresolved":true,"context_lines":[{"line_number":25,"context_line":"        \"policycoreutils\": \"python3-policycoreutils\","},{"line_number":26,"context_line":"        \"policycoreutils-python\": \"policycoreutils-python-utils\""},{"line_number":27,"context_line":"      },"},{"line_number":28,"context_line":"      \"9\": {"},{"line_number":29,"context_line":"        \"libselinux-python\": \"python3-libselinux\","},{"line_number":30,"context_line":"        \"policycoreutils\": \"python3-policycoreutils\","},{"line_number":31,"context_line":"        \"policycoreutils-python\": \"policycoreutils-python-utils\""},{"line_number":32,"context_line":"       }"},{"line_number":33,"context_line":"    }"},{"line_number":34,"context_line":"  }"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"4f0998e6_b00ba9c1","line":31,"range":{"start_line":28,"start_character":0,"end_line":31,"end_character":1},"updated":"2021-10-06 07:54:46.000000000","message":"i think we no longer need this one since that merged [1] - so these are now the defaults with [2]\n\n\n[1] https://opendev.org/openstack/diskimage-builder/commit/017ba12eb099dc858aa63a276ffa09d70ef7fc40 \n[2] https://opendev.org/openstack/diskimage-builder/src/commit/82aa8c516db773f35a4dcfa64fc06cad1fc33e0f/diskimage_builder/elements/redhat-common/package-installs.yaml#L14-L16","commit_id":"13ea624f7692dc6948c79393c33e6c435dcf3a62"}]}