Changed used error type for missing uid to be in line with what Lars uses in the operator framework (which in turn is copied from the kubernetes client crate).

Author: soenkeliebau

src/provider/error.rs

@@ -43,6 +43,6 @@ pub enum StackableError {
         "The following config maps were specified in a pod but not found: {missing_config_maps:?}"
     )]
     MissingConfigMapsError { missing_config_maps: Vec<String> },
-    #[error("An object received from Kubernetes didn't contain a required field: {field_name}")]
-    IllegalKubeObject { field_name: String },
+    #[error("Object is missing key: {key}")]
+    MissingObjectKey { key: &'static str },
 }

src/provider/mod.rs

@@ -14,7 +14,7 @@ use log::{debug, error};
 
 use crate::provider::error::StackableError;
 use crate::provider::error::StackableError::{
-    CrdMissing, IllegalKubeObject, KubeError, PodValidationError,
+    CrdMissing, KubeError, MissingObjectKey, PodValidationError,
 };
 use crate::provider::repository::package::Package;
 use crate::provider::states::downloading::Downloading;
@@ -164,8 +164,8 @@ impl Provider for StackableProvider {
         let service_uid = if let Some(uid) = pod.as_kube_pod().metadata.uid.as_ref() {
             uid.to_string()
         } else {
-            return Err(anyhow::Error::new(IllegalKubeObject {
-                field_name: "uid".to_string(),
+            return Err(anyhow::Error::new(MissingObjectKey {
+                key: ".metadata.uid",
             }));
         };
         let parcel_directory = self.parcel_directory.clone();