- Devel - oVirt List Archives

importing fixtures in ovirt-system-tests
by Yedidyah Bar David 08 Jul '21

08 Jul '21

Hi all, We have several different styles of where/how to import fixtures in ovirt-system-tests: - import directly inside the test code - import in conftest.py - import in fixtures modules - For all of above, both 'import *' and importing specific fixtures I think we should try to agree on a specific style and then follow it. One drawback of importing directly in test/fixtures code is that it's then impossible to override them in conftest.py. A drawback of importing '*' and/or doing this in conftest.py is that you might inadvertently import more than you want, or this might happen eventually (after more stuff are added), that this makes it harder to find what uses what, and that it risks unintended collisions in names - as opposed to intended overrides. A related issue is having to update many places if you add/change something. If there is some kind of "best practices" document somewhere that people are happy with, perhaps we should follow it. Otherwise, we should come up with our own. Personally I think I'd like to have a single file with all the imports, of specific fixtures (not '*'), and import this file from conftest.py of all the suites. Didn't actually try this and no idea what complications it might bring. Comments/ideas/opinions/decisions are welcome :-) Best regards, -- Didi

3 2

ppc64le mock environment is broken
by Sandro Bonazzola 07 Jul '21

07 Jul '21

Looking at https://jenkins.ovirt.org/blue/organizations/jenkins/ovirt-release_standard… It shows: # /usr/bin/yum --installroot /var/lib/mock/centos-stream-8-ppc64le-6cf77c626743751e199a195f6b050912-bootstrap-10125/root/ --releasever 8 install dnf dnf-plugins-core --setopt=tsflags=nocontexts command being executed and failing with: [2021-07-07T15:18:32.577Z] --> Finished Dependency Resolution [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: python(abi) = 3.6 [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: python3-libdnf >= 0.61.1 [2021-07-07T15:18:32.577Z] Error: Package: 1:python3-dateutil-2.6.1-6.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: python(abi) = 3.6 [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-plugins-core-4.0.21-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: python3-dbus [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: python3-libcomps >= 0.1.8 [2021-07-07T15:18:32.577Z] Error: Package: dnf-data-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: libreport-filesystem [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: libmodulemd >= 2.9.3 [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-plugins-core-4.0.21-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: python(abi) = 3.6 [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-plugins-core-4.0.21-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: python3-hawkey >= 0.46.1 [2021-07-07T15:18:32.577Z] Error: Package: dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: /bin/sh [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.577Z] Requires: python3-rpm >= 4.14.2-35 [2021-07-07T15:18:32.577Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.578Z] Requires: python3-hawkey >= 0.61.1 [2021-07-07T15:18:32.578Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.578Z] Requires: python3-libdnf [2021-07-07T15:18:32.578Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.578Z] Requires: /usr/libexec/platform-python [2021-07-07T15:18:32.578Z] Error: Package: python3-six-1.11.0-8.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.578Z] Requires: python(abi) = 3.6 [2021-07-07T15:18:32.578Z] Error: Package: python3-dnf-4.7.0-1.el8.noarch (centos-base-el8stream) [2021-07-07T15:18:32.578Z] Requires: python3-gpg [2021-07-07T15:18:32.578Z] You could try using --skip-broken to work around the problem [2021-07-07T15:18:32.578Z] You could try running: rpm -Va --nofiles --nodigest I initially thought it was a repo issue on centos stream side and reported it at https://bugzilla.redhat.com/show_bug.cgi?id=1979601 But it sorted out it's a CI configuration issue. The proof repositories are fine: LC_ALL=C sudo dnf --disablerepo=* --repofrompath=baseos1, http://mirror.centos.org/centos/8/BaseOS/ppc64le/os/ --enablerepo=baseos1 --releasever=8 --installroot=/tmp/test --nogpgcheck install :qdnf dnf-plugins-core --setopt=tsflags=nocontexts --setopt=arch=ppc64le correctly install the needed packages. This is blocking CI on ppc64le architecture. -- 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.*

1 4

Please resync centos-stream-el8stream/base mirror
by Sandro Bonazzola 07 Jul '21

07 Jul '21

Looks like the mirror needs a re-sync: [2021-07-07T06:24:14.859Z] [MIRROR] libpmem-1.6.1-1.el8.x86_64.rpm: Status code: 404 for http://mirrors.phx.ovirt.org/repos/yum/centos-stream-el8stream/base/Package… (IP: 66.187.230.49) [2021-07-07T06:24:14.859Z] [MIRROR] libpmem-1.6.1-1.el8.x86_64.rpm: Status code: 404 for http://mirrors.phx.ovirt.org/repos/yum/centos-stream-el8stream/base/Package… (IP: 66.187.230.49) [2021-07-07T06:24:14.859Z] [MIRROR] libpmem-1.6.1-1.el8.x86_64.rpm: Status code: 404 for http://mirrors.phx.ovirt.org/repos/yum/centos-stream-el8stream/base/Package… (IP: 66.187.230.49) [2021-07-07T06:24:14.859Z] [FAILED] libpmem-1.6.1-1.el8.x86_64.rpm: Status code: 404 for http://mirrors.phx.ovirt.org/repos/yum/centos-stream-el8stream/base/Package… (IP: 66.187.230.49) [2021-07-07T06:24:14.859Z] Error: Error downloading packages: [2021-07-07T06:24:14.859Z] Status code: 404 for http://mirrors.phx.ovirt.org/repos/yum/centos-stream-el8stream/base/Package… (IP: 66.187.230.49) Thanks -- 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.*

4 3

Reminder: oVirt conference call for papers closes in 20 days
by Sandro Bonazzola 05 Jul '21

05 Jul '21

A kind reminder that oVirt conference call for papers closes in 20 days! Looking forward to user stories, developers onboarding journeys, integration stories, new features presentation, old features nobody knows about because nobody presented them before and more you see fit. You can find call for paper at https://blogs.ovirt.org/ovirt-2021-online-conference/ Thanks, -- 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.*

1 0

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