Re: [PATCH v2 3/7] libmultipath: add a protocol subsection to multipath.conf

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2022-04-14 at 09:27 +0200, Martin Wilck wrote:
> On Wed, 2022-04-13 at 23:27 -0500, Benjamin Marzinski wrote:
> > Some storage arrays can be accessed using multiple protocols at the
> > same
> > time.  In these cases, users may want to set path attributes
> > differently, depending on the protocol that the path is using. To
> > allow
> > this, add a protocol subsection to the overrides section in
> > multipath.conf, which allows select path-specific options to be
> > set.
> > This commit simply adds the subsection, and handles merging
> > matching
> > entries. Future patches will make use of the section.
> > 
> > Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
> 
> Looks good, just a minor nit below.

As this is really minor and I'm going to be out of office next week:

Reviewed-by: Martin Wilck <mwilck@xxxxxxxx>

... and I'm going to push this series to my "queue" branch.

Perhaps you want to send an add-on patch with a symbolic value later.

Regards
Martin

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/dm-devel





[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux