Nir Soffer has posted comments on this change.

Change subject: api: use the new devel_warn helper
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/62210/2//COMMIT_MSG
Commit Message:

Line 5: CommitDate: 2016-08-24 17:34:59 +0200
Line 6: 
Line 7: api: use the new devel_warn helper
Line 8: 
Line 9: Switch from the warnings module to the new devel_warn function.
> why not to stop using this module everywhere in vdsm?
This is the plan, see next patches.

The only use Francesco missed is in testrunner, hopefully he will address this 
in this series.
Line 10: 
Line 11: Change-Id: Idc85dde59e09a5882e9d586fb0d37d2434f0a351
Line 12: Related-To: https://bugzilla.redhat.com/1364149
Line 13: Bug-Url: https://bugzilla.redhat.com/1369822


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idc85dde59e09a5882e9d586fb0d37d2434f0a351
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to