[virt-tools-list] [virt-manager PATCH] installertreemedia, unattended: Only use kernel args from libosinfo

[Pavel Hrdina]

On Mon, Apr 29, 2019 at 02:13:21PM +0200, Fabiano Fidêncio wrote:
> On Mon, Apr 29, 2019 at 1:42 PM Pavel Hrdina <phrdina at redhat.com> wrote:
> >
> > On Thu, Apr 25, 2019 at 03:42:36PM +0200, Fabiano Fidêncio wrote:
> > > When performing a tree based unattended installation, some distros
> > > require extra arguments to be passed to the kernel command line.
> > >
> > > Although those extra arguments are handled by
> > > _OsVariant::get_kernel_url_arg(), it's libosinfo responsability to
> > > properly inform them to the management apps.
> > >
> > > With that in mind and knowing that osinfo-db has been updated
> > > accordingly, we can fully rely on the command line returned by libosinfo
> > > and stop appending arguments returned by get_kernel_url_arg() when
> > > performing unattended installations.
> >
> > What happens if there is osinfo-db version installed that supports
> > unattended installation but the cmdline is still broken?
> 
> Then the cmdline would be broken.
> However, as there's *no* released code with unattended installation
> functionality, I think that those changes may be fine, Mainly because
> once we do release virt-manager we'd add a Requires: osinfo-db >=
> specific version.

Do we really want to have that strict dependency?  It would make sense
for some core functionality but this is a nice to have feature so I
would rather have it optional.

Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20190429/3a9aabd6/attachment.sig>