[lnst: 6/6] Cover letter for "PerfRepo support"

[Jiří Pírko]

commit 8b7d6bf386272ff3f677a48f8fa55d2d048395d8
Merge: f56fc9e 3b11e39
Author: Jiri Pirko <jiri(a)resnulli.us&gt;
Date:   Fri Apr 17 15:40:04 2015 +0200

    Cover letter for "PerfRepo support"
    
    Ondrej says:
    
    The following set of patches implements basic support for talking to PerfRepo.
    There still are some placeholders for functions that are not important for now
    and bugs are to be expected.
    
    There is a short example work flow example in the last patch of the set but you
    still need to have a PerfRepo instance available to try everything out, I'll be
    sending a follow-up email to interested parties that will explain in more detail
    how to try this out.
    
    Any comments and ideas are always welcome.

 lnst/Common/Utils.py                       |   52 +++
 lnst/Controller/NetTestController.py       |    3 +-
 lnst/Controller/NetTestResultSerializer.py |    3 +-
 lnst/Controller/PerfRepo.py                |  532 ++++++++++++++++++++++++++++
 lnst/Controller/SlavePool.py               |   17 +-
 lnst/Controller/Task.py                    |  127 +++++++
 6 files changed, 728 insertions(+), 6 deletions(-)
---

[Ondrej Lichtner]

One thing I forgot... this adds a new dependency on a Python library
'requests', on Fedora, this is installed by default, however on RHEL
machines you need to install it using:
yum install python-requests

Without it you'll probably experience some crashes, sorry if this caused
any problems.

On Fri, Apr 17, 2015 at 01:42:44PM +0000, Jiří Pírko wrote:
>  commit 8b7d6bf386272ff3f677a48f8fa55d2d048395d8
>  Merge: f56fc9e 3b11e39
>  Author: Jiri Pirko <jiri(a)resnulli.us&gt;
>  Date:   Fri Apr 17 15:40:04 2015 +0200
>  
>      Cover letter for "PerfRepo support"
>      
>      Ondrej says:
>      
>      The following set of patches implements basic support for talking to PerfRepo.
>      There still are some placeholders for functions that are not important for now
>      and bugs are to be expected.
>      
>      There is a short example work flow example in the last patch of the set but you
>      still need to have a PerfRepo instance available to try everything out, I'll be
>      sending a follow-up email to interested parties that will explain in more detail
>      how to try this out.
>      
>      Any comments and ideas are always welcome.
>  
>   lnst/Common/Utils.py                       |   52 +++
>   lnst/Controller/NetTestController.py       |    3 +-
>   lnst/Controller/NetTestResultSerializer.py |    3 +-
>   lnst/Controller/PerfRepo.py                |  532 ++++++++++++++++++++++++++++
>   lnst/Controller/SlavePool.py               |   17 +-
>   lnst/Controller/Task.py                    |  127 +++++++
>   6 files changed, 728 insertions(+), 6 deletions(-)
>  ---
>  _______________________________________________
>  LNST-developers mailing list
>  LNST-developers(a)lists.fedorahosted.org
>  https://lists.fedorahosted.org/mailman/listinfo/lnst-developers

[Jiri Pirko]

Mon, Apr 20, 2015 at 10:38:05AM CEST, olichtne(a)redhat.com wrote:
> One thing I forgot... this adds a new dependency on a Python library
> 'requests', on Fedora, this is installed by default, however on RHEL
> machines you need to install it using:
> yum install python-requests 
Will add this to the spec file. Thanks. 

Talking about this, are there any other dependencies that need to be
added there?


> Without it you'll probably experience some crashes, sorry if this caused
> any problems.
>
> On Fri, Apr 17, 2015 at 01:42:44PM +0000, Jiří Pírko wrote:
> > commit 8b7d6bf386272ff3f677a48f8fa55d2d048395d8
> > Merge: f56fc9e 3b11e39
> > Author: Jiri Pirko <jiri(a)resnulli.us&gt;
> > Date:   Fri Apr 17 15:40:04 2015 +0200
> > 
> >     Cover letter for "PerfRepo support"
> >     
> >     Ondrej says:
> >     
> >     The following set of patches implements basic support for talking to PerfRepo.
> >     There still are some placeholders for functions that are not important for now
> >     and bugs are to be expected.
> >     
> >     There is a short example work flow example in the last patch of the set but you
> >     still need to have a PerfRepo instance available to try everything out, I'll
> be
> >     sending a follow-up email to interested parties that will explain in more detail
> >     how to try this out.
> >     
> >     Any comments and ideas are always welcome.
> > 
> >  lnst/Common/Utils.py                       |   52 +++
> >  lnst/Controller/NetTestController.py       |    3 +-
> >  lnst/Controller/NetTestResultSerializer.py |    3 +-
> >  lnst/Controller/PerfRepo.py                |  532 ++++++++++++++++++++++++++++
> >  lnst/Controller/SlavePool.py               |   17 +-
> >  lnst/Controller/Task.py                    |  127 +++++++
> >  6 files changed, 728 insertions(+), 6 deletions(-)
> > ---
> > _______________________________________________
> > LNST-developers mailing list
> > LNST-developers(a)lists.fedorahosted.org
> > https://lists.fedorahosted.org/mailman/listinfo/lnst-developers
> _______________________________________________
> LNST-developers mailing list
> LNST-developers(a)lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/lnst-developers

[Ondrej Lichtner]

On Mon, Apr 20, 2015 at 11:35:44AM +0200, Jiri Pirko wrote:
>  Mon, Apr 20, 2015 at 10:38:05AM CEST, olichtne(a)redhat.com wrote:
>  >One thing I forgot... this adds a new dependency on a Python library
>  >'requests', on Fedora, this is installed by default, however on RHEL
>  >machines you need to install it using:
>  >yum install python-requests
>  
>  Will add this to the spec file. Thanks. 
>  
>  Talking about this, are there any other dependencies that need to be
>  added there? 
I don't think so, but I'll try running a with clean virt machines just
to be sure.

>  
>  
>  >
>  >Without it you'll probably experience some crashes, sorry if this caused
>  >any problems.
>  >
>  >On Fri, Apr 17, 2015 at 01:42:44PM +0000, Jiří Pírko wrote:
>  >> commit 8b7d6bf386272ff3f677a48f8fa55d2d048395d8
>  >> Merge: f56fc9e 3b11e39
>  >> Author: Jiri Pirko <jiri(a)resnulli.us&gt;
>  >> Date:   Fri Apr 17 15:40:04 2015 +0200
>  >> 
>  >>     Cover letter for "PerfRepo support"
>  >>     
>  >>     Ondrej says:
>  >>     
>  >>     The following set of patches implements basic support for talking to
> PerfRepo.
>  >>     There still are some placeholders for functions that are not important for
> now
>  >>     and bugs are to be expected.
>  >>     
>  >>     There is a short example work flow example in the last patch of the set but
> you
>  >>     still need to have a PerfRepo instance available to try everything out,
> I'll be
>  >>     sending a follow-up email to interested parties that will explain in more
> detail
>  >>     how to try this out.
>  >>     
>  >>     Any comments and ideas are always welcome.
>  >> 
>  >>  lnst/Common/Utils.py                       |   52 +++
>  >>  lnst/Controller/NetTestController.py       |    3 +-
>  >>  lnst/Controller/NetTestResultSerializer.py |    3 +-
>  >>  lnst/Controller/PerfRepo.py                |  532 ++++++++++++++++++++++++++++
>  >>  lnst/Controller/SlavePool.py               |   17 +-
>  >>  lnst/Controller/Task.py                    |  127 +++++++
>  >>  6 files changed, 728 insertions(+), 6 deletions(-)
>  >> ---
>  >> _______________________________________________
>  >> LNST-developers mailing list
>  >> LNST-developers(a)lists.fedorahosted.org
>  >> https://lists.fedorahosted.org/mailman/listinfo/lnst-developers
>  >_______________________________________________
>  >LNST-developers mailing list
>  &gt;LNST-developers(a)lists.fedorahosted.org
>  >https://lists.fedorahosted.org/mailman/listinfo/lnst-developers

[Ondrej Lichtner]

On Mon, Apr 20, 2015 at 12:39:03PM +0200, Ondrej Lichtner wrote:
>  On Mon, Apr 20, 2015 at 11:35:44AM +0200, Jiri Pirko wrote:
>  > Mon, Apr 20, 2015 at 10:38:05AM CEST, olichtne(a)redhat.com wrote:
>  > >One thing I forgot... this adds a new dependency on a Python library
>  > >'requests', on Fedora, this is installed by default, however on RHEL
>  > >machines you need to install it using:
>  > >yum install python-requests
>  > 
>  > Will add this to the spec file. Thanks. 
>  > 
>  > Talking about this, are there any other dependencies that need to be
>  > added there?
>  
>  I don't think so, but I'll try running a with clean virt machines just
>  to be sure. 

So I've tried this with lnst-slave on minimal RHEL6 and RHEL7 installs
and we need at least these:
tar
bzip2
gcc
bridge-utils
python-pyroute2
lxml
python-requests

Tomorrow I'll try to run the Controller from a minimal install as well,
to see if anything else is missing


One of the regression tests also needs pstree which is a useless
dependency so I'll fix that separately.

I also found that running a test with team configuration on RHEL6 leaves
lnst-slave in an inconsistent state (at least for a while), not an
important bug since you shouldn't be testing team on RHEL6, but I
definitely should look into it further.

-Ondrej