changelog, some fixes after merge, cargo clippy

Author: xeniape

Cargo.lock

@@ -26,5 +26,5 @@ pub struct DemoUninstallParameters {
     pub demo_namespace: String,
 
     pub skip_operators: bool,
-    pub skip_crds: bool
+    pub skip_crds: bool,
 }

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

@@ -1,6 +1,6 @@
-use kube::api::{ApiResource, GroupVersionKind};
 use serde::{Deserialize, Serialize};
 use snafu::{OptionExt, ResultExt, Snafu};
+use stackable_operator::kube::api::{ApiResource, GroupVersionKind};
 use tracing::{Span, debug, info, instrument, warn};
 use tracing_indicatif::span_ext::IndicatifSpanExt as _;
 #[cfg(feature = "openapi")]
@@ -251,7 +251,11 @@ impl DemoSpec {
 
         // Delete remaining objects not namespace scoped
         client
-            .delete_all_objects_with_label("stackable.tech/demo", &uninstall_parameters.demo_name, None)
+            .delete_all_objects_with_label(
+                "stackable.tech/demo",
+                &uninstall_parameters.demo_name,
+                None,
+            )
             .await
             .context(DeleteObjectSnafu)?;
 

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

@@ -24,5 +24,5 @@ pub struct StackUninstallParameters {
     pub stack_namespace: String,
 
     pub skip_operators: bool,
-    pub skip_crds: bool
+    pub skip_crds: bool,
 }

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

@@ -1,7 +1,7 @@
-use kube::api::{ApiResource, GroupVersionKind};
 use serde::{Deserialize, Serialize};
 use serde_yaml::Mapping;
 use snafu::{OptionExt, ResultExt, Snafu};
+use stackable_operator::kube::api::{ApiResource, GroupVersionKind};
 use tracing::{Span, debug, info, instrument, log::warn};
 use tracing_indicatif::span_ext::IndicatifSpanExt as _;
 #[cfg(feature = "openapi")]
@@ -248,14 +248,17 @@ impl StackSpec {
 
         // Delete remaining objects not namespace scoped
         client
-            .delete_all_objects_with_label("stackable.tech/stack", &uninstall_parameters.stack_name, None)
+            .delete_all_objects_with_label(
+                "stackable.tech/stack",
+                &uninstall_parameters.stack_name,
+                None,
+            )
             .await
             .context(DeleteObjectSnafu)?;
 
         // Delete operators and the operator namespace
         if !uninstall_parameters.skip_operators {
-            self
-                .uninstall_release(release_list, &uninstall_parameters.operator_namespace)
+            self.uninstall_release(release_list, &uninstall_parameters.operator_namespace)
                 .await?;
 
             client

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

@@ -3,24 +3,27 @@ use std::{collections::BTreeMap, string::FromUtf8Error};
 use reqwest::StatusCode;
 use serde::Deserialize;
 use snafu::{OptionExt, ResultExt, Snafu};
-use tokio::{
-    sync::RwLock,
-    time::{self, sleep},
-};
 use stackable_operator::{
     crd::listener::v1alpha1::Listener,
-    k8s_openapi::api::{
-        apps::v1::{Deployment, StatefulSet},
-        core::v1::{Endpoints, Namespace, Node, Secret, Service},
+    k8s_openapi::{
+        api::{
+            apps::v1::{Deployment, StatefulSet},
+            core::v1::{Endpoints, Namespace, Node, Secret, Service},
+        },
+        apiextensions_apiserver::pkg::apis::apiextensions::v1::CustomResourceDefinition,
     },
     kube::{
         self, Api, Discovery, ResourceExt,
         api::{DeleteParams, ListParams, Patch, PatchParams, PostParams},
         core::{DynamicObject, GroupVersionKind, ObjectList, ObjectMeta, TypeMeta},
-        discovery::{ApiCapabilities, ApiResource, Scope},
+        discovery::{self, ApiCapabilities, ApiResource, Scope},
     },
     kvp::Labels,
 };
+use tokio::{
+    sync::RwLock,
+    time::{self, sleep},
+};
 use tracing::{Span, info, instrument};
 use tracing_indicatif::{indicatif_eprintln, span_ext::IndicatifSpanExt as _};
 

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

@@ -4,12 +4,17 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+### Added
+
+- Add `uninstall` subcommand for `demo`/`stack` commands ([#429]).
+
 ### Changed
 
 - Bump Rust to `1.93.0` as well as dependencies ([#426]).
 - Bump Go to `1.26.0` as well as dependencies ([#426]).
 
 [#426]: https://github.com/stackabletech/stackable-cockpit/pull/426
+[#429]: https://github.com/stackabletech/stackable-cockpit/pull/429
 
 ## [1.2.2] - 2025-12-03
 

rust/stackablectl/CHANGELOG.md

@@ -21,9 +21,7 @@ use stackable_cockpit::{
     },
     xfer,
 };
-use stackable_operator::{
-    kvp::{LabelError, Labels},
-};
+use stackable_operator::kvp::{LabelError, Labels};
 use tracing::{Span, debug, info, instrument};
 use tracing_indicatif::{self, span_ext::IndicatifSpanExt as _};
 
@@ -548,8 +546,8 @@ async fn uninstall_cmd(
             demo_name: args.demo_name.clone(),
             operator_namespace: args.namespaces.operator_namespace.clone(),
             demo_namespace: args.namespaces.namespace.clone(),
-            skip_operators: args.skip_operators_and_crds.clone(),
-            skip_crds: args.skip_operators_and_crds.clone(),
+            skip_operators: args.skip_operators_and_crds,
+            skip_crds: args.skip_operators_and_crds,
         },
         &client,
         transfer_client,

rust/stackablectl/src/cmds/demo.rs

@@ -21,9 +21,7 @@ use stackable_cockpit::{
     },
     xfer,
 };
-use stackable_operator::{
-    kvp::{LabelError, Labels},
-};
+use stackable_operator::kvp::{LabelError, Labels};
 use tracing::{Span, debug, info, instrument};
 use tracing_indicatif::span_ext::IndicatifSpanExt as _;
 
@@ -499,8 +497,8 @@ async fn uninstall_cmd(
                         stack_name: args.stack_name.clone(),
                         operator_namespace: args.namespaces.operator_namespace.clone(),
                         stack_namespace: args.namespaces.namespace.clone(),
-                        skip_operators: args.skip_operators_and_crds.clone(),
-                        skip_crds: args.skip_operators_and_crds.clone(),
+                        skip_operators: args.skip_operators_and_crds,
+                        skip_crds: args.skip_operators_and_crds,
                     },
                     &client,
                     transfer_client,