mulkieran
mulkieran
@jbaublitz I see that it is what stratisd is asking for. I don't think we need to do anything now that it's explained. Feel free to close if you're satisfied.
Content is at least as important as format. Identifiers which help sort out which thinpool, what filesystem, etc. are very important. It is really rough having to try to infer...
We have lot of functions that are very clear on the pool UUID, though, and could report that. That would really help us distinguish. The UUID is a persistent name...
See https://github.com/stratis-storage/stratisd/pull/1521 for another invariant about the size of the regions allocated to the variable length metadata.
Note that the addition of types in #1534 automatically enforces some of the invariants of interest.
It makes sense to decide the cache block size per pool, rather than have a constant for all pools.
Note that PR #1521 adds some FIXMEs, to help make it obvious where this could go wrong.
I took a stab at the filesystem's one and that turns out to require some thought. It's impossible to implement it in ```StratPool::create_filesystems``` unless ```ThinPool::create_filesystems``` is changed so that its...
This has drifted too far down my stack, I don't expect I'll get back to it before January.
@jbaublitz This is a puzzle problem, and not really urgent. If it conflicts w/ #1614, please mark it as blocked and leave it alone for now.