Fix/k8s helm health (#363)

* fix k8s processor not getting health requests

Author: JannikSt

crates/orchestrator/src/plugins/node_groups/mod.rs

@@ -248,7 +248,7 @@ impl NodeGroupsPlugin {
         let mut total_available = healthy_nodes.len();
 
         for config in &available_configurations {
-            println!("Checking configuration: {:?}", config);
+            debug!("Checking configuration: {:?}", config);
             while total_available >= config.min_group_size {
                 let initial_available = total_available;
 
@@ -347,14 +347,18 @@ impl NodeGroupsPlugin {
         let mut interval = tokio::time::interval(Duration::from_secs(duration));
 
         loop {
+            let start = std::time::Instant::now();
             interval.tick().await;
+
             if let Err(e) = self.try_form_new_groups() {
                 error!("Error in group management: {}", e);
             }
             if let Some(heartbeats) = &self.node_groups_heartbeats {
                 heartbeats.update_node_groups();
             }
-            log::info!("Group management loop completed");
+
+            let elapsed = start.elapsed();
+            log::info!("Group management loop completed in {:?}", elapsed);
         }
     }
 

crates/shared/src/models/node.rs

@@ -306,7 +306,7 @@ impl FromStr for ComputeRequirements {
         Ok(requirements)
     }
 }
-use log::info;
+use log::{debug, info};
 
 impl ComputeSpecs {
     /// Checks if the current compute specs meet the given requirements.
@@ -354,7 +354,7 @@ impl ComputeSpecs {
         if !requirements.gpu.is_empty() {
             // Requirements specify GPUs, so the node must have a GPU spec...
             let Some(spec_gpu) = &self.gpu else {
-                info!("GPU requirements not met: GPU required but none available");
+                debug!("GPU requirements not met: GPU required but none available");
                 return false;
             };
             // ...and that GPU spec must meet *at least one* of the requirement options.
@@ -363,7 +363,7 @@ impl ComputeSpecs {
                 .iter()
                 .any(|req_gpu| spec_gpu.meets(req_gpu))
             {
-                info!("GPU requirements not met");
+                debug!("GPU requirements not met");
                 return false;
             }
         }

deployment/k8s/orchestrator-chart/templates/orchestrator-deployment.yaml

@@ -1,7 +1,6 @@
 {{- define "orchestrator.container" }}
 - name: {{ .root.Values.name | default "orchestrator" }}
   image: {{ .root.Values.image }}
-  {{- if .api }}
   ports:
     - containerPort: 8090
   readinessProbe:
@@ -16,7 +15,6 @@
       port: 8090
     initialDelaySeconds: 15
     periodSeconds: 20
-  {{- end }}
   resources:
     requests:
       memory: "256Mi"