[Freeipa-devel] [PATCH] 102 Fix automountkey-add summary

Martin Kosek mkosek at redhat.com
Wed Jul 27 16:26:51 UTC 2011


On Wed, 2011-07-27 at 11:13 -0400, Rob Crittenden wrote:
> Martin Kosek wrote:
> > On Wed, 2011-07-27 at 10:35 -0400, Rob Crittenden wrote:
> >> Martin Kosek wrote:
> >>> On Wed, 2011-07-27 at 11:28 +0200, Martin Kosek wrote:
> >>>> The summary value was set to invalid primary key - automount map
> >>>> instead of automount key.
> >>>>
> >>>> https://fedorahosted.org/freeipa/ticket/1524
> >>>>
> >>>
> >>> The other commands' summary may be wrong in some cases as well. Updated
> >>> patch fixes summary for all automountkey commands.
> >>>
> >>> Martin
> >>
> >> nack
> >>
> >> ipalib/plugins/automount.py:926: [E0602, automountkey_show.execute]
> >> Undefined variable 'key'
> >>
> >> I think you meant to set result['value'] = options['automountkey']
> >>
> >> rob
> >
> > Ah, that's embarrassing. That happens when one makes "just one small
> > change before commit". Attaching fixed patch.
> >
> > Martin
> 
> ack

Pushed to master, ipa-2-0 (this version had to be merged).

Martin




More information about the Freeipa-devel mailing list