Skip to content

[11.0] Backport of profiler #1465

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
pedrobaeza merged 15 commits into from
May 3, 2019
Merged

[11.0] Backport of profiler #1465

pedrobaeza merged 15 commits into from
May 3, 2019

Conversation

@etobella
Copy link
Member

@etobella etobella commented Dec 26, 2018 •
edited
Loading

Backport for profiler from #1369
@moylop260 @jbeficent

@etobella
Copy link
Member Author

etobella commented Dec 27, 2018

Travis is failing due to #1466

@etobella etobella force-pushed the 11.0-mig-profiler branch 2 times, most recently from 6d6be53 to 2ecdda2 Compare December 27, 2018 15:20
@JordiBForgeFlow
Copy link
Member

JordiBForgeFlow commented Jan 9, 2019

@etobella now that #1466 is merged, can you rebase to check that Travis is green?

@etobella
Copy link
Member Author

etobella commented Jan 10, 2019

@jbeficent Now it is green and rebased.

JordiBForgeFlow
JordiBForgeFlow approved these changes Feb 28, 2019
View reviewed changes
Copy link
Member

@JordiBForgeFlow JordiBForgeFlow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have used it now for several months and works great!

@JordiBForgeFlow
Copy link
Member

JordiBForgeFlow commented Feb 28, 2019

@moylop260 this one LGTM. If you are good with it too we can merge!

@etobella
Copy link
Member Author

etobella commented Mar 1, 2019

I will apply commit 5355ab8 from v12 in order to make it similar,
Also, I made some tests in order to improve the coverage, maybe it is interesting to apply them on v12 @moylop260

@pedrobaeza pedrobaeza added this to the 11.0 milestone Mar 7, 2019
pedrobaeza
pedrobaeza approved these changes Mar 7, 2019
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please squash a bit the commit history of initial ones.

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Mar 7, 2019

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@rousseldenis
Copy link
Contributor

rousseldenis commented May 2, 2019

@etobella Could you squash ?

moylop260 and others added 15 commits May 2, 2019 10:03
@moylop260 @etobella
[ADD] profiler: Integration of python cprofile and postgresql logging…
70e82a9 
… collector for Odoo
@moylop260 @etobella
[ADD] profiler: Integration of python cprofile and postgresql logging…
3392541 
… collector for Odoo
@moylop260 @etobella
[REF] profiler: Forcing english log otuput and UTC time
9936a02
@moylop260 @etobella
[REF] profiler: Uses standard unix log_directory
522f537
@moylop260 @etobella
[REF] Add patch to kw fc
a5a0d3c
@moylop260 @etobella
Revert "[REF] Add patch to kw fc"
899efe5 
This reverts commit a023ae2.
@moylop260 @etobella
[REF] add view TODO: context is not working
91eb289
@moylop260 @etobella
[REF] profiler: Using BytesIO instead of StringIO
e9748aa
@moylop260 @etobella
conf file
53682eb
@hugho-ad @etobella
[REF] profiler: Filter profiling lines by active_id
4a04ada
@hugho-ad @etobella
[FIX] dual compatibility py2.7 py3.5 (OCA#3)
94d3fc3
@moylop260 @etobella
[REF] profiler: Using same output from brew install pgbadger
20d43d9
@etobella
[REF] profiler: Patching rpccalls
4b65bbc
@moylop260 @etobella
[FIX] profiler: Open form view without errors if the users is not sup…
a1408af 
…eradmin for psql

Signed-off-by: Moisés López <moylop260@vauxoo.com>
@etobella
[MIG] To 11.0
98e43d9
@etobella etobella force-pushed the 11.0-mig-profiler branch from 0eba7e0 to 98e43d9 Compare May 2, 2019 08:04
@etobella
Copy link
Member Author

etobella commented May 2, 2019

I have squashed my commits except the patching rpccalls, as it already exists on 12.0

@rousseldenis
Copy link
Contributor

rousseldenis commented May 2, 2019

@OCA/core-maintainers

@pedrobaeza pedrobaeza merged commit 8e17343 into OCA:11.0 May 3, 2019
@moylop260
Copy link
Contributor

moylop260 commented May 4, 2019

@etobella
Thanks for finishing the work

Also, I made some tests in order to improve the coverage, maybe it is interesting to apply them on v12 @moylop260

Yes, of course.
Sorry for delay about my PR
Could you help me to finish my own PR or creating another one for v12.0, please?

This was referenced Nov 10, 2021
Closed
Merged
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
@bt-admin
Merge pull request OCA#1465 from OCA/16.0
8e3a7a9 
Syncing from upstream OCA/server-tools (16.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pedrobaeza pedrobaeza pedrobaeza approved these changes

+1 more reviewer

@JordiBForgeFlow JordiBForgeFlow JordiBForgeFlow approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved ready to merge

Projects

None yet

Milestone

11.0

Development

Successfully merging this pull request may close these issues.

7 participants

@etobella @JordiBForgeFlow @OCA-git-bot @rousseldenis @moylop260 @pedrobaeza @hugho-ad