On Tue, Jan 31, 2012 at 08:29:15PM -0500, Stephen Gallagher wrote:
On Tue, 2012-01-31 at 15:56 +0100, Jakub Hrozek wrote:
> On Wed, Jan 25, 2012 at 08:23:23PM +0100, Jakub Hrozek wrote:
> > On Fri, Jan 20, 2012 at 01:22:19PM +0100, Pavel Březina wrote:
> > > Dne 18.1.2012 10:30, Jakub Hrozek napsal(a):
> > > >On Wed, Jan 18, 2012 at 10:16:14AM +0100, Jakub Hrozek wrote:
> > > >>On Wed, Jan 18, 2012 at 10:05:23AM +0100, Jan Zelený wrote:
> > > >>>>https://fedorahosted.org/sssd/ticket/1116
> > > >>>>
> > > >>>>Adds option 'sudo_timed' to responder.
> > > >>>
> > > >>>Ack. I didn't perform any real-life testing but the patch
seems simple enough.
> > > >>>
> > > >>>Thanks
> > > >>>Jan
> > > >>
> > > >>Real life testing cannot be performed until the sudo side is
ready.
> > > >>
> > > >>That's also why I asked Pavel to hold sending out this patch
for review
> > > >>until he can actuall test it.
> > > >
> > > >Also, nack. The new option needs to be added into the config API.
> > >
> > > I think this should be done as part of #1144 [1] or #1144 should be
> > > done as a part of this patch (preferably because this is going to be
> > > the only responder related option so far).
> > >
> > > [1]
https://fedorahosted.org/sssd/ticket/1144
> >
> > You're right, I didn't realize it was the only option. Then please
> > include the changes into this patch and we can close both tickets.
> >
> > Also, this patch needs rebasing on top of the latest iteration of
> > "cn=defaults" patches.
>
> I grabbed Pavel's WIP patch, and made some fixes (to separate the work,
> attached as Patch #2).
This doesn't apply atop the current master. Please rebase it. (It also
looks like maybe there's a missing patch in the series?)
Yes, sorry, I had the in-memory cache in my tree before the timed
patches. Instead of rebasing the timed patches on top of master and then
the other way around, I dediced to finish the in-memory patch.
I'm attaching the patches again, there are no changes in the code, just
rebase on top of the changed in the in-memory patch.