From: odidev Date: Tue, 28 Jul 2020 13:42:16 +0000 (+0000) Subject: Arm64 Support to Github Actions For Testing And Development Of Wheels X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=131ef520bbf863ad9b65ca3bf3c2d2d214405702;p=thirdparty%2Fsqlalchemy%2Fsqlalchemy.git Arm64 Support to Github Actions For Testing And Development Of Wheels Signed-off-by: odidev --- diff --git a/.github/workflows/create-wheels.yaml b/.github/workflows/create-wheels.yaml index 1861353a68..506e9986f6 100644 --- a/.github/workflows/create-wheels.yaml +++ b/.github/workflows/create-wheels.yaml @@ -283,3 +283,124 @@ jobs: run: | pip install -U twine twine upload --skip-existing dist/*manylinux* + + make-wheel-linux-arm64: + name: ${{ matrix.python-version }}-arm64-${{ matrix.os }} + runs-on: ${{ matrix.os }} + strategy: + matrix: + os: + - "ubuntu-latest" + python-version: + # the versions are - as specified in PEP 425. + - cp36-cp36m + - cp37-cp37m + - cp38-cp38 + + fail-fast: false + + steps: + - name: Checkout repo + uses: actions/checkout@v2 + + - name: Get python version + id: linux-py-version + env: + py_tag: ${{ matrix.python-version }} + # the command `echo "::set-output ...` is used to create an step output that can be used in following steps + # this is from https://github.community/t5/GitHub-Actions/Using-the-output-of-run-inside-of-if-condition/td-p/33920 + run: | + version="${py_tag: 2:1}.${py_tag: 3:1}" + echo $version + echo "::set-output name=python-version::$version" + + - name: Set up Python + uses: actions/setup-python@v2 + with: + python-version: ${{ steps.linux-py-version.outputs.python-version }} + architecture: ${{ matrix.architecture }} + + - name: Remove tag_build from setup.cfg + # sqlalchemy has `tag_build` set to `dev` in setup.cfg. We need to remove it before creating the weel + # otherwise it gets tagged with `dev0` + shell: pwsh + # This is equivalent to the sed commands: + # `sed -i '/tag_build=dev/d' setup.cfg` + # `sed -i '/tag_build = dev/d' setup.cfg` + + # `-replace` uses a regexp match + # alternative form: `(get-content setup.cfg) | foreach-object{$_ -replace "tag_build.=.dev",""} | set-content setup.cfg` + run: | + (cat setup.cfg) | %{$_ -replace "tag_build.?=.?dev",""} | set-content setup.cfg + + - name: Enable qemu + run: | + docker run --rm --privileged hypriot/qemu-register + + - name: Create wheel for manylinux2014 + # this step uses the image provided by pypa here https://github.com/pypa/manylinux to generate the wheels on linux + # the action uses the image for manylinux2014 but can generate also a manylinux1 wheel + # change the tag of this image to change the image used + uses: RalfG/python-wheels-manylinux-build@v0.3.1-manylinux2014_aarch64 + # this action generates 2 wheels in dist/. linux and manylinux2014 + with: + # python-versions is the output of the previous step and is in the form -. Eg cp37-cp37mu + python-versions: ${{ matrix.python-version }} + build-requirements: "setuptools>=44 wheel>=0.34" + # Create the wheel using --no-use-pep517 since locally we have pyproject + # This flag should be removed once sqlalchemy supports pep517 + # `--no-deps` is used to only generate the wheel for the current library. Redundant in sqlalchemy since it has no dependencies + pip-wheel-args: "-w ./dist --no-use-pep517 -v --no-deps" + + - name: Check created wheel + # check that the wheel is compatible with the current installation. + # - runs the tests + uses: docker://quay.io/pypa/manylinux2014_aarch64 + with: + args: | + bash -c "export PATH=/opt/python/${{ matrix.python-version }}/bin:$PATH; + pip install -f wheelhouse --no-index sqlalchemy; + python -c 'from sqlalchemy import cprocessors, cresultproxy, cutils'; + pip install pytest pytest-xdist; + pytest -n2 -q test -k 'not MockReconnectTest' --nomemory;" + + - name: Get wheel names + id: wheel-name + shell: bash + # the wheel creation step generates 2 wheels: linux and manylinux2014 + # Pypi accepts only the manylinux versions + run: | + cd dist + echo ::set-output name=wheel2014::`ls *manylinux2014*` + + - name: Upload wheel manylinux2014 to release + # upload the generated manylinux2014 wheel to the github release. Only a single file per step can be uploaded at the moment + uses: actions/upload-release-asset@v1 + env: + GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} + with: + # this is a release to the event is called release: https://help.github.com/en/actions/reference/events-that-trigger-workflows#release-event-release + # the release event has the structure of this response https://developer.github.com/v3/repos/releases/#create-a-release + upload_url: ${{ github.event.release.upload_url }} + asset_path: dist/${{ steps.wheel-name.outputs.wheel2014 }} + asset_name: ${{ steps.wheel-name.outputs.wheel2014 }} + asset_content_type: application/zip # application/octet-stream + + - name: Set up Python for twine + # twine on py2 is very old and is no longer updated, so we change to python 3.8 before upload + uses: actions/setup-python@v2 + with: + python-version: "3.8" + + - name: Publish wheel + # the action https://github.com/marketplace/actions/pypi-publish runs only on linux and we cannot specify + # additional options + # We upload manylinux2014 arm64 wheels. pip will download the appropriate one according to the system. + env: + TWINE_USERNAME: __token__ + # replace TWINE_PASSWORD with token for real pypi + # TWINE_PASSWORD: ${{ secrets.test_pypi_token }} + TWINE_PASSWORD: ${{ secrets.pypi_token }} + run: | + pip install -U twine + twine upload --skip-existing dist/*manylinux* diff --git a/.github/workflows/run-on-pr.yaml b/.github/workflows/run-on-pr.yaml index 53b4f5a095..b2a2672aea 100644 --- a/.github/workflows/run-on-pr.yaml +++ b/.github/workflows/run-on-pr.yaml @@ -14,7 +14,7 @@ env: TOX_WORKERS: -n2 jobs: - run-test: + run-test-amd64: name: ${{ matrix.python-version }}-${{ matrix.build-type }}-${{ matrix.architecture }}-${{ matrix.os }} runs-on: ${{ matrix.os }} strategy: @@ -51,3 +51,30 @@ jobs: - name: Run tests run: tox -e github-${{ matrix.build-type }} -- -q --nomemory ${{ matrix.pytest-args }} + + run-test-arm64: + name: ${{ matrix.python-version }}-${{ matrix.build-type }}-arm64-ubuntu-latest + runs-on: ubuntu-latest + strategy: + matrix: + python-version: + - "3.8" + build-type: + - "cext" + - "nocext" + fail-fast: true + + steps: + - name: Checkout repo + uses: actions/checkout@v2 + + - name: Enable qemu + run: | + docker run --rm --privileged hypriot/qemu-register + + - name: Run tests + uses: docker://arm64v8/python:3.8-buster + with: + args: | + bash -c "pip install -U pip setuptools tox; + tox -e github-${{ matrix.build-type }} -- -q --nomemory;" diff --git a/.github/workflows/run-test.yaml b/.github/workflows/run-test.yaml index 6a10b9626f..3d13c80558 100644 --- a/.github/workflows/run-test.yaml +++ b/.github/workflows/run-test.yaml @@ -87,3 +87,36 @@ jobs: - name: Run tests run: tox -e github-${{ matrix.build-type }} -- -q --nomemory ${{ matrix.pytest-args }} + + run-test-arm64: + name: ${{ matrix.python-version }}-${{ matrix.build-type }}-arm64-ubuntu-latest + runs-on: ubuntu-latest + strategy: + matrix: + python-version: + - "3.6" + - "3.7" + - "3.8" + build-type: + - "cext" + - "nocext" + + fail-fast: false + + steps: + - name: Checkout repo + uses: actions/checkout@v2 + + - name: Enable qemu + run: | + docker run --rm --privileged hypriot/qemu-register + + - name: Run tests + uses: docker://arm64v8/ubuntu:bionic + with: + args: | + bash -c "pyver=python${{ matrix.python-version }}; + apt-get update -qq; + apt-get install -qq coreutils $pyver python3-pip lib$pyver-dev; + $pyver -m pip install -U pip setuptools tox; + $pyver -m tox -e github-${{ matrix.build-type }} -- -q --nomemory;"