[SSSD] [PATCH] sss_override: Add restart requirements to man page
Pavel Reichl
preichl at redhat.com
Fri Oct 23 07:54:49 UTC 2015
On 10/23/2015 09:10 AM, Lukas Slebodnik wrote:
> 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.pavel at 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.
Change the author before pushing then.
BTW do I deserve something for my effort? :-)
BTW2: second patch was not acked at all, first should be reacked.
>
> LS
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>
More information about the sssd-devel
mailing list