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: set and unset osd flags #5460

Merged
merged 2 commits into from
Jun 29, 2020
Merged

Conversation

guits
Copy link
Collaborator

@guits guits commented Jun 25, 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.

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

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

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)
@guits guits added the DNM Do NOT merge label Jun 25, 2020
@guits guits added this to the 3.3z7 milestone Jun 25, 2020
@dsavineau dsavineau changed the title [skip ci] switch-to-containers: set and unset osd flags switch-to-containers: set and unset osd flags Jun 25, 2020
@dsavineau
Copy link
Contributor

jenkins test centos-non_container-switch_to_containers

@dsavineau
Copy link
Contributor

jenkins test centos-non_container-switch_to_containers

@guits guits modified the milestones: 3.3z7, 3.3z6 Jun 29, 2020
@dsavineau dsavineau removed the DNM Do NOT merge label Jun 29, 2020
@guits guits merged commit 8b8fa74 into stable-3.2 Jun 29, 2020
@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