- Devel - oVirt List Archives

ovirt-engine-4.4.7 was branched
by Tal Nisan 30 Jun '21

30 Jun '21

Hi everyone, Today we've tagged what will be 4.4.7 RC build, to maintain stability and to allow work on 4.4.8 we have branched 4.4.7 in case of bugs found on the RC Build that need to be fixed. The general guidelines: For your 4.4.8 work - just push to master For 4.4.7 work (blockers/exceptions only!) - push to master and once it's merged backport to the ovirt-engine-4.4.7.z branch Thanks!

1 0

Vdsm: ovirt-4.4.7 branch created
by Milan Zamazal 30 Jun '21

30 Jun '21

Hi, ovirt-4.4.7 branch has been created for 4.4.7 backports. You can merge 4.4.8 stuff to master now. Regards, Milan

1 0

ovirt-engine has been tagged (ovirt-engine-4.4.7.6)
by Tal Nisan 30 Jun '21

30 Jun '21

1 0

oVirt CI mirror for centos stream advancedvirt-common repo
by Nir Soffer 30 Jun '21

30 Jun '21

I updated vdsm libvirt requirement to 7.0.0-14. The package exists in centos stream and rhel so the change is correct, but the install check fails in the CI. I found that adding the repo to check-patch.repos works: https://gerrit.ovirt.org/c/vdsm/+/115039/4/automation/check-patch.repos But depending on mirror.centos.org does not feel like the right way. I think keeping a local mirror is the right way. It looks like the stdci docs[1] are not maintained for a while, listing mirrors for fedora 30 and centos 7. Looking in the mirrors jobs[2] we have advanced-virtualization for centos[3] but it holds old versions (6.x). Can we add a local mirror for this repo? [1] https://ovirt-infra-docs.readthedocs.io/en/latest/CI/List_of_mirrors/index.… [2] https://jenkins.ovirt.org/search/?q=system-sync_mirrors [3] https://buildlogs.centos.org/centos/8/virt/x86_64/advanced-virtualization/P… Nir

3 3

cloning LVM partitions
by duparchy＠esrf.fr 29 Jun '21

29 Jun '21

Hi, What are the internal commands used to clone LVM partitions ? (you can simply indicate me the source code where this is done) Thanks

3 6

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