fix: Remove repository requirement to support oci helm charts

dsieradzki · Feb 14, 2024 · 8cdca24 · 8cdca24
1 parent 052cc68
commit 8cdca24
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/core/src/dispatcher/usecase/common.rs b/core/src/dispatcher/usecase/common.rs
@@ -484,8 +484,13 @@ pub(crate) mod apps {
             .upgrade_or_install()
             .create_namespace()
             .namespace(&app.namespace)
-            .name(&app.release_name)
-            .chart(&app.chart_name, &app.chart_name);
+            .name(&app.release_name);
+
+        if app.repository.is_empty() {
+            command_builder = command_builder.chart(&app.chart_name);
+        } else {
+            command_builder = command_builder.chart_with_repo(&app.chart_name, &app.chart_name);
+        }
 
         if !app.values.is_empty() {
             command_builder =

diff --git a/helm-client/src/command_builder.rs b/helm-client/src/command_builder.rs
@@ -82,7 +82,12 @@ impl InstallCommand {
         res.parts.push(name.to_string());
         res
     }
-    pub fn chart(self, repo: &str, name: &str) -> Self {
+    pub fn chart(self, name: &str) -> Self {
+        let mut res = self;
+        res.parts.push(name.to_string());
+        res
+    }
+    pub fn chart_with_repo(self, repo: &str, name: &str) -> Self {
         let mut res = self;
         res.parts.push(format!("{}/{}", repo, name));
         res