<?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>10937</bug_id>
          
          <creation_ts>2017-01-17 17:34:17 +0000</creation_ts>
          <short_desc>remove from oe-selftest buildsets the step for building an image prior the test session</short_desc>
          <delta_ts>2017-12-06 17:25:13 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>5</classification_id>
          <classification>Infrastructure</classification>
          <product>AutoBuilder</product>
          <component>autobuilder</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>enhancement</bug_severity>
          <target_milestone>2.4 M3</target_milestone>
          <dependson>11118</dependson>
          <blocked>10035</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Benjamin Esquivel">benjamin.esquivel</reporter>
          <assigned_to name="Joshua Lock">joshuagloe</assigned_to>
          <cc>infras.ab.watcher</cc>
    
    <cc>Infras.watcher</cc>
    
    <cc>mariano.lopez</cc>
    
    <cc>monserratx.sedeno.bustos</cc>
    
    <cc>pidge</cc>
    
    <cc>sonia.leon.bautista</cc>
          
          
          <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>69861</commentid>
    <comment_count>0</comment_count>
    <who name="Benjamin Esquivel">benjamin.esquivel</who>
    <bug_when>2017-01-17 17:34:17 +0000</bug_when>
    <thetext>since the tests should be self-sufficient in their dependencies for images built prior testing, having a step prior the testing for bitbake core-image-minimal makes no sense.

Removing this step would improve the testing time for over half an hour and should have zero impact over the testing. If there are tests failing then those should be fixed to be self-sufficient.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>70170</commentid>
    <comment_count>1</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2017-01-26 11:34:54 +0000</bug_when>
    <thetext>M2 rc1 has been built, moving this to M3</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>70917</commentid>
    <comment_count>2</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2017-02-23 16:57:03 +0000</bug_when>
    <thetext>We&apos;ve seen multiple hangs when testing this change, it&apos;s clear there&apos;s more work to be done than just removing the BuildImage step. It&apos;s not yet clear what that work is.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>71204</commentid>
    <comment_count>3</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2017-03-13 14:36:27 +0000</bug_when>
    <thetext>This didn&apos;t make it for 2.3 M3, as it&apos;s an enhancement push to 2.4 M1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75567</commentid>
    <comment_count>4</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2017-08-01 18:59:14 +0000</bug_when>
    <thetext>Only Failure was test_checkpkg (distrodata.Distrodata), which is known to be flaky.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75652</commentid>
    <comment_count>5</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2017-08-04 15:55:11 +0000</bug_when>
    <thetext>morty builds fail with this change, they seem to require the BuildImages step have run before oe-selftest is invoked.

I replicated this locally, with clean build directories and a checkout of the morty branch:
i) invoking oe-selftest on the clean build directory results in the build being &quot;stuck&quot; at the &quot;Running tests...&quot; output of oe-selftest
ii) building core-image-minimal in a clean build directory, then invoking oe-selftest things proceed as expected.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75727</commentid>
    <comment_count>6</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2017-08-08 15:16:25 +0000</bug_when>
    <thetext>We&apos;ll achieve this by replacing the &quot;core-image-minimal&quot; BuildImages target with a placeholder value, for releases older than Pyro we&apos;ll replace the placeholder with core-image-minimal and for newer release we&apos;ll clear the placeholder.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>75982</commentid>
    <comment_count>7</comment_count>
    <who name="Joshua Lock">joshuagloe</who>
    <bug_when>2017-08-18 10:51:37 +0000</bug_when>
    <thetext>Code and configuration changes made and waiting for idle period on the AB to deploy.

http://git.yoctoproject.org/clean/cgit.cgi/yocto-autobuilder/commit/?id=086d06f1f3ae9d01d9987d6239676d01e220c64f
http://git.yoctoproject.org/clean/cgit.cgi/yocto-autobuilder/commit/?id=eec3de41d07cc17cc776c726123e866aa54d13b8</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>