diff --git a/.gitea/workflows/check.yml b/.gitea/workflows/check.yml new file mode 100644 index 0000000..b790ff0 --- /dev/null +++ b/.gitea/workflows/check.yml @@ -0,0 +1,14 @@ +name: Run Check Script +on: + push: + pull_request: + +jobs: + check: + runs-on: rust-cargo + steps: + - name: Checkout code + uses: actions/checkout@v4 + + - name: Run check script + run: bash check.sh diff --git a/examples/lamp/src/main.rs b/examples/lamp/src/main.rs index feac05d..74df0d8 100644 --- a/examples/lamp/src/main.rs +++ b/examples/lamp/src/main.rs @@ -4,9 +4,7 @@ use harmony::{ maestro::Maestro, modules::{ lamp::{LAMPConfig, LAMPScore}, - monitoring::monitoring_alerting::{ - AlertChannel, MonitoringAlertingStackScore, WebhookServiceType, - }, + monitoring::monitoring_alerting::{AlertChannel, MonitoringAlertingStackScore}, }, topology::{K8sAnywhereTopology, Url}, }; @@ -50,10 +48,6 @@ async fn main() { let mut monitoring_stack_score = MonitoringAlertingStackScore::new(); monitoring_stack_score.namespace = Some(lamp_stack.config.namespace.clone()); - monitoring_stack_score.alert_channel = Some(AlertChannel::WebHookUrl { - url: url, - webhook_service_type: WebhookServiceType::Discord, - }); maestro.register_all(vec![Box::new(lamp_stack), Box::new(monitoring_stack_score)]); // Here we bootstrap the CLI, this gives some nice features if you need them diff --git a/harmony/src/modules/monitoring/config.rs b/harmony/src/modules/monitoring/config.rs index c06377c..1477905 100644 --- a/harmony/src/modules/monitoring/config.rs +++ b/harmony/src/modules/monitoring/config.rs @@ -2,7 +2,6 @@ use serde::Serialize; use super::monitoring_alerting::AlertChannel; - #[derive(Debug, Clone, Serialize)] pub struct KubePrometheusConfig { pub namespace: String, diff --git a/harmony/src/modules/monitoring/discord_alert_manager.rs b/harmony/src/modules/monitoring/discord_alert_manager.rs index 5eaffa0..a3519e8 100644 --- a/harmony/src/modules/monitoring/discord_alert_manager.rs +++ b/harmony/src/modules/monitoring/discord_alert_manager.rs @@ -26,8 +26,10 @@ environment: Some(HelmChartScore { namespace: Some(NonBlankString::from_str(&config.namespace).unwrap()), release_name: NonBlankString::from_str(&name).unwrap(), - chart_name: NonBlankString::from_str("oci://hub.nationtech.io/library/alertmanager-discord") - .unwrap(), + chart_name: NonBlankString::from_str( + "oci://hub.nationtech.io/library/alertmanager-discord", + ) + .unwrap(), chart_version: None, values_overrides: None, values_yaml: Some(values.to_string()), diff --git a/harmony/src/modules/monitoring/mod.rs b/harmony/src/modules/monitoring/mod.rs index d880a67..914ae07 100644 --- a/harmony/src/modules/monitoring/mod.rs +++ b/harmony/src/modules/monitoring/mod.rs @@ -1,4 +1,4 @@ +mod config; +mod discord_alert_manager; mod kube_prometheus; pub mod monitoring_alerting; -mod discord_alert_manager; -mod config; diff --git a/harmony/src/modules/monitoring/monitoring_alerting.rs b/harmony/src/modules/monitoring/monitoring_alerting.rs index 0e7c2d4..325f9d3 100644 --- a/harmony/src/modules/monitoring/monitoring_alerting.rs +++ b/harmony/src/modules/monitoring/monitoring_alerting.rs @@ -116,7 +116,7 @@ impl MonitoringAlertingStackInterpret { outcomes.push(outcome); } for result in outcomes { - result?; + result?; } Ok(Outcome::success("All alert channels deployed".to_string()))