<?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>8757</bug_id>
          
          <creation_ts>2015-12-03 20:49:18 +0000</creation_ts>
          <short_desc>Automate the process of checking for dead links in recipes</short_desc>
          <delta_ts>2016-03-21 14:18:04 +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.1</version>
          <rep_platform>x86</rep_platform>
          <op_sys>Multiple</op_sys>
          <bug_status>VERIFIED</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.1 M1</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Juro Bystricky">juro.bystricky</reporter>
          <assigned_to name="Beth Flanagan">elizabeth.flanagan</assigned_to>
          <cc>benjamin.esquivel</cc>
    
    <cc>bluelightning</cc>
    
    <cc>bogdanx.a.voiculescu</cc>
    
    <cc>elizabeth.flanagan</cc>
    
    <cc>infras.ab.watcher</cc>
    
    <cc>Infras.watcher</cc>
    
    <cc>ross.burton</cc>
          
          <qa_contact name="Bogdan Alexandru Voiculescu">bogdanx.a.voiculescu</qa_contact>
          <cf_os>---</cf_os>
          <cf_regression_type>---</cf_regression_type>
          
          <cf_docchange>Don&apos;t know</cf_docchange>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>57016</commentid>
    <comment_count>0</comment_count>
    <who name="Juro Bystricky">juro.bystricky</who>
    <bug_when>2015-12-03 20:49:18 +0000</bug_when>
    <thetext>URL links in recipes can become obsoleted over the time. Recipes with dead links should be detected by a link-checker running on AB as part of QA.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>57105</commentid>
    <comment_count>1</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2015-12-07 19:37:12 +0000</bug_when>
    <thetext>Presumably you mean SRC_URI?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>57107</commentid>
    <comment_count>2</comment_count>
    <who name="Juro Bystricky">juro.bystricky</who>
    <bug_when>2015-12-07 20:56:39 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; Presumably you mean SRC_URI?

Yes, I guess I should have been more specific. SRC_URI may become obsoleted (containing a dead link), but that may be detected quite a bit later, typically when building images from scratch. You will most likely have no problems, as the SRC_URI content will be fetched from a mirror, but still this should be detected and recipes updated accordingly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>57109</commentid>
    <comment_count>3</comment_count>
    <who name="Paul Eggleton">bluelightning</who>
    <bug_when>2015-12-07 21:41:48 +0000</bug_when>
    <thetext>So in concrete terms, I think this would be a simple matter of running:

  bitbake -c checkuri world

(On a regular basis).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>57110</commentid>
    <comment_count>4</comment_count>
    <who name="Juro Bystricky">juro.bystricky</who>
    <bug_when>2015-12-07 21:47:29 +0000</bug_when>
    <thetext>(In reply to comment #1)
&gt; Presumably you mean SRC_URI?

Yes, I guess I should have been more specific. SRC_URI may become obsoleted (containing a dead link), but that may be detected quite a bit later, typically when building images from scratch. You will most likely have no problems, as the SRC_URI content will be fetched from a mirror, but still this should be detected and recipes updated accordingly.
(In reply to comment #3)
&gt; So in concrete terms, I think this would be a simple matter of running:
&gt; 
&gt;   bitbake -c checkuri world
&gt; 
&gt; (On a regular basis).

Yes, something like that. The whole idea is to have this check automated (and presumably followed up by SWAT)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>57744</commentid>
    <comment_count>5</comment_count>
    <who name="Beth Flanagan">elizabeth.flanagan</who>
    <bug_when>2016-01-12 13:52:57 +0000</bug_when>
    <thetext>http://git.yoctoproject.org/cgit/cgit.cgi/yocto-autobuilder/commit/?id=7823bff4fd66b835dea3e4d13dfb0017f234b24b</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>59934</commentid>
    <comment_count>6</comment_count>
    <who name="Bogdan Alexandru Voiculescu">bogdanx.a.voiculescu</who>
    <bug_when>2016-03-14 10:32:08 +0000</bug_when>
    <thetext>Hi Beth,
I can not find this step on AB. 
https://autobuilder.yoctoproject.org/main/builders/nightly/builds/772

Please verify if patch is applied in production.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60157</commentid>
    <comment_count>7</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2016-03-18 15:34:21 +0000</bug_when>
    <thetext>Its not a build set but a buildset: https://autobuilder.yoctoproject.org/main/builders/nightly-checkuri.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60158</commentid>
    <comment_count>8</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2016-03-18 15:35:16 +0000</bug_when>
    <thetext>To save time, verifying myself.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60180</commentid>
    <comment_count>9</comment_count>
    <who name="Bogdan Alexandru Voiculescu">bogdanx.a.voiculescu</who>
    <bug_when>2016-03-21 06:35:38 +0000</bug_when>
    <thetext>Hi Ross,
The commit comment was:  
&quot;
nightly.conf: add checkuri step
[YOCTO #8757]

Add a world -c checkuri step to nightly.

Signed-off-by: Elizabeth Flanagan &lt;elizabeth.flanagan@intel.com&gt;&quot;

for this file&quot; /buildset-config.controller/nightly.conf

So I looked for a build step in nightly.conf job. Also I looked for this commit to be executed in production; meaning this command &quot;bitbake -c checkuri world&quot;

The &quot;bitbake -k universe -c checkuri&quot; command is running for a while, but it is a different commit on a different buildset. I agree &quot;bitbake -k universe -c checkuri&quot; is covering &quot;bitbake -c checkuri world&quot;, and it is a solution to verify URL links in recipes. 
Still it is a different solution and not from this enhacement.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60194</commentid>
    <comment_count>10</comment_count>
    <who name="Ross Burton">ross.burton</who>
    <bug_when>2016-03-21 11:01:47 +0000</bug_when>
    <thetext>You&apos;re right but that was just the first iteration that landed in git, it was revised in this commit to be what is currently on the AB:

commit 1c7b54394cf5cc94eca80b5092d0d8d7a4944102
Author: Elizabeth Flanagan &lt;elizabeth.flanagan@intel.com&gt;
Date:   Mon Jan 18 11:28:36 2016 +0000

    nightly-checkuri.conf: move checkuri to nowait

    checkuri takes about an hour to complete. moving it from
    nightly to it&apos;s own buildset.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>60206</commentid>
    <comment_count>11</comment_count>
    <who name="Bogdan Alexandru Voiculescu">bogdanx.a.voiculescu</who>
    <bug_when>2016-03-21 14:18:04 +0000</bug_when>
    <thetext>Ross, thank you for clarification!</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>