Skip to content

Conversation

@acuarica
Copy link
Contributor

@acuarica acuarica commented Jun 21, 2025

Description:

This PR introduces the OPCODELOGGER_ENABLED to enable/disable the opcodeLogger tracer.

Related issue(s):

Fixes #3868.

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@acuarica acuarica added this to the 0.70.0 milestone Jun 21, 2025
@acuarica acuarica self-assigned this Jun 21, 2025
@acuarica acuarica added the enhancement New feature or request label Jun 21, 2025
@lfdt-bot
Copy link

lfdt-bot commented Jun 21, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@github-actions
Copy link

github-actions bot commented Jun 21, 2025

Test Results

 22 files  +  2  305 suites  +49   18m 54s ⏱️ +47s
668 tests +  2  661 ✅ ±  0  5 💤 ±0  2 ❌ +2 
884 runs  +202  877 ✅ +200  5 💤 ±0  2 ❌ +2 

For more details on these failures, see this check.

Results for commit ff0dd5b. ± Comparison against base commit 928f5c7.

♻️ This comment has been updated with latest results.

acuarica added 3 commits June 25, 2025 01:27
Signed-off-by: Luis Mastrangelo <[email protected]>
Signed-off-by: Luis Mastrangelo <[email protected]>
Signed-off-by: Luis Mastrangelo <[email protected]>
@acuarica acuarica force-pushed the 3868-flag-to-enabledisable-opcodelogger branch from b151d81 to ff0dd5b Compare June 24, 2025 23:27
@acuarica acuarica marked this pull request as ready for review June 24, 2025 23:28
@acuarica acuarica requested review from a team as code owners June 24, 2025 23:28
@acuarica acuarica requested a review from simzzz June 24, 2025 23:28
Copy link
Member

@natanasow natanasow left a comment

Choose a reason for hiding this comment

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

LGTM

@acuarica acuarica merged commit 8ff3d14 into main Jun 25, 2025
43 of 45 checks passed
@acuarica acuarica deleted the 3868-flag-to-enabledisable-opcodelogger branch June 25, 2025 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Introduce a configuration flag to enable/disable opcodeLogger in debug_traceTransaction

5 participants