I like the above. One small suggestion: how about returning just the list of pool names since for the most part `#pool{}` should be an internal detail.

Along those lines, I was thinking that after merging your patch I would add something like `get_pool_name` to use instead of `dump_pool`.

If you update with you revised rm_group as discussed here, I'll get this merged.