NationTech/harmony
6
0
Fork 1
Code Issues 32 Pull Requests 7 Actions 7 Packages Projects Releases 1 Wiki Activity

monitoring-alerting #30

Merged
wjro merged 12 commits from monitoring-alerting into master 2025-05-06 17:50:57 +00:00
Conversation 0 Commits 12 Files Changed 6 +306
wjro commented 2025-05-05 16:50:30 +00:00
Owner
Copy Link
No description provided.
wjro added 4 commits 2025-05-05 16:50:31 +00:00
wjro added 1 commit 2025-05-05 17:40:40 +00:00
wjro added 1 commit 2025-05-05 19:38:42 +00:00
wjro added 1 commit 2025-05-05 20:37:23 +00:00
taha reviewed 2025-05-06 14:17:17 +00:00
harmony/src/modules/monitoring/monitoring_alerting.rs Outdated
@ -0,0 +16,4 @@
#[derive(Debug)]
pub struct MonitoringAlertingStackScore {
pub monitoring_stack: Vec<Box<dyn Score<MonitoringAlertingTopology>>>,
taha commented 2025-05-06 14:17:17 +00:00
Collaborator
Copy Link

Why does the MonitoringAlertingStackScore need a Vec<Box<dyn Score<MonitoringAlertingTopology>>>? It seems like it's never used for anything.

A score's variables should be what is needed to set it up, i.e. any values, rules, etc you want to pass in for the interpret to use to spin up the score

Why does the `MonitoringAlertingStackScore` need a `Vec<Box<dyn Score<MonitoringAlertingTopology>>>`? It seems like it's never used for anything. A score's variables should be what is needed to set it up, i.e. any values, rules, etc you want to pass in for the interpret to use to spin up the score
wjro added 3 commits 2025-05-06 15:54:33 +00:00
wjro added 1 commit 2025-05-06 16:23:18 +00:00
wjro added 1 commit 2025-05-06 17:43:05 +00:00
wjro merged commit 4a9b95acad into master 2025-05-06 17:50:57 +00:00
johnride deleted branch monitoring-alerting 2025-05-06 17:55:41 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Architecture Decision Record

   
Good first issue

   
RFC
No Label
Architecture Decision Record
Good first issue
RFC
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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#30
No description provided.
Delete Branch "monitoring-alerting"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?