Dan Kenigsberg has submitted this change and it was merged. Change subject: profiling: add support for memory profiling ......................................................................
profiling: add support for memory profiling There are many python memory profilers, but most of them are tailored for interactive usage, so additional integration work is needed to use them in a daemon, like VDSM. This patch adds support for dowser: <http://www.aminus.net/wiki/Dowser> <https://pypi.python.org/pypi/dowser/0.2> which has the following benefits which makes it in a better position to be used with VDSM: + self contained, includes WEB UI + allows live monitoring + tailored for cherrypy usage, so friendly towards server applications. + easily portable (pure package) It must be noted that dowser has some drawbacks as well - significat dependencies, both in number and in size (with respect to VDSM standards): cherrypy, PIL - based on gc module As per the cpu profile already added, this code is meant to be used only as debug aid or in development environments. The feature is controlled by a config tunable and disabled by default; the feature disable itself if any dependency is missed. Change-Id: Ib56b65513e0118b68cd43791bf655c928d6a26e2 Signed-off-by: Francesco Romani <from...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/32019 Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M debian/vdsm-python.install M lib/vdsm/config.py.in M lib/vdsm/profiling/Makefile.am A lib/vdsm/profiling/memory.py M lib/vdsm/profiling/profile.py M vdsm.spec.in 6 files changed, 124 insertions(+), 0 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/32019 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib56b65513e0118b68cd43791bf655c928d6a26e2 Gerrit-PatchSet: 21 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <toni+ov...@midokura.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
