Skip to content

Commit

Permalink
Use local_s3 for github ci
Browse files Browse the repository at this point in the history
  • Loading branch information
delucchi-cmu committed Oct 3, 2024
1 parent 3e4613c commit 3f3f552
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 10 deletions.
13 changes: 8 additions & 5 deletions .github/workflows/smoke-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,12 +37,15 @@ jobs:
- name: List dependencies
run: |
pip list
- name: Run ABFS unit tests with pytest
- name: Run local s3 with pytest
run: |
python -m pytest tests --cloud abfs
env:
ABFS_LINCCDATA_ACCOUNT_NAME: ${{ secrets.LINCC_ABFS_ACCOUNT_NAME }}
ABFS_LINCCDATA_ACCOUNT_KEY: ${{ secrets.LINCC_ABFS_ACCOUNT_KEY }}
python -m pytest tests --cloud local_s3
# - name: Run ABFS unit tests with pytest
# run: |
# python -m pytest tests --cloud abfs
# env:
# ABFS_LINCCDATA_ACCOUNT_NAME: ${{ secrets.LINCC_ABFS_ACCOUNT_NAME }}
# ABFS_LINCCDATA_ACCOUNT_KEY: ${{ secrets.LINCC_ABFS_ACCOUNT_KEY }}
- name: Send status to Slack app (LSDB CI Reporter)
if: ${{ failure() && github.event_name != 'workflow_dispatch' }}
id: slack
Expand Down
13 changes: 8 additions & 5 deletions .github/workflows/testing-and-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,12 @@ jobs:
python -m pip install --upgrade pip
pip install -e .[dev]
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
- name: Run ABFS unit tests with pytest
- name: Run local s3 with pytest
run: |
python -m pytest tests --cloud abfs
env:
ABFS_LINCCDATA_ACCOUNT_NAME: ${{ secrets.LINCC_ABFS_ACCOUNT_NAME }}
ABFS_LINCCDATA_ACCOUNT_KEY: ${{ secrets.LINCC_ABFS_ACCOUNT_KEY }}
python -m pytest tests --cloud local_s3
# - name: Run ABFS unit tests with pytest
# run: |
# python -m pytest tests --cloud abfs
# env:
# ABFS_LINCCDATA_ACCOUNT_NAME: ${{ secrets.LINCC_ABFS_ACCOUNT_NAME }}
# ABFS_LINCCDATA_ACCOUNT_KEY: ${{ secrets.LINCC_ABFS_ACCOUNT_KEY }}

0 comments on commit 3f3f552

Please sign in to comment.