<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 19/06/15 09:28, Fraser Tweedale
      wrote:<br>
    </div>
    <blockquote
      cite="mid:20150619072838.GR29330@dhcp-40-8.bne.redhat.com"
      type="cite">
      <pre wrap="">The attached patches fix upgrade issues when pki is also updated
from pre 10.2.4.

pki dependency is bumped to 10.2.5 - the official builds should be
done Friday (US time) but it is available from my copr[1].  If
someone wants to add to official freeipa COPR in meantime the SRPM
is here[2].

[1] <a class="moz-txt-link-freetext" href="https://copr.fedoraproject.org/coprs/ftweedal/freeipa/">https://copr.fedoraproject.org/coprs/ftweedal/freeipa/</a>
[2] <a class="moz-txt-link-freetext" href="https://ftweedal.fedorapeople.org/pki-core-10.2.5-0.2.fc21.src.rpm">https://ftweedal.fedorapeople.org/pki-core-10.2.5-0.2.fc21.src.rpm</a>

Thanks,
Fraser
</pre>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
    </blockquote>
    Thank you.<br>
    <br>
    1)<br>
    I cannot apply patches.<br>
    <br>
    2)<br>
    IMO patch 0020 was fixed with my patch 266<br>
    <br>
    3)<br>
    This print should not be there<br>
    +<br>
    +    print cs_cfg<br>
    +    for profile_id in profile_ids:<br>
    <br>
    4)<br>
    This is unused variable,  it is defined later<br>
    +   cs_cfg = None<br>
    <br>
    5)<br>
    Can you add there log.error or log.debug instead of pass please?<br>
    +        # enable the profile<br>
    +        try:<br>
    +            profile_api.enable_profile(profile_id)<br>
    +        except errors.RemoteRetrieveError:<br>
    +            pass<br>
    <br>
    I will test it later.<br>
    <pre class="moz-signature" cols="72">-- 
Martin Basti</pre>
  </body>
</html>