Skip to content

Conversation

@vrigal
Copy link
Collaborator

@vrigal vrigal commented Sep 26, 2022

@vrigal
Copy link
Collaborator Author

vrigal commented Sep 26, 2022

I tested with a simple script and got those logs:

$ ./test_log.py
2022-09-26 15:10:51.000241 [ERROR   ] test_log.py: AAAAAAAA
2022-09-26 15:10:51.000241 [ERROR   ] test_log.py: BBBBBBBB
$ ./test_log.py # with the modification
2022-09-26 15:10:55.000179 [ERROR   ] test_log.py: msg='AAAAAAAA' extra={'stdout': 'out text', 'anything': 'any'}
2022-09-26 15:10:55.000179 [ERROR   ] test_log.py: msg='BBBBBBBB' extra={}

@vrigal vrigal requested review from abpostelnicu and marco-c October 3, 2022 12:32
@abpostelnicu abpostelnicu merged commit 71ba053 into mozilla:master Oct 4, 2022
marco-c added a commit to marco-c/code-coverage that referenced this pull request Mar 9, 2023
marco-c added a commit to marco-c/code-coverage that referenced this pull request Mar 9, 2023
marco-c added a commit to mozilla/code-coverage that referenced this pull request Mar 9, 2023
* Use sentry_sdk instead of raven and improve Sentry grouping

Fixes #1521 by bringing in the changes from:
- mozilla/code-review#1253
- mozilla/code-review#1276
- mozilla/code-review#1333
- mozilla/code-review#1365

* Rename sentry_dsn parameter to SENTRY_DSN to match the name in code-review and reduce code differences between the log.py files
marco-c added a commit to mozilla/code-coverage that referenced this pull request Mar 9, 2023
* Use sentry_sdk instead of raven and improve Sentry grouping

Fixes #1521 by bringing in the changes from:
- mozilla/code-review#1253
- mozilla/code-review#1276
- mozilla/code-review#1333
- mozilla/code-review#1365

* Rename sentry_dsn parameter to SENTRY_DSN to match the name in code-review and reduce code differences between the log.py files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extend mercurial logging

2 participants