Skip to content

Add new tool "crau-monitor"#206

Merged
ueno merged 3 commits intolatchset:mainfrom
ueno:wip/tools
Jan 25, 2026
Merged

Add new tool "crau-monitor"#206
ueno merged 3 commits intolatchset:mainfrom
ueno:wip/tools

Conversation

@ueno
Copy link
Copy Markdown
Contributor

@ueno ueno commented Dec 11, 2025

It is a common use-case to monitor cryptographic usage on the system
at real time. This adds a dedicated CLI tool, crau-monitor, by
combining the functionalities of event-broker and client. Unlike
those, crau-monitor doesn't require any system service but directly
reads the log file.

Copy link
Copy Markdown
Collaborator

@t184256 t184256 left a comment

Choose a reason for hiding this comment

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

Inline: less of a review, more of a mixture of high-level questions with learning Rust questions.

Comment thread monitor/src/config.rs
use std::time::Duration;
use toml::{Table, Value};

const CONFIG: &str = "/etc/crypto-auditing/monitor.conf";
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I'm not sure it needs a config, what's would be a compelling reason to use one you're anticipating?

Comment thread monitor/src/monitor.rs Outdated
Comment thread monitor/src/monitor.rs Outdated
Comment thread monitor/src/monitor.rs Outdated
Comment thread monitor/src/monitor.rs Outdated
Comment thread monitor/src/monitor.rs
@ueno ueno force-pushed the wip/tools branch 4 times, most recently from dff8e6a to a8a56d4 Compare December 12, 2025 09:10
Copy link
Copy Markdown
Collaborator

@t184256 t184256 left a comment

Choose a reason for hiding this comment

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

no more concerns other than the CREATE events

ueno added 3 commits January 25, 2026 09:25
Signed-off-by: Daiki Ueno <dueno@redhat.com>
It is a common use-case to monitor cryptographic usage on the system
at real time. This adds a dedicated CLI tool, crau-monitor, by
combining the functionalities of event-broker and client. Unlike
those, crau-monitor doesn't require any system service but directly
reads the log file.

Signed-off-by: Daiki Ueno <dueno@redhat.com>
Signed-off-by: Daiki Ueno <dueno@redhat.com>
@ueno
Copy link
Copy Markdown
Contributor Author

ueno commented Jan 25, 2026

Thanks for the review, merging. There will be a follow up on further refactoring of the shared code between log-parser and monitor on tracking contexts.

@ueno ueno merged commit 5392989 into latchset:main Jan 25, 2026
8 checks passed
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.

2 participants