feat(metrics): total sequenced transactions

[einar-oplabs]

No description provided.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.0%. Comparing base (6d30f52) to head (cec150c).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...rates/node/service/src/actors/sequencer/metrics.rs	0.0%	4 Missing ⚠️
crates/node/service/src/actors/sequencer/actor.rs	0.0%	3 Missing ⚠️
crates/protocol/protocol/src/attributes.rs	0.0%	3 Missing ⚠️
crates/node/service/src/metrics/mod.rs	0.0%	2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (6d30f52) and HEAD (cec150c). Click for more details.

HEAD has 23 uploads less than BASE

Flag	BASE (6d30f52)	HEAD (cec150c)
proof	11	0
e2e	11	0
unit	2	1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[einar-oplabs]

There seem to be buggy behavior in the current version of the inc! macro. It has 3 arms, but it seems only arm 0 and 1 have been used in code. There are no tests. When trying to trigger the arm 2 (the one that allows for an increment parameter), it is actually matched by arm 1 instead, but then fails with leftover tokens. If I swap arm 1 and 2 it succeeds. However, I don't if that changes the behavior of other macro calls, so we need to feel comfortable before merging this one.

It is out of scope for this issue to debug and write tests for the metric macros.

[einar-oplabs]

I decided to not using the macro all together. I now consider the branch done and ready to merge. Please check everything looks right on your side @jelias2.

[op-will]

Looks good!