[ovirt-devel] Re: Propose Sharon Gratch as oVirt FE maintainer

2020-03-10 Thread Andrej Krejcir
+1 On Tue, 10 Mar 2020 at 13:28, Liran Rotenberg wrote: > +1 > > On Tue, Mar 10, 2020 at 2:17 PM Laura Wright wrote: > >> +1 >> >> On Tue, Mar 10, 2020 at 7:22 AM Scott Dickerson >> wrote: >> >>> +1 >>> >>> On Tue, Mar 10, 2020, 3:53 AM Tomas Jelinek wrote: >>> Hey all, Sharon

[ovirt-devel] Re: OST network suite is failing

2020-02-06 Thread Andrej Krejcir
This patch should fix the NPE: https://gerrit.ovirt.org/#/c/106794/ On Thu, 6 Feb 2020 at 10:59, Benny Zlotnik wrote: > I think it was fixed[1], is the patch rebased? > > [1] https://gerrit.ovirt.org/#/c/106772/ > > On Thu, Feb 6, 2020 at 11:53 AM Dominik Holler wrote: > > > > Hello Tal and An

[ovirt-devel] Missing java-client-kubevirt dependency for ovirt-engine

2020-01-30 Thread Andrej Krejcir
Hi, Installing the latest ovirt-engine master requires package: java-client-kubevirt >= 0.2.0 But the master nightly snapshot contains only version: java-client-kubevirt-0.1.0-1.20200102130005.git0e3be11 On the patch that bumped the version, there is a comment from the CI, that the change was no

[ovirt-devel] Re: Implicit Affinity Label checkbox

2020-01-27 Thread Andrej Krejcir
Hi, The behavior of affinity labels has changed since version 4.3.6.6. The 'Implicit affinity group' checkbox is there to enable the legacy behavior, it is not visible in version 4.4 or later. Now affinity labels do not have any direct effect, they only group VMs or hosts, and it is possible to a

[ovirt-devel] Re: Propose Lucia Jelinkova as oVirt FE maintainer

2019-11-26 Thread Andrej Krejcir
On Tue, 26 Nov 2019 at 15:21, Michal Skrivanek wrote: > Hi all, > > I’d like to propose Lucia Jelinkova as a frontend maintainer. We’re short > on GWT and ui-extensions maintenance recently and we need to raise up more > people who can share the responsibility and keep our frontend in a good > sh

[ovirt-devel] Re: SDK: Editing VM memory size and max memory attribute

2019-11-21 Thread Andrej Krejcir
Hi, the memory unit in the API is byte, but then later it is converted to MiB. These low numbers round down to 0, which causes the error. Here is the documentation: http://ovirt.github.io/ovirt-engine-api-model/4.2/#types/vm/attributes/memory Regards, Andrej On Wed, 20 Nov 2019 at 13:21, wro

[ovirt-devel] Recent ovirt-engine master patches are failing CI check-merged with UNSTABLE

2019-09-30 Thread Andrej Krejcir
Hi, recently, patches merged to ovirt-engine are failing check-merged because of FindBugs. It seems the first patch that fails is: https://gerrit.ovirt.org/#/c/98623/ Here is the jenkins failure: https://jenkins.ovirt.org/job/ovirt-engine_master_check-merged-el7-x86_64/10106/ I haven't checked a

[ovirt-devel] Re: Setting reason for VM shutdown

2019-09-26 Thread Andrej Krejcir
Hi, this is an existing RFE, and it will be finished soon: https://bugzilla.redhat.com/show_bug.cgi?id=1450351 Best regards, Andrej On Thu, 26 Sep 2019 at 13:16, wrote: > Hi all, > > I have a test case as follows: > > Enable "require reason for vm maintenance" on Cluster > Shutdown vm > Ensur

[ovirt-devel] Re: URGENT - ovirt-engine master is failing for the past 9 days

2019-09-02 Thread Andrej Krejcir
This patch fixes the issue: https://gerrit.ovirt.org/#/c/103019/ On Mon, 2 Sep 2019 at 10:16, Andrej Krejcir wrote: > I will post a fix later today. Sorry for the delay, It took me a while to > figure out what is the problem. > > On Sun, 1 Sep 2019 at 14:58, Tal Nisan wrote: >

[ovirt-devel] Re: URGENT - ovirt-engine master is failing for the past 9 days

2019-09-02 Thread Andrej Krejcir
I will post a fix later today. Sorry for the delay, It took me a while to figure out what is the problem. On Sun, 1 Sep 2019 at 14:58, Tal Nisan wrote: > OST failing for 9 days is totally unacceptable, the only reason I'm not > sending a revert patch is that it'll make a bloody mess with the upg

[ovirt-devel] Re: OST upgrade-from-release-suite-master update_cluster_versions failing

2019-08-28 Thread Andrej Krejcir
This patch is not the cause, it only removes unused code. But there was another patch set merged recently, that is probably the cause. I'm looking into it. Andrej On Tue, 27 Aug 2019 at 16:25, Dafna Ron wrote: > Adding Andrej Krejcir and Lucia = > Patch https://gerrit.ovirt.org

[ovirt-devel] Re: getStats exception in Engine

2019-07-31 Thread Andrej Krejcir
Hi, There is a bug[1] to fix this in mom. In mom version 0.5.13, the division operator was changed to behave as in python 3. [1] https://bugzilla.redhat.com/show_bug.cgi?id=1733546 Andrej On Wed, 31 Jul 2019 at 08:20, Ales Musil wrote: > Hi, > yesterday we have found out that there is except

[ovirt-devel] Re: ovirt 4.1 backup/restore

2019-05-13 Thread Andrej Krejcir
Hi, The RPMs for the current and past versions are here: https://resources.ovirt.org/pub/ The 4.1 version can be installed in the same way as described in the documentation: https://www.ovirt.org/documentation/install-guide/chap-Installing_oVirt.html But instead of installing 'ovirt-release43.rp

[ovirt-devel] Re: URGENT - ovirt-engine broken for 3 days Re: Subject: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 05-05-2019 ] [ upgrade_hosts ]

2019-05-09 Thread Andrej Krejcir
me know. >>> Note that ovirt-engine project is broken and if we cannot find the cause >>> relatively fast we should consider reverting the patch to allow a new >>> package to be built in CQ with other changes that were submitted. >>> >>> Thanks, >>&g

[ovirt-devel] Re: Subject: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 05-05-2019 ] [ upgrade_hosts ]

2019-05-07 Thread Andrej Krejcir
After running a few OSTs manually, it seems that the patch is the cause. Investigating... On Tue, 7 May 2019 at 14:58, Andrej Krejcir wrote: > Hi, > > The issue is probably not caused by the patch. > > This log line means that the VM does not exist in the DB: > > 2019-05-07

[ovirt-devel] Re: Subject: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 05-05-2019 ] [ upgrade_hosts ]

2019-05-07 Thread Andrej Krejcir
Hi, The issue is probably not caused by the patch. This log line means that the VM does not exist in the DB: 2019-05-07 06:02:04,215-04 WARN [org.ovirt.engine.core.bll.MigrateMultipleVmsCommand] (EE-ManagedThreadFactory-commandCoordinator-Thread-2) [33485140] Validation of action 'MigrateMultipl

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 13-02-2019 ] [ 001_initialize_engine.initialize_engine ]

2019-02-13 Thread Andrej Krejcir
Hi, I already posted a patch to fix it: https://gerrit.ovirt.org/#/c/97732/ We can merge it once CI finishes. Andrej On Wed, 13 Feb 2019 at 15:22, Dafna Ron wrote: > Hi, > we are failing on project ovirt-engine in master branch on initialize > engine > > Change suspected: https://gerrit.ovirt

[ovirt-devel] Re: Getting info about qemu guest agent into the engine

2018-11-19 Thread Andrej Krejcir
On Fri, 16 Nov 2018 at 17:09, Michal Skrivanek wrote: > > > > On 16 Nov 2018, at 13:21, Tomasz Baranski wrote: > > > > On 18/11/16 11:39, Michal Skrivanek wrote: > >> > >> > >>> On 16 Nov 2018, at 11:22, Tomasz Barański wrote: > >>> > >>> Hello, wise people! > >>> > >>> As a part of BZ1615830[1

[ovirt-devel] Failing compilation of engine to RPM using build-artefacts.sh

2018-11-05 Thread Andrej Krejcir
Hi, I use a VM with latest CentOS to compile engine to RPM. Steps: - git clone repo from my dev machine to the VM - run ovirt-engine/automation/build-artifacts.sh This used to work, but since about 2 weeks ago the build fails with an error message: + %py_byte_compile /usr/bin/python2 /root/rpmb

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

2018-10-22 Thread Andrej Krejcir
Hi, I hit this problem again. Otopi sends many zero bytes before a response. This time, on an engine installed from master RPM, not dev environment. The engine and host machines are latest Centos 7. Is there some workaround for deploying the host? Thanks, Andrej On Thu, 20 Sep 2018 at 08:59, Y

[ovirt-devel] Is using CompensationContext in child commands broken?

2018-08-16 Thread Andrej Krejcir
Hi, I'm changing UpdateClusterCommand to be non-transactive, and it would use CompensationContext for reverting DB operations. Reading through CommandBase code, I noticed the following flow seems to be broken: 1. A command without transaction is executed. 2. It executes a child command, with the

[ovirt-devel] Re: [ OST Failure Report ] [ oVirt Master (ovirt-engine) ] [ 23-07-2018 ] [ 008_basic_ui_sanity.start_grid ]

2018-07-24 Thread Andrej Krejcir
On Mon, 23 Jul 2018 at 15:03, Martin Perina wrote: > > > On Mon, Jul 23, 2018 at 1:32 PM, Dafna Ron wrote: > >> Hi, >> >> the issue seems to be that host-1 stopped responding and I can see some >> fluetd errors which we should look at. >> >> Jira opened to track this issue: >> https://ovirt-jira

[ovirt-devel] Re: ovirt-system-tests_he-basic-iscsi-suite-master fails due to timeout waiting for StoragePool to go up after vdsmd restart

2018-07-16 Thread Andrej Krejcir
Hi, The jenkins job is no longer available, I will run the test manually and see why it fails. Maybe this patch will fix the problem with the test: https://gerrit.ovirt.org/#/c/92734/ It ensures that all services are stopped before starting them again. The purpose of the test is to try restartin

[ovirt-devel] Re: Strange error in CI when checking an engine patch

2018-07-03 Thread Andrej Krejcir
After updating my build environment, the same error happens also locally. I run it in debugger and found the problem. The test case is not using mockito correctly. On Tue, 3 Jul 2018 at 15:14, Andrej Krejcir wrote: > Hi, > > I'm getting a strange error from CI on one of my patches

[ovirt-devel] Strange error in CI when checking an engine patch

2018-07-03 Thread Andrej Krejcir
Hi, I'm getting a strange error from CI on one of my patches[1]. The error only happens in CI, local build passes OK. [ERROR] Tests run: 29, Failures: 0, Errors: 4, Skipped: 0, Time elapsed: 0.073 s <<< FAILURE! - in org.ovirt.engine.core.bll.quota.QuotaManagerTest [ERROR] testConsumeStorageQuot

[ovirt-devel] Re: Recent he-basic-ansible-suite-4.2 failures

2018-07-02 Thread Andrej Krejcir
Hi, The first problem with 'Waiting for agent to be ready' looks like a race condition between HE broker and vdsm. The broker calls vdsm in the same moment that vdsm restarts but the broker keeps waiting for the result that never arrives. The timeout in the call is set to 15 minutes, but the test

[ovirt-devel] Changing unit labels from GB, MB, KB to GiB, MiB, KiB

2018-04-26 Thread Andrej Krejcir
Hi, In the ovirt-engine UI, we use units KB, MB, GB with the meaning of powers of 1024 bytes, instead of powers of 1000. Some time ago, there was a bug[1] that has changed some of the GB labels to GiB, but not in all places. This causes inconsistent messages in the UI, where one part of a sentence

Re: [ovirt-devel] Subject: [ OST Failure Report ] [ oVirt Master ] [Jan 15th 2018 ] [ 004_basic_sanity.verify_glance_import ]

2018-01-18 Thread Andrej Krejcir
The NPE was fixed by patch: https://gerrit.ovirt.org/#/c/86249/ On 15 January 2018 at 15:47, Dafna Ron wrote: > Hi, > > We had a failure in test : 004_basic_sanity.verify_glance_import >

Re: [ovirt-devel] [oVirt 4.2 Localization Question #5] VM SLA Policy command does not change anything.

2017-10-26 Thread Andrej Krejcir
Yes, the user should not be able to get this error message. The message means that the operation to update the SLA policies does not actually update any policy and has no effect. Andrej On 26 October 2017 at 10:11, Martin Sivak wrote: > Hi, > > I think the only way to get this message is to up

Re: [ovirt-devel] Failure in CQ testing

2017-10-09 Thread Andrej Krejcir
The HE setup is failing because one patch [1] was merged sooner than another needed patch [2]. In the time between, the setup does not work. With the patch [2], the he-basic suite passes: http://jenkins.ovirt.org/job/ovirt-system-tests_manual/1330/ Andrej [1] - https://gerrit.ovirt.org/#/c/78081

Re: [ovirt-devel] OST Failure he-basic

2017-10-02 Thread Andrej Krejcir
The fix has been merged. Andrej On 2 October 2017 at 17:29, Andrej Krejcir wrote: > Yes, that patch is the cause of the problem. It will be fixed by [1]. > > [1] - https://gerrit.ovirt.org/#/c/81404/ > > On 2 October 2017 at 17:19, Yaniv Kaul wrote: > >> >> >

Re: [ovirt-devel] OST Failure he-basic

2017-10-02 Thread Andrej Krejcir
Yes, that patch is the cause of the problem. It will be fixed by [1]. [1] - https://gerrit.ovirt.org/#/c/81404/ On 2 October 2017 at 17:19, Yaniv Kaul wrote: > > > On Mon, Oct 2, 2017 at 6:05 PM, Dominik Holler wrote: > >> Hi all, >> OST he-basic is currently failing [1] during setup "hosted-e