June 2021 - Devel - oVirt List Archives

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

Gerrit hook adds unrelated patches to bugs
by Nir Soffer 22 Jun '21

22 Jun '21

Gerrit hook is wrongly looking for https://bugzilla.redhat.com/ URLs in the commit message, and adding the patch to the bug. Example patch: https://gerrit.ovirt.org/c/vdsm/+/115339 I had to clean up the bug after the broken hook (see screenshot). The hook should really look only in the single URL in (one or more) Bug-Url headers: Bug-Url: https://bugzilla.redhat.com/ I reported this years ago (I think for Related-To:), and I remember we had a patch fixing this issue, but for some reason it was lost. Nir

3 2