<html> <head> <meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type"> </head> <body bgcolor="#FFFFFF" text="#000000"> <div class="moz-cite-prefix">On 04/12/2013 12:31 AM, Ana Krivokapic wrote: </div> <blockquote cite="mid:516739D8.5030702@redhat.com" type="cite">On 04/11/2013 05:57 PM, Rob Crittenden wrote: <blockquote type="cite">Petr Viktorin wrote: <blockquote type="cite">On 04/11/2013 12:01 AM, Rob Crittenden wrote: <blockquote type="cite">Ana Krivokapic wrote: <blockquote type="cite">Hello, This patch set deprecates HBAC source hosts from IPA. See commit messages and the design page[1] for details. <a class="moz-txt-link-freetext" href="https://fedorahosted.org/freeipa/ticket/3528">https://fedorahosted.org/freeipa/ticket/3528</a> [1] <a class="moz-txt-link-freetext" href="http://www.freeipa.org/page/V3/HBACSourceHosts">http://www.freeipa.org/page/V3/HBACSourceHosts</a> </blockquote> Been a while since I've run the UI but I get an error in FF 18: Timestamp: 04/10/2013 05:43:31 PM Error: TypeError: e.messages is undefined Source File: <a class="moz-txt-link-freetext" href="https://rawhide2.greyoak.com/ipa/ui/js/freeipa/app.js">https://rawhide2.greyoak.com/ipa/ui/js/freeipa/app.js</a> Line: 1 </blockquote> This probably means that you didn't rebuild the UI since 42300eb. Try git clean and a fresh rebuild, or use tools in install/ui/util/ (Petr Vobornik is the person to ask about those). </blockquote> Seems to have been a remnant of a previous build. I'm guessing that the UI build directories aren't covered by a clean/distclean. I manually removed some files from build and it works now. <blockquote type="cite"> <blockquote type="cite">The other changes seem to operate fine. I tested with an older client and got reasonable error messages back when I tried to do the various sourcehost things. I got an unknown error message with --add-sourcehost but it did include the text that the command is deprecated so I think this is acceptable. There isn't a lot we can do, I'm sorry we didn't add this exception in the beginning. I do wonder if we should leave the warning in hbactest if sourcehost is set though, for those cases where there are already options set. </blockquote> </blockquote> So this question still remains, should we leave the sourcehost warning in hbactest for another release or two? rob _______________________________________________ Freeipa-devel mailing list <a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a> <a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a> </blockquote> Rob and I agreed on IRC to put the sourcehost warning back in hbactest for now. I updated the patch 0016 accordingly. (It also needed a slight rebase, due to API changes that happened in the meantime.) New version is attached. <fieldset class="mimeAttachmentHeader"></fieldset> <pre wrap="">_______________________________________________ Freeipa-devel mailing list <a class="moz-txt-link-abbreviated" href="mailto:Freeipa-devel@redhat.com">Freeipa-devel@redhat.com</a> <a class="moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/freeipa-devel">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre> </blockquote> After further discussion on IRC, we decided there is no reason to keep the warning after all. I am attaching a rebased version of the original patch 0016. <pre class="moz-signature" cols="72">-- Regards, Ana Krivokapic Associate Software Engineer FreeIPA team Red Hat Inc.</pre> </body> </html>