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

[skip ci] Fix hosts field in rolling_update playbook when mds are processed #5744

Merged
merged 1 commit into from Sep 8, 2020

Conversation

fmount
Copy link
Contributor

@fmount fmount commented Sep 8, 2020

In the OSP context, during the rolling update the playbook fails
with the following error:

'''
ERROR! The field 'hosts' has an invalid value, which includes an
undefined variable. The error was: list object has no element 0
'''

This PR just change the hosts field providing a valid mons group
value.

Closes: https://bugzilla.redhat.com/1876803
Signed-off-by: Francesco Pantano fpantano@redhat.com

In the OSP context, during the rolling update the playbook fails
with the following error:

'''
ERROR! The field 'hosts' has an invalid value, which includes an
undefined variable. The error was: list object has no element 0
'''

This PR just change the hosts field providing a valid mons group
value.

Closes: https://bugzilla.redhat.com/1876803
Signed-off-by: Francesco Pantano <fpantano@redhat.com>
@gfidente
Copy link
Contributor

gfidente commented Sep 8, 2020

lgmt, it's the same syntax we have in [1] which passes when group is empty

  1. https://github.com/ceph/ceph-ansible/blob/master/infrastructure-playbooks/rolling_update.yml#L436

@guits
Copy link
Collaborator

guits commented Sep 8, 2020

jenkins test centos-container-update

@guits
Copy link
Collaborator

guits commented Sep 8, 2020

jenkins test centos-container-external_clients

@guits
Copy link
Collaborator

guits commented Sep 8, 2020

jenkins test centos-container-lvm_batch

@guits
Copy link
Collaborator

guits commented Sep 8, 2020

jenkins test centos-non_container-switch_to_containers

@guits
Copy link
Collaborator

guits commented Sep 8, 2020

jenkins test centos-non_container-external_clients

@guits
Copy link
Collaborator

guits commented Sep 8, 2020

jenkins test centos-non_container-collocation

@dsavineau
Copy link
Contributor

jenkins test centos-non_container-collocation

@dsavineau
Copy link
Contributor

jenkins test centos-non_container-switch_to_containers

@dsavineau
Copy link
Contributor

unrelated CI failures and the update scenarios are passing

@dsavineau dsavineau changed the title Fix hosts field in rolling_update playbook when mds are processed [skip ci] Fix hosts field in rolling_update playbook when mds are processed Sep 8, 2020
@dsavineau dsavineau merged commit e65f9a5 into ceph:master Sep 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants