kubectx must fail when it fails to retrieve namespaces/contexts about kubectx

[ahmetb]

The script already has -o pipefail set :)

See the stackoverflow link above. I think it's because we're calling it like for c in $(get_contexts) which isn't failing properly b/c it's in sub-shell.

[ahmetb]

Some of this is fixed in 6610d70, but here's a case that still swallows some errors:

$ brew unlink kubectl # or make kubectl unavailable to the kubectx

# kubens
/Users/ahmetb/.homebrew/bin/kubens: line 48: kubectl: command not found
/Users/ahmetb/.homebrew/bin/kubens: line 39: kubectl: command not found
/Users/ahmetb/.homebrew/bin/kubens: line 52: kubectl: command not found

instead, the first occurrence of the error should've caused script to exit.

I found that if I add set -e to the caller functions, it fails as expected, but I probably shouldn't add set -e to all functions like that.

[2rs2ts]

Isn't the problem from your OP that you don't have pipefail set?

% false | echo hi
hi
% echo $?
0
% set -o pipefail
% false | echo hi
hi
% echo $?
1