Skip to content

Commit 81698a9

Browse files
committed
fix(action): Run Evaluator after Advisor
Policy rules for security vulnerabilities do not work as expected prior to this change as the Evaluator is executed before the Advisor e.g. rules are checked before known vulnerabilities for packages are retrieved. Signed-off-by: Thomas Steenbergen <[email protected]>
1 parent fe5f27a commit 81698a9

File tree

1 file changed

+25
-25
lines changed

1 file changed

+25
-25
lines changed

action.yml

Lines changed: 25 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -454,12 +454,14 @@ runs:
454454
[[ -f $ORT_RESULTS_SCANNER_PATH ]] && \
455455
ln -frs $ORT_RESULTS_SCANNER_PATH $ORT_RESULTS_CURRENT_PATH || \
456456
echo -e "\e[1;31m File $ORT_RESULTS_SCANNER_PATH not found."
457-
- name: Run ORT Evaluator
458-
id: ort-evaluator
457+
- name: Run ORT Advisor
458+
id: ort-advisor
459459
shell: bash
460-
if: contains(inputs.run, 'evaluator')
460+
if: contains(inputs.run, 'advisor')
461+
env:
462+
ORT_ADVISORS: ${{ inputs.advisors }}
461463
run: |
462-
echo -e "\e[1;33m Running ORT Evaluator... "
464+
echo -e "\e[1;33m Running ORT Advisor... "
463465
docker run \
464466
--mount type=bind,source=$HOME,target=/home/ort \
465467
-e JDK_JAVA_OPTIONS="--illegal-access=warn -Xmx5120m" \
@@ -469,25 +471,24 @@ runs:
469471
$ORT_DOCKER_IMAGE \
470472
--$ORT_LOG_LEVEL \
471473
$ORT_CLI_ARGS \
472-
evaluate \
474+
advise \
473475
-i ${ORT_RESULTS_CURRENT_PATH/$USER/ort} \
474476
-o ${ORT_RESULTS_PATH/$USER/ort} \
477+
-a $ORT_ADVISORS \
475478
-f JSON \
476-
${ORT_CLI_EVALUATE_ARGS} || ORT_CLI_EVALUATE_EXIT_CODE=$? \
477-
&& export ORT_CLI_EVALUATE_EXIT_CODE="${ORT_CLI_EVALUATE_EXIT_CODE:-0}" \
479+
${ORT_CLI_ADVISE_ARGS} || ORT_CLI_ADVISE_EXIT_CODE=$? \
480+
&& export ORT_CLI_ADVISE_EXIT_CODE="${ORT_CLI_ADVISE_EXIT_CODE:-0}" \
478481
&& printenv >> "$GITHUB_ENV"
479-
[[ -f $ORT_RESULTS_EVALUATOR_PATH ]] && \
480-
ln -frs $ORT_RESULTS_EVALUATOR_PATH $ORT_RESULTS_CURRENT_PATH || \
481-
echo -e "\e[1;31m File $ORT_RESULTS_EVALUATOR_PATH not found."
482-
[[ $ORT_CLI_EVALUATE_EXIT_CODE -ne 2 ]] && exit ${ORT_CLI_EVALUATE_EXIT_CODE} || exit 0
483-
- name: Run ORT Advisor
484-
id: ort-advisor
482+
[[ -f $ORT_RESULTS_ADVISOR_PATH ]] && \
483+
ln -frs $ORT_RESULTS_ADVISOR_PATH $ORT_RESULTS_CURRENT_PATH || \
484+
echo -e "\e[1;31m File $ORT_RESULTS_ADVISOR_PATH not found."
485+
[[ $ORT_CLI_ADVISE_EXIT_CODE -ne 2 ]] && exit ${ORT_CLI_ADVISE_EXIT_CODE} || exit 0
486+
- name: Run ORT Evaluator
487+
id: ort-evaluator
485488
shell: bash
486-
if: contains(inputs.run, 'advisor')
487-
env:
488-
ORT_ADVISORS: ${{ inputs.advisors }}
489+
if: contains(inputs.run, 'evaluator')
489490
run: |
490-
echo -e "\e[1;33m Running ORT Advisor... "
491+
echo -e "\e[1;33m Running ORT Evaluator... "
491492
docker run \
492493
--mount type=bind,source=$HOME,target=/home/ort \
493494
-e JDK_JAVA_OPTIONS="--illegal-access=warn -Xmx5120m" \
@@ -497,18 +498,17 @@ runs:
497498
$ORT_DOCKER_IMAGE \
498499
--$ORT_LOG_LEVEL \
499500
$ORT_CLI_ARGS \
500-
advise \
501+
evaluate \
501502
-i ${ORT_RESULTS_CURRENT_PATH/$USER/ort} \
502503
-o ${ORT_RESULTS_PATH/$USER/ort} \
503-
-a $ORT_ADVISORS \
504504
-f JSON \
505-
${ORT_CLI_ADVISE_ARGS} || ORT_CLI_ADVISE_EXIT_CODE=$? \
506-
&& export ORT_CLI_ADVISE_EXIT_CODE="${ORT_CLI_ADVISE_EXIT_CODE:-0}" \
505+
${ORT_CLI_EVALUATE_ARGS} || ORT_CLI_EVALUATE_EXIT_CODE=$? \
506+
&& export ORT_CLI_EVALUATE_EXIT_CODE="${ORT_CLI_EVALUATE_EXIT_CODE:-0}" \
507507
&& printenv >> "$GITHUB_ENV"
508-
[[ -f $ORT_RESULTS_ADVISOR_PATH ]] && \
509-
ln -frs $ORT_RESULTS_ADVISOR_PATH $ORT_RESULTS_CURRENT_PATH || \
510-
echo -e "\e[1;31m File $ORT_RESULTS_ADVISOR_PATH not found."
511-
[[ $ORT_CLI_ADVISE_EXIT_CODE -ne 2 ]] && exit ${ORT_CLI_ADVISE_EXIT_CODE} || exit 0
508+
[[ -f $ORT_RESULTS_EVALUATOR_PATH ]] && \
509+
ln -frs $ORT_RESULTS_EVALUATOR_PATH $ORT_RESULTS_CURRENT_PATH || \
510+
echo -e "\e[1;31m File $ORT_RESULTS_EVALUATOR_PATH not found."
511+
[[ $ORT_CLI_EVALUATE_EXIT_CODE -ne 2 ]] && exit ${ORT_CLI_EVALUATE_EXIT_CODE} || exit 0
512512
- name: Run ORT Reporter
513513
id: ort-reporter
514514
shell: bash

0 commit comments

Comments
 (0)