emo/thin-provisioning-tools
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[thin_repair, thin_dump] --repair falg wasn't being passed through properly.

Browse Source 
Recent regression.
This commit is contained in:
Joe Thornber 2019-05-27 09:04:55 +01:00
parent 9e87ad8298
commit 65264911c1
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
thin-provisioning/metadata_dumper.cc
View File

@ -754,8 +754,6 @@ thin_provisioning::metadata_repair(block_manager<>::ptr bm, emitter::ptr e)
// for ourselves. We've had a few cases where people have // for ourselves. We've had a few cases where people have
// activated a pool on multiple hosts at once, which results in // activated a pool on multiple hosts at once, which results in
// the superblock being over written. // the superblock being over written.
gatherer g(*bm); gatherer g(*bm);
auto tm = open_tm(bm, superblock_detail::SUPERBLOCK_LOCATION); auto tm = open_tm(bm, superblock_detail::SUPERBLOCK_LOCATION);
auto p = g.find_best_roots(*tm); auto p = g.find_best_roots(*tm);
@ -786,7 +784,7 @@ thin_provisioning::metadata_repair(block_manager<>::ptr bm, emitter::ptr e)
dump_options opts; dump_options opts;
details_extractor de(opts); details_extractor de(opts);
device_tree_detail::damage_visitor::ptr dd_policy(details_damage_policy(false)); device_tree_detail::damage_visitor::ptr dd_policy(details_damage_policy(true));
walk_device_tree(*md->details_, de, *dd_policy); walk_device_tree(*md->details_, de, *dd_policy);
e->begin_superblock("", md->sb_.time_, e->begin_superblock("", md->sb_.time_,
@ -799,7 +797,7 @@ thin_provisioning::metadata_repair(block_manager<>::ptr bm, emitter::ptr e)
{ {
mapping_tree_detail::damage_visitor::ptr md_policy(mapping_damage_policy(true)); mapping_tree_detail::damage_visitor::ptr md_policy(mapping_damage_policy(true));
mapping_tree_emitter mte(opts, md, e, de.get_details(), mapping_damage_policy(false)); mapping_tree_emitter mte(opts, md, e, de.get_details(), mapping_damage_policy(true));
walk_mapping_tree(*md->mappings_top_level_, mte, *md_policy); walk_mapping_tree(*md->mappings_top_level_, mte, *md_policy);
} }