Skip to content

Commit 5fffd35

Browse files
clean up monitoring script (#270)
1 parent 17c87ea commit 5fffd35

File tree

2 files changed

+3
-7
lines changed

2 files changed

+3
-7
lines changed

startupscript/butane/006-start-proxy-agent.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,6 @@ docker run \
6161
--compute-platform="${COMPUTE_PLATFORM^^}" \
6262
--shim-path="${SHIM_PATH}" \
6363
--rewrite-websocket-host="${REWRITE_WEBSOCKET_HOST}" \
64-
--enable-monitoring-script="${ENABLE_MONITORING_SCRIPT:-false}" \
64+
--enable-monitoring-script=true \
6565
--websocket-transport=true \
6666
"${OPTIONS[@]}"

startupscript/dataproc/startup.sh

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -828,8 +828,6 @@ IS_NON_GOOGLE_ACCOUNT="$(curl "${USER_SERVICE_URL}/api/profile?path=non_google_a
828828
| jq '.value')"
829829
readonly IS_NON_GOOGLE_ACCOUNT
830830

831-
ENABLE_VWB_PROXY="$(get_metadata_value "instance/attributes/enable-dataproc-vwb-proxy")"
832-
readonly ENABLE_VWB_PROXY
833831
# Retrieve Workbench proxy agent image
834832
PROXY_AGENT_IMAGE="$(get_metadata_value "instance/attributes/proxy-agent-image")"
835833
readonly PROXY_AGENT_IMAGE
@@ -853,7 +851,7 @@ readonly PROXY_TYPE_GOOGLE_AGENT_WITH_VWB_PROXY="GOOGLE_AGENT_WITH_VWB_PROXY"
853851

854852
# Decide proxy type
855853
PROXY_TYPE=""
856-
if [[ ${ENABLE_VWB_PROXY} == "true" && -n "${PROXY_AGENT_IMAGE}" ]]; then
854+
if [[ -n "${PROXY_AGENT_IMAGE}" ]]; then
857855
PROXY_TYPE="${PROXY_TYPE_VWB}"
858856
elif [[ ${IS_NON_GOOGLE_ACCOUNT} == "true" ]]; then
859857
PROXY_TYPE="${PROXY_TYPE_GOOGLE_AGENT_WITH_VWB_PROXY}"
@@ -867,8 +865,6 @@ if [[ "${PROXY_TYPE}" != "${PROXY_TYPE_GOOGLE}" ]]; then
867865
###################################
868866
if [[ "${PROXY_TYPE}" == "${PROXY_TYPE_VWB}" ]]; then
869867
emit "Using Workbench Proxy"
870-
ENABLE_MONITORING_SCRIPT="$(get_metadata_value "instance/attributes/enable-dataproc-monitoring-script")"
871-
readonly ENABLE_MONITORING_SCRIPT
872868

873869
readonly WORKSPACE_LINK_EL='<a id="workspace" class="forum" target="_blank" href="'"${UI_BASE_URL}/workspaces/${TERRA_WORKSPACE}"'"'">${TERRA_WORKSPACE}</a>"
874870

@@ -927,7 +923,7 @@ docker run \
927923
--backend="${RESOURCE_ID}" \
928924
--session-cookie-name="_xsrf" \
929925
--inject-banner="\$(cat ${PROXY_AGENT_BANNER})" \
930-
--enable-monitoring-script="${ENABLE_MONITORING_SCRIPT:-false}" \
926+
--enable-monitoring-script=true \
931927
--websocket-transport=true
932928
EOF
933929
# Grant execute permission to the startup script

0 commit comments

Comments
 (0)