)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"324a83c4435b0ee1f4e0c7e915c77a9a9f2f31ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5d2cfd33_3c0c7005","updated":"2023-01-17 21:15:13.000000000","message":"I think we should switch to curl-minimal for all redhat family from 9 onwards","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":34411,"name":"Neil Hanlon","email":"neil@shrug.pw","username":"nhanlon"},"change_message_id":"40234048ef8b46e060c863dcb06d9ca281ced946","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"18d52431_2b158576","updated":"2023-01-17 23:03:35.000000000","message":"I\u0027m in agreement with @sbaker - we should move to minimal for c9s+.\n\nIf users want/need the full curl, they can `dnf swap curl-minimal curl`. I had a local branch I hadn\u0027t yet sent for review which does this in the containerfile, which is another option, but I still prefer just keeping minimal.","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"6265a17934b91b223918911a663243b2e0cf7365","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ad1e5412_3b976628","updated":"2023-01-12 10:37:47.000000000","message":"do we need this also for centos stream 9 ? seems like tripleo had similar issue - https://bugs.launchpad.net/tripleo/+bug/1953156","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"9deadac80846339a5bb019c27677c75f4c8333d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"29715f37_71cf0fc3","updated":"2023-01-12 13:45:28.000000000","message":"kayobe-seed-images-rocky9 (non-voting) will be fixed with https://review.opendev.org/c/openstack/diskimage-builder/+/869424","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"28f8bca5a7e5848a761e58071c4efb6b6eb0d7c6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9c68ef93_d7a3211c","in_reply_to":"18d52431_2b158576","updated":"2023-01-18 05:51:46.000000000","message":"AFAICS we basically don\u0027t have any sane way to say \"curl|curl-minimal\" (we could do something like have a dummy cache-url package that depends on curl, that seems like it would be satisifed if either was installed, but that\u0027s not \"sane\" 😊 ).\n\nWhatever choice we make seems like it could be wrong.  I\u0027m almost of the opinion that we can\u0027t solve this with package-installs/pkg-map.  \n\nI feel like it would be OK to drop this and assume that the base system comes with curl -- replace it with a note in README that curl is required and probably a if [[ ! -f /usr/bin/curl ]]; then \"something useful\"; fi type message ...","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":23084,"name":"Bartosz Bezak","email":"bartosz@stackhpc.com","username":"b.bezak"},"change_message_id":"94f2d9eddfc2e8a0f0371d17e52df7d30bfb37d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e039cc31_8e1af585","in_reply_to":"29715f37_71cf0fc3","updated":"2023-01-12 13:45:55.000000000","message":"sorry wrong window :)","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"451b3f0419c16e925b6a159ca0b5f868bde99d10","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5b0377e5_62e1c4db","in_reply_to":"49e5b5b3_ccfffc4f","updated":"2023-01-20 05:16:39.000000000","message":"I\u0027m open to ideas, but i think the best move is not to play -- I9ccebe2dbf3a8682dab60c2070c5f78849e01446","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":34411,"name":"Neil Hanlon","email":"neil@shrug.pw","username":"nhanlon"},"change_message_id":"6fae17ce1f8065608251a595f57dbc3aeea74b95","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"49e5b5b3_ccfffc4f","in_reply_to":"9c68ef93_d7a3211c","updated":"2023-01-18 13:41:22.000000000","message":"in the RHEL world we can say e.g. `dnf install /usr/bin/curl` or have a `requires` against that, for which both curl and curl-minimal fulfill.\n\nIt seems for rocky, the cache-url element is what is requiring curl, so I suppose we could also conditionalize it there somehow?","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"3ba8d1be90d30fa94fbbed9d5b328da86704cfb4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"12182825_5c2eee4a","in_reply_to":"9c6961ff_317ae5b3","updated":"2023-01-17 06:58:56.000000000","message":"rockylinux 9, centos stream 9 and probably rhel 9 ubi images come with curl-minimal, id order to install curl, we would need to uninstall curl-minimal.\nI don\u0027t have an opinion if we should stick to curl-minimal or to curl, but curl-minimal works for all Kolla container uses - so probably it also is ok for all other uses (it just supports FTP/HTTP/HTTPS, not all the other protocols that full blown curl does).\nThird option is to stick to distro default, but allow the user to specify he wants curl, and then remove curl-minimal ;-)\nSee https://fedoraproject.org/wiki/Changes/CurlMinimal_as_Default","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"41c4a6a6899ffebaeceeaaa6dd243a2267a787c4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9c6961ff_317ae5b3","in_reply_to":"ad1e5412_3b976628","updated":"2023-01-16 20:39:26.000000000","message":"hrm, we have [1] that pulls in the full curl for the yum-minimal elements, which I think would get 9-stream.\n\nI wonder if it is a better idea to standardise by installing the full curl package?  \n\nWould just installing \"curl\" at [2] get that working?\n\n[1] https://opendev.org/openstack/diskimage-builder/src/branch/master/diskimage_builder/elements/yum-minimal/root.d/08-yum-chroot#L298\n[2] https://opendev.org/openstack/diskimage-builder/src/branch/master/diskimage_builder/elements/rocky-container/containerfiles/9#L3","commit_id":"51adbcceff05d7a909b23be7974d14b3f6e2f0fd"}]}