Skip to content

Commit c69a5c8

Browse files
fix: Run uv commands from repo root to use correct virtualenv
- registry-rest-api-tests: Run pytest from repo root instead of sdk/python to avoid uv creating a new venv in the sdk/python directory - feast-operator: Use uv run python from repo root for test-datasources to access the feast package installed in the root virtualenv Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 2d54924 commit c69a5c8

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

.github/workflows/registry-rest-api-tests.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,8 +145,7 @@ jobs:
145145
- name: Setup and Run Registry Rest API tests
146146
run: |
147147
echo "Running Registry REST API tests..."
148-
cd sdk/python
149-
uv run pytest tests/integration/registration/rest_api/test_registry_rest_api.py --integration -s
148+
uv run pytest sdk/python/tests/integration/registration/rest_api/test_registry_rest_api.py --integration -s
150149
151150
- name: Clean up docker images
152151
if: always()

infra/feast-operator/Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,10 +131,10 @@ test-upgrade:
131131
test-previous-version:
132132
go test -timeout 60m ./test/previous-version/ -v -ginkgo.v
133133

134-
# Requires python3
134+
# Requires python3 with feast installed (use uv run in CI)
135135
.PHONY: test-datasources
136136
test-datasources:
137-
python3 test/data-source-types/data-source-types.py
137+
cd ../.. && uv run python infra/feast-operator/test/data-source-types/data-source-types.py
138138
go test ./test/data-source-types/
139139

140140
.PHONY: lint

0 commit comments

Comments
 (0)