Dan Kenigsberg has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
Escalating score.
--
To view, visit http://gerrit.ovirt.org/26113
To unsubscribe, visit
Dan Kenigsberg has submitted this change and it was merged.
Change subject: profiling: Add an application wide profile
..
profiling: Add an application wide profile
The built in Python profilers are good for single thread
Dan Kenigsberg has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9: Code-Review+2
Escalating score.
--
To view, visit http://gerrit.ovirt.org/26113
To unsubscribe,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 10:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_install_rpm_sanity_gerrit/547/
Francesco Romani has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
(1 comment)
http://gerrit.ovirt.org/#/c/26113/9/lib/vdsm/profile.py
File lib/vdsm/profile.py:
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
(1 comment)
http://gerrit.ovirt.org/#/c/26113/9/lib/vdsm/profile.py
File lib/vdsm/profile.py:
Line 45:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8476/ : SUCCESS
Francesco Romani has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9: Code-Review+1
I was thinking to avoid to the blowing up, by coercing an unexpected value,
like
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8476/ : SUCCESS
Federico Simoncelli has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
+1 for the initiative! I'll reviw this later. Thanks Nir.
--
To view, visit
Adam Litke has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26113
To unsubscribe, visit
Douglas Schilling Landgraf has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
(1 comment)
http://gerrit.ovirt.org/#/c/26113/9/lib/vdsm/profile.py
File
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
(1 comment)
http://gerrit.ovirt.org/#/c/26113/9/lib/vdsm/profile.py
File lib/vdsm/profile.py:
Line 35:
Douglas Schilling Landgraf has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9: Code-Review+1
(1 comment)
http://gerrit.ovirt.org/#/c/26113/9/lib/vdsm/profile.py
File
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 8:
Changes:
- Move profiling details to profile module
- Address Saggi comments regarding stopping the
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 7:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8474/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 8:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8475/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9: Verified+1
This patch add the new profile module to the distribution and fix a missing
import.
I
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8476/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8476/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8476/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8476/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
The tests failures are not relevant (segfault in Python while running libvirt
test).
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 9:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/8476/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6:
After talking with Saggi, we agreed to:
- Move profiling logic to separate module under
Saggi Mizrahi has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6: Code-Review-1
(2 comments)
http://gerrit.ovirt.org/#/c/26113/6/vdsm/vdsm
File vdsm/vdsm:
Line 88:
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6:
(2 comments)
Thanks, will clean up.
http://gerrit.ovirt.org/#/c/26113/6/vdsm/vdsm
File vdsm/vdsm:
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6:
(1 comment)
http://gerrit.ovirt.org/#/c/26113/6/vdsm/vdsm
File vdsm/vdsm:
Line 84: try:
Line 85:
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6:
Rebase on dependent patch.
--
To view, visit http://gerrit.ovirt.org/26113
To unsubscribe, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 6:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6964/ : SUCCESS
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 5:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6934/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 5:
Rebased over dependent patch.
--
To view, visit http://gerrit.ovirt.org/26113
To unsubscribe, visit
Yaniv Bronhaim has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/26113
To unsubscribe, visit
Yaniv Bronhaim has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 2: Code-Review-1
you must to have yappi package on your local setup when running vdsm, and then
we
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 2:
There is no such package, and it will take ages until we can get it into
Fedora/EL. So either we ship
Yaniv Bronhaim has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 2: -Code-Review
removing my score. i suggest to start the packaging procedure anyway
--
To view,
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 3:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6892/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 3:
Added profile_format option.
--
To view, visit http://gerrit.ovirt.org/26113
To unsubscribe, visit
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 3: Verified+1
Verified running with and without profiling.
--
To view, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 4:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6894/ : SUCCESS
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 4:
Rebased over dependent patch. Verified on RHEL 6.5 and Fedora 19.
--
To view, visit
Nir Soffer has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 2: Verified+1
Move profile to /var/run/vdsm, protecting from symlink attacks.
Verified by running with and
oVirt Jenkins CI Server has posted comments on this change.
Change subject: profiling: Add an application wide profile
..
Patch Set 2:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/6876/ : SUCCESS
43 matches
Mail list logo