Skip to content

bench: nixos service fixes & analysis improvements #4509

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 6 commits into from
Oct 5, 2022
Merged

bench: nixos service fixes & analysis improvements #4509

merged 6 commits into from
Oct 5, 2022

Conversation

deepfire
Copy link
Contributor

@deepfire deepfire commented Oct 5, 2022 •
edited
Loading

  1. extend defServiceModule to allow configurable tracing of arguments and return values.
  2. fix the cardano-tracer service systemd section
  3. update tx-generator service to cardano-node service changes
  4. add withCardanoTracer option to cardano-node service
  5. fixes and improvements in locli

@deepfire deepfire changed the title bench: cardano-tracer enablement bench: nixos service fixes & analysis improvements Oct 5, 2022
@deepfire
Copy link
Contributor Author

deepfire commented Oct 5, 2022

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Oct 5, 2022

Build succeeded:

@iohk-bors iohk-bors bot merged commit 8ced4de into master Oct 5, 2022
@iohk-bors iohk-bors bot deleted the bench-master branch October 5, 2022 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fmaste fmaste fmaste approved these changes

@mgmeier mgmeier Awaiting requested review from mgmeier

@denisshevchenko denisshevchenko Awaiting requested review from denisshevchenko

@jutaro jutaro Awaiting requested review from jutaro

@MarcFontaine MarcFontaine Awaiting requested review from MarcFontaine

@cleverca22 cleverca22 Awaiting requested review from cleverca22

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@deepfire @fmaste