cleaned up some comments, removed Tgz case

stackabletech · Dec 19, 2024 · b9b6100 · b9b6100
1 parent b0fbdfc
commit b9b6100
Show file tree

Hide file tree

Showing 11 changed files with 31 additions and 46 deletions.
diff --git a/extra/completions/_stackablectl b/extra/completions/_stackablectl
diff --git a/extra/completions/stackablectl.bash b/extra/completions/stackablectl.bash
diff --git a/extra/completions/stackablectl.fish b/extra/completions/stackablectl.fish
diff --git a/extra/completions/stackablectl.nu b/extra/completions/stackablectl.nu
diff --git a/rust/stackable-cockpit/src/platform/manifests.rs b/rust/stackable-cockpit/src/platform/manifests.rs
@@ -62,7 +62,6 @@ pub enum Error {
 
 pub trait InstallManifestsExt {
     // TODO (Techassi): This step shouldn't care about templating the manifests nor fetching them from remote
-    // TODO aken: do we support helm charts from registries?
     #[instrument(skip_all)]
     #[allow(async_fn_in_trait)]
     async fn install_manifests(
@@ -95,7 +94,7 @@ pub trait InstallManifestsExt {
                         helm_chart.name, helm_chart.version
                     );
 
-                    // TODO aken: assuming all manifest helm charts refer to repos not registries
+                    // Assumption: that all manifest helm charts refer to repos not registries
                     helm::add_repo(&helm_chart.repo.name, &helm_chart.repo.url).context(
                         AddHelmRepositorySnafu {
                             repo_name: helm_chart.repo.name.clone(),

diff --git a/rust/stackable-cockpit/src/platform/operator/mod.rs b/rust/stackable-cockpit/src/platform/operator/mod.rs
@@ -194,7 +194,6 @@ impl OperatorSpec {
         let chart_source = match chart_source {
             ChartSourceType::OCI => HELM_OCI_REGISTRY.to_string(),
             ChartSourceType::Repo => self.helm_repo_name(),
-            ChartSourceType::Tgz => "TODO".to_string(),
         };
 
         // Install using Helm
@@ -237,9 +236,6 @@ pub enum ChartSourceType {
 
     /// Nexus repositories: resolution (dev, test, stable) is based on the version and thus may be operator-specific
     Repo,
-
-    /// Archive
-    Tgz,
 }
 
 #[cfg(test)]