Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Barak Korren
On 19 October 2017 at 10:50, Allon Mureinik wrote: > The missing deps issue happened again this morning [1]: > > Why are you looking at OST check-patch job? it has little to do with how OST runs when it is used to check other projects (For example it runs all suits as

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Dan Kenigsberg
On Thu, Oct 19, 2017 at 10:29 AM, Martin Perina wrote: > > > On Thu, Oct 19, 2017 at 7:35 AM, Dan Kenigsberg wrote: >> >> On Wed, Oct 18, 2017 at 2:40 PM, Daniel Belenky >> wrote: >>> >>> Hi all, >>> >>> The following test is failing:

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Martin Perina
On Thu, Oct 19, 2017 at 11:04 AM, Martin Perina wrote: > > > On Thu, Oct 19, 2017 at 10:58 AM, Dan Kenigsberg > wrote: > >> On Thu, Oct 19, 2017 at 10:29 AM, Martin Perina >> wrote: >> > >> > >> > On Thu, Oct 19, 2017 at 7:35 AM, Dan

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Martin Perina
On Thu, Oct 19, 2017 at 10:53 AM, Eyal Edri wrote: > > > On Thu, Oct 19, 2017 at 11:51 AM, Barak Korren wrote: > >> >> >> On 19 October 2017 at 11:43, Eyal Edri wrote: >> >>> >>> >>> On Thu, Oct 19, 2017 at 10:50 AM, Allon Mureinik

Re: [ovirt-devel] [ovirt-users] Cockpit oVirt support

2017-10-19 Thread Marek Libra
Regarding libvirt, there's fallback to Libvirt provider (part of cockpit-machines) for VMs which are not managed by oVirt. For the oVirt ones, oVirt API handles all the actions. It's not yet implemented, but I'm considering to fallback to Libvirt for selected actions in case the oVirt API can't

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Barak Korren
On 19 October 2017 at 11:43, Eyal Edri wrote: > > > On Thu, Oct 19, 2017 at 10:50 AM, Allon Mureinik > wrote: > >> The missing deps issue happened again this morning [1]: >> >> Traceback (most recent call last): >> File

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Martin Perina
On Thu, Oct 19, 2017 at 10:58 AM, Dan Kenigsberg wrote: > On Thu, Oct 19, 2017 at 10:29 AM, Martin Perina > wrote: > > > > > > On Thu, Oct 19, 2017 at 7:35 AM, Dan Kenigsberg > wrote: > >> > >> On Wed, Oct 18, 2017 at 2:40 PM, Daniel

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Martin Perina
On Thu, Oct 19, 2017 at 7:35 AM, Dan Kenigsberg wrote: > On Wed, Oct 18, 2017 at 2:40 PM, Daniel Belenky > wrote: > >> Hi all, >> >> *The following test is failing:* 002_bootstrap.verify_add_hosts >> *All logs from failing job >>

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Allon Mureinik
The missing deps issue happened again this morning [1]: Traceback (most recent call last): File "/tmp/ovirt-q04eYYi5Ym/pythonlib/otopi/context.py", line 133, in _executeMethod method['method']() File "/tmp/ovirt-q04eYYi5Ym/otopi-plugins/otopi/packagers/yumpackager.py", line 256, in

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Eyal Edri
On Thu, Oct 19, 2017 at 11:51 AM, Barak Korren wrote: > > > On 19 October 2017 at 11:43, Eyal Edri wrote: > >> >> >> On Thu, Oct 19, 2017 at 10:50 AM, Allon Mureinik >> wrote: >> >>> The missing deps issue happened again this morning

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Dan Kenigsberg
On Thu, Oct 19, 2017 at 8:35 AM, Dan Kenigsberg wrote: > On Wed, Oct 18, 2017 at 2:40 PM, Daniel Belenky wrote: >> >> Hi all, >> >> The following test is failing: 002_bootstrap.verify_add_hosts >> All logs from failing job >> Only 2 engine patches

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Gal Ben Haim
Taken from the ansible-playbook log of host-0: TASK [ovirt-host-deploy-firewalld : Enable firewalld rules] failed: [lago-basic-suite-master-host-0] (item={u'service': u'glusterfs'}) => {"changed": false, "failed": true, "item": {"service": "glusterfs"}, "msg": "ERROR:

Re: [ovirt-devel] [ovirt-users] Cockpit oVirt support

2017-10-19 Thread Michal Skrivanek
> On 19 Oct 2017, at 09:06, Marek Libra wrote: > > Regarding libvirt, there's fallback to Libvirt provider (part of > cockpit-machines) for VMs which are not managed by oVirt. It’s layered, so the oVirt specifics are on top of the libvirt-based code. So the whole VM

Re: [ovirt-devel] Cockpit oVirt support

2017-10-19 Thread Michal Skrivanek
> On 18 Oct 2017, at 13:32, Barak Korren wrote: > > > > On 18 October 2017 at 10:24, Michal Skrivanek > wrote: > Hi all, > I’m happy to announce that we finally finished initial contribution of oVirt >

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Eyal Edri
On Thu, Oct 19, 2017 at 10:50 AM, Allon Mureinik wrote: > The missing deps issue happened again this morning [1]: > > Traceback (most recent call last): > File "/tmp/ovirt-q04eYYi5Ym/pythonlib/otopi/context.py", line 133, in > _executeMethod > method['method']() >

Re: [ovirt-devel] Cockpit oVirt support

2017-10-19 Thread Michal Skrivanek
> On 18 Oct 2017, at 11:42, Roy Golan wrote: > > > > On Wed, 18 Oct 2017 at 10:25 Michal Skrivanek > wrote: > Hi all, > I’m happy to announce that we finally finished initial contribution of oVirt > specific support into the Cockpit

Re: [ovirt-devel] Cockpit oVirt support

2017-10-19 Thread Roy Golan
On Thu, 19 Oct 2017 at 14:02 Michal Skrivanek wrote: > > > On 18 Oct 2017, at 11:42, Roy Golan wrote: > > > > > > > > On Wed, 18 Oct 2017 at 10:25 Michal Skrivanek < > michal.skriva...@redhat.com> wrote: > > Hi all, > > I’m happy to announce that

Re: [ovirt-devel] [ovirt-users] Cockpit oVirt support

2017-10-19 Thread Yaniv Kaul
On Thu, Oct 19, 2017 at 3:06 PM, Roy Golan wrote: > > > On Thu, 19 Oct 2017 at 14:02 Michal Skrivanek > wrote: > >> >> > On 18 Oct 2017, at 11:42, Roy Golan wrote: >> > >> > >> > >> > On Wed, 18 Oct 2017 at 10:25 Michal

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Martin Perina
So the real issue on adding a host is the same as I've today described in [2] and most probably caused by [3] (I reverted engine in my dev env prior this patch and host deploy finished successfully). Allon, do you have time to post a fix? If not I'll try to dig into your change and related

Re: [ovirt-devel] How to build a particular module of code

2017-10-19 Thread Greg Sheremeta
On Thu, Oct 19, 2017 at 3:08 PM, Roy Golan wrote: > > > On Thu, 19 Oct 2017 at 20:12 Greg Sheremeta wrote: > >> Since I needed to do a bunch of ovirt-engine recompiles today and I'm >> disabling checks to make it go faster, I figured I'd share: >> >> make

Re: [ovirt-devel] How to build a particular module of code

2017-10-19 Thread Roy Golan
On Thu, 19 Oct 2017 at 20:12 Greg Sheremeta wrote: > Since I needed to do a bunch of ovirt-engine recompiles today and I'm > disabling checks to make it go faster, I figured I'd share: > > make install-dev PREFIX=/home/greg/ovirt-engine >

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Allon Mureinik
Bloody hell. The original was also completely broken, and worked by chance. Damn it. This should fix it: https://gerrit.ovirt.org/#/c/82989/ On Thu, Oct 19, 2017 at 3:49 PM, Martin Perina wrote: > So the real issue on adding a host is the same as I've today described in >

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Allon Mureinik
Fix merged based on Alona and Martin's reviews. It seems to do the trick with my testing on my local engine, let's hope that's really it. On Thu, Oct 19, 2017 at 4:37 PM, Allon Mureinik wrote: > Bloody hell. The original was also completely broken, and worked by > chance.

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Barak Korren
On 19 October 2017 at 17:48, Allon Mureinik wrote: > Fix merged based on Alona and Martin's reviews. > It seems to do the trick with my testing on my local engine, let's hope > that's really it. > > Umm... It does not seem to be merged yet... -- Barak Korren RHV DevOps

[ovirt-devel] [webadmin] Fixed a bug when pressing Enter while focused on dialog field

2017-10-19 Thread Vojtech Szocs
Hi, when there's a dialog open and you're focused on some input (e.g. text) field and then press Enter, dialog's model is now properly updated with input's value before the dialog View's flush() method runs. See https://gerrit.ovirt.org/#/c/82944/ for details. Regards, Vojtech

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt Master ] [ bootstrap.verify_add_hosts ] [ 18/10/17 ]

2017-10-19 Thread Allon Mureinik
not my finest hour. thanks, Barak, it's merged now. On Thu, Oct 19, 2017 at 6:29 PM, Barak Korren wrote: > > > On 19 October 2017 at 17:48, Allon Mureinik wrote: > >> Fix merged based on Alona and Martin's reviews. >> It seems to do the trick with my

Re: [ovirt-devel] How to build a particular module of code

2017-10-19 Thread Greg Sheremeta
Since I needed to do a bunch of ovirt-engine recompiles today and I'm disabling checks to make it go faster, I figured I'd share: make install-dev PREFIX=/home/greg/ovirt-engine DEV_EXTRA_BUILD_FLAGS="-Danimal.sniffer.skip -Dcheckstyle.skip -Dgwt.compiler.localWorkers=1"

Re: [ovirt-devel] [webadmin] Fixed a bug when pressing Enter while focused on dialog field

2017-10-19 Thread Dan Kenigsberg
On Thu, Oct 19, 2017 at 6:20 PM, Vojtech Szocs wrote: > Hi, > > when there's a dialog open and you're focused on some input (e.g. text) > field and then press Enter, dialog's model is now properly updated with > input's value before the dialog View's flush() method runs. See >