Comment 11 for bug 1864279

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

FYI, for Bionic this isn't a trivial backport.
The change as it was added later would need at least
  commit 932862b8 conf: Rework and rename virDomainDeviceFindControllerModel
  commit 6ae6ffd8 qemu: Introduce qemuDomainFindSCSIControllerModel
Maybe more.
It is not un-doable, but it gets into an area where slowly but surely the regression risk might out weight the benefit. 18.04 is out for almowst 3 years now, coming up only now indicates it can't be the most evil issue possible. Many (but sadly not all) use cases have workarounds by specifying explicit IDs.
And finally - at least for the case reported here fixing it in Bionic wouldn't even help as you want/need it for UCA based on later versions.

On the bright side, the versions in UCA that you ask for have those reworks applied. There the code should match more easily.

i'm willing to give this a deeper look IF someone actually cares about this in Bionic (not Bionuc-UCA-*). So I'll set the bug task there to Won't fix and would ask anyone affected to update the bug to make an argument for why we really need it (that reasoning will be needed for an SRU anyway).