<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Your patch works for me. I think your place for do_image_qa is even
    better.<br>
    <br>
    Tomasz<br>
    <div class="moz-signature">
      <br>
    </div>
    <div class="moz-cite-prefix">On 19.09.2017 18:22, Burton, Ross
      wrote:<br>
    </div>
    <blockquote type="cite"
cite="mid:CAJTo0La10Uta7_QSA-QM3N7iczvRrOHONy5CnHw78Gj-JhFFpQ@mail.gmail.com">
      <div dir="ltr">On 19 September 2017 at 17:07, Burton, Ross <span
          dir="ltr">&lt;<a href="mailto:ross.burton@intel.com"
            target="_blank" moz-do-not-send="true">ross.burton@intel.com</a>&gt;</span>
        wrote:<br>
        <div class="gmail_extra">
          <div class="gmail_quote">
            <blockquote class="gmail_quote" style="margin:0 0 0
              .8ex;border-left:1px #ccc solid;padding-left:1ex">
              <div dir="ltr"><span class="">On 19 September 2017 at
                  11:36, Tomasz Meresiński <span dir="ltr">&lt;<a
                      href="mailto:tomasz.meresinski@comarch.pl"
                      target="_blank" moz-do-not-send="true">tomasz.meresinski@comarch.pl</a>&gt;</span>
                  wrote:<br>
                </span>
                <div class="gmail_extra">
                  <div class="gmail_quote"><span class="">
                      <blockquote class="gmail_quote" style="margin:0 0
                        0 .8ex;border-left:1px #ccc
                        solid;padding-left:1ex">After do_image_complete
                        IMAGE_ROOTFS directory is empty so qa checks do
                        not work</blockquote>
                      <div><br>
                      </div>
                    </span>
                    <div>Important context: this only happens if rm_work
                      is enabled.  I'm wondering if the task should be
                      moved earlier so its between rootfs and image, as
                      it's actually a *rootfs* QA hook (its expected
                      that functions examine the contents of
                      IMAGE_ROOTFS).</div>
                    <div><br>
                    </div>
                    <div>I think we need to push the task back a bit in
                      the tree and put it in between rootfs and image. 
                      This way if it fails the image generation doesn't
                      happen, and rm_work won't break it.</div>
                  </div>
                </div>
              </div>
            </blockquote>
            <div><br>
            </div>
            <div>I've just sent an alternative patch the list, can you
              verify that it solves your problem and doesn't break your
              existing image QA functions?</div>
            <div><br>
            </div>
            <div>Cheers,</div>
            <div>Ross </div>
          </div>
          <br>
        </div>
      </div>
    </blockquote>
    <br>
  </body>
</html>