This looks fine, but I'm not sure why we aren't using "virsh pool-dumpxml", which would report the units consistently in the first place. For example, I see the following:
... <capacity unit='bytes'>61966548992</capacity> <allocation unit='bytes'>35070156800</allocation> <available unit='bytes'>26896392192</available> ...
I'm going to approve anyway, since it seems like that work would be well beyond the scope of this branch.
« Back to merge proposal
This looks fine, but I'm not sure why we aren't using "virsh pool-dumpxml", which would report the units consistently in the first place. For example, I see the following:
... >61966548992< /capacity> >35070156800< /allocation> >26896392192< /available>
<capacity unit='bytes'
<allocation unit='bytes'
<available unit='bytes'
...
I'm going to approve anyway, since it seems like that work would be well beyond the scope of this branch.