Nir Soffer has posted comments on this change.

Change subject: persist: /etc/multipath/wwwids
......................................................................


Patch Set 2: -Code-Review

Douglas, I think this file is created by multipath on the first time it runs. 
Did you test that with this patch, the default multipath wwids file is created?

It should look like this:

 # Multipath wwids, Version : 1.0
 # NOTE: This file is automatically maintained by multipath and multipathd.
 # You should not need to edit this file in normal circumstances.
 #
 # Valid WWIDs:

It would be nice if we can add a note about persisting this file to
the comment.

-- 
To view, visit https://gerrit.ovirt.org/40187
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7f6b5a6b0bed6521503b972866d93a7da5cd28fe
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-node-plugin-vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to