On (22/10/15 13:35), Pavel Reichl wrote: >On 10/22/2015 12:27 PM, Pavel Březina wrote: >>On 10/22/2015 11:23 AM, Pavel Reichl wrote: >>> >>> >>>On 10/21/2015 12:44 PM, Pavel Březina wrote: >>>>On 10/21/2015 12:39 PM, Pavel Reichl wrote: >>>>>Hello, >>>>> >>>>>while reviewing integration tests for sss_override Nick noticed that man >>>>>page does not mention restart requirements. Attached patch (attempts to) >>>>>address that. >>>>> >>>>>Thanks! >>>> >>>>LGTM >>> >>>Pavel it seems to me that after calling user-del SSSD must be also >>>restarted, right? >> >>No. But you may be hitting memory cache. > >Oh, thanks. Please see updated patches. > >
>From 073155493b06f3805f282843143a4a020e63bfbf Mon Sep 17 00:00:00 2001 >From: Pavel Reichl <reichl.pa...@gmail.com> >Date: Wed, 21 Oct 2015 12:28:37 +0200 >Subject: [PATCH 1/2] sss_override: Add restart requirements to man page > >--- > src/man/sss_override.8.xml | 11 +++++++++-- > 1 file changed, 9 insertions(+), 2 deletions(-) > >diff --git a/src/man/sss_override.8.xml b/src/man/sss_override.8.xml >index >24c38936984946b3284d1523a7321a7e7f3d7982..5be4295183a78a9e6625b6689cb1003ce1c01541 > 100644 >--- a/src/man/sss_override.8.xml >+++ b/src/man/sss_override.8.xml >@@ -34,8 +34,15 @@ > and groups. This change takes effect only on local machine. > </para> > <para> >- Overrides data are stored in SSSD cache. If the cache is deleted >- all local overrides are lost. >+ Overrides data are stored in the SSSD cache. If the cache is >deleted, >+ all local overrides are lost. Please note that after the first >+ override is created using any of the following >+ <emphasis>user-add</emphasis>, <emphasis>group-add</emphasis>, >+ <emphasis>user-import</emphasis> or >+ <emphasis>group-import</emphasis> command. SSSD needs to be >+ restarted to take effect. >+ <emphasis>sss_override</emphasis> prints message when a restart is >+ required. > </para> > </refsect1> The patch is different iw we compare to the initial version. I think Dad deserves a credit so the author should be Dan. Will you prepare new patch or shoudl we change it before pushing. LS _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
