impl_monitoring_alerting_kube_prometheus #64

Merged
wjro merged 9 commits from impl_monitoring_alerting_kube_prometheus into master 2025-06-24 18:54:17 +00:00
Owner
No description provided.
wjro added 12 commits 2025-06-20 19:49:27 +00:00
feat: deploys cluster monitoring from MonitoringAlertingScore
Some checks failed
Run Check Script / check (push) Failing after 44s
8d219c648a
fix: cargo fmt, remove old use from lamp example
All checks were successful
Run Check Script / check (push) Successful in 1m49s
Run Check Script / check (pull_request) Successful in 1m48s
7c92a08132
wip Alerting abstractions architecture
Some checks failed
Run Check Script / check (push) Failing after 42s
Run Check Script / check (pull_request) Failing after 41s
c16276b62b
wip: setting up base for monitoring
Some checks failed
Run Check Script / check (push) Failing after 43s
Run Check Script / check (pull_request) Failing after 43s
f6c146b14b
feat: add monitoring system and alerting
Some checks failed
Run Check Script / check (push) Failing after 45s
Run Check Script / check (pull_request) Failing after 44s
9412c0529a
- Implemented a basic monitoring system with alerting capabilities.
- Introduced `MonitoringAlertingScore` and related traits (`MonitoringSystem`, `ScrapeTarget`, `AlertRule`).
- Added `MonitoringAlertingInterpret` to handle the execution of monitoring configurations.
- Defined interfaces for Prometheus integration (PrometheusReceiver, PrometheusRule, PrometheusScrapeTarget).
- Introduced `CloneBox` trait to enable cloning boxed trait objects.
- Refactored Prometheus related traits to require `Debug` implementation.
- Implemented basic placeholder logic for Prometheus configuration and alerting.
- Updated existing traits to include `Send` and `Sync` where appropriate.
wip:installable/installer
Some checks failed
Run Check Script / check (push) Failing after 43s
Run Check Script / check (pull_request) Failing after 43s
ba536885bd
WIP: working on implementation of monitoring alert score for HelmKubePrometheus
Some checks failed
Run Check Script / check (push) Failing after 44s
Run Check Script / check (pull_request) Failing after 43s
3ad8cd60f8
chore: clean up unused old code, organized traits, impls, etc into logical locations
All checks were successful
Run Check Script / check (push) Successful in 1m46s
Run Check Script / check (pull_request) Successful in 1m46s
1771f47eca
fix: git conflict
All checks were successful
Run Check Script / check (pull_request) Successful in 1m31s
e344eddeb7
fix: deleted outdated monitoring and alerting score
All checks were successful
Run Check Script / check (pull_request) Successful in 1m59s
273befc61c
taha force-pushed impl_monitoring_alerting_kube_prometheus from 98675dc5e8 to dec4b76b57 2025-06-20 20:29:30 +00:00 Compare
wjro added 1 commit 2025-06-24 13:46:12 +00:00
removed unnecessary files
All checks were successful
Run Check Script / check (pull_request) Successful in 1m46s
55da0cb1ea
johnride requested changes 2025-06-24 18:13:53 +00:00
@@ -0,0 +1,18 @@
FROM php:8.4-apache
Owner

Don't commit that file it is autogenerated by Harmony. It should be stored somewhere that is gititgnored, which is not the case right now but still - don't commit.

Don't commit that file it is autogenerated by Harmony. It should be stored somewhere that is gititgnored, which is not the case right now but still - don't commit.
@@ -0,0 +1,16 @@
Owner

Don't commit

Don't commit
@@ -17,3 +20,3 @@
#[async_trait]
impl<S: AlertSender, T: Topology> Interpret<T> for AlertingInterpret<S> {
impl<S: AlertSender + Installable<T>, T: Topology + HelmCommand> Interpret<T>
Owner

T should not be bound to HelmCommand here

T should not be bound to HelmCommand here
@@ -147,0 +198,4 @@
let alert_manager_yaml =
serde_yaml::to_string(&alert_manager_values).expect("Failed to serialize YAML");
debug!("serialed alert manager: \n {:#}", alert_manager_yaml);
Owner

typo

typo
@@ -16,0 +41,4 @@
}
}
// //before we talked about having a trait installable and a trait installer for the topology
Owner

Delete comment

Delete comment
wjro added 1 commit 2025-06-24 18:51:16 +00:00
fix: change request from pr
All checks were successful
Run Check Script / check (pull_request) Successful in 1m23s
61c1d22b55
wjro merged commit f437c40428 into master 2025-06-24 18:54:17 +00:00
wjro deleted branch impl_monitoring_alerting_kube_prometheus 2025-06-24 18:54:18 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: NationTech/harmony#64
No description provided.