> On 12 Jul 2016, at 09:19, Yaniv Kaul <yk...@redhat.com> wrote:
> 
> 
> 
> On Tue, Jul 12, 2016 at 10:12 AM, Francesco Romani <from...@redhat.com 
> <mailto:from...@redhat.com>> wrote:
> 
> 
> ----- Original Message -----
> > From: "Dan Kenigsberg" <dan...@redhat.com <mailto:dan...@redhat.com>>
> > To: "Nadav Goldin" <ngol...@redhat.com <mailto:ngol...@redhat.com>>, 
> > from...@redhat.com <mailto:from...@redhat.com>
> > Cc: "Eyal Edri" <ee...@redhat.com <mailto:ee...@redhat.com>>, "Nir Soffer" 
> > <nsof...@redhat.com <mailto:nsof...@redhat.com>>, "Yaniv Kaul" 
> > <yk...@redhat.com <mailto:yk...@redhat.com>>, "Martin
> > Perina" <mper...@redhat.com <mailto:mper...@redhat.com>>, "Tolik Litovsky" 
> > <tlito...@redhat.com <mailto:tlito...@redhat.com>>, "infra" 
> > <infra@ovirt.org <mailto:infra@ovirt.org>>
> > Sent: Monday, July 11, 2016 10:12:48 PM
> > Subject: Re: Build failed in Jenkins: ovirt_master_system-tests #243
> >
> > On Thu, Jul 07, 2016 at 07:00:35PM +0300, Nadav Goldin wrote:
> > > Seems like [1], as ovirt-srv19  has fresh new FC24 installation,
> > > virtlogd is not enabled by default:
> > > ● virtlogd.service - Virtual machine log manager
> > >    Loaded: loaded (/usr/lib/systemd/system/virtlogd.service; indirect;
> > > vendor preset: disabled)
> > >    Active: inactive (dead)
> > >      Docs: man:virtlogd(8)
> > >            http://libvirt.org <http://libvirt.org/>
> > > we can add it to puppet for now.
> > >
> > > [1] https://bugzilla.redhat.com/show_bug.cgi?id=1290357 
> > > <https://bugzilla.redhat.com/show_bug.cgi?id=1290357>
> >
> > Francesco, shouldn't vdsm require virtlogd explicitly?
> 
> Disclaimer: I didn't read yet the rest of the thread.
> 
> We choose not to, trying to prevent issues:
> https://gerrit.ovirt.org/#/c/55189/1 <https://gerrit.ovirt.org/#/c/55189/1>
> Related-To: https://bugzilla.redhat.com/show_bug.cgi?id=1318902 
> <https://bugzilla.redhat.com/show_bug.cgi?id=1318902>
> 
> I thought and still do it's quite a mistake - if we think a feature of our 
> friends from libvirt is immature - we should tell them that, hopefully 
> accompanied by bugs.

It’s not so much feature itself, rather it’s integration into OSes. I think 
there was enough feedback, a random google search shows a load of issues 
reported at the time this dependency was introduced, so I hope they learned 
from it

> I'd like to believe they don't enable features by default unless they believe 
> the features are mature and ready for general consumption. If that's not the 
> case, we should talk with them.

IMO it was a bit hasty on their part, and we just didn’t have time to resolve 
all the virtlogd daemon management in RHEL and RHEVH in time

Thanks,
michal


> Y.
> 
> 
> We want to reenable and require it in the near future:
> https://bugzilla.redhat.com/show_bug.cgi?id=1321010 
> <https://bugzilla.redhat.com/show_bug.cgi?id=1321010>
> 
> HTH,
> 
> --
> Francesco Romani
> RedHat Engineering Virtualization R & D
> Phone: 8261328
> IRC: fromani
> 

_______________________________________________
Infra mailing list
Infra@ovirt.org
http://lists.ovirt.org/mailman/listinfo/infra

Reply via email to