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

quincy: pybind/mgr/autoscaler: Donot show NEW PG_NUM value if autoscaler is not on #47925

Merged
merged 1 commit into from Dec 5, 2022
Merged

quincy: pybind/mgr/autoscaler: Donot show NEW PG_NUM value if autoscaler is not on #47925

merged 1 commit into from Dec 5, 2022

Conversation

rzarzynski
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/56648

backport of #46761
parent tracker: https://tracker.ceph.com/issues/56136

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

@rzarzynski
pybind/mgr/autoscaler: Donot show NEW PG_NUM value if autoscaler is n…
e40b5dc 
…ot on

When noautscale is set, autoscale-status shows NEW PG_NUM
value if pool pg_num is more than 96. If autoscaler is in
off or warn mode for the pool then donot adjust the final
pg count for the pool.

Fixes: https://tracker.ceph.com/issues/56136

Signed-off-by: Prashant D <pdhange@redhat.com>
(cherry picked from commit 46ce0be)
@rzarzynski rzarzynski added this to the quincy milestone Sep 2, 2022
pdvian
pdvian approved these changes Sep 14, 2022
View reviewed changes
Copy link

@pdvian pdvian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @rzarzynski.

@ljflores
Copy link
Contributor

ljflores commented Dec 5, 2022

Rados suite review: http://pulpito.front.sepia.ceph.com/?branch=wip-yuri5-testing-2022-11-18-1554-quincy

Failures, unrelated:
1. https://tracker.ceph.com/issues/57311
2. https://tracker.ceph.com/issues/58146
3. https://tracker.ceph.com/issues/58046
4. https://tracker.ceph.com/issues/58097
5. https://tracker.ceph.com/issues/56000
6. https://tracker.ceph.com/issues/52321
7. https://tracker.ceph.com/issues/57754

Details:
1. rook: ensure CRDs are installed first - Ceph - Orchestrator
2. test_cephadm.sh: Error: Error initializing source docker://quay.ceph.io/ceph-ci/ceph:master - Ceph - Orchestrator
3. qa/workunits/rados/test_librados_build.sh: specify redirect in curl command
4. qa/workunits/post-file.sh: kex_exchange_identification: read: Connection reset by peer - Infrastructure
5. task/test_nfs: ERROR: Daemon not found: mds.a.smithi060.ujwxef. See cephadm ls - Ceph - Orchestrator
6. qa/tasks/rook times out: 'check osd count' reached maximum tries (90) after waiting for 900 seconds - Ceph - Orchestrator
7. test_envlibrados_for_rocksdb.sh: update-alternatives: error: alternative path /usr/bin/gcc-11 doesn't exist - Ceph - RADOS

@yuriw yuriw merged commit db5da35 into ceph:quincy Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pdvian pdvian pdvian approved these changes

@neha-ojha neha-ojha neha-ojha approved these changes

@kamoltat kamoltat Awaiting requested review from kamoltat

Assignees
No one assigned
Labels
core needs-qa pybind wip-yuri5-testing
Projects
None yet
Milestone
quincy
5 participants
@rzarzynski @ljflores @pdvian @neha-ojha @yuriw