kubernetes
/
minikube
mirror of https://github.com/kubernetes/minikube.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Rewrite unsetScript to fix a variety of bugs

pull/6540/head
Thomas Stromberg 2020-02-06 17:01:05 -08:00
parent 0cac457126
commit 5d59e83b64
2 changed files with 90 additions and 160 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

97
cmd/minikube/cmd/env.go
View File

@ -50,12 +50,11 @@ var envTmpl = fmt.Sprintf("{{ .Prefix }}%s{{ .Delimiter }}{{ .DockerTLSVerify }}
const ( const (
fishSetPfx = "set -gx " fishSetPfx = "set -gx "
fishSetSfx = "\";\n" fishSetSfx = "\"\n"
fishSetDelim = " \"" fishSetDelim = " \""
fishUnsetPfx = "set -e " fishUnsetPfx = "set -e "
fishUnsetSfx = ";\n" fishUnsetSfx = "\n"
fishUnsetDelim = ""
psSetPfx = "$Env:" psSetPfx = "$Env:"
psSetSfx = "\"\n" psSetSfx = "\"\n"
@ -63,7 +62,6 @@ const (
psUnsetPfx = `Remove-Item Env:\\` psUnsetPfx = `Remove-Item Env:\\`
psUnsetSfx = "\n" psUnsetSfx = "\n"
psUnsetDelim = ""
cmdSetPfx = "SET " cmdSetPfx = "SET "
cmdSetSfx = "\n" cmdSetSfx = "\n"
@ -87,7 +85,6 @@ const (
bashUnsetPfx = "unset " bashUnsetPfx = "unset "
bashUnsetSfx = "\n" bashUnsetSfx = "\n"
bashUnsetDelim = ""
nonePfx = "" nonePfx = ""
noneSfx = "\n" noneSfx = "\n"
@ -124,7 +121,7 @@ type NoProxyGetter interface {
type EnvNoProxyGetter struct{} type EnvNoProxyGetter struct{}
func generateUsageHint(profile, sh string) string { func generateUsageHint(profile, sh string) string {
const usgPlz = "Please run command bellow to point your shell to minikube's docker-daemon :" const usgPlz = "To point your shell to minikube's docker-daemon, run:"
var usgCmd = fmt.Sprintf("minikube -p %s docker-env", profile) var usgCmd = fmt.Sprintf("minikube -p %s docker-env", profile)
var usageHintMap = map[string]string{ var usageHintMap = map[string]string{
"bash": fmt.Sprintf(` "bash": fmt.Sprintf(`
@ -135,16 +132,13 @@ func generateUsageHint(profile, sh string) string {
# %s # %s
# eval (%s) # eval (%s)
`, usgPlz, usgCmd), `, usgPlz, usgCmd),
"powershell": fmt.Sprintf(` "powershell": fmt.Sprintf(`# %s
# %s
# & %s | Invoke-Expression # & %s | Invoke-Expression
`, usgPlz, usgCmd), `, usgPlz, usgCmd),
"cmd": fmt.Sprintf(` "cmd": fmt.Sprintf(`REM %s
REM %s
REM @FOR /f "tokens=*" %%i IN ('%s') DO @%%i REM @FOR /f "tokens=*" %%i IN ('%s') DO @%%i
`, usgPlz, usgCmd), `, usgPlz, usgCmd),
"emacs": fmt.Sprintf(` "emacs": fmt.Sprintf(`;; %s
;; %s
;; (with-temp-buffer (shell-command "%s" (current-buffer)) (eval-buffer)) ;; (with-temp-buffer (shell-command "%s" (current-buffer)) (eval-buffer))
`, usgPlz, usgCmd), `, usgPlz, usgCmd),
} }
@ -213,46 +207,6 @@ func shellCfgSet(ec EnvConfig, envMap map[string]string) *ShellConfig {
return s return s
} }
// shellCfgUnset generates context variables for "docker-env -u"
func shellCfgUnset(ec EnvConfig) *ShellConfig {
s := &ShellConfig{
UsageHint: generateUsageHint(ec.profile, ec.shell),
}
if ec.noProxy {
s.NoProxyVar, s.NoProxyValue = defaultNoProxyGetter.GetNoProxyVar()
}
switch ec.shell {
case "fish":
s.Prefix = fishUnsetPfx
s.Suffix = fishUnsetSfx
s.Delimiter = fishUnsetDelim
case "powershell":
s.Prefix = psUnsetPfx
s.Suffix = psUnsetSfx
s.Delimiter = psUnsetDelim
case "cmd":
s.Prefix = cmdUnsetPfx
s.Suffix = cmdUnsetSfx
s.Delimiter = cmdUnsetDelim
case "emacs":
s.Prefix = emacsUnsetPfx
s.Suffix = emacsUnsetSfx
s.Delimiter = emacsUnsetDelim
case "none":
s.Prefix = nonePfx
s.Suffix = noneSfx
s.Delimiter = noneDelim
s.UsageHint = ""
default:
s.Prefix = bashUnsetPfx
s.Suffix = bashUnsetSfx
s.Delimiter = bashUnsetDelim
}
return s
}
// GetNoProxyVar gets the no_proxy var // GetNoProxyVar gets the no_proxy var
func (EnvNoProxyGetter) GetNoProxyVar() (string, string) { func (EnvNoProxyGetter) GetNoProxyVar() (string, string) {
// first check for an existing lower case no_proxy var // first check for an existing lower case no_proxy var
@ -379,8 +333,43 @@ func setScript(ec EnvConfig, w io.Writer) error {
// setScript writes out a shell-compatible 'docker-env unset' script // setScript writes out a shell-compatible 'docker-env unset' script
func unsetScript(ec EnvConfig, w io.Writer) error { func unsetScript(ec EnvConfig, w io.Writer) error {
tmpl := template.Must(template.New("envConfig").Parse(envTmpl)) vars := []string{
return tmpl.Execute(w, shellCfgUnset(ec)) constants.DockerTLSVerifyEnv,
constants.DockerHostEnv,
constants.DockerCertPathEnv,
constants.MinikubeActiveDockerdEnv,
}
if ec.noProxy {
k, _ := defaultNoProxyGetter.GetNoProxyVar()
if k != "" {
vars = append(vars, k)
}
}
var sb strings.Builder
switch ec.shell {
case "fish":
for _, v := range vars {
sb.WriteString(fmt.Sprintf("%s%s%s", fishUnsetPfx, v, fishUnsetSfx))
}
case "powershell":
sb.WriteString(fmt.Sprintf("%s%s%s", psUnsetPfx, strings.Join(vars, " Env:\\\\"), psUnsetSfx))
case "cmd":
for _, v := range vars {
sb.WriteString(fmt.Sprintf("%s%s%s%s", cmdUnsetPfx, v, cmdUnsetDelim, cmdUnsetSfx))
}
case "emacs":
for _, v := range vars {
sb.WriteString(fmt.Sprintf("%s%s%s%s", emacsUnsetPfx, v, emacsUnsetDelim, emacsUnsetSfx))
}
case "none":
sb.WriteString(fmt.Sprintf("%s%s%s", nonePfx, strings.Join(vars, " "), noneSfx))
default:
sb.WriteString(fmt.Sprintf("%s%s%s", bashUnsetPfx, strings.Join(vars, " "), bashUnsetSfx))
}
_, err := w.Write([]byte(sb.String()))
return err
} }
// dockerURL returns a the docker endpoint URL for an ip/port pair. // dockerURL returns a the docker endpoint URL for an ip/port pair.

125
cmd/minikube/cmd/env_test.go
View File

@ -47,16 +47,10 @@ export DOCKER_HOST="tcp://127.0.0.1:2376"
export DOCKER_CERT_PATH="/certs" export DOCKER_CERT_PATH="/certs"
export MINIKUBE_ACTIVE_DOCKERD="bash" export MINIKUBE_ACTIVE_DOCKERD="bash"
# Please run command bellow to point your shell to minikube's docker-daemon : # To point your shell to minikube's docker-daemon, run:
# eval $(minikube -p bash docker-env) # eval $(minikube -p bash docker-env)
`, `,
`unset DOCKER_TLS_VERIFY `unset DOCKER_TLS_VERIFY DOCKER_HOST DOCKER_CERT_PATH MINIKUBE_ACTIVE_DOCKERD
unset DOCKER_HOST
unset DOCKER_CERT_PATH
unset MINIKUBE_ACTIVE_DOCKERD
# Please run command bellow to point your shell to minikube's docker-daemon :
# eval $(minikube -p bash docker-env)
`, `,
}, },
{ {
@ -67,36 +61,27 @@ export DOCKER_HOST="tcp://[fe80::215:5dff:fe00:a903]:2376"
export DOCKER_CERT_PATH="/certs" export DOCKER_CERT_PATH="/certs"
export MINIKUBE_ACTIVE_DOCKERD="ipv6" export MINIKUBE_ACTIVE_DOCKERD="ipv6"
# Please run command bellow to point your shell to minikube's docker-daemon : # To point your shell to minikube's docker-daemon, run:
# eval $(minikube -p ipv6 docker-env) # eval $(minikube -p ipv6 docker-env)
`, `,
`unset DOCKER_TLS_VERIFY `unset DOCKER_TLS_VERIFY DOCKER_HOST DOCKER_CERT_PATH MINIKUBE_ACTIVE_DOCKERD
unset DOCKER_HOST
unset DOCKER_CERT_PATH
unset MINIKUBE_ACTIVE_DOCKERD
# Please run command bellow to point your shell to minikube's docker-daemon :
# eval $(minikube -p ipv6 docker-env)
`, `,
}, },
{ {
EnvConfig{profile: "fish", shell: "fish", driver: "kvm2", hostIP: "127.0.0.1", certsDir: "/certs"}, EnvConfig{profile: "fish", shell: "fish", driver: "kvm2", hostIP: "127.0.0.1", certsDir: "/certs"},
nil, nil,
`set -gx DOCKER_TLS_VERIFY "1"; `set -gx DOCKER_TLS_VERIFY "1"
set -gx DOCKER_HOST "tcp://127.0.0.1:2376"; set -gx DOCKER_HOST "tcp://127.0.0.1:2376"
set -gx DOCKER_CERT_PATH "/certs"; set -gx DOCKER_CERT_PATH "/certs"
set -gx MINIKUBE_ACTIVE_DOCKERD "fish"; set -gx MINIKUBE_ACTIVE_DOCKERD "fish"
# Please run command bellow to point your shell to minikube's docker-daemon : # To point your shell to minikube's docker-daemon, run:
# eval (minikube -p fish docker-env) # eval (minikube -p fish docker-env)
`, `,
`set -e DOCKER_TLS_VERIFY; `set -e DOCKER_TLS_VERIFY
set -e DOCKER_HOST; set -e DOCKER_HOST
set -e DOCKER_CERT_PATH; set -e DOCKER_CERT_PATH
set -e MINIKUBE_ACTIVE_DOCKERD; set -e MINIKUBE_ACTIVE_DOCKERD
# Please run command bellow to point your shell to minikube's docker-daemon :
# eval (minikube -p fish docker-env)
`, `,
}, },
{ {
@ -106,18 +91,11 @@ set -e MINIKUBE_ACTIVE_DOCKERD;
$Env:DOCKER_HOST = "tcp://192.168.0.1:2376" $Env:DOCKER_HOST = "tcp://192.168.0.1:2376"
$Env:DOCKER_CERT_PATH = "/certs" $Env:DOCKER_CERT_PATH = "/certs"
$Env:MINIKUBE_ACTIVE_DOCKERD = "powershell" $Env:MINIKUBE_ACTIVE_DOCKERD = "powershell"
# To point your shell to minikube's docker-daemon, run:
# Please run command bellow to point your shell to minikube's docker-daemon :
# & minikube -p powershell docker-env | Invoke-Expression # & minikube -p powershell docker-env | Invoke-Expression
`, `,
`Remove-Item Env:\\DOCKER_TLS_VERIFY `Remove-Item Env:\\DOCKER_TLS_VERIFY Env:\\DOCKER_HOST Env:\\DOCKER_CERT_PATH Env:\\MINIKUBE_ACTIVE_DOCKERD
Remove-Item Env:\\DOCKER_HOST
Remove-Item Env:\\DOCKER_CERT_PATH
Remove-Item Env:\\MINIKUBE_ACTIVE_DOCKERD
# Please run command bellow to point your shell to minikube's docker-daemon :
# & minikube -p powershell docker-env | Invoke-Expression
`, `,
}, },
{ {
@ -127,8 +105,7 @@ Remove-Item Env:\\MINIKUBE_ACTIVE_DOCKERD
SET DOCKER_HOST=tcp://192.168.0.1:2376 SET DOCKER_HOST=tcp://192.168.0.1:2376
SET DOCKER_CERT_PATH=/certs SET DOCKER_CERT_PATH=/certs
SET MINIKUBE_ACTIVE_DOCKERD=cmd SET MINIKUBE_ACTIVE_DOCKERD=cmd
REM To point your shell to minikube's docker-daemon, run:
REM Please run command bellow to point your shell to minikube's docker-daemon :
REM @FOR /f "tokens=*" %i IN ('minikube -p cmd docker-env') DO @%i REM @FOR /f "tokens=*" %i IN ('minikube -p cmd docker-env') DO @%i
`, `,
@ -136,9 +113,6 @@ REM @FOR /f "tokens=*" %i IN ('minikube -p cmd docker-env') DO @%i
SET DOCKER_HOST= SET DOCKER_HOST=
SET DOCKER_CERT_PATH= SET DOCKER_CERT_PATH=
SET MINIKUBE_ACTIVE_DOCKERD= SET MINIKUBE_ACTIVE_DOCKERD=
REM Please run command bellow to point your shell to minikube's docker-daemon :
REM @FOR /f "tokens=*" %i IN ('minikube -p cmd docker-env') DO @%i
`, `,
}, },
{ {
@ -148,17 +122,13 @@ REM @FOR /f "tokens=*" %i IN ('minikube -p cmd docker-env') DO @%i
(setenv "DOCKER_HOST" "tcp://192.168.0.1:2376") (setenv "DOCKER_HOST" "tcp://192.168.0.1:2376")
(setenv "DOCKER_CERT_PATH" "/certs") (setenv "DOCKER_CERT_PATH" "/certs")
(setenv "MINIKUBE_ACTIVE_DOCKERD" "emacs") (setenv "MINIKUBE_ACTIVE_DOCKERD" "emacs")
;; To point your shell to minikube's docker-daemon, run:
;; Please run command bellow to point your shell to minikube's docker-daemon :
;; (with-temp-buffer (shell-command "minikube -p emacs docker-env" (current-buffer)) (eval-buffer)) ;; (with-temp-buffer (shell-command "minikube -p emacs docker-env" (current-buffer)) (eval-buffer))
`, `,
`(setenv "DOCKER_TLS_VERIFY" nil) `(setenv "DOCKER_TLS_VERIFY" nil)
(setenv "DOCKER_HOST" nil) (setenv "DOCKER_HOST" nil)
(setenv "DOCKER_CERT_PATH" nil) (setenv "DOCKER_CERT_PATH" nil)
(setenv "MINIKUBE_ACTIVE_DOCKERD" nil) (setenv "MINIKUBE_ACTIVE_DOCKERD" nil)
;; Please run command bellow to point your shell to minikube's docker-daemon :
;; (with-temp-buffer (shell-command "minikube -p emacs docker-env" (current-buffer)) (eval-buffer))
`, `,
}, },
{ {
@ -170,18 +140,11 @@ export DOCKER_CERT_PATH="/certs"
export MINIKUBE_ACTIVE_DOCKERD="bash-no-proxy" export MINIKUBE_ACTIVE_DOCKERD="bash-no-proxy"
export NO_PROXY="127.0.0.1" export NO_PROXY="127.0.0.1"
# Please run command bellow to point your shell to minikube's docker-daemon : # To point your shell to minikube's docker-daemon, run:
# eval $(minikube -p bash-no-proxy docker-env) # eval $(minikube -p bash-no-proxy docker-env)
`, `,
`unset DOCKER_TLS_VERIFY `unset DOCKER_TLS_VERIFY DOCKER_HOST DOCKER_CERT_PATH MINIKUBE_ACTIVE_DOCKERD NO_PROXY
unset DOCKER_HOST
unset DOCKER_CERT_PATH
unset MINIKUBE_ACTIVE_DOCKERD
unset NO_PROXY127.0.0.1
# Please run command bellow to point your shell to minikube's docker-daemon :
# eval $(minikube -p bash-no-proxy docker-env)
`, `,
}, },
{ {
@ -193,41 +156,26 @@ export DOCKER_CERT_PATH="/certs"
export MINIKUBE_ACTIVE_DOCKERD="bash-no-proxy-lower" export MINIKUBE_ACTIVE_DOCKERD="bash-no-proxy-lower"
export no_proxy="127.0.0.1" export no_proxy="127.0.0.1"
# Please run command bellow to point your shell to minikube's docker-daemon : # To point your shell to minikube's docker-daemon, run:
# eval $(minikube -p bash-no-proxy-lower docker-env) # eval $(minikube -p bash-no-proxy-lower docker-env)
`, `,
`unset DOCKER_TLS_VERIFY `unset DOCKER_TLS_VERIFY DOCKER_HOST DOCKER_CERT_PATH MINIKUBE_ACTIVE_DOCKERD no_proxy
unset DOCKER_HOST
unset DOCKER_CERT_PATH
unset MINIKUBE_ACTIVE_DOCKERD
unset no_proxy127.0.0.1
# Please run command bellow to point your shell to minikube's docker-daemon :
# eval $(minikube -p bash-no-proxy-lower docker-env)
`, `,
}, },
{ {
EnvConfig{profile: "bash-no-proxy-idempotent", shell: "bash", driver: "kvm2", hostIP: "127.0.0.1", certsDir: "/certs", noProxy: true}, EnvConfig{profile: "powershell-no-proxy-idempotent", shell: "powershell", driver: "hyperv", hostIP: "192.168.0.1", certsDir: "/certs", noProxy: true},
&FakeNoProxyGetter{"no_proxy", "127.0.0.1"}, &FakeNoProxyGetter{"no_proxy", "192.168.0.1"},
`export DOCKER_TLS_VERIFY="1" `$Env:DOCKER_TLS_VERIFY = "1"
export DOCKER_HOST="tcp://127.0.0.1:2376" $Env:DOCKER_HOST = "tcp://192.168.0.1:2376"
export DOCKER_CERT_PATH="/certs" $Env:DOCKER_CERT_PATH = "/certs"
export MINIKUBE_ACTIVE_DOCKERD="bash-no-proxy-idempotent" $Env:MINIKUBE_ACTIVE_DOCKERD = "powershell-no-proxy-idempotent"
export no_proxy="127.0.0.1" $Env:no_proxy = "192.168.0.1"
# To point your shell to minikube's docker-daemon, run:
# Please run command bellow to point your shell to minikube's docker-daemon : # & minikube -p powershell-no-proxy-idempotent docker-env | Invoke-Expression
# eval $(minikube -p bash-no-proxy-idempotent docker-env)
`, `,
`unset DOCKER_TLS_VERIFY `Remove-Item Env:\\DOCKER_TLS_VERIFY Env:\\DOCKER_HOST Env:\\DOCKER_CERT_PATH Env:\\MINIKUBE_ACTIVE_DOCKERD Env:\\no_proxy
unset DOCKER_HOST
unset DOCKER_CERT_PATH
unset MINIKUBE_ACTIVE_DOCKERD
unset no_proxy127.0.0.1
# Please run command bellow to point your shell to minikube's docker-daemon :
# eval $(minikube -p bash-no-proxy-idempotent docker-env)
`, `,
}, },
{ {
@ -239,18 +187,11 @@ export DOCKER_CERT_PATH="/certs"
export MINIKUBE_ACTIVE_DOCKERD="sh-no-proxy-add" export MINIKUBE_ACTIVE_DOCKERD="sh-no-proxy-add"
export NO_PROXY="192.168.0.1,10.0.0.4,127.0.0.1" export NO_PROXY="192.168.0.1,10.0.0.4,127.0.0.1"
# Please run command bellow to point your shell to minikube's docker-daemon : # To point your shell to minikube's docker-daemon, run:
# eval $(minikube -p sh-no-proxy-add docker-env) # eval $(minikube -p sh-no-proxy-add docker-env)
`, `,
`unset DOCKER_TLS_VERIFY `unset DOCKER_TLS_VERIFY DOCKER_HOST DOCKER_CERT_PATH MINIKUBE_ACTIVE_DOCKERD NO_PROXY
unset DOCKER_HOST
unset DOCKER_CERT_PATH
unset MINIKUBE_ACTIVE_DOCKERD
unset NO_PROXY192.168.0.1,10.0.0.4
# Please run command bellow to point your shell to minikube's docker-daemon :
# eval $(minikube -p sh-no-proxy-add docker-env)
`, `,
}, },
} }