stratis-storage / stratisd

Easy to use local storage management for Linux.
https://stratis-storage.github.io
Mozilla Public License 2.0
794 stars 55 forks source link

Add D-Bus method to return Stratis pool-level metadata #3599

Closed mulkieran closed 4 months ago

mulkieran commented 4 months ago

Related https://github.com/stratis-storage/project/issues/692

mulkieran commented 4 months ago

I'm not convinced that it would be wrong to have a parameter to modify what the method returns. For example, what if in future we do want to get what the metadata on the devices is and compare it with the metadata to write. We do not need to implement this right now, but we might want the flexibility to add it without adding another method.

mulkieran commented 4 months ago

@jbaublitz https://github.com/stratis-storage/stratisd/pull/3606 which caches the BDA is up for your review.