Skip to content

Commit

Permalink
Merge pull request vespa-engine#23970 from vespa-engine/revert-23962-…
Browse files Browse the repository at this point in the history
…arnej/simpler-security-env

Revert "switch to script-utils for security-env"
  • Loading branch information
freva committed Sep 7, 2022
2 parents ceb337d + 26b723b commit 5f19d58
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 9 deletions.
5 changes: 1 addition & 4 deletions client/go/script-utils/main.go
Expand Up @@ -27,8 +27,6 @@ func main() {
switch action {
case "export-env":
vespa.ExportDefaultEnvToSh()
case "security-env":
vespa.ExportSecurityEnvToSh()
case "ipv6-only":
if vespa.HasOnlyIpV6() {
os.Exit(0)
Expand All @@ -45,7 +43,6 @@ func main() {
cobra.Execute()
default:
fmt.Fprintf(os.Stderr, "unknown action '%s'\n", action)
fmt.Fprintln(os.Stderr, "actions: export-env, ipv6-only, security-env")
fmt.Fprintln(os.Stderr, "(also: vespa-deploy, vespa-logfmt)")
fmt.Fprintln(os.Stderr, "actions: export-env, ipv6-only, vespa-deploy, vespa-logfmt")
}
}
4 changes: 2 additions & 2 deletions security-tools/src/main/sh/vespa-curl-wrapper
@@ -1,7 +1,7 @@
#!/usr/bin/env bash
# Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.

# Uses security-env to call curl with paths to credentials.
# Uses vespa-security-env to call curl with paths to credentials.
# This script should be installed in libexec only. It is not public api.

# BEGIN environment bootstrap section
Expand Down Expand Up @@ -79,7 +79,7 @@ findhost

set -e

eval $($VESPA_HOME/libexec/vespa/script-utils security-env)
eval $($VESPA_HOME/bin/vespa-security-env)

CURL_PARAMETERS=("$@")

Expand Down
2 changes: 1 addition & 1 deletion vespaclient/src/sh/vespa-get-cluster-state
Expand Up @@ -74,6 +74,6 @@ findhost

# END environment bootstrap section

eval $($VESPA_HOME/libexec/vespa/script-utils security-env)
eval $($VESPA_HOME/bin/vespa-security-env)

exec $VESPA_HOME/libexec/vespa/GetClusterState.pl "$@"
2 changes: 1 addition & 1 deletion vespaclient/src/sh/vespa-get-node-state
Expand Up @@ -74,6 +74,6 @@ findhost

# END environment bootstrap section

eval $($VESPA_HOME/libexec/vespa/script-utils security-env)
eval $($VESPA_HOME/bin/vespa-security-env)

exec $VESPA_HOME/libexec/vespa/GetNodeState.pl "$@"
2 changes: 1 addition & 1 deletion vespaclient/src/sh/vespa-set-node-state
Expand Up @@ -74,6 +74,6 @@ findhost

# END environment bootstrap section

eval $($VESPA_HOME/libexec/vespa/script-utils security-env)
eval $($VESPA_HOME/bin/vespa-security-env)

exec $VESPA_HOME/libexec/vespa/SetNodeState.pl "$@"

0 comments on commit 5f19d58

Please sign in to comment.