On 11/23/20 4:24 PM, Martin Perina wrote:
On Mon, Nov 23, 2020 at 11:20 AM Marcin Sobczyk <[email protected]
<mailto:[email protected]>> wrote:
On 11/20/20 12:15 PM, Martin Perina wrote:
>
>
> On Fri, Nov 20, 2020 at 12:02 PM Marcin Sobczyk
<[email protected] <mailto:[email protected]>
> <mailto:[email protected] <mailto:[email protected]>>> wrote:
>
>
>
> On 11/20/20 11:50 AM, Martin Perina wrote:
> >
> >
> > On Fri, Nov 20, 2020 at 11:46 AM Marcin Sobczyk
> <[email protected] <mailto:[email protected]>
<mailto:[email protected] <mailto:[email protected]>>
> > <mailto:[email protected] <mailto:[email protected]>
<mailto:[email protected] <mailto:[email protected]>>>> wrote:
> >
> > Posted https://gerrit.ovirt.org/#/c/112302/
<https://gerrit.ovirt.org/#/c/112302/>
> <https://gerrit.ovirt.org/#/c/112302/
<https://gerrit.ovirt.org/#/c/112302/>>
> > <https://gerrit.ovirt.org/#/c/112302/
<https://gerrit.ovirt.org/#/c/112302/>
> <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/>
> <https://gerrit.ovirt.org/#/c/112291/
<https://gerrit.ovirt.org/#/c/112291/>>
> > <https://gerrit.ovirt.org/#/c/112291/
<https://gerrit.ovirt.org/#/c/112291/>
> <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.
>
>
> 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
It seems that the tip of the branch points to a commit that already
includes Eitan's change that removed 'network_suite_4.3' from
'stdci.yaml' [6],
so we can't run this pipeline in CI.
Could you please change the branch to point to the commit just before
that? [7]
Readded: https://gerrit.ovirt.org/#/c/112324/
<https://gerrit.ovirt.org/#/c/112324/>
Thanks. The CI is not running yet though.
Tried a manual runner:
https://jenkins.ovirt.org/job/standard-manual-runner/1619/parameters/
but it doesn't see the pipeline.
Ehud, could you please take a look at it?
Regards, Marcin
[6] https://gerrit.ovirt.org/#/c/112291/
<https://gerrit.ovirt.org/#/c/112291/>
[7] a2a462ef2f409ce5ef293adef6ecbb0c86c9d4a7
>
> >
> >
> > On 10/12/20 6:28 PM, Michal Skrivanek wrote:
> > >> On 12 Oct 2020, at 14:49, Marcin Sobczyk
> <[email protected] <mailto:[email protected]>
<mailto:[email protected] <mailto:[email protected]>>
> > <mailto:[email protected]
<mailto:[email protected]> <mailto:[email protected]
<mailto:[email protected]>>>>
> 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/>
> <https://gerrit.ovirt.org/#/c/111643/
<https://gerrit.ovirt.org/#/c/111643/>>
> > <https://gerrit.ovirt.org/#/c/111643/
<https://gerrit.ovirt.org/#/c/111643/>
> <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>
>
<https://gerrit.ovirt.org/#/c/111643/6/basic-suite-master/control.sh
<https://gerrit.ovirt.org/#/c/111643/6/basic-suite-master/control.sh>>
> >
>
<https://gerrit.ovirt.org/#/c/111643/6/basic-suite-master/control.sh
<https://gerrit.ovirt.org/#/c/111643/6/basic-suite-master/control.sh>
>
<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>
>
<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>>
> >
>
<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>
>
<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>
> <https://gerrit.ovirt.org/108809
<https://gerrit.ovirt.org/108809>>
> > <https://gerrit.ovirt.org/108809
<https://gerrit.ovirt.org/108809>
> <https://gerrit.ovirt.org/108809
<https://gerrit.ovirt.org/108809>>>
> > >> [5] https://gerrit.ovirt.org/110097
<https://gerrit.ovirt.org/110097>
> <https://gerrit.ovirt.org/110097
<https://gerrit.ovirt.org/110097>>
> > <https://gerrit.ovirt.org/110097
<https://gerrit.ovirt.org/110097>
> <https://gerrit.ovirt.org/110097
<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.
--
Martin Perina
Manager, Software Engineering
Red Hat Czech s.r.o.
_______________________________________________
Devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
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/[email protected]/message/CF5ZVF64J6MF77XJ4BJI5YY7KCHGY7ZB/
