Re: [ovirt-devel] [rhev-devel] Fwd: [ACTION REQUIRED] update ovirt-js-dependencies

2017-04-25 Thread Greg Sheremeta
Not yet, but in a few days: https://gerrit.ovirt.org/#/c/76023/ It's ok to install the fc25 one. On Tue, Apr 25, 2017 at 11:59 AM, Jakub Niedermertl wrote: > Hi Greg, > > I can't find package ovirt-js-dependencies-0.0.4 for fedora 24. Is there > any? > > Thanks > Jakub >

Re: [ovirt-devel] [rhev-devel] Fwd: [ACTION REQUIRED] update ovirt-js-dependencies

2017-04-25 Thread Jakub Niedermertl
Package for fc25 works well. I was just surprised that `dnf update` was not enough when I have 'tested' repo added. Thanks for the patch for package for fc24. On Tue, Apr 25, 2017 at 6:08 PM, Greg Sheremeta wrote: > Not yet, but in a few days:

Re: [ovirt-devel] OST failure NoClassDefFoundError

2017-04-25 Thread Oved Ourfali
I guess you need to upgrade vdsm-jsonrpc-java. Did you try that? On Tue, Apr 25, 2017 at 12:57 AM, Roy Golan wrote: > I get this on basic suite from a built RPM[1]: > > Caused by: java.lang.NoClassDefFoundError: org/apache/commons/lang/StringUtils > at >

Re: [ovirt-devel] OST failure NoClassDefFoundError

2017-04-25 Thread Roy Golan
Its OST, I have no option to upgrade On Tue, Apr 25, 2017 at 8:59 AM Oved Ourfali wrote: > I guess you need to upgrade vdsm-jsonrpc-java. > Did you try that? > > On Tue, Apr 25, 2017 at 12:57 AM, Roy Golan wrote: > >> I get this on basic suite from a

Re: [ovirt-devel] [ACTION SUGGESTED] can't start VMs on master? run engine-setup

2017-04-25 Thread Arik Hadas
On Mon, Apr 24, 2017 at 7:37 PM, Greg Sheremeta wrote: > Hi, > > I noticed today that I couldn't start a VM on master. Turns out I needed > to run engine-setup to pull in some database changes (I assume). For what > it's worth, the stacktrace [1] didn't say anything about

Re: [ovirt-devel] OST failure NoClassDefFoundError

2017-04-25 Thread Piotr Kliczewski
I seems that there is engine side patch missing which module.xml update. Please rebase [2] on top of the HEAD and it should be fine. On Tue, Apr 25, 2017 at 8:49 AM, Roy Golan wrote: > Its OST, I have no option to upgrade > > On Tue, Apr 25, 2017 at 8:59 AM Oved Ourfali

Re: [ovirt-devel] Subject: [ OST Failure Report ] [ oVirt master ] [ 24-04-2017 ] [import_template_from_glance]

2017-04-25 Thread Fred Rolland
Looking at it On Tue, Apr 25, 2017 at 12:11 AM, Nadav Goldin wrote: > Test failed: add_secondary_storage_domains/import_template_from_glance > > Link to suspected patches: https://gerrit.ovirt.org/#/c/74382/ > > Link to Job:

Re: [ovirt-devel] Which is the official oVirt docker repository, 'ovirt' or 'ovirtorg'?

2017-04-25 Thread Barak Korren
On 18 April 2017 at 16:22, Juan Hernández wrote: > Hello, > > Both exist: > > https://hub.docker.com/u/ovirt > https://hub.docker.com/u/ovirtorg > > The first, 'ovirt', has some branding, but it is empty, no images. > > The second, 'ovirtorg', has no branding, but hosts

Re: [ovirt-devel] Subject: [ OST Failure Report ] [ oVirt master ] [ 24-04-2017 ] [import_template_from_glance]

2017-04-25 Thread Fred Rolland
Eli hi, It seems there is some issue in the squash patch [1]. Regarding the issue found by the OST, if you start from a fresh DB, wrong values will be inserted in the "spec_params" column in the vm_device table. [2] We will get '58ca7b19-0071-00c0-01d6-0212' instead of a map like {

Re: [ovirt-devel] Subject: [ OST Failure Report ] [ oVirt master ] [ 24-04-2017 ] [import_template_from_glance]

2017-04-25 Thread Eli Mesika
Hi Please review the fixing patch https://gerrit.ovirt.org/#/c/76013/2 On Tue, Apr 25, 2017 at 3:00 PM, Fred Rolland wrote: > Eli hi, > > It seems there is some issue in the squash patch [1]. > > Regarding the issue found by the OST, if you start from a fresh DB, wrong >

Re: [ovirt-devel] Subject: [ OST Failure Report ] [ oVirt master ] [ 24-04-2017 ] [import_template_from_glance]

2017-04-25 Thread Eli Mesika
The fix for that regression was merged to master, please sync and check again On Tue, Apr 25, 2017 at 5:14 PM, Eli Mesika wrote: > Hi > > Please review the fixing patch > https://gerrit.ovirt.org/#/c/76013/2 > > On Tue, Apr 25, 2017 at 3:00 PM, Fred Rolland

Re: [ovirt-devel] Which is the official oVirt docker repository, 'ovirt' or 'ovirtorg'?

2017-04-25 Thread Nir Soffer
On Tue, Apr 25, 2017 at 2:23 PM Barak Korren wrote: > On 18 April 2017 at 16:22, Juan Hernández wrote: > > Hello, > > > > Both exist: > > > > https://hub.docker.com/u/ovirt > > https://hub.docker.com/u/ovirtorg > > > > The first, 'ovirt', has some

Re: [ovirt-devel] Subject: [ OST Failure Report ] [ oVirt master ] [ 24-04-2017 ] [import_template_from_glance]

2017-04-25 Thread Gil Shinar
Hi Eli, When was it merged? I looked at the patch in your former message and it is not merged. Master is still failing on add_secondary_storage_domains Thanks Gil On Tue, Apr 25, 2017 at 5:19 PM, Eli Mesika wrote: > The fix for that regression was merged to master, please

Re: [ovirt-devel] Which is the official oVirt docker repository, 'ovirt' or 'ovirtorg'?

2017-04-25 Thread Barak Korren
On 25 April 2017 at 17:47, Nir Soffer wrote: > On Tue, Apr 25, 2017 at 2:23 PM Barak Korren wrote: > > ovirtorg was opened by Dan because we could not find who owns ovirt, and > it is used for vdsm testing images and other stuff. > > We plan to move the

[ovirt-devel] [VDSM] Using pylint to improve your code

2017-04-25 Thread Nir Soffer
Hi all, Dan merge today a patch adding pylint-diff make target. With this target, you can check new code quickly: $ time make pylint ... real 1m31.133s user 1m29.934s sys 0m0.849s $ time make pylint-diff ... real 0m12.364s

Re: [ovirt-devel] [rhev-devel] Fwd: [ACTION REQUIRED] update ovirt-js-dependencies

2017-04-25 Thread Jakub Niedermertl
Hi Greg, I can't find package ovirt-js-dependencies-0.0.4 for fedora 24. Is there any? Thanks Jakub On Sun, Apr 23, 2017 at 6:24 PM, Greg Sheremeta wrote: > > -- Forwarded message -- > From: Greg Sheremeta > Date: Sun, Apr 23, 2017 at