use Label struct for label

Author: xeniape

rust/stackable-cockpit/src/platform/demo/spec.rs

@@ -1,5 +1,6 @@
 use serde::{Deserialize, Serialize};
 use snafu::{OptionExt, ResultExt, Snafu};
+use stackable_operator::kvp::{Label, LabelError};
 use tracing::{Span, debug, info, instrument, warn};
 use tracing_indicatif::span_ext::IndicatifSpanExt as _;
 #[cfg(feature = "openapi")]
@@ -59,6 +60,9 @@ pub enum Error {
 
     #[snafu(display("failed to delete object"))]
     DeleteObject { source: k8s::Error },
+
+    #[snafu(display("failed to build label"))]
+    BuildLabel { source: LabelError },
 }
 
 impl InstallManifestsExt for DemoSpec {}
@@ -245,8 +249,8 @@ impl DemoSpec {
         // Delete remaining objects not namespace scoped
         client
             .delete_all_objects_with_label(
-                "stackable.tech/demo",
-                &uninstall_parameters.demo_name,
+                Label::try_from(("stackable.tech/demo", &uninstall_parameters.demo_name))
+                    .context(BuildLabelSnafu)?,
                 None,
             )
             .await

rust/stackable-cockpit/src/platform/stack/spec.rs

@@ -1,6 +1,7 @@
 use serde::{Deserialize, Serialize};
 use serde_yaml::Mapping;
 use snafu::{OptionExt, ResultExt, Snafu};
+use stackable_operator::kvp::{Label, LabelError};
 use tracing::{Span, debug, info, instrument, log::warn};
 use tracing_indicatif::span_ext::IndicatifSpanExt as _;
 #[cfg(feature = "openapi")]
@@ -74,6 +75,9 @@ pub enum Error {
 
     #[snafu(display("failed to delete object"))]
     DeleteObject { source: k8s::Error },
+
+    #[snafu(display("failed to build label"))]
+    BuildLabel { source: LabelError },
 }
 
 /// This struct describes a stack with the v2 spec
@@ -243,8 +247,8 @@ impl StackSpec {
         // Delete remaining objects not namespace scoped
         client
             .delete_all_objects_with_label(
-                "stackable.tech/stack",
-                &uninstall_parameters.stack_name,
+                Label::try_from(("stackable.tech/stack", &uninstall_parameters.stack_name))
+                    .context(BuildLabelSnafu)?,
                 None,
             )
             .await

rust/stackable-cockpit/src/utils/k8s/client.rs

@@ -18,7 +18,7 @@ use stackable_operator::{
         core::{DynamicObject, GroupVersionKind, ObjectList, ObjectMeta, TypeMeta},
         discovery::{self, ApiCapabilities, ApiResource, Scope},
     },
-    kvp::Labels,
+    kvp::{Label, Labels},
 };
 use tokio::{
     sync::RwLock,
@@ -323,8 +323,7 @@ impl Client {
     /// If no namespace is provided, deletes all clusterwide objects with the given label.
     pub async fn delete_all_objects_with_label(
         &self,
-        label_key: &str,
-        label_value: &str,
+        label: Label,
         namespace: Option<&str>,
     ) -> Result<(), Error> {
         let api_resources = self
@@ -353,8 +352,8 @@ impl Client {
 
             if let Some(objectlist) = objects {
                 for object in objectlist {
-                    if let Some(value) = object.labels().get(label_key) {
-                        if value.eq(label_value) {
+                    if let Some(value) = object.labels().get(&label.key().to_string()) {
+                        if value.eq(&label.value().to_string()) {
                             self.delete_object(
                                 &object.metadata.name.unwrap(),
                                 &api_resource,