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>
 
-- 
2.4.3

>From 8606d19c1d685b75123086963506e27eb7133b1c Mon Sep 17 00:00:00 2001
From: Pavel Reichl <reichl.pa...@gmail.com>
Date: Thu, 22 Oct 2015 13:25:45 +0200
Subject: [PATCH 2/2] sss_override: Removed overrides might be in memcache

---
 src/man/sss_override.8.xml | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/man/sss_override.8.xml b/src/man/sss_override.8.xml
index 5be4295183a78a9e6625b6689cb1003ce1c01541..6d6d28477c1304072484bc9def5461fe6332e8ed 100644
--- a/src/man/sss_override.8.xml
+++ b/src/man/sss_override.8.xml
@@ -80,7 +80,10 @@
                 </term>
                 <listitem>
                     <para>
-                        Remove user overrides.
+                        Remove user overrides. However be aware that overridden
+                        attributes might be returned from memory cache. Please
+                        see SSSD option <emphasis>memcache_timeout</emphasis>
+                        for more details.
                     </para>
                 </listitem>
             </varlistentry>
@@ -150,7 +153,10 @@
                 </term>
                 <listitem>
                     <para>
-                        Remove group overrides.
+                        Remove group overrides. However be aware that overridden
+                        attributes might be returned from memory cache. Please
+                        see SSSD option <emphasis>memcache_timeout</emphasis>
+                        for more details.
                     </para>
                 </listitem>
             </varlistentry>
-- 
2.4.3

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to