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

fix/argo #133

Merged
letian merged 10 commits from fix/argo into master 2025-09-08 14:04:13 +00:00
Conversation 0 Commits 10 Files Changed 3 +6 -5
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 95c2e1dd72 - Show all commits

11
harmony/src/modules/prometheus/k8s_prometheus_alerting_score.rs
View File

@ -197,11 +197,12 @@ impl K8sPrometheusCRDAlertingInterpret {
} }
async fn ensure_grafana_operator(&self) -> Result<Outcome, InterpretError> { async fn ensure_grafana_operator(&self) -> Result<Outcome, InterpretError> {
if self.crd_exists("grafanas.grafana.integreatly.org").await { // if self.crd_exists("grafanas.grafana.integreatly.org").await {
debug!("grafana CRDs already exist — skipping install."); // debug!("grafana CRDs already exist — skipping install.");
return Ok(Outcome::success("Grafana CRDs already exist".to_string())); // return Ok(Outcome::success("Grafana CRDs already exist".to_string()));
} // }
//may not need to check this since the grafana operator is namespaceScope=true -> it
//will deploy with rolebindings rather than cluster role bindings
letian marked this conversation as resolved Outdated
letian commented 2025-09-05 19:37:36 +00:00
Outdated
Review
Copy Link

still need to keep that commented?

still need to keep that commented?
let _ = Command::new("helm") let _ = Command::new("helm")
.args([ .args([
"repo", "repo",