Incorrect handling of virsh `domblkinfo` output
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Triaged
|
Medium
|
Igor Brovtsin |
Bug Description
(originally found by ~peppepetra)
In some cases (network storage devices?), virsh `domblkinfo` command will return the following output:
```
Capacity: -
Allocation: -
Physical: -
```
(reference: https:/
This will cause rack to fail the virsh discovery with the following traceback:
```
2023-06-27 13:30:41 provisioningser
Traceback (most recent call last):
File "/snap/
File "/snap/
File "/snap/
File "/snap/
--- <exception caught here> ---
File "/snap/
result = result.
File "/snap/
return g.throw(self.type, self.value, self.tb)
File "/snap/
File "/snap/
result = inContext.theWork()
File "/snap/
File "/snap/
return self.currentCon
File "/snap/
return func(*args,**kw)
File "/snap/
result = func(*args, **kwargs)
File "/snap/
size = self.get_
File "/snap/
return int(self.
```
The workaround is to get rid of such storage devices and perform the discovery again.
I think we might want to handle '-' response separately. For a proper fix, we need to clarify in which cases virsh returns such responses and figure out how exactly do we want to handle storage in those cases.
Changed in maas: | |
milestone: | none → 3.5.0 |