server-tools icon indicating copy to clipboard operation
server-tools copied to clipboard

[14.0] Sentry APM integration

Open sebastienbeau opened this issue 3 years ago • 7 comments

Working progress

The aim of this PR is to integrate correctly Sentry APM feature.

  • [x] generate transaction with human readable name for http
  • [x] generate transaction with human readable name for cron process
  • [x] track queue job correctly
  • [x] integrate span for database
  • [x] integrate span for external request
  • [x] improve logger integration (add event_log_level and breadcrum_log_level)
  • [ ] write test

Note: I have work on a previous integration with elastic APM : https://github.com/akretion/odoo-elasticapm

sebastienbeau avatar Dec 05 '22 22:12 sebastienbeau

Hi @barsi, @moylop260, @versada, @fernandahf, @naglis, some modules you are maintaining are being modified, check this out!

OCA-git-bot avatar Dec 05 '22 22:12 OCA-git-bot

If you are interested you can play with it, it's working just need to write the unit test

@barsi, @moylop260, @versada, @fernandahf, @naglis, @hparfr, @PierrickBrun, @dnplkndll, @themreza, @kirtangajjar

sebastienbeau avatar Dec 13 '22 09:12 sebastienbeau

Transaction name improvement

image

Tag improvement : For every http method : db, model, method are automatically added as tag

image

Tag improvement : For queue job db, job.model, job.method are automatically added as tag

image

sebastienbeau avatar Dec 13 '22 10:12 sebastienbeau

@sebastienbeau can you resolve conflicts?

ioans73 avatar Jun 22 '23 10:06 ioans73

ping @sebastienbeau

ioans73 avatar Jun 29 '23 11:06 ioans73

@sebastienbeau @moylop260 Could we merge this? To be able to migrate it to version 16.0 and propose possible improvements

ioans73 avatar Jul 10 '23 11:07 ioans73