Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cents: add support for CentOS 9 #70

Merged
merged 2 commits into from
Jul 11, 2024
Merged

cents: add support for CentOS 9 #70

merged 2 commits into from
Jul 11, 2024

Conversation

epuertat
Copy link
Member

@epuertat epuertat commented Jan 9, 2023

Signed-off-by: Ernesto Puerta [email protected]

@epuertat epuertat force-pushed the centos_9 branch 2 times, most recently from 8d8b97f to 7916c3c Compare January 9, 2023 12:49
@nizamial09 nizamial09 force-pushed the centos_9 branch 4 times, most recently from 75355ce to bf64a6a Compare June 4, 2024 09:06
FROM quay.io/centos/centos:stream$CENTOS_VERSION as ceph-base
ARG CENTOS_VERSION

RUN dnf --disablerepo '*' --enablerepo=extras swap centos-linux-repos centos-stream-repos -y
RUN if [ "$CENTOS_VERSION" = "8" ]; then \
dnf --disablerepo '*' --enablerepo=extras centos-linux-repos centos-stream-repos swap -y; \
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the reason of the current failure:

Suggested change
dnf --disablerepo '*' --enablerepo=extras centos-linux-repos centos-stream-repos swap -y; \
dnf --disablerepo '*' --enablerepo="extras centos-linux-repos centos-stream-repos" swap -y; \

Copy link
Member

@nizamial09 nizamial09 Jul 11, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you @epuertat I might have forgotten to update but I removed centos 8 completely and went with centos9. centos8 was failing to pull in its dependencies for appstreams and others...

@nizamial09 nizamial09 force-pushed the centos_9 branch 6 times, most recently from 072ebd9 to c875781 Compare June 16, 2024 14:21
centos8 is failing contionusly to pull and install its repos so this is
an attempt to remove it completely and see if it passes

Signed-off-by: Nizamudeen A <[email protected]>
@nizamial09
Copy link
Member

i am merging this for now since all the builds are breaking for a while. With this we will only have the ceph-reef image in a broken state which i can take a look separately

 > [ 6/10] RUN bash -x /ceph/install-deps.sh     && dnf clean packages:
87.45 cortx-motr-2.0.0-1_git3252d623_any.el8.x86_64.r  27 MB/s |  17 MB     00:00    
87.71 cortx-motr-devel-2.0.0-1_git3252d623_any.el8.x8 2.7 MB/s | 718 kB     00:00    
88.97 Error: 
88.97  Problem: conflicting requests
88.97   - nothing provides python(abi) = 2.7 needed by cortx-motr-2.0.0-1_git3252d623_any.el8.x86_64 from @commandline
88.97   - nothing provides /usr/bin/python2 needed by cortx-motr-2.0.0-1_git3252d623_any.el8.x86_64 from @commandline
88.97   - nothing provides libpython2.7.so.1.0()(64bit) needed by cortx-motr-2.0.0-1_git3252d623_any.el8.x86_64 from @commandline
88.97   - nothing provides rubygem-net-ssh needed by cortx-motr-2.0.0-1_git3252d623_any.el8.x86_64 from @commandline
88.97 (try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)
89.01 + rm -fr /tmp/install-deps.7

@nizamial09 nizamial09 merged commit 6d996d1 into main Jul 11, 2024
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants