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

switch_to_containers: don't set noup flag #5425

Merged
merged 2 commits into from
Jun 18, 2020
Merged

Conversation

guits
Copy link
Collaborator

@guits guits commented Jun 17, 2020

We shouldn't set this flag when running switch_to_containers playbook.
Otherwise the playbook fails waiting for pgs to be clean.

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

Signed-off-by: Guillaume Abrioux gabrioux@redhat.com
(cherry picked from commit b91d60d)

@guits guits changed the title [skip ci] switch_to_containers: don't set noup flag switch_to_containers: don't set noup flag Jun 17, 2020
@guits
Copy link
Collaborator Author

guits commented Jun 17, 2020

jenkins test centos-non_container-switch_to_containers

@guits
Copy link
Collaborator Author

guits commented Jun 17, 2020

jenkins test centos-non_container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Jun 17, 2020

jenkins test centos-container-all_daemons

@guits guits force-pushed the guits-bp-5243-3.2 branch 2 times, most recently from d8f3747 to 0e95393 Compare June 17, 2020 15:30
@dsavineau
Copy link
Contributor

jenkins test centos-non_container-switch_to_containers

@dsavineau dsavineau added this to the 3.3z6 milestone Jun 17, 2020
The workflow in this playbook should be the same than in rolling_update,
we should first set noout and nodeep-scrub flags before migrating the
first osd and unset osd flags after the last osd is migrated.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit 2cfaa05)
We shouldn't set this flag when running switch_to_containers playbook.
Otherwise the playbook fails waiting for pgs to be clean.

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

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
(cherry picked from commit b91d60d)
@dsavineau dsavineau changed the title switch_to_containers: don't set noup flag [skip ci] switch_to_containers: don't set noup flag Jun 17, 2020
@dsavineau dsavineau changed the title [skip ci] switch_to_containers: don't set noup flag switch_to_containers: don't set noup flag Jun 17, 2020
@dsavineau
Copy link
Contributor

jenkins test centos-non_container-switch_to_containers

@guits guits merged commit b7ec4a9 into stable-3.2 Jun 18, 2020
@guits guits deleted the guits-bp-5243-3.2 branch June 18, 2020 07:56
@guits guits modified the milestones: 3.3z6, 3.3z7 Jun 25, 2020
@guits guits restored the guits-bp-5243-3.2 branch June 25, 2020 15:12
@dsavineau dsavineau deleted the guits-bp-5243-3.2 branch June 29, 2020 13:25
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.

None yet

2 participants