]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
osd, tools: refactor OSDMap::calc_pg_upmaps (simplify the code) 44002/head
authorJosh Salomon <josh.salomon@gmail.com>
Thu, 13 Jan 2022 02:23:07 +0000 (02:23 +0000)
committerLaura Flores <lflores@redhat.com>
Thu, 13 Jan 2022 02:25:14 +0000 (02:25 +0000)
commit86d6d110b4892bbb5b708e1aac55b70bc2b8fe7f
treeb1fb642f54abf84311995afc152eb901b4e89c5c
parente5c3c680d43c84415375262f12127fc424a1d0de
osd, tools: refactor OSDMap::calc_pg_upmaps (simplify the code)

This is the first commit in a series of commits that aims at adding a primary balancer to Ceph and improving the current upmap balancer functionality. This first commit focuses on simplifying (refactoring) the code of `calc_pg_upmaps` so it is easier to change in the future. This PR keeps the existing functionality as-is and does not change anything but the code structure.

As part of the work is major refactoring of OSDMap::calc_pg_upmaps, the first thing is adding an --upmap-seed param to osdmaptool so test results can be compared without the random factor.

Other changes made:
    - Divided sections of `OSDMap::calc_pg_upmaps` into their own separate functions
    - Renamed tmp to tmp_osd_map
    - Changed all the occurances of 'first' and 'second' in the function to more meaningful names.

Signed-off-by: Josh Salomon <josh.salomon@gmail.com>
src/osd/OSDMap.cc
src/osd/OSDMap.h
src/tools/osdmaptool.cc