feat: scrape targets to be able to get snmp alerts from machines to prometheus #171

Merged
wjro merged 3 commits from feat/scrape_target into master 2025-10-22 15:33:25 +00:00
Owner
No description provided.
wjro added 1 commit 2025-10-20 15:45:49 +00:00
feat: scrape targets to be able to get snmp alerts from machines to prometheus
Some checks failed
Run Check Script / check (pull_request) Has been cancelled
a815f6ac9c
letian reviewed 2025-10-20 18:10:14 +00:00
@ -21,6 +21,7 @@ pub struct AlertingInterpret<S: AlertSender> {
pub sender: S,
pub receivers: Vec<Box<dyn AlertReceiver<S>>>,
pub rules: Vec<Box<dyn AlertRule<S>>>,
pub scrape_target: Vec<Box<dyn ScrapeTarget<S>>>,
Owner

Should it be scrape_targets? as it's a vector

Should it be `scrape_targets`? as it's a vector
letian marked this conversation as resolved
letian reviewed 2025-10-20 18:12:41 +00:00
@ -31,6 +31,7 @@ impl<T: Topology + HelmCommand + TenantManager> Score<T> for HelmPrometheusAlert
sender: KubePrometheus { config },
receivers: self.receivers.clone(),
rules: self.rules.clone(),
scrape_target: vec![],
Owner

Should we actually configure something here? Because we pass an empty list it doesn't seem used.

Should we actually configure something here? Because we pass an empty list it doesn't seem used.
Author
Owner

I will make this an Option<Vec> since it is not used in this score

I will make this an Option<Vec<ScrapeTarget>> since it is not used in this score
letian marked this conversation as resolved
wjro added 1 commit 2025-10-20 18:44:42 +00:00
fix: made targets plural and changed scrape targets to option in AlertingInterpret
Some checks failed
Run Check Script / check (pull_request) Has been cancelled
cb66b7592e
wjro added 1 commit 2025-10-22 15:33:05 +00:00
Merge branch 'master' into feat/scrape_target
Some checks are pending
Run Check Script / check (pull_request) Waiting to run
3174645c97
wjro merged commit 05205f4ac1 into master 2025-10-22 15:33:25 +00:00
wjro deleted branch feat/scrape_target 2025-10-22 15:33:25 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: NationTech/harmony#171
No description provided.