Skip to content

Commit

Permalink
test node20 arm64
Browse files Browse the repository at this point in the history
Signed-off-by: Peter Zhu <[email protected]>
  • Loading branch information
peterzhuamazon committed Jan 22, 2025
1 parent 302e78f commit f5d6af9
Showing 1 changed file with 39 additions and 38 deletions.
77 changes: 39 additions & 38 deletions .github/workflows/python-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,55 +9,56 @@ jobs:
with:
product: opensearch

python-tests:
strategy:
matrix:
os:
- macos-14 # arm64
- macos-13 # x64
- windows-latest
runs-on: ${{ matrix.os }}
env:
PYTHON_VERSION: 3.9
steps:
- uses: actions/checkout@v4
- name: Set up Python ${{ env.PYTHON_VERSION }}
uses: actions/setup-python@v3
with:
python-version: ${{ env.PYTHON_VERSION }}
- name: Install Pipenv and Dependencies
run: |
python -m pip install --upgrade pipenv wheel
pipenv install --deploy --dev
- name: Check for Sorted Imports
run: |
pipenv run isort --check .
- name: Enforce Style Guide
run: |
pipenv run flake8 .
- name: Run Type Checker
run: |
pipenv run mypy .
- name: Run Tests with Coverage
run: |
pipenv run coverage run -m pytest --cov=./src --cov-report=xml
- name: Upload Coverage Report
uses: codecov/codecov-action@v2
with:
files: ./coverage.xml
# python-tests:

Check warning on line 12 in .github/workflows/python-tests.yml

View workflow job for this annotation

GitHub Actions / yaml-lint

12:1 [comments-indentation] comment not indented like content
# strategy:
# matrix:
# os:
# - macos-14 # arm64
# - macos-13 # x64
# - windows-latest
# runs-on: ${{ matrix.os }}
# env:
# PYTHON_VERSION: 3.9
# steps:
# - uses: actions/checkout@v4
# - name: Set up Python ${{ env.PYTHON_VERSION }}
# uses: actions/setup-python@v3
# with:
# python-version: ${{ env.PYTHON_VERSION }}
# - name: Install Pipenv and Dependencies
# run: |
# python -m pip install --upgrade pipenv wheel
# pipenv install --deploy --dev
# - name: Check for Sorted Imports
# run: |
# pipenv run isort --check .
# - name: Enforce Style Guide
# run: |
# pipenv run flake8 .
# - name: Run Type Checker
# run: |
# pipenv run mypy .
# - name: Run Tests with Coverage
# run: |
# pipenv run coverage run -m pytest --cov=./src --cov-report=xml
# - name: Upload Coverage Report
# uses: codecov/codecov-action@v2
# with:
# files: ./coverage.xml

python-tests-linux:
needs: Get-CI-Image-Tag
strategy:
matrix:
os:
- ubuntu-24.04-arm # arm64-preview

Check warning on line 54 in .github/workflows/python-tests.yml

View workflow job for this annotation

GitHub Actions / yaml-lint

54:30 [comments] too few spaces before comment
- ubuntu-latest # x64
# - ubuntu-latest # x64

Check warning on line 55 in .github/workflows/python-tests.yml

View workflow job for this annotation

GitHub Actions / yaml-lint

55:1 [comments-indentation] comment not indented like content
runs-on: ${{ matrix.os }}
container:
# using the same image which is used by opensearch-build team to build the OpenSearch Distribution
# this image tag is subject to change as more dependencies and updates will arrive over time
image: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-version-linux }}
#image: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-version-linux }}

Check warning on line 60 in .github/workflows/python-tests.yml

View workflow job for this annotation

GitHub Actions / yaml-lint

60:8 [comments] missing starting space in comment
image: peterzhuamazon/ci-runner:test-gcc12-binutils242-openblas327-arm64-node20
# need to switch to root so that github actions can install runner binary on container without permission issues.
options: ${{ needs.Get-CI-Image-Tag.outputs.ci-image-start-options }}

Expand Down

0 comments on commit f5d6af9

Please sign in to comment.