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] facts: fix 'set_fact rgw_instances with rgw multisite' #5804

Merged
merged 1 commit into from Sep 18, 2020

Conversation

guits
Copy link
Collaborator

@guits guits commented Sep 18, 2020

the current condition doesn't work, as soon as the first iteration is
done the condition makes next iterations skip since rgw_instances got
set with the first iteration.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1859872

Signed-off-by: Guillaume Abrioux gabrioux@redhat.com

@guits guits changed the title [skip ic] facts: fix 'set_fact rgw_instances with rgw multisite' facts: fix 'set_fact rgw_instances with rgw multisite' Sep 18, 2020
@guits
Copy link
Collaborator Author

guits commented Sep 18, 2020

jenkins test centos-non_container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Sep 18, 2020

jenkins test centos-container-all_in_one

1 similar comment
@guits
Copy link
Collaborator Author

guits commented Sep 18, 2020

jenkins test centos-container-all_in_one

@guits guits force-pushed the guits-bzboomrang branch 2 times, most recently from 0675009 to 1b9ae69 Compare September 18, 2020 12:51
the current condition doesn't work, as soon as the first iteration is
done the condition makes next iterations skip since `rgw_instances` got
set with the first iteration.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1859872

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
@dsavineau
Copy link
Contributor

jenkins test centos-non_container-rgw_multisite

@dsavineau
Copy link
Contributor

jenkins test centos-container-rgw_multisite

@dsavineau dsavineau changed the title facts: fix 'set_fact rgw_instances with rgw multisite' [skip ci] facts: fix 'set_fact rgw_instances with rgw multisite' Sep 18, 2020
@dsavineau dsavineau merged commit ff19c1d into master Sep 18, 2020
@dsavineau dsavineau deleted the guits-bzboomrang branch September 18, 2020 14:14
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

2 participants