Merge pull request #65 from Ramilito/fix/set_default_multifile

fix: set default context

src/commands.rs

@@ -1,5 +1,6 @@
 use crate::config;
 use crate::error::SetContextError;
+use crate::KUBECONFIG;
 
 use std::{
     io::Cursor,
@@ -24,17 +25,20 @@ pub fn set_default_namespace(ns: &str, ctx: &str, target: &Path) {
         .unwrap();
 }
 
-pub fn set_default_context(ctx: &str, target: &Path) {
-    Command::new("kubectl")
+pub fn set_default_context(ctx: &str) {
+    let output = Command::new("kubectl")
         .arg("config")
-        .arg(format!("--kubeconfig={}", target.to_string_lossy()))
         .arg("use-context")
         .arg(ctx)
+        .env("KUBECONFIG", KUBECONFIG.as_str())
         .stdout(Stdio::null())
-        .spawn()
-        .unwrap()
-        .wait()
-        .unwrap();
+        .stderr(Stdio::piped())
+        .output()
+        .expect("Failed to execute command");
+
+    if !output.status.success() {
+        eprintln!("Error: {}", String::from_utf8_lossy(&output.stderr));
+    }
 }
 
 pub fn get_namespaces() -> Vec<String> {

src/modes.rs

@@ -40,7 +40,7 @@ pub fn default_context(args: Cli) -> Result<(), Error> {
         })
         .map(|(_, path)| path.clone())
     {
-        commands::set_default_context(&ctx, &target);
+        commands::set_default_context(&ctx);
         // TODO: We should move the target to the front of the line instead of inserting a
         // duplicate
         println!("{}:{}", target.to_string_lossy(), KUBECONFIG.as_str());