On 11/20/20 11:50 AM, Martin Perina wrote:


On Fri, Nov 20, 2020 at 11:46 AM Marcin Sobczyk <msobc...@redhat.com <mailto:msobc...@redhat.com>> wrote:

    Posted https://gerrit.ovirt.org/#/c/112302/
    <https://gerrit.ovirt.org/#/c/112302/> to remove all 4.3 suites.
    Please review.


Could you please remove network suite 4.3 from that patch? Network team would like to keep to verify 4.3 hosts functionality. And they have already removed 4.3 execution from check-patch https://gerrit.ovirt.org/#/c/112291/ <https://gerrit.ovirt.org/#/c/112291/>
The plan is to execute 4.3 network suite only nightly
Done. If there is still a need to run a 4.3 suite nightly, I'd be pushing towards
creating a 4.3 branch and removing the code from master though.



    On 10/12/20 6:28 PM, Michal Skrivanek wrote:
    >> On 12 Oct 2020, at 14:49, Marcin Sobczyk <msobc...@redhat.com
    <mailto:msobc...@redhat.com>> wrote:
    >>
    >> Hi all,
    >>
    >> after minimizing the usage of lago in basic suite,
    >> and some minor adjustments in the network suite, we are finally
    >> able to remove lago OST plugin as a dependency [1].
    >>
    >> This however comes with a price of keeping lots of ugly
    ifology, i.e. [2][3].
    >> There's big disparity between OST runs we have on el7 and el8.
    >> There's also tons of symlink-based code sharing between suites
    - be it 4.3
    >> suites and master suites or simply different types of suites.
    >> The basic suite has its own 'test_utils', which is copied/symlinked
    >> in multiple places. There's also 'ost_utils', which is really
    messy ATM.
    >> It's very hard to keep track and maintain all of this...
    >>
    >> At this moment, we are able to run basic suite and network suite
    >> on el8, with prebuilt ost-images and without lago plugin.
    >> HE suites should be the next step. We have patches that make them
    >> py3-compatible that probably still need some attention [4][5].
    >> We don't have any prebuilt HE ost-images, but this will be handled
    >> in the nearest future.
    >>
    >> I think it's good time to detach ourselves from the legacy stuff
    >> and start with a clean slate. My proposition would be to branch
    >> out 4.3 in ovirt-system-tests and not use py2/el7 in the master
    >> branch at all. This would allow us to focus on py3, el8 and
    ost-images
    >> efforts while keeping the legacy stuff intact.
    >>
    >> WDYT?
    > Great. We don’t really need 4.3 that much anymore.
    >
    >> Regards, Marcin
    >>
    >> [1] https://gerrit.ovirt.org/#/c/111643/
    <https://gerrit.ovirt.org/#/c/111643/>
    >> [2]
    https://gerrit.ovirt.org/#/c/111643/6/basic-suite-master/control.sh
    <https://gerrit.ovirt.org/#/c/111643/6/basic-suite-master/control.sh>
    >> [3]
    
https://gerrit.ovirt.org/#/c/111643/6/basic-suite-master/test-scenarios/conftest.py
    
<https://gerrit.ovirt.org/#/c/111643/6/basic-suite-master/test-scenarios/conftest.py>
    >> [4] https://gerrit.ovirt.org/108809
    <https://gerrit.ovirt.org/108809>
    >> [5] https://gerrit.ovirt.org/110097
    <https://gerrit.ovirt.org/110097>
    >>



--
Martin Perina
Manager, Software Engineering
Red Hat Czech s.r.o.
_______________________________________________
Devel mailing list -- devel@ovirt.org
To unsubscribe send an email to devel-le...@ovirt.org
Privacy Statement: https://www.ovirt.org/privacy-policy.html
oVirt Code of Conduct: 
https://www.ovirt.org/community/about/community-guidelines/
List Archives: 
https://lists.ovirt.org/archives/list/devel@ovirt.org/message/SMVY2RASJKDWATMVCCTYE3KRJN4E6MOL/

Reply via email to