Observability stack + Other changes - #1150
Open
HatemMn wants to merge 34 commits into
Open
Conversation
fix: fixes fix: commit fix: manual edits fix: another fix
feat: WIP on feat/audit_and_siem chore: adr feat: fix the limits doc: doc is hard test: integration
feat: fix final flaws feat: fix tests that didn't pass fix: ci fix feat: 2 feat: ci fear: update reverse proxy docs fix: iris fix: another fix fix: restaure iris
This commit basically hashes usernames
- UserId.username is now a typed UserId (not String) after develop's UserId refactor: use Display instead of clone(). - AuditFailureMode::default() disambiguated from Default::default() per clippy::default_trait_access (pedantic).
This reverts commit 79299d2.
retrieve_objects() now takes ObjectHandle<'_> instead of &str after develop's database API refactor.
… const-fn, .gitignore, ADR title)
…g (split out of feat/audit_integration)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR is a collection of commits deferred from #1115 , thanks to @Manuthor
Everything here is monitoring and OTLP telemetry, nothing about audit or SIEM.
The mise monitoring test spins up the OTel collector, VictoriaMetrics and Grafana and checks the metrics actually show up on the Prometheus endpoint. I also renamed otlp-metrics.md to otlp-telemetry.md and split the OTLP section out of logging.md, then reorganized nav.yml and SUMMARY.md so logging, log reference, OTLP and the monitoring stack all sit under one "Observability" entry instead of being scattered across three separate sections.
There's also a small orphaned file, otlp-audit-collector-config.yaml, left over from an earlier attempt at pushing audit events through OTLP before we switched to file tailing. It's unused now but I kept it here rather than in the audit PR since it's still OTLP-shaped.
Three unrelated one-liners rode along from the same branch history and didn't seem worth splitting further: a const fn tweak on the TTLV serializer's too
These files need to be reviewed one by one before any merge