Skip to content

Refactor and enhance hbrelay architecture and audit integration#1005

Merged
cgalibern merged 9 commits intoopensvc:mainfrom
cgalibern:dev2
Apr 7, 2026
Merged

Refactor and enhance hbrelay architecture and audit integration#1005
cgalibern merged 9 commits intoopensvc:mainfrom
cgalibern:dev2

Conversation

@cgalibern
Copy link
Copy Markdown
Contributor

Summary

This pull request introduces several improvements in the hbrelay component and audit subsystem integration to enhance functionality and maintainability.

Details

The following updates have been made:

  • hbrelay Enhancements:

    • Refactored hbrelay configuration into a self-contained cfg struct.
    • Added support for reloading configurations (e.g., password changes) via dynamic client updates (refreshClient).
    • Improved event subscription for handling dynamic updates effectively.
    • Added context-specific prefixes to log messages for better clarity.
    • Renamed peerDropWorker to peerDropper for naming consistency.
    • Adjusted logging levels (e.g., from Tracef to Debugf) for appropriate message verbosity.
  • Audit Enhancements:

    • Integrated hbaudit package capabilities to improve audit traceability.
    • Refactored HandleAuditStart and HandleAuditStop methods to accept variadic subsystem matchers, simplifying and standardizing subsystem name handling.

…subsystem matchers.

- Remove redundant checks with `slices.Contains`.
- Standardize and simplify subsystem naming logic.
- Encapsulate hbrelay-specific configuration into a dedicated `cfg` struct.
- Streamline client creation and password handling with `refreshClient`.
- Introduce event subscription within `cfg` to manage dynamic updates.
- Enhance logging and audit integration for better traceability.
@cgalibern cgalibern merged commit 2da6581 into opensvc:main Apr 7, 2026
1 check 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