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

Not possible to run rolling update from Quincy to Reef (Stable-8.0) with a custom cluster name #7635

Open
ksr82 opened this issue Jan 21, 2025 · 1 comment

Comments

@ksr82
Copy link

ksr82 commented Jan 21, 2025

Bug Report

The tasks
ceph-ansible/infrastructure-playbooks/rolling_update.yml -> name: Set cluster configs
/ceph-ansible/roles/ceph-mon/tasks/main.yml -> name: Set cluster configs

Fail durring rolling update from Quincy to Reef if cluster name is different from "ceph"

What you expected to happen:
I would expect the tasks to take the cluster variable in the group_vars/all.yml into account

I was able to fix it by adding: cluster: "{{ cluster }}" to the tasks like so:

  • name: Set cluster configs
    ceph_config:
    action: set
    who: "{{ item.0.key }}"
    option: "{{ item.1.key }}"
    value: "{{ item.1.value }}"
    cluster: "{{ cluster }}"
    when: item.1.value != omit
    loop: "{{ ceph_cluster_conf | dict2dict }}"
    environment:
    CEPH_CONTAINER_IMAGE: "{{ ceph_docker_registry + '/' + ceph_docker_image + ':' + ceph_docker_image_tag if containerized_deployment | bool else None }}"
    CEPH_CONTAINER_BINARY: "{{ container_binary }}"

I would love to fix it myself and create a pull request for it, but I can't figure out how to - Seems like I'm not allowed to push brances to the project.

@guits
Copy link
Collaborator

guits commented Jan 22, 2025

As far as I can tell, custom cluster names have not been supported for quite a while.
That being said the code still allows to do that, I believe.

I would love to fix it myself and create a pull request for it, but I can't figure out how to - Seems like I'm not allowed to push brances to the project.

You need to fork the repository and create your own branch, then submit a pull request from your branch targeting the main branch of ceph-ansible

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

No branches or pull requests

2 participants