Skip to content

Commit 36aae4b

Browse files
RobertSamoilescuRobertSamoilescu
committed
Fix syntax error
1 parent a2cc30a commit 36aae4b

File tree

2 files changed

+104
-104
lines changed

2 files changed

+104
-104
lines changed

‎.github/workflows/tests.yml‎

Lines changed: 103 additions & 103 deletions
Original file line numberDiff line numberDiff line change
@@ -9,51 +9,51 @@ on:
99
branches: [master]
1010

1111
jobs:
12-
generate:
13-
strategy:
14-
fail-fast: false
15-
matrix:
16-
python-version: ["3.9", "3.10"]
17-
target: ["dataplane", "model-repository"]
18-
runs-on: ubuntu-22.04
19-
steps:
20-
- uses: actions/checkout@v4
21-
- name: Set up Python ${{ matrix.python-version }}
22-
uses: actions/setup-python@v5
23-
with:
24-
python-version: ${{ matrix.python-version }}
25-
- name: Install Poetry
26-
uses: snok/install-poetry@v1
27-
with:
28-
virtualenvs-create: false
29-
- name: Install dependencies
30-
run: poetry install --sync --only dev
31-
- name: Generate ${{ matrix.target }}
32-
run: make generate-${{ matrix.target }}
33-
- name: Check for changes
34-
run: make lint-no-changes
12+
# generate:
13+
# strategy:
14+
# fail-fast: false
15+
# matrix:
16+
# python-version: ["3.9", "3.10"]
17+
# target: ["dataplane", "model-repository"]
18+
# runs-on: ubuntu-22.04
19+
# steps:
20+
# - uses: actions/checkout@v4
21+
# - name: Set up Python ${{ matrix.python-version }}
22+
# uses: actions/setup-python@v5
23+
# with:
24+
# python-version: ${{ matrix.python-version }}
25+
# - name: Install Poetry
26+
# uses: snok/install-poetry@v1
27+
# with:
28+
# virtualenvs-create: false
29+
# - name: Install dependencies
30+
# run: poetry install --sync --only dev
31+
# - name: Generate ${{ matrix.target }}
32+
# run: make generate-${{ matrix.target }}
33+
# - name: Check for changes
34+
# run: make lint-no-changes
3535

36-
lint:
37-
strategy:
38-
matrix:
39-
python-version:
40-
- "3.9"
41-
- "3.10"
42-
runs-on: ubuntu-22.04
43-
steps:
44-
- uses: actions/checkout@v4
45-
- name: Set up Python ${{ matrix.python-version }}
46-
uses: actions/setup-python@v5
47-
with:
48-
python-version: ${{ matrix.python-version }}
49-
- name: Install Poetry
50-
uses: snok/install-poetry@v1
51-
with:
52-
virtualenvs-create: false
53-
- name: Install dependencies
54-
run: poetry install --sync --only dev
55-
- name: Lint
56-
run: make lint
36+
# lint:
37+
# strategy:
38+
# matrix:
39+
# python-version:
40+
# - "3.9"
41+
# - "3.10"
42+
# runs-on: ubuntu-22.04
43+
# steps:
44+
# - uses: actions/checkout@v4
45+
# - name: Set up Python ${{ matrix.python-version }}
46+
# uses: actions/setup-python@v5
47+
# with:
48+
# python-version: ${{ matrix.python-version }}
49+
# - name: Install Poetry
50+
# uses: snok/install-poetry@v1
51+
# with:
52+
# virtualenvs-create: false
53+
# - name: Install dependencies
54+
# run: poetry install --sync --only dev
55+
# - name: Lint
56+
# run: make lint
5757

5858
test-mlserver:
5959
strategy:
@@ -65,7 +65,7 @@ jobs:
6565
# NOTE: There's no pre-built `grpcio` wheel for Python 3.11 yet
6666
# https://github.com/grpc/grpc/issues/32454
6767
python-version:
68-
- "3.9"
68+
# - "3.9"
6969
- "3.10"
7070
is-pr:
7171
- ${{ github.event_name == 'pull_request' }}
@@ -99,66 +99,66 @@ jobs:
9999
run: poetry install --only test
100100
- name: Test
101101
run: |
102-
if [[ ${{ runner.os == 'macOS' }} ]]; then
102+
if [[ "$(uname)" == 'Darwin' ]]; then
103103
source ./hack/activate-libomp-macos.sh
104-
poetry run tox -e mlserver
104+
poetry run tox -e mlserver -- -- -n auto
105105
else
106106
poetry run tox -e mlserver -- -- -n auto
107107
fi
108108
109109
110-
test-runtimes:
111-
strategy:
112-
fail-fast: false
113-
matrix:
114-
os:
115-
- ubuntu-22.04
116-
- macos-13
117-
python-version:
118-
- "3.9"
119-
- "3.10"
120-
tox-environment:
121-
- sklearn
122-
- xgboost
123-
- lightgbm
124-
- mlflow
125-
- huggingface
126-
- alibi-explain
127-
- alibi-detect
128-
- catboost
129-
is-pr:
130-
- ${{ github.event_name == 'pull_request' }}
131-
# exclude:
132-
# # MacOS tests take a lot of time, so we will run them only on merge
133-
# # From https://github.com/orgs/community/discussions/26253
134-
# - is-pr: true
135-
# os: macos-13
136-
runs-on: ${{ matrix.os }}
137-
steps:
138-
- uses: actions/checkout@v4
139-
- name: Install OpenMP (in MacOS)
140-
if: runner.os == 'macOS'
141-
run: brew install libomp
142-
- name: Set up Python ${{ matrix.python-version }}
143-
uses: actions/setup-python@v5
144-
with:
145-
python-version: ${{ matrix.python-version }}
146-
- uses: conda-incubator/setup-miniconda@v3
147-
with:
148-
auto-update-conda: true
149-
auto-activate-base: false
150-
- name: Install Poetry
151-
uses: snok/install-poetry@v1
152-
with:
153-
virtualenvs-create: false
154-
- name: Install dependencies
155-
run: poetry install --only test
156-
- name: Test
157-
run: |
158-
if [[ ${{ runner.os == 'macOS' }} ]]; then
159-
source ./hack/activate-libomp-macos.sh
160-
fi
161-
poetry run tox -c ./runtimes/${{ matrix.tox-environment }} -- -- -n auto
110+
# test-runtimes:
111+
# strategy:
112+
# fail-fast: false
113+
# matrix:
114+
# os:
115+
# - ubuntu-22.04
116+
# - macos-13
117+
# python-version:
118+
# - "3.9"
119+
# - "3.10"
120+
# tox-environment:
121+
# - sklearn
122+
# - xgboost
123+
# - lightgbm
124+
# - mlflow
125+
# - huggingface
126+
# - alibi-explain
127+
# - alibi-detect
128+
# - catboost
129+
# is-pr:
130+
# - ${{ github.event_name == 'pull_request' }}
131+
# # exclude:
132+
# # # MacOS tests take a lot of time, so we will run them only on merge
133+
# # # From https://github.com/orgs/community/discussions/26253
134+
# # - is-pr: true
135+
# # os: macos-13
136+
# runs-on: ${{ matrix.os }}
137+
# steps:
138+
# - uses: actions/checkout@v4
139+
# - name: Install OpenMP (in MacOS)
140+
# if: runner.os == 'macOS'
141+
# run: brew install libomp
142+
# - name: Set up Python ${{ matrix.python-version }}
143+
# uses: actions/setup-python@v5
144+
# with:
145+
# python-version: ${{ matrix.python-version }}
146+
# - uses: conda-incubator/setup-miniconda@v3
147+
# with:
148+
# auto-update-conda: true
149+
# auto-activate-base: false
150+
# - name: Install Poetry
151+
# uses: snok/install-poetry@v1
152+
# with:
153+
# virtualenvs-create: false
154+
# - name: Install dependencies
155+
# run: poetry install --only test
156+
# - name: Test
157+
# run: |
158+
# if [[ "$(uname)" == 'Darwin' ]]; then
159+
# source ./hack/activate-libomp-macos.sh
160+
# fi
161+
# poetry run tox -c ./runtimes/${{ matrix.tox-environment }} -- -- -n auto
162162

163163
# Ensure that having all the runtimes installed together works
164164
test-all-runtimes:
@@ -170,7 +170,7 @@ jobs:
170170
- ubuntu-22.04
171171
- macos-13
172172
python-version:
173-
- "3.9"
173+
# - "3.9"
174174
- "3.10"
175175
runs-on: ${{ matrix.os }}
176176
steps:
@@ -208,10 +208,10 @@ jobs:
208208
run: poetry install --only test
209209
- name: Test
210210
run: |
211-
if [[ ${{ runner.os == 'macOS' }} ]]; then
211+
if [[ "$(uname)" == 'Darwin' ]]; then
212212
source ./hack/activate-libomp-macos.sh
213-
poetry run tox -e all-runtimes
213+
poetry run tox -e all-runtimes -- -- -n auto
214214
else
215215
poetry run tox -e all-runtimes -- -- -n auto
216-
if
216+
fi
217217

‎tox.ini‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ commands =
4040
{toxinidir}/runtimes/mllib \
4141
{toxinidir}/runtimes/lightgbm \
4242
{toxinidir}/runtimes/mlflow \
43-
{toxinidir}/runtimes/huggingface \
43+
{toxinidir}/runtimes/huggingface
4444
set_env =
4545
GITHUB_SERVER_URL = {env:GITHUB_SERVER_URL:https\://github.com}
4646
GITHUB_REPOSITORY = {env:GITHUB_REPOSITORY:SeldonIO/MLServer}

0 commit comments

Comments
 (0)