<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugzilla.yoctoproject.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugzilla.yoctoproject.org/"
          
          maintainer="it-coreprojects-helpdesk@linuxfoundation.org"
>

    <bug>
          <bug_id>10067</bug_id>
          
          <creation_ts>2016-08-02 20:20:51 +0000</creation_ts>
          <short_desc>Host sanity check fails for CentOS 7</short_desc>
          <delta_ts>2016-08-10 13:06:15 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>7</classification_id>
          <classification>Build System, Metadata &amp; Runtime</classification>
          <product>OE-Core</product>
          <component>core</component>
          <version>2.2</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>Medium+</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.2 M3</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Bill Randle">bill.randle</reporter>
          <assigned_to name="Jose Perez C">jose.perez.carranza</assigned_to>
          <cc>bluelightning</cc>
    
    <cc>meta.mr.watcher</cc>
    
    <cc>meta.watcher</cc>
    
    <cc>randy.macleod</cc>
    
    <cc>sgw</cc>
    
    <cc>yi.zhao</cc>
          
          <qa_contact name="David Rodriguez">david.israelx.rodriguez.castellanos</qa_contact>
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>No (bug/feature does not impact docs)</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>64797</commentid>
    <comment_count>0</comment_count>
    <who name="Bill Randle">bill.randle</who>
    <bug_when>2016-08-02 20:20:51 +0000</bug_when>
    <thetext>sanity.conf includes a list of test distros in the variable SANITY_TESTED_DISTROS that have been validated for builds. One of the entries in the list is &quot;CentOS-7.*&quot; However, I still see warning messages on the AB:
  &quot;WARNING: Host distribution &quot;CentOSLinux-7.2.1511&quot; has not been validated...&quot;

The distro id is figured out by meta/lib/oe/lsb.py based first on the output of the &apos;lsb_release&apos; command, or if not present, then parsing some known files, such as /etc/redhat-release. In the case of CentOS 7, /etc/redhat-release contains &quot;CentOS Linux release 7.2.1511 (Core)&quot;. This parsed as &quot;CentOS Linux&quot; (id) and &quot;7.2.1511&quot; (release).

Further down in the code, it deletes all non-alphanumeric characters in the id, which compresses it to &quot;CentOSLinux&quot;, without the space.

So, bottom line, the &quot;CentOS-7.*&quot; entry in the sanity list should be replaced with &quot;CentOSLinux-7.*&quot;.

Other distributions should be checked to see if the id string contains a (collapsed) space and adjust the sanity list name accordingly.

In addition, I noticed the supported Yocto released are included in the sanity list, but 2.1 is missing. Should it be added to the list?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64799</commentid>
    <comment_count>1</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2016-08-02 20:30:50 +0000</bug_when>
    <thetext>I&apos;m receiving the same warning on CentOS 7 here.

Yes we should add the 2.1 release and 2.2 (and drop unsupported older versions) - those are there for the build appliance. This was something that the QA team were supposed to be doing for every release, along with updating the other distributions in the list - though it doesn&apos;t matter who actually does it as long as it gets done.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64801</commentid>
    <comment_count>2</comment_count>
    <who name="Bill Randle">bill.randle</who>
    <bug_when>2016-08-02 20:34:02 +0000</bug_when>
    <thetext>Jose Perez Carranza volunteered to fix it up.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64804</commentid>
    <comment_count>3</comment_count>
    <who name="Jose Perez C">jose.perez.carranza</who>
    <bug_when>2016-08-02 20:36:53 +0000</bug_when>
    <thetext>This was a process that was not clearly defined in the past but now is defined an documented here https://wiki.yoctoproject.org/wiki/Distro_Testing_Plan , I will investigate if another OS distributions are giving the same warning and update the list accordingly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64843</commentid>
    <comment_count>4</comment_count>
    <who name="Jose Perez C">jose.perez.carranza</who>
    <bug_when>2016-08-03 17:23:18 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; I&apos;m receiving the same warning on CentOS 7 here.
&gt; 
&gt; Yes we should add the 2.1 release and 2.2 (and drop unsupported older
&gt; versions) - those are there for the build appliance. This was something that
&gt; the QA team were supposed to be doing for every release, along with updating
&gt; the other distributions in the list - though it doesn&apos;t matter who actually
&gt; does it as long as it gets done.

2.2 should be added? or this should be added until is officially released ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64850</commentid>
    <comment_count>5</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2016-08-03 20:20:45 +0000</bug_when>
    <thetext>I don&apos;t think it does harm to add it in advance - but the most important thing is it gets added in time for the release.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64864</commentid>
    <comment_count>6</comment_count>
    <who name="Yi Zhao">yi.zhao</who>
    <bug_when>2016-08-04 02:16:36 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; I&apos;m receiving the same warning on CentOS 7 here.
&gt; 
&gt; Yes we should add the 2.1 release and 2.2 (and drop unsupported older
&gt; versions) - those are there for the build appliance. This was something that
&gt; the QA team were supposed to be doing for every release, along with updating
&gt; the other distributions in the list - though it doesn&apos;t matter who actually
&gt; does it as long as it gets done.

(In reply to comment #4)
&gt; (In reply to comment #1)
&gt; &gt; I&apos;m receiving the same warning on CentOS 7 here.
&gt; &gt; 
&gt; &gt; Yes we should add the 2.1 release and 2.2 (and drop unsupported older
&gt; &gt; versions) - those are there for the build appliance. This was something that
&gt; &gt; the QA team were supposed to be doing for every release, along with updating
&gt; &gt; the other distributions in the list - though it doesn&apos;t matter who actually
&gt; &gt; does it as long as it gets done.
&gt; 
&gt; 2.2 should be added? or this should be added until is officially released ?

Please add Poky-2.1 in the list and backport to krogoth branch. There is a warning in build appliance for krogoth</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>64905</commentid>
    <comment_count>7</comment_count>
    <who name="Jose Perez C">jose.perez.carranza</who>
    <bug_when>2016-08-04 20:22:58 +0000</bug_when>
    <thetext>Patch sent to the mailing list 

https://lists.yoctoproject.org/pipermail/poky/2016-August/010643.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>65016</commentid>
    <comment_count>8</comment_count>
    <who name="Jose Perez C">jose.perez.carranza</who>
    <bug_when>2016-08-10 13:05:13 +0000</bug_when>
    <thetext>Patch is now in master on below commit: 

http://git.yoctoproject.org/clean/cgit.cgi/poky/commit/?id=dd53025d24ef7ab948414cc0141f65f48eaacfa2</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>