Skip to content
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

RTView: important fixes #4259

Merged
merged 1 commit into from
Aug 5, 2022
Merged

RTView: important fixes #4259

merged 1 commit into from
Aug 5, 2022

Conversation

denisshevchenko
Copy link
Contributor

@denisshevchenko denisshevchenko commented Aug 2, 2022 •
edited
Loading

This PR introduces a lot of small fixes/improvements, including:

  1. Better backup/restoring for historical data.
  2. More efficient work with node's name (taken from NodeInfo).
  3. Names-based logging instead of ids-based logging.
  4. Better timeline charts.
  5. "Static" colors for the nodes.

@denisshevchenko denisshevchenko added RTView The service for monitoring over Cardano nodes tracing labels Aug 2, 2022
@denisshevchenko denisshevchenko self-assigned this Aug 2, 2022
@denisshevchenko denisshevchenko force-pushed the rtview-fixes branch 5 times, most recently from 0caf1be to 5fc1c11 Compare August 3, 2022 13:28
@denisshevchenko denisshevchenko marked this pull request as ready for review August 4, 2022 18:02
@denisshevchenko
Copy link
Contributor Author

bors r+

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Aug 5, 2022

Build succeeded:

@iohk-bors iohk-bors bot merged commit 13a50c1 into master Aug 5, 2022
@iohk-bors iohk-bors bot deleted the rtview-fixes branch August 5, 2022 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deepfire deepfire deepfire approved these changes

@Jimbo4350 Jimbo4350 Awaiting requested review from Jimbo4350

@newhoggy newhoggy Awaiting requested review from newhoggy

Assignees

@denisshevchenko denisshevchenko

Labels
RTView The service for monitoring over Cardano nodes
Projects
None yet
Milestone
No milestone
Development

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