Skip to content

Commit 201057e

Browse files
authored
Merge python-integration.yml into python-ci.yml (#1963)
<!-- Thanks for opening a pull request! --> <!-- In the case this PR will resolve an issue, please replace ${GITHUB_ISSUE_ID} below with the actual Github issue id. --> <!-- Closes #${GITHUB_ISSUE_ID} --> Closes #1942 # Rationale for this change This condenses similar CI logic into one place # Are these changes tested? # Are there any user-facing changes? No <!-- In the case of user-facing changes, please add the changelog label. -->
1 parent dfae694 commit 201057e

File tree

2 files changed

+19
-62
lines changed

2 files changed

+19
-62
lines changed

.github/workflows/python-ci.yml

+19-1
Original file line numberDiff line numberDiff line change
@@ -65,4 +65,22 @@ jobs:
6565
- name: Linters
6666
run: make lint
6767
- name: Tests
68-
run: make test-coverage
68+
run: make test-coverage-unit
69+
70+
integration-test:
71+
runs-on: ubuntu-22.04
72+
strategy:
73+
matrix:
74+
python: ['3.9', '3.10', '3.11', '3.12']
75+
76+
steps:
77+
- uses: actions/checkout@v4
78+
- name: Install system dependencies
79+
run: sudo apt-get update && sudo apt-get install -y libkrb5-dev # for kerberos
80+
- name: Install
81+
run: make install
82+
- name: Run integration tests
83+
run: make test-coverage-integration
84+
- name: Show debug logs
85+
if: ${{ failure() }}
86+
run: docker compose -f dev/docker-compose.yml logs

.github/workflows/python-integration.yml

-61
This file was deleted.

0 commit comments

Comments
 (0)