Skip to content

Commit

Permalink
Merge pull request #17 from nicholastmosher/namespaces
Browse files Browse the repository at this point in the history
Search all namespaces for installed charts when no namespace is given
  • Loading branch information
sehz authored Jun 18, 2021
2 parents fc8466b + a7fd3c4 commit 7346841
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 4 deletions.
4 changes: 3 additions & 1 deletion Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[package]
name = "fluvio-helm"
version = "0.4.2"
version = "0.4.3"
edition = "2018"
license = "Apache-2.0"
authors = ["Fluvio Contributors <[email protected]>"]
Expand Down
11 changes: 9 additions & 2 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -337,8 +337,15 @@ impl HelmClient {
.arg(exact_match)
.arg("--output")
.arg("json");
if let Some(ns) = namespace {
command.args(&["--namespace", ns]);

match namespace {
Some(ns) => {
command.args(&["--namespace", ns]);
}
None => {
// Search all namespaces
command.args(&["-A"]);
}
}

let output = command.result()?;
Expand Down

0 comments on commit 7346841

Please sign in to comment.