Re: [ovirt-devel] useless warnings in engine.log

2015-07-15 Thread Omer Frenkel
- Original Message - > From: "Alexander Wels" > To: devel@ovirt.org > Cc: "Omer Frenkel" > Sent: Wednesday, July 15, 2015 4:09:02 PM > Subject: Re: [ovirt-devel] useless warnings in engine.log > > On Wednesday, July 15, 2015 08:23:20 AM Omer Frenkel wrote: > > Hi, > > lately i noticed

Re: [ovirt-devel] useless warnings in engine.log

2015-07-15 Thread Alexander Wels
On Wednesday, July 15, 2015 08:23:20 AM Omer Frenkel wrote: > Hi, > lately i noticed my engine.log filled with useless warnings, every 30 secs: > 2015-07-15 15:03:38,961 WARN > [org.ovirt.engine.core.branding.BrandingFilter] (default task-38) [] > Detected Locale is null, setting to default 'en_US

[ovirt-devel] useless warnings in engine.log

2015-07-15 Thread Omer Frenkel
Hi, lately i noticed my engine.log filled with useless warnings, every 30 secs: 2015-07-15 15:03:38,961 WARN [org.ovirt.engine.core.branding.BrandingFilter] (default task-38) [] Detected Locale is null, setting to default 'en_US' 2015-07-15 15:04:08,964 WARN [org.ovirt.engine.core.branding.Brand

Re: [ovirt-devel] Checkstyle changes in oVirt

2015-07-15 Thread Piotr Kliczewski
On Wed, Jul 15, 2015 at 9:51 AM, Eli Mesika wrote: > Well done > However, InteliJ instructions in > http://www.ovirt.org/Building_oVirt_Engine/IDE are very old and should be > aligned with current InteliJ version > Tal , can you update this as well? > Great job!!! It seems that eclipse instruc

Re: [ovirt-devel] Checkstyle changes in oVirt

2015-07-15 Thread Eli Mesika
Well done However, InteliJ instructions in http://www.ovirt.org/Building_oVirt_Engine/IDE are very old and should be aligned with current InteliJ version Tal , can you update this as well? - Original Message - > From: "Ramesh Nachimuthu" > To: "Tal Nisan" , "rhev-devel" , > devel@ovir

[ovirt-devel] engine-setup new dep: ovirt-setup-lib

2015-07-15 Thread Sandro Bonazzola
Just a quick note for letting you know that ovirt-engine and ovirt-hosted-engine-setup are going to require a new package: ovirt-setup-lib[1] It includes common code to the 2 setups. It will be available in next nightly snapshot. [1] https://gerrit.ovirt.org/#/admin/projects/ovirt-setup-lib --

Re: [ovirt-devel] VM status checks - request for comments

2015-07-15 Thread Michal Skrivanek
On Jul 15, 2015, at 08:13 , Oved Ourfali wrote: > Hi > > Nice initiative! > I reviewed some suspect infra-related flows, and added comments. > I must say that it is not very clear what you're looking for. > In one of the cases I reviewed, for example, there was a comment explaining > the logic