On Mon, Jan 12, 2015 at 10:22:51AM +0200, Mykola Golub wrote: > On Sun, Jan 11, 2015 at 09:33:57AM -0800, Sage Weil wrote: > > > By the way I took another look and I'm not sure that it is worth > > duplicating all of the tree logic for a tree view. It seems easier to > > either include this optionally in the tree output (the utilzation calc is > > simpler than the tree traversal stack)... or generalize it somehow? > > Note, we already have duplication, at least CrushWrapper::dump_tree() > and OSDMap::print_tree(). I will work on generalization, I think some > tree dumper in CrushWrapper.
I have updated the code. https://github.com/ceph/ceph/pull/3347 -- Mykola Golub -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
