On 12/06/2016 04:22 PM, Benjamin Marzinski wrote:
> On Tue, Dec 06, 2016 at 04:11:42PM +0100, Xose Vazquez Perez wrote:
>> On 10/29/2016 04:55 AM, Benjamin Marzinski wrote:
>>
>>> Multipathd uses retrigger_tries to give udev more chances to to fill in
>>> the uid_attribute, so that the path device is correctly set up in the
>>> udev database. However the multipath command can't do this, so it should
>>> just immediately give up on udev, and try to get the wwid directly.
>>>
>>> Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
>>> ---
>>> multipath/main.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/multipath/main.c b/multipath/main.c
>>> index ee00fdb..06add30 100644
>>> --- a/multipath/main.c
>>> +++ b/multipath/main.c
>>> @@ -521,6 +521,7 @@ main (int argc, char *argv[])
>>> if (!conf)
>>> exit(1);
>>> multipath_conf = conf;
>>> + conf->retrigger_tries = 0;
>>> while ((arg = getopt(argc, argv, ":adchl::FfM:v:p:b:BritquwW")) != EOF ) {
>>> switch(arg) {
>>> case 1: printf("optarg : %s\n",optarg);
>>>
>>
>> I don't know how, but this patch(?) is overwriting the default value:
>
> That's because multipath and mutipathd need to do different things.
> However, I can change the patch so that 'multipath -t' still reports the
> configured value.
Could you do a patch to fix it?
Thank you.
>>
>> libmultipath/defaults.h:#define DEFAULT_RETRIGGER_TRIES 3
>> libmultipath/config.c: conf->retrigger_tries = DEFAULT_RETRIGGER_TRIES;
>>
>> # multipath -t
>> defaults {
>> [...]
>> retrigger_tries 0
>> [...]
>> }
>> [...]
>
--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel
