Skip to content

Commit

Permalink
ceph-volume lvm.strategies fix a bug where incorrect template was used
Browse files Browse the repository at this point in the history
The template wouldn't render the total OSDs and the removed section was
repeated

Signed-off-by: Alfredo Deza <adeza@redhat.com>
(cherry picked from commit f473bce)
  • Loading branch information
Alfredo Deza authored and andrewschoen committed Aug 28, 2018
1 parent e56d02b commit 4855ac0
Showing 1 changed file with 2 additions and 7 deletions.
Expand Up @@ -129,15 +129,10 @@ def report_pretty(self):
db_size = str(disk.Size(b=(vg_extents['sizes'])))

string = ""
string += templates.ssd_volume_group.format(
targets='block.db',
total_lv_size=str(self.total_ssd_size),
total_lvs=vg_extents['parts'],
block_lv_size=db_size,
block_db_devices=', '.join([ssd['path'] for ssd in self.ssds]),
lv_size=str(disk.Size(b=(vg_extents['sizes']))),
string += templates.total_osds.format(
total_osds=len(self.hdds)
)

string += templates.ssd_volume_group.format(
target='block.db',
total_lv_size=str(self.total_ssd_size),
Expand Down

0 comments on commit 4855ac0

Please sign in to comment.