- Devel - oVirt List Archives

selenium.common.exceptions.TimeoutException: Message: Clusters menu is not displayed
by Yedidyah Bar David 29 Jun '21

29 Jun '21

On Wed, Jun 23, 2021 at 1:03 PM Code Review <gerrit(a)ovirt.org> wrote: > > From Jenkins CI <jenkins(a)ovirt.org>: > > Jenkins CI has posted comments on this change. ( https://gerrit.ovirt.org/c/ovirt-system-tests/+/115191 ) > > Change subject: Make the ansible_inventory fixture backend-independent > ...................................................................... > > > Patch Set 16: Continuous-Integration-1 > > Build Failed > > https://jenkins.ovirt.org/job/ovirt-system-tests_standard-check-patch/17451/ : FAILURE Hi all, Can someone please have a look and decide if this failure is reasonable? The screenshot of the failed "Open cluster list view" does not show anything suspicious at all [1], to me, but not sure how it should otherwise look - e.g. should it include a pointer, which might imply where we tried to move, etc. It timed out 3 minutes after a successful login [2]. If it's not supposed to include a pointer, perhaps we should try to make selenium do include a pointer in the screenshots - I have a feeling this can help debug other UI issues. But I don't know this code at all... Thanks and best regards, [1] https://jenkins.ovirt.org/job/ovirt-system-tests_standard-check-patch/17451… [2] https://jenkins.ovirt.org/job/ovirt-system-tests_standard-check-patch/17451… > > > -- > To view, visit https://gerrit.ovirt.org/c/ovirt-system-tests/+/115191 > To unsubscribe, or for help writing mail filters, visit https://gerrit.ovirt.org/settings > > Gerrit-Project: ovirt-system-tests > Gerrit-Branch: master > Gerrit-Change-Id: Id07012f8fc3a972a3b62d32f2271d5747514e13a > Gerrit-Change-Number: 115191 > Gerrit-PatchSet: 16 > Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> > Gerrit-Reviewer: Andrej Cernek <acernek(a)redhat.com> > Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> > Gerrit-Reviewer: Dafna Ron <dron(a)redhat.com> > Gerrit-Reviewer: Dusan Fodor <dfodor(a)redhat.com> > Gerrit-Reviewer: Gal Ben Haim <galbh2(a)gmail.com> > Gerrit-Reviewer: Galit Rosenthal <grosenth(a)redhat.com> > Gerrit-Reviewer: Jenkins CI <jenkins(a)ovirt.org> > Gerrit-Reviewer: Marcin Sobczyk <msobczyk(a)redhat.com> > Gerrit-Reviewer: Name of user not set #1001916 > Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> > Gerrit-Comment-Date: Wed, 23 Jun 2021 10:03:15 +0000 > Gerrit-HasComments: No > Gerrit-Has-Labels: Yes > Gerrit-MessageType: comment > -- Didi

3 4

2FA requirement for oVirt GitHub org?
by Janos Pasztor 28 Jun '21

28 Jun '21

Hey folks, I just noticed that we don't have 2FA enforcement on the oVirt GitHub org. [image: image.png] Is this just an oversight or is it intentional? May I propose to turn 2FA enforcement on? Janos

5 5

Broken commit-msg hook in gerrit
by Nir Soffer 25 Jun '21

25 Jun '21

In ovirt-imageio developer docs, we ask to install the commit-msg hook from gerrit: wget -P .git/hooks https://gerrit.ovirt.org/tools/hooks/commit-msg This downloads now a broken script that does not add a ChangeId header. In the past I think it returned something like the attached file, without the code for adding signed-off-by. I suggest to replace the current file at: https://gerrit.ovirt.org/tools/hooks/commit-msg Or if we want to keep the default file for some reason, add: https://gerrit.ovirt.org/tools/ovirt/hooks/commit-msg This file contains the code for adding the signed-off-by header, so so developers do not need to add this manually: # Add Signed-off-by trailer. sob=$(git var GIT_AUTHOR_IDENT | sed -n 's/^$.*>$.*$/Signed-off-by: \1/p') git interpret-trailers --in-place --trailer "$sob" "$1" Later we need to replace this ugly shell script with proper code. Nir

1 1

Add "ci system-test" command
by Nir Soffer 24 Jun '21

24 Jun '21

Similar to "ci build", "ci test", "ci merge" add a new command that triggers OST run. Running OST is tied now in vdsm (and engine?) to Code-Review: +2. This causes trouble and does not allow non-maintainers to use the convenient OST infrastructure. Expected flow: 1. User add a comment with "ci system-test" 2. OST flow building and running OST triggered

4 5

basic OST: Error initializing source docker, unauthorized: incorrect username or password
by Yedidyah Bar David 24 Jun '21

24 Jun '21

Hi all, On Mon, Jun 21, 2021 at 8:55 PM <jenkins(a)jenkins.phx.ovirt.org> wrote: > > Project: https://jenkins.ovirt.org/job/ovirt-system-tests_basic-suite-master_nightly/ > Build: https://jenkins.ovirt.org/job/ovirt-system-tests_basic-suite-master_nightly… This is a week-old build, but some (but not all) of the check-patch OST runs I did yesterday failed the same way, e.g. [1][2]: ====================================================================== failed on setup with "ost_utils.shell.ShellError: Command failed with rc=125. Stdout: Stderr: Trying to pull docker.io/selenium/hub:3.141.59-20210422... unable to retrieve auth token: invalid username/password: unauthorized: incorrect username or password Error: 1 error occurred: * Error initializing source docker://selenium/hub:3.141.59-20210422: unable to retrieve auth token: invalid username/password: unauthorized: incorrect username or password" ====================================================================== Is this a known issue? Perhaps related to docker's rate-limiting? [1] https://jenkins.ovirt.org/job/ovirt-system-tests_standard-check-patch/17434/ [2] https://jenkins.ovirt.org/job/ovirt-system-tests_standard-check-patch/17434… Best regards, -- Didi

2 3

Add OST flag to all oVirt projects
by Nir Soffer 23 Jun '21

23 Jun '21

Similar to Continuous-Integration flag, OST needs its own flag. OST will set this flag when OST build started from gerrit was started. We want to make this field mandatory for merging patches, like Continuous-Integration. Like Continuous-Integration maintainers should be able to override the value after OST completed, or set the value before OST started.

1 0

AV 8.4 for CentOS Linux
by Sandro Bonazzola 23 Jun '21

23 Jun '21

Hi all, New versions of AV packages in CentOS Virt SIG for CentOS Linux 8.4 are available on https://buildlogs.centos.org/centos/8/virt/x86_64/advanced-virtualization/ . Please help with testing so we can release it. hivex-1.3.18-21.el8 libguestfs-1.44.0-2.el8 libguestfs-winsupport-8.2-1.el8 libiscsi-1.18.0-8.el8 libnbd-1.6.0-3.el8 libosinfo-1.8.0-1.el8 libtpms-0.7.4-4.20201106git2452a24dab.el8 libvirt-dbus-1.3.0-2.el8 libvirt-7.0.0-14.1.el8 libvirt-python-7.0.0-1.el8 nbdkit-1.24.0-1.el8 netcf-0.2.8-12.el8 perl-Sys-Virt-7.0.0-1.el8 python-pyvmomi-6.7.1-7.el8 qemu-kvm-5.2.0-16.el8 seabios-1.14.0-1.el8 sgabios-0.20170427git-3.el8 SLOF-20200717-1.gite18ddad8.el8 supermin-5.2.1-1.el8 swtpm-0.4.2-1.20201201git2df14e3.el8 virglrenderer-0.8.2-1.el8 virt-v2v-1.42.0-10.el8 If no negative feedback will be reported, the plan is to tag it for release tomorrow. -- Sandro Bonazzola MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV Red Hat EMEA <https://www.redhat.com/> sbonazzo(a)redhat.com <https://www.redhat.com/> *Red Hat respects your work life balance. Therefore there is no need to answer this email out of your office hours.*

8 15

ovirt-engine has been tagged (ovirt-engine-4.4.7.5)
by Tal Nisan 23 Jun '21

23 Jun '21

1 0

ansible OST: Failed to hot-plug disk
by Yedidyah Bar David 23 Jun '21

23 Jun '21

Hi all, Please see [1], failed twice so far: "msg": "Fault reason is \"Operation Failed\". Fault detail is \"[Failed to hot-plug disk]\". HTTP response code is 400." Thanks and best regards, [1] https://jenkins.ovirt.org/job/ovirt-system-tests_ansible-suite-master/1972/ -- Didi

2 2

planned Jenkins restart
by Evgheni Dereveanchin 23 Jun '21

23 Jun '21

Hi everyone, I'll be performing a planned Jenkins restart within the next hour. No new CI jobs will be scheduled during this maintenance period. I will inform you once it is back online. -- Regards, Evgheni Dereveanchin

1 1