----- Original Message ----- > From: "Dan Kenigsberg" <[email protected]> > To: "Liron Aravot" <[email protected]> > Cc: [email protected], [email protected], [email protected], > [email protected], "Michal Skrivanek" > <[email protected]>, "Vinzenz Feenstra" <[email protected]>, "Allon > Mureinik" <[email protected]> > Sent: Monday, September 1, 2014 11:23:45 PM > Subject: Re: feature review - ReportGuestDisksLogicalDeviceName > > On Sun, Aug 31, 2014 at 07:20:04AM -0400, Liron Aravot wrote: > > Feel free to review the the following feature. > > > > http://www.ovirt.org/Features/ReportGuestDisksLogicalDeviceName > > Thanks for posting this feature page. Two things worry me about this > feature. The first is timing. It is not reasonable to suggest an API > change, and expect it to get to ovirt-3.5.0. We are two late anyway. > > The other one is the suggested API. You suggest placing volatile and > optional infomation in getVMList. It won't be the first time that we > have it (guestIPs, guestFQDN, clientIP, and displayIP are there) but > it's foreign to the notion of "conf" reported by getVMList() - the set > of parameters needed to recreate the VM.
At first sight this seems something belonging to getVmStats (which is reporting already other guest agent information). -- Federico _______________________________________________ Users mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/users
