<div dir="ltr">Hi,<div><br></div><div>The config outputed should be usable, so a comment could be printed for the keywords you suggest printing <undef> as the default for.</div><div><br></div><div>Example:</div><div><br></div><div># <span style="font-size:19.2px">no_path_retry <comment explaining the contextual default behaviour></span></div><div><span style="font-size:19.2px"><br></span></div><div><span style="font-size:19.2px">or simply</span></div><div><span style="font-size:19.2px"><br></span></div><div><span style="font-size:19.2px"># no_path_retry <contextual, see man page></span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 25, 2017 at 8:42 PM, Benjamin Marzinski <span dir="ltr"><<a href="mailto:bmarzins@redhat.com" target="_blank">bmarzins@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Apr 25, 2017 at 04:26:06PM +0200, Xose Vazquez Perez wrote:<br>
> Hi,<br>
><br>
> These values:<br>
<br>
What do you think of these?<br>
> no_path_retry<br>
<br>
The default if no_path_retry is set is to use the features line to<br>
determine what to do when there are no paths. we could have it<br>
print something like<br>
<br>
no_path_retry <undef><br>
<br>
> dev_loss_tmo<br>
<br>
The default here is to keep whatever the sysfs value currently is. We<br>
could print<br>
<br>
dev_loss_tmo <undef><br>
<br>
or<br>
<br>
dev_loss_tmo <sysfs><br>
<br>
or something else. I'm open to ideas.<br>
<br>
> reservation_key<br>
<br>
The default here is nothing<br>
<br>
reservation_key <undef><br>
<br>
probably makes the most sense<br>
<br>
> partition_delimite<br>
<br>
The default here is an empty string<br>
<br>
partition_delm ""<br>
<br>
is probably the right answer.<br>
<br>
> delay_watch_checks<br>
> delay_wait_checks<br>
<br>
These clearly have well defined default values and should be set.<br>
<br>
<br>
-Ben<br>
<div class="HOEnZb"><div class="h5"><br>
> are missing from the "defaults section" of multipath -t output.<br>
> --<br>
> dm-devel mailing list<br>
> <a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a><br>
> <a href="https://www.redhat.com/mailman/listinfo/dm-devel" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/dm-devel</a><br>
<br>
--<br>
dm-devel mailing list<br>
<a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/dm-devel" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/dm-devel</a><br>
</div></div></blockquote></div><br></div>