Skip to content

Commit 7eff817

Browse files
[Setup] [Run] bugfix: emergency fix to restore CI/CD functionality (llm-d#228)
* [Setup] [Run] bugfix: emergency fix to restore CI/CD functionality * Make the stack endpoint URL on run conditional. --------- Signed-off-by: maugustosilva <maugusto.silva@gmail.com>
1 parent 7bd8bf1 commit 7eff817

File tree

7 files changed

+26
-15
lines changed

7 files changed

+26
-15
lines changed

.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,3 +58,5 @@ venv/
5858
ENV/
5959
env.bak/
6060
venv.bak/
61+
62+
scenarios/none.sh

setup/env.sh

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -281,14 +281,18 @@ if [[ ! -z $LLMDBENCH_DEPLOY_SCENARIO ]]; then
281281
else
282282
export LLMDBENCH_SCENARIO_FULL_PATH=$(echo ${LLMDBENCH_MAIN_DIR}/scenarios/$LLMDBENCH_DEPLOY_SCENARIO'.sh' | $LLMDBENCH_CONTROL_SCMD 's^.sh.sh^.sh^g')
283283
fi
284-
if [[ -f $LLMDBENCH_SCENARIO_FULL_PATH ]]; then
285-
source $LLMDBENCH_SCENARIO_FULL_PATH
286-
elif [[ $LLMDBENCH_SCENARIO_FULL_PATH == "${LLMDBENCH_MAIN_DIR}/scenarios/none.sh" ]]; then
287-
true
288-
else
289-
echo "❌ Scenario file \"$LLMDBENCH_SCENARIO_FULL_PATH\" could not be found."
290-
exit 1
291-
fi
284+
else
285+
export LLMDBENCH_SCENARIO_FULL_PATH="${LLMDBENCH_MAIN_DIR}/scenarios/none.sh"
286+
touch ${LLMDBENCH_MAIN_DIR}/scenarios/none.sh
287+
fi
288+
289+
if [[ -f $LLMDBENCH_SCENARIO_FULL_PATH ]]; then
290+
source $LLMDBENCH_SCENARIO_FULL_PATH
291+
elif [[ $LLMDBENCH_SCENARIO_FULL_PATH == "${LLMDBENCH_MAIN_DIR}/scenarios/none.sh" ]]; then
292+
touch ${LLMDBENCH_MAIN_DIR}/scenarios/none.sh
293+
else
294+
echo "❌ Scenario file \"$LLMDBENCH_SCENARIO_FULL_PATH\" could not be found."
295+
exit 1
292296
fi
293297

294298
if [[ "$LLMDBENCH_VLLM_MODELSERVICE_GAIE_PRESETS" == /* ]]; then

setup/run.sh

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,6 +194,7 @@ for method in ${LLMDBENCH_DEPLOY_METHODS//,/ }; do
194194

195195
for model in ${LLMDBENCH_DEPLOY_MODEL_LIST//,/ }; do
196196
export LLMDBENCH_DEPLOY_CURRENT_MODEL=$(model_attribute $model model)
197+
export LLMDBENCH_DEPLOY_CURRENT_MODELID=$(model_attribute $model modelid)
197198

198199
export LLMDBENCH_RUN_HARNESS_LAUNCHER_NAME=llmdbench-${LLMDBENCH_HARNESS_NAME}-launcher
199200

@@ -251,7 +252,11 @@ for method in ${LLMDBENCH_DEPLOY_METHODS//,/ }; do
251252
exit 1
252253
fi
253254

254-
export LLMDBENCH_HARNESS_STACK_ENDPOINT_URL="http://${LLMDBENCH_HARNESS_STACK_ENDPOINT_NAME}${LLMDBENCH_VLLM_COMMON_FQDN}:${LLMDBENCH_HARNESS_STACK_ENDPOINT_PORT}"
255+
if [[ $LLMDBENCH_CONTROL_ENVIRONMENT_TYPE_MODELSERVICE_ACTIVE -eq 1 ]]; then
256+
export LLMDBENCH_HARNESS_STACK_ENDPOINT_URL="http://${LLMDBENCH_HARNESS_STACK_ENDPOINT_NAME}${LLMDBENCH_VLLM_COMMON_FQDN}:${LLMDBENCH_HARNESS_STACK_ENDPOINT_PORT}/${LLMDBENCH_DEPLOY_CURRENT_MODELID}"
257+
else
258+
export LLMDBENCH_HARNESS_STACK_ENDPOINT_URL="http://${LLMDBENCH_HARNESS_STACK_ENDPOINT_NAME}${LLMDBENCH_VLLM_COMMON_FQDN}:${LLMDBENCH_HARNESS_STACK_ENDPOINT_PORT}"
259+
fi
255260
announce "ℹ️ Stack Endpoint URL detected is \"$LLMDBENCH_HARNESS_STACK_ENDPOINT_URL\""
256261

257262
if [[ $LLMDBENCH_CONTROL_DRY_RUN -eq 1 ]]; then

setup/steps/02_ensure_gateway_provider.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ if [[ $LLMDBENCH_CONTROL_ENVIRONMENT_TYPE_MODELSERVICE_ACTIVE -eq 1 ]]; then
3636
fi
3737

3838
else
39-
announce "❗No privileges to setup Gateway Provider. Will assume an user with proper privileges already performed this action."
39+
announce "❗No privileges to setup Gateway Provider. Will assume a user with proper privileges already performed this action."
4040
fi
4141
fi
4242

setup/steps/07_deploy_setup.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
source ${LLMDBENCH_CONTROL_DIR}/env.sh
33

44
if [[ $LLMDBENCH_CONTROL_ENVIRONMENT_TYPE_MODELSERVICE_ACTIVE -eq 1 ]]; then
5-
5+
66
# make sure llm-d-modelservice helm repo is available
77
llmdbench_execute_cmd "$LLMDBENCH_CONTROL_HCMD repo add ${LLMDBENCH_VLLM_MODELSERVICE_CHART_NAME} ${LLMDBENCH_VLLM_MODELSERVICE_HELM_REPOSITORY_URL} --force-update" ${LLMDBENCH_CONTROL_DRY_RUN} ${LLMDBENCH_CONTROL_VERBOSE}
88
llmdbench_execute_cmd "$LLMDBENCH_CONTROL_HCMD repo update" ${LLMDBENCH_CONTROL_DRY_RUN} ${LLMDBENCH_CONTROL_VERBOSE}
@@ -62,7 +62,7 @@ releases:
6262
managedBy: helmfile
6363
EOF
6464

65-
((model_number++))
65+
model_number=$((model_number + 1))
6666
done
6767
announce "✅ Completed gaie deployment"
6868
else

setup/steps/08_deploy_gaie.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ EOF
5050

5151
unset LLMDBENCH_DEPLOY_CURRENT_MODEL_ID_LABEL
5252

53-
((model_number++))
53+
model_number=$((model_number + 1))
5454
done
5555
announce "✅ Completed model deployment"
5656
else

setup/steps/09_deploy_via_modelservice.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -302,8 +302,8 @@ EOF
302302
unset LLMDBENCH_DEPLOY_CURRENT_MODEL
303303
unset LLMDBENCH_DEPLOY_CURRENT_MODEL_ID
304304
unset LLMDBENCH_DEPLOY_CURRENT_MODEL_ID_LABEL
305-
306-
((model_number++))
305+
306+
model_number=$((model_number + 1))
307307
done
308308
announce "✅ modelservice completed model deployment"
309309

0 commit comments

Comments
 (0)