On Tue, 2014-11-04 at 11:17 -0500, Nathaniel McCallum wrote: > On Wed, 2014-10-29 at 09:34 -0400, Nathaniel McCallum wrote: > > On Wed, 2014-10-29 at 12:21 +0100, Petr Viktorin wrote: > > > On 10/29/2014 10:37 AM, Martin Kosek wrote: > > > > On 10/28/2014 09:59 PM, Nathaniel McCallum wrote: > > > >> On Thu, 2014-10-23 at 18:07 -0400, Nathaniel McCallum wrote: > > > >>> This patch gives the administrator variables to control the size of > > > >>> the authentication and synchronization windows for OTP tokens. > > > >>> > > > >>> https://fedorahosted.org/freeipa/ticket/4511 > > > >>> > > > >>> NOTE: There is one known issue with this patch which I don't know how > > > >>> to > > > >>> solve. This patch changes the schema in > > > >>> install/share/60ipaconfig.ldif. > > > >>> On an upgrade, all of the new attributeTypes appear correctly. > > > >>> However, > > > >>> the modifications to the pre-existing objectClass do not show up on > > > >>> the > > > >>> server. What am I doing wrong? > > > >>> > > > >>> After modifying ipaGuiConfig manually, everything in this patch works > > > >>> just fine. > > > >> > > > >> This new version takes into account the new (proper) OIDs and attribute > > > >> names. > > > > > > > > Thanks Nathaniel! > > > > > > > >> The above known issue still remains. > > > > > > > > Petr3, any idea what could have gone wrong? ObjectClass MAY list > > > > extension > > > > should work just fine, AFAIK. > > > > > > You added a blank line to the LDIF file. This is an entry separator, so > > > the objectClasses after the blank line don't belong to cn=schema, so > > > they aren't considered in the update. > > > Without the blank line it works fine. > > > > Thanks for the catch! > > > > Here is a version without the blank line. > > I forgot to remove the old steps defines. This patch performs this > cleanup.
Can I get a review on this soon? I need to make other changes in these files for another bug and I'd prefer to not have to shuffle between patches. Nathaniel _______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel