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.

Sign up for GitHub

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

Jump to bottom

common: disable/enable pg_autoscaler #6591

Merged
merged 1 commit into from Jul 20, 2021
Merged

common: disable/enable pg_autoscaler #6591

merged 1 commit into from Jul 20, 2021

Conversation

guits
Copy link
Collaborator

@guits guits commented Jun 14, 2021 •
edited by dsavineau

The PG autoscaler can disrupt the PG checks so the idea here is to
disable it and re-enable it back after the restart is done.

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

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

@guits guits force-pushed the disable_pg_autoscaler branch 4 times, most recently from 558c093 to d51af04 Compare June 15, 2021 12:22
@guits
Copy link
Collaborator Author

guits commented Jun 15, 2021

jenkins test centos-container-all_in_one

@guits
Copy link
Collaborator Author

guits commented Jun 15, 2021

jenkins test centos-non_container-switch_to_containers

@guits
Copy link
Collaborator Author

guits commented Jun 15, 2021

jenkins test centos-container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Jun 15, 2021

jenkins test centos-container-purge

@guits guits force-pushed the disable_pg_autoscaler branch 2 times, most recently from 59e1242 to 3ad735f Compare June 15, 2021 13:18
@guits
Copy link
Collaborator Author

guits commented Jun 15, 2021

jenkins test centos-non_container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Jun 15, 2021

jenkins test centos-container-all_daemons

@guits
Copy link
Collaborator Author

guits commented Jun 15, 2021

jenkins test centos-non_container-switch_to_containers

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Jun 30, 2021
@guits guits removed the stale label Jul 7, 2021
@dsavineau
Copy link
Contributor

@guits do you plan to add the balancer stuff in this PR too or in another PR ?

@dsavineau
Copy link
Contributor

jenkins test centos-non_container-lvm_osds

@dsavineau
Copy link
Contributor

jenkins test centos-non_container-all_in_one

@dsavineau
Copy link
Contributor

jenkins test centos-container-all_in_one

@dsavineau
Copy link
Contributor

jenkins test centos-non_container-lvm_osds

1 similar comment
@guits
Copy link
Collaborator Author

guits commented Jul 19, 2021

jenkins test centos-non_container-lvm_osds

@guits guits force-pushed the disable_pg_autoscaler branch 5 times, most recently from 6adcddf to 160a151 Compare July 19, 2021 22:56
@guits
Copy link
Collaborator Author

guits commented Jul 19, 2021

jenkins test centos-container-update

@guits
Copy link
Collaborator Author

guits commented Jul 19, 2021

jenkins test centos-non_container-lvm_osds

@guits
common: disable/enable pg_autoscaler
bdd6aa7 
The PG autoscaler can disrupt the PG checks so the idea here is to
disable it and re-enable it back after the restart is done.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
@guits
Copy link
Collaborator Author

guits commented Jul 20, 2021

jenkins test centos-non_container-lvm_osds

3 similar comments
@dsavineau
Copy link
Contributor

jenkins test centos-non_container-lvm_osds

@guits
Copy link
Collaborator Author

guits commented Jul 20, 2021

jenkins test centos-non_container-lvm_osds

@guits
Copy link
Collaborator Author

guits commented Jul 20, 2021

jenkins test centos-non_container-lvm_osds

@guits guits merged commit 1303611 into master Jul 20, 2021
@guits guits deleted the disable_pg_autoscaler branch July 20, 2021 05:37
This was referenced Jul 20, 2021
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dsavineau dsavineau Awaiting requested review from dsavineau

Assignees
No one assigned
Labels
backport-stable-4.0 backport-stable-5.0 backport-stable-6.0 do-not-close fixes a bz
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@guits @dsavineau