Vered Volansky has posted comments on this change.

Change subject: Changing vdsm-tool and vdsmd pre-tasks outputs to be more 
meaningful to user
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/23253/2/lib/vdsm/tool/configurator.py
File lib/vdsm/tool/configurator.py:

Line 248:     if not ret:
Line 249:         msg = \
Line 250:             """
Line 251: 
Line 252: One of the modules is not configured to VDSM.
Didn't address my comment... - s/to/with
Line 253: To configure the module use the following:
Line 254: 'vdsm-tool configure [module_name]'.
Line 255: 
Line 256: If all modules are not configured try to use:


-- 
To view, visit http://gerrit.ovirt.org/23253
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If29f41081a70c0734cd78dab36f86c794a44b2d3
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Vered Volansky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to