Change in vdsm[master]: fc-scan: Use utilities from vdsm library.

nsoffer at redhat.com nsoffer at redhat.com
Mon Mar 16 17:15:42 UTC 2015


Nir Soffer has posted comments on this change.

Change subject: fc-scan: Use utilities from vdsm library.
......................................................................


Patch Set 2:

Since this is part of vdsm, it seems wasteful to avoid using vdsm infrastructure.

If we like to make this more general tool separate from vdsm, we will probably rewrite this in C, so sharing code with vdsm will not be an option :-)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic48748d6a43d41e034e16cb4f636ebe627881590
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list