Skip to content

Commit 3297bd8

Browse files
Removed AGENT_CONFIG from config.js and some changes
Signed-off-by: Aayush Chouhan <[email protected]>
1 parent 0dbc642 commit 3297bd8

File tree

2 files changed

+12
-20
lines changed

2 files changed

+12
-20
lines changed

config.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -242,12 +242,6 @@ if (process.env.CONTAINER_PLATFORM || process.env.LOCAL_MD_SERVER) {
242242

243243
config.ROOT_KEY_MOUNT = '/etc/noobaa-server/root_keys';
244244

245-
//////////////////
246-
// AGENT CONFIG //
247-
//////////////////
248-
config.AGENT_CONFIG_PATH = process.env.AGENT_CONFIG_PATH || '/etc/agent-config/agent_config';
249-
config.AGENT_CONFIG = process.env.AGENT_CONFIG || _get_data_from_file(config.AGENT_CONFIG_PATH);
250-
251245
///////////////
252246
// DB CONFIG //
253247
///////////////

src/deploy/NVA_build/noobaa_init.sh

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -123,26 +123,24 @@ run_internal_process() {
123123
prepare_agent_conf() {
124124
AGENT_CONF_FILE="/noobaa_storage/agent_conf.json"
125125

126-
# get AGENT_CONFIG from env var or config.js (volume mount)
127-
if [ -z ${AGENT_CONFIG} ]; then
128-
cd /root/node_modules/noobaa-core/
129-
AGENT_CONFIG=$(node -p 'require("./config").AGENT_CONFIG || ""')
130-
fi
126+
[ -f "$AGENT_CONF_FILE" ] && return 0
127+
128+
# get AGENT_CONFIG from env var or file
129+
AGENT_CONFIG_PATH=${AGENT_CONFIG_PATH:-"/etc/agent-config/agent_config"}
130+
AGENT_CONFIG=${AGENT_CONFIG:-$(cat "$AGENT_CONFIG_PATH" 2>/dev/null || echo "")}
131131

132132
if [ -z "${AGENT_CONFIG}" ]; then
133-
echo "AGENT_CONFIG is required. AGENT_CONFIG is missing. Exit"
133+
echo "AGENT_CONFIG is required. AGENT_CONFIG is not found in env or $AGENT_CONFIG_PATH. Exit"
134134
exit 1
135135
fi
136136

137-
echo "Got agent_conf"
138-
if [ ! -f $AGENT_CONF_FILE ]; then
139-
if echo "${AGENT_CONFIG}" | jq . >/dev/null 2>&1; then
140-
echo "${AGENT_CONFIG}" >${AGENT_CONF_FILE}
141-
else
142-
openssl enc -base64 -d -A <<<${AGENT_CONFIG} >${AGENT_CONF_FILE}
143-
fi
137+
# write agent config - decode base64 if not a valid JSON format
138+
if ! echo "${AGENT_CONFIG}" | jq . >"$AGENT_CONF_FILE" 2>/dev/null; then
139+
openssl enc -base64 -d -A <<<"${AGENT_CONFIG}" >"$AGENT_CONF_FILE" || {
140+
echo "AGENT_CONFIG format is invalid. AGENT_CONFIG must be valid JSON or base64 encoded JSON. Exit"
141+
exit 1
142+
}
144143
fi
145-
echo "Written agent_conf.json"
146144
}
147145

148146
prepare_server_pvs() {

0 commit comments

Comments
 (0)