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

Ceph - Libvirt handle multiple pool #162

Open
GaetanF opened this issue Nov 18, 2024 · 1 comment · May be fixed by #167
Open

Ceph - Libvirt handle multiple pool #162

GaetanF opened this issue Nov 18, 2024 · 1 comment · May be fixed by #167

Comments

@GaetanF
Copy link

GaetanF commented Nov 18, 2024

Hello,

After checking the ceph block on to_xml function of server.rb file. It's not possible to handle multiple Ceph pool on the /etc/foreman/ceph.conf file.

if ceph_args && volume.pool_name.include?(ceph_args["libvirt_ceph_pool"])

It will possible to handle them by splitting using comma the configuration string entry and check with them like this :
if ceph_args && ceph_args["libvirt_ceph_pool"]&.split(",")&.include?(volume.pool_name)

Regards

@ekohl
Copy link
Contributor

ekohl commented Dec 20, 2024

Since you already suggested the solution, would you be willing to submit a PR for this?

@GaetanF GaetanF linked a pull request Dec 23, 2024 that will close this issue
@ekohl ekohl linked a pull request Dec 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants