Skip to content

Enable persistence telemetry - #480

Open
jakduch wants to merge 1 commit into
Dynflow:masterfrom
jakduch:fix/463-persistence-telemetry
Open

jakduch wants to merge 1 commit into
Dynflow:masterfrom
jakduch:fix/463-persistence-telemetry

Conversation

@jakduch

@jakduch jakduch commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • load the existing persistence telemetry instrumentation
  • prepend the instrumentation module to the correct persistence class
  • cover all current persistence operations and remove duplicate entries
  • add a regression test that verifies persistence calls emit the registered metric

Testing

  • bundle exec ruby -Itest test/persistence_telemetry_test.rb
  • bundle exec rubocop lib/dynflow.rb lib/dynflow/debug/telemetry/persistence.rb test/persistence_telemetry_test.rb
  • full Ruby 3.3 / SQLite test suite: 403 runs, 1490 assertions, 0 failures, 0 errors

Fixes #463

@jakduch
jakduch force-pushed the fix/463-persistence-telemetry branch from c83357e to 468eb6c Compare September 26, 2026 11:42

This branch has not been deployed

No deployments
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.

Possibly dead code in lib/dynflow/debug/telemetry/persistence.rb

1 participant