[ovirt-devel] Re: Branching out 4.3 in ovirt-system-tests

2020-11-20 Thread Martin Perina 
On Fri, Nov 20, 2020 at 12:02 PM Marcin Sobczyk  wrote:

>
>
> On 11/20/20 11:50 AM, Martin Perina wrote:
> >
> >
> > On Fri, Nov 20, 2020 at 11:46 AM Marcin Sobczyk  > > wrote:
> >
> > Posted 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/
> > 
> > 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.
>

Definitely, I've just created ovirt-engine-4.3 from current master, so we
have current status saved and your removal patch can be merged to master

>
> >
> >
> > On 10/12/20 6:28 PM, Michal Skrivanek wrote:
> > >> On 12 Oct 2020, at 14:49, Marcin Sobczyk  > > 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/
> > 
> > >> [2]
> > 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
> > 
> > >> [5] https://gerrit.ovirt.org/110097
> > 
> > >>
> >
> >
> >
> > --
> > Martin Perina
> > Manager, Software Engineering
> > Red Hat Czech s.r.o.
>
>

-- 
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/UUM22ELODDGOT7J5IOI77ESBPYYPJYE6/

[ovirt-devel] Re: Branching out 4.3 in ovirt-system-tests

2020-11-20 Thread Marcin Sobczyk 



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



On Fri, Nov 20, 2020 at 11:46 AM Marcin Sobczyk > wrote:


Posted 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/ 


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 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/

>> [2]
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


>> [4] https://gerrit.ovirt.org/108809

>> [5] 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/

[ovirt-devel] Re: Branching out 4.3 in ovirt-system-tests

2020-11-20 Thread Martin Perina 
On Fri, Nov 20, 2020 at 11:46 AM Marcin Sobczyk  wrote:

> Posted 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/
The plan is to execute 4.3 network suite only nightly


> On 10/12/20 6:28 PM, Michal Skrivanek wrote:
> >> On 12 Oct 2020, at 14:49, Marcin Sobczyk  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/
> >> [2] 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
> >> [4] https://gerrit.ovirt.org/108809
> >> [5] 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/YBGMKULBGCOJTNFBUKZCPZWXUSYTFZEL/