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

Various updates to support newer OSes #1953

Merged
merged 12 commits into from
Jun 12, 2024
Merged

Various updates to support newer OSes #1953

merged 12 commits into from
Jun 12, 2024

Conversation

zmc
Copy link
Member

@zmc zmc commented Jun 4, 2024

This isn't exhaustive; these are issues I ran into when updating ceph-devstack to work with centos 9 hosts.

zmc added 7 commits June 4, 2024 13:48
When attempting to provision a CentOS node, if the user leaves out ".stream",
try searching for that as well.

Signed-off-by: Zack Cerza <[email protected]>
Until it can get updated.

Signed-off-by: Zack Cerza <[email protected]>
Signed-off-by: Zack Cerza <[email protected]>
zmc added 2 commits June 4, 2024 14:41
... instead of hardcoding, which is more error-prone.

Signed-off-by: Zack Cerza <[email protected]>
This fixes an inconsistency where we'd change "centosX.stream" to "centosX" when
a job starts, submitting that to paddles.

Signed-off-by: Zack Cerza <[email protected]>
VallariAg
VallariAg previously approved these changes Jun 11, 2024
@zmc
Copy link
Member Author

zmc commented Jun 12, 2024

@VallariAg I found a bug that affected CentOS Stream reimages; the last two commits fix it. Could you review those as well? Thanks!

@kamoltat
Copy link
Member

Tested on my local docker container setup, works great and fixed the galaxy ansible failure for me

Copy link
Member

@VallariAg VallariAg left a comment

Choose a reason for hiding this comment

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

Looks good - works for me too!

@zmc zmc merged commit c1b06c2 into main Jun 12, 2024
8 checks passed
@zmc zmc deleted the newer-oses branch June 12, 2024 15:55
zmc added a commit to zmc/ceph that referenced this pull request Jun 19, 2024
idryomov pushed a commit to idryomov/ceph that referenced this pull request Jun 19, 2024
idryomov pushed a commit to idryomov/ceph that referenced this pull request Jun 19, 2024
idryomov pushed a commit to idryomov/ceph that referenced this pull request Jun 20, 2024
See: https://sentry.ceph.com/share/issue/21ed88d705854238bdafbf6711e795ee/
They're strings, not floats.
This surfaced as a result of ceph/teuthology#1953

Signed-off-by: Zack Cerza <[email protected]>
(cherry picked from commit 260a45a)
idryomov pushed a commit to idryomov/ceph that referenced this pull request Jun 20, 2024
See: https://sentry.ceph.com/share/issue/21ed88d705854238bdafbf6711e795ee/
They're strings, not floats.
This surfaced as a result of ceph/teuthology#1953

Signed-off-by: Zack Cerza <[email protected]>
(cherry picked from commit 260a45a)
idryomov pushed a commit to idryomov/ceph that referenced this pull request Jun 20, 2024
See: https://sentry.ceph.com/share/issue/21ed88d705854238bdafbf6711e795ee/
They're strings, not floats.
This surfaced as a result of ceph/teuthology#1953

Signed-off-by: Zack Cerza <[email protected]>
(cherry picked from commit 260a45a)
yuriw pushed a commit to ceph/ceph-ci that referenced this pull request Jun 27, 2024
See: https://sentry.ceph.com/share/issue/21ed88d705854238bdafbf6711e795ee/
They're strings, not floats.
This surfaced as a result of ceph/teuthology#1953

Signed-off-by: Zack Cerza <[email protected]>
(cherry picked from commit 260a45a)
(cherry picked from commit c675506)
yuriw pushed a commit to ceph/ceph-ci that referenced this pull request Jul 1, 2024
See: https://sentry.ceph.com/share/issue/21ed88d705854238bdafbf6711e795ee/
They're strings, not floats.
This surfaced as a result of ceph/teuthology#1953

Signed-off-by: Zack Cerza <[email protected]>
(cherry picked from commit 260a45a)
(cherry picked from commit 7fb2281)
yuriw pushed a commit to ceph/ceph-ci that referenced this pull request Jul 2, 2024
See: https://sentry.ceph.com/share/issue/21ed88d705854238bdafbf6711e795ee/
They're strings, not floats.
This surfaced as a result of ceph/teuthology#1953

Signed-off-by: Zack Cerza <[email protected]>
(cherry picked from commit 260a45a)
(cherry picked from commit 7fb2281)
idryomov pushed a commit to ceph/ceph that referenced this pull request Jul 8, 2024
See: https://sentry.ceph.com/share/issue/21ed88d705854238bdafbf6711e795ee/
They're strings, not floats.
This surfaced as a result of ceph/teuthology#1953

Signed-off-by: Zack Cerza <[email protected]>
(cherry picked from commit 260a45a)
(cherry picked from commit 7fb2281)
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.

3 participants