[ovirt-devel] Re: Dealing with TaskID in vdsm api

2018-09-17 Thread Germano Veit Michel
Hi Marcin, Thanks for replying. In short, this is what I am doing: ~~~ from vdsm import client cli = client.connect('localhost', 54321, use_tls=True) cli.SDM.update_volume(job_id=job, vol_info=vol, vol_attr=changes) ~~~ This last line does an API call. A taskID is generated in vdsm, I can only s

[ovirt-devel] Re: PLEASE READ: Stop merging to ovirt-engine master until further notice - you can now merge

2018-09-17 Thread Dafna Ron
Hi, Thanks to all that helped to resolve the issue. ovirt-engine passed in master CQ [1] and we have a new package in tested [2] so you can now go back to normal merging processes [1] https://jenkins.ovirt.org/view/Change%20queue%20jobs/job/ovirt-master_change-queue-tester/10340/ [2] https://reso

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Dafna Ron
I had to re-trigger didi's fix due to a issue in Jenkins but I can now confirm that the issue is resolved in ovirt-engine and that we have a new ovirt-engine package in tested for el7: https://jenkins.ovirt.org/view/Change%20queue%20jobs/job/ovirt-master_change-queue-tester/10340/ https://resource

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Yuval Turgeman
Ok, regarding the tar issue, there's another solution - since commons-compress hard coded the blocking factor to 1, while the default in tar is 20, we could continue creating the tar as we do today, but add a bunch of zeros to the end of the tarball. [yturgema@piggie ~/aa]$ ls -l ovirt-host-deploy

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Martin Perina
On Mon, 17 Sep 2018, 16:25 Ravi Shankar Nori, wrote: > host-deploy is still broken on master fc28 > Yes, there are multiple issues on FC28, but the question is if this fixed OST on CentOS? > On Mon, Sep 17, 2018 at 8:01 AM, Yuval Turgeman > wrote: > >> I'm pretty sure I verified this on el7 a

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Ravi Shankar Nori
host-deploy is still broken on master fc28 On Mon, Sep 17, 2018 at 8:01 AM, Yuval Turgeman wrote: > I'm pretty sure I verified this on el7 as well, i'll check again, but > thinking about it, tar will stop when it gets to the first empty block, so > if the record size on the engine's side is larg

[ovirt-devel] Re: why we only save lv that seg_start_pe=="0" in _reloadlvs

2018-09-17 Thread Nir Soffer
On Mon, Sep 17, 2018 at 3:51 PM pengyixiang wrote: > Hello, everyone! > if we reload lvs, why we don't update lvs that seg_start_pe != "0" ? > there's in [1] with red mark. > what seg_start_pe means? > It is kind of hidden in lvm manual, but if you read carefully lvs(8): -o|--optio

[ovirt-devel] why we only save lv that seg_start_pe=="0" in _reloadlvs

2018-09-17 Thread pengyixiang
Hello, everyone! if we reload lvs, why we don't update lvs that seg_start_pe != "0" ? there's in [1] with red mark. what seg_start_pe means? If this lv is a vm template, may seg_start_pe is not zero, and we should ignore it? [1] vdsm/storage/lvm.py def _reloadlvs(self, vgName, lvNames=N

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (vdsm) ] [ 14-08-2018 ] [ 006_migrations.migrate_vm ]

2018-09-17 Thread Dafna Ron
vdsm is passing in master so I think the regression is fixed. On Mon, Sep 17, 2018 at 12:45 PM Marcin Sobczyk wrote: > Hi, > > this has been addressed and fixed by https://gerrit.ovirt.org/#/c/94347/ > > Regards, Marcin > On 9/14/18 2:15 PM, Dafna Ron wrote: > > looking deeper, cause can also b

[ovirt-devel] Re: Dealing with TaskID in vdsm api

2018-09-17 Thread Marcin Sobczyk
Hi, I'm currently doing some work around vdsm API so maybe I can. I need some more detailed description of your problem though - a pseudo-code of the calls you're making with explanation what exactly is the problem would be ideal. Marcin On 9/17/18 7:04 AM, Germano Veit Michel wrote: Hello

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Yuval Turgeman
I'm pretty sure I verified this on el7 as well, i'll check again, but thinking about it, tar will stop when it gets to the first empty block, so if the record size on the engine's side is large and the end is filled with zeros, -b1 will make it stop at the first empty block so the next read on the

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (vdsm) ] [ 14-08-2018 ] [ 006_migrations.migrate_vm ]

2018-09-17 Thread Marcin Sobczyk
Hi, this has been addressed and fixed by https://gerrit.ovirt.org/#/c/94347/ Regards, Marcin On 9/14/18 2:15 PM, Dafna Ron wrote: looking deeper, cause can also be: https://gerrit.ovirt.org/#/c/94249/ On Fri, Sep 14, 2018 at 1:08 PM, Dafna Ron > wrote: Hi,

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Galit Rosenthal
Didi, Is this what you are looking for https://ovirt-jira.atlassian.net/browse/OVIRT-2259 ? Galit On Mon, Sep 17, 2018 at 1:54 PM Dafna Ron wrote: > I think that in ovirt-engine we currently only build to centos. > since we have not had an engine build for 2 weeks (on master) I think we > shoul

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Sandro Bonazzola
Il giorno lun 17 set 2018 alle ore 13:27 Martin Perina ha scritto: > > > On Mon, Sep 17, 2018 at 12:47 PM Dafna Ron wrote: > >> I think that in ovirt-engine we currently only build to centos. >> since we have not had an engine build for 2 weeks (on master) I think we >> should merge and worry ab

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Martin Perina
On Mon, Sep 17, 2018 at 12:47 PM Dafna Ron wrote: > I think that in ovirt-engine we currently only build to centos. > since we have not had an engine build for 2 weeks (on master) I think we > should merge and worry about fc28 once it would be relevant. > So let's get the change reverted and let

[ovirt-devel] PLEASE READ: Stop merging to ovirt-engine master until further notice

2018-09-17 Thread Dafna Ron
Hello to all, I asked to stop merging last week but it seemed to have been ignored so writing again. *please stop merging to ovirt-engine master until further notice.* the project has been failing for the last two weeks on various different regressions. You continue to merge changes which means

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Dafna Ron
I think that in ovirt-engine we currently only build to centos. since we have not had an engine build for 2 weeks (on master) I think we should merge and worry about fc28 once it would be relevant. the failure we have now could be another regression missed since the project has been broken for two

[ovirt-devel] Re: Error Java SDK Issue??

2018-09-17 Thread Geschwentner, Patrick
Hi! Is the patch also already in the maven-repository: https://mvnrepository.com/artifact/org.ovirt.engine.api/sdk/4.2.4 I still get the error in this code: vmService = vmsService.vmService(vm.id()); ListRequest diskListRequest= vmService.diskAttachmentsService().list(); ListResponse

[ovirt-devel] Re: Error Java SDK Issue??

2018-09-17 Thread joey
Hi, Not contained in v4.2.4 yet. The patch has just got merged, and maybe a new release still needs some time. If having local dev environment, you could compile the sdk with the latest revision. Or, I could provide you with a compiled jar file for temporary use. :) > On Sep 17, 2018, at 17:

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Yedidyah Bar David
On Mon, Sep 17, 2018 at 11:49 AM Dafna Ron wrote: > > Didi, Marin, any update on the patch? Yes - it passed. Actually failed, but only after host-deploy: https://jenkins.ovirt.org/view/oVirt%20system%20tests/job/ovirt-system-tests_manual/3189/ I'd rather not merge it as-is, because it will brea

[ovirt-devel] ovirt-engine has been tagged (ovirt-engine-4.2.7)

2018-09-17 Thread Tal Nisan
___ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-le...@ovirt.org Privacy Statement: https://www.ovirt.org/site/privacy-policy/ oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archive

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 14-09-2018 ] [ 002_bootstrap.add_hosts ]

2018-09-17 Thread Dafna Ron
Didi, Marin, any update on the patch? On Sun, Sep 16, 2018 at 11:09 AM Yedidyah Bar David wrote: > On Sun, Sep 16, 2018 at 12:53 PM Yedidyah Bar David > wrote: > > > > On Fri, Sep 14, 2018 at 6:06 PM Martin Perina > wrote: > > > > > > > > > > > > On Fri, Sep 14, 2018 at 4:51 PM, Ravi Shankar

[ovirt-devel] Re: [CQ Failure Report] [oVirt Master - ovirt-engine] [16/9/18] [basic-suite.el7.x86_64]

2018-09-17 Thread Dafna Ron
this has been reported last week and Martin/Didi have been working to solve this. On Sun, Sep 16, 2018 at 11:35 AM, Ehud Yonasi wrote: > Hey everyone, > > We have a failure in CQ on the test of verify_add_hosts. > > Link to error: > https://pastebin.com/Bwqn19Pn > > Link to the CQ job: > http:/

[ovirt-devel] vdsm has been tagged (v4.20.40)

2018-09-17 Thread Milan Zamazal
Vdsm 4.20.40 for oVirt 4.2.7 rc1 ___ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-le...@ovirt.org Privacy Statement: https://www.ovirt.org/site/privacy-policy/ oVirt Code of Conduct: https://www.ovirt.org/community/about/co