[yocto] [[PATCH][yocto-autobuilder] 2/2] buildsteps/RunOeSelftest.py: Set native_lsb to universal

[Joshua G Lock]

On Tue, 2016-07-12 at 11:21 -0500, Aníbal Limón wrote:
> 
> On 07/12/2016 11:09 AM, Joshua G Lock wrote:
> > On Mon, 2016-07-11 at 16:37 -0500, Aníbal Limón wrote:
> > > The error report is generated into AB master and currently we
> > > don't have a method to get AB worker distro, so set to universal.
> > 
> > Can you elaborate a bit here? Why don't we have a method available
> > to
> > get the worker distro? Is it because we don't ensure all of the
> > builders in the cluster include lsb_release?
> 
> Currently, we don't have an easy way to get the worker distro an
> buildstep is needed in order TODO that but isn't trivial.
> 
> I propose to have at step at buildinit for get the worker
> information.
> At this time only will be distro. I removed the get_lsb_distro()
> because
> it gets the distro of the master instead of the worker.

Could we just extract the value of NATIVELSBSTRING[1] from the build
header in the logs?

Regards,

Joshua

1. http://www.yoctoproject.org/docs/2.1/ref-manual/ref-manual.html#var-
NATIVELSBSTRING

> 
> I added a bug for this task,
> 
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=9917
> 
> Cheers,
> 	alimon
> 
> 
> > 
> > Are our worker distro versions new enough that we can rely on os-
> > release?
> > 
> > http://manpages.ubuntu.com/manpages/xenial/en/man5/os-release.5.htm
> > l
> > 
> > Regards,
> > 
> > Joshua
> > 
> > > 
> > > Signed-off-by: Aníbal Limón <anibal.limon at linux.intel.com>
> > > ---
> > >  lib/python2.7/site-
> > > packages/autobuilder/buildsteps/RunOeSelftest.py
> > > > 4 +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/lib/python2.7/site-
> > > packages/autobuilder/buildsteps/RunOeSelftest.py
> > > b/lib/python2.7/site-
> > > packages/autobuilder/buildsteps/RunOeSelftest.py
> > > index 5b9e408..809bcc3 100644
> > > --- a/lib/python2.7/site-
> > > packages/autobuilder/buildsteps/RunOeSelftest.py
> > > +++ b/lib/python2.7/site-
> > > packages/autobuilder/buildsteps/RunOeSelftest.py
> > > @@ -52,7 +52,9 @@ class RunOeSelftest(ShellCommand):
> > >          report['machine'] = 'universal'
> > >          report['build_sys'] = "%s-%s" % (platform.machine(),
> > >                  platform.system().lower())
> > > -        report['nativelsb'] = get_lsb_distro() or 'universal'
> > > +        # XXX: Set to universal because isn't easy to get
> > > Autobuilder
> > > +        # worker distro.
> > > +        report['nativelsb'] = 'universal' 
> > >          report['distro'] = 'universal'
> > >          report['target_sys'] = 'universal'
> > >  
> > > -- 
> > > 2.1.4
> > > 
>