install: Use RHEL repo for Rocky Linux#557
Conversation
86f94af added Rocky Linux as a recognized distro and thus the script sets up the repo[0]. However the repo doesn't currently contain the required packages, and thus the installation fails. Rocky being fully compatible with RHEL, using its repo leads to a working installation. This aligns with the current advice from Rocky's own docs[1]. Fixes docker#264 docker#373 docker#468 docker#469 [0]: https://download.docker.com/linux/rocky/ [1]: https://docs.rockylinux.org/10/gemstones/containers/docker/ Signed-off-by: Bastien Traverse <neitsab@esrevart.net>
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
|
I'm not sure we should do this; we deliberately don't handle most derivatives; The plan is to provide add packages for Rocky, but not all are available yet, so wondering if we should either leave the script as-is, or adding a warning ( + delay?) to allow the user to cancel the script. Any thoughts, @vvoland ? |
|
TBF this is just a workaround so as not to leave the current situation broken; IMO the script had better state Rocky was unsupported until the repo was effectively setup, and only then 86f94af would have made sense. I drafted this PR because the replies to my comment on this commit invited to do so, but it is definitely a not-ideal workaround.
This is indeed the root cause and the first issue I wanted to report, but I couldn't find where to do do so (my question below the aforementioned commit didn't get a reply). It would indeed make more a lot more sense to finish building up the packages if that's planned/in doing and then have 86f94af. Leaving the script as-is results in a less-than-satisfying user experience:
That's basically me yesterday, and might generate a bit of useless fuss. |
vvoland
left a comment
There was a problem hiding this comment.
This will break docker sbx installation which supports Rocky 8: https://github.com/docker/sbx-releases
Official Rocky packages for Docker Engine/CLI are in the works and should be available soon, so I think it's just not worth the time to bother with such workarounds.
That's great to here, but until then I still think 86f94af should be reverted to avoid the making the install script fail due to the absence of packages in the added repo. Feel free to close this PR then :-) |
…stall Docker's get.docker.com script sets up a Rocky-specific repo (https://download.docker.com/linux/rocky/) that has no packages published, causing Docker installation to fail on both Rocky 8 and Rocky 9 monitor nodes. Add a Rocky-specific fallback that uses the CentOS Docker repo instead, which is fully compatible with Rocky Linux. This aligns with Rocky's own official documentation and the proposed upstream fix. Affects two code paths: - Monitor node setup via SSH (sdcm/cluster.py) - Cloud-init provisioning (sdcm/provision/common/utils.py) Upstream: docker/docker-install#557
Docker's get.docker.com script sets up a Rocky-specific repo (https://download.docker.com/linux/rocky/) that has no packages published, causing Docker installation to fail on Rocky monitor nodes. Add a Rocky-specific fallback that uses the CentOS Docker repo instead, which is fully compatible with Rocky Linux. This aligns with Rocky's own official documentation and the proposed upstream fix. Upstream: docker/docker-install#557
Docker's get.docker.com script sets up a Rocky-specific repo (https://download.docker.com/linux/rocky/) that has no packages published, causing Docker installation to fail on Rocky monitor nodes. Add a Rocky-specific fallback that uses the CentOS Docker repo instead, which is fully compatible with Rocky Linux. This aligns with Rocky's own official documentation and the proposed upstream fix. Upstream: docker/docker-install#557
Docker's get.docker.com script sets up a Rocky-specific repo (https://download.docker.com/linux/rocky/) that has no packages published, causing Docker installation to fail on Rocky monitor nodes. Add a Rocky-specific fallback that uses the CentOS Docker repo instead, which is fully compatible with Rocky Linux. This aligns with Rocky's own official documentation and the proposed upstream fix. Upstream: docker/docker-install#557
Rocky Linux minimal images have two issues that break monitor node setup: 1. Docker's get.docker.com script configures a Rocky-specific repo disabled (https://download.docker.com/linux/rocky/) that has no packages published. Add a fallback that uses the CentOS Docker repo instead, which is fully compatible. 2. wget is not included in Rocky minimal images but is needed by download_scylla_monitoring() to fetch the monitoring stack archive. Add it to RHEL/CentOS prerequisites. Upstream: docker/docker-install#557
Rocky Linux minimal images have two issues that break monitor node setup: 1. Docker's get.docker.com script configures a Rocky-specific repo disabled (https://download.docker.com/linux/rocky/) that has no packages published. Add a fallback that uses the CentOS Docker repo instead, which is fully compatible. 2. wget is not included in Rocky minimal images but is needed by download_scylla_monitoring() to fetch the monitoring stack archive. Add it to RHEL/CentOS prerequisites. Upstream: docker/docker-install#557
Rocky Linux minimal images have two issues that break monitor node setup: 1. Docker's get.docker.com script configures a Rocky-specific repo disabled (https://download.docker.com/linux/rocky/) that has no packages published. Add a fallback that uses the CentOS Docker repo instead, which is fully compatible. 2. wget is not included in Rocky minimal images but is needed by download_scylla_monitoring() to fetch the monitoring stack archive. Add it to RHEL/CentOS prerequisites. Upstream: docker/docker-install#557
86f94af added Rocky Linux as a recognized distro and thus the script sets up the repo0. However the repo doesn't currently contain the required packages, and thus the installation fails.
Rocky being fully compatible with RHEL, using its repo leads to a working installation. This aligns with the current advice from Rocky's own docs1.
Fixes #264 #373 #468 #469
- What I did
Add a test changing the value of
$repo_file_urlwhen rocky is detected to use rhel repo instead.- How I did it
Thinking a bit and writing as much
- How to verify it
By reading the code?
- Description for the changelog
install: improve Rocky Linux support
- A picture of a cute animal (not mandatory but encouraged)