Skip to content

Commit

Permalink
fix the get test matrix issue (#139)
Browse files Browse the repository at this point in the history
Signed-off-by: chensuyue <suyue.chen@intel.com>
  • Loading branch information
chensuyue authored Jun 6, 2024
1 parent 47faf79 commit 1f827d4
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
7 changes: 4 additions & 3 deletions .github/workflows/microservice-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ on:
- comps/**
- "!**.md"
- "!**.txt"
- .github/workflows/microservice.yml
- .github/workflows/microservice-test.yml

# If there is a new commit, the previous jobs will be canceled
concurrency:
Expand All @@ -34,8 +34,9 @@ jobs:
id: get-test-matrix
run: |
set -xe
changed_files=$(git diff --name-only ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }} \
| grep 'comps/' | grep -vE '*.md|*.txt|comps/cores')
merged_commit=$(git log -1 --format='%H')
changed_files="$(git diff --name-only ${{ github.event.pull_request.base.sha }} ${merged_commit} \
| grep 'comps/' | grep -vE '*.md|*.txt|comps/cores')" || true
services=$(printf '%s\n' "${changed_files[@]}" | cut -d'/' -f2 | grep -vE '*.py' | sort -u)
run_matrix="{\"include\":["
for service in ${services}; do
Expand Down
3 changes: 1 addition & 2 deletions tests/test_llms.sh
Original file line number Diff line number Diff line change
Expand Up @@ -26,15 +26,14 @@ function start_service() {

# check whether tgi is fully ready
n=0
until [[ "$n" -ge 100 ]] || [[ $ready == true ]]; do
until [[ "$n" -ge 100 ]]; do
docker logs test-comps-llm-tgi-endpoint > test-comps-llm-tgi-endpoint.log
n=$((n+1))
if grep -q Connected test-comps-llm-tgi-endpoint.log; then
break
fi
sleep 5s
done
# rm -f test-comps-llm-tgi-endpoint.log
sleep 5s
}

Expand Down

0 comments on commit 1f827d4

Please sign in to comment.