Nir Soffer has posted comments on this change.

Change subject: vdsm: drop the warnings module configuration
......................................................................


Patch Set 4:

(1 comment)

https://gerrit.ovirt.org/#/c/62212/4//COMMIT_MSG
Commit Message:

Line 6: 
Line 7: vdsm: drop the warnings module configuration
Line 8: 
Line 9: We don't use the warnings module anymore in Vdsm, so we
Line 10: don't need this configuration anymore and we can drop it.
This change is correct but bot because we don't use the warnings module, but 
because we don't use it for reporting inconsistency issues.

We can leave the warnings module in vdscli, but we don't need to configure 
anything for it. Application using this library may like to configure warnings 
filter to show warnings if they want to show warnings.
Line 11: 
Line 12: Change-Id: I791882a2dd8ba2dda9135b087bca33610db8a20d
Line 13: Related-To: https://bugzilla.redhat.com/1364149
Line 14: Bug-Url: https://bugzilla.redhat.com/1369822


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I791882a2dd8ba2dda9135b087bca33610db8a20d
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to