Merge pull request #206 from aktech/build-package

Add workflow to build and test ml-metadata

Author: XinranTang

.github/reusable-build/action.yml

@@ -0,0 +1,37 @@
+name: Resusable steps to build ml-metadata
+
+inputs:
+  python-version:
+    description: 'Python version'
+    required: true
+runs:
+  using: 'composite'
+  steps:
+
+  - name: Set up Python ${{ inputs.python-version }}
+    uses: actions/setup-python@v5
+    with:
+      python-version: ${{ inputs.python-version }}
+
+  - name: Upgrade pip
+    shell: bash
+    run: |
+      python -m pip install --upgrade pip pytest
+
+  - name: Build the package for Python ${{ inputs.python-version }}
+    shell: bash
+    run: |
+      version="${{ inputs.python-version }}"
+      DOCKER_SERVICE=manylinux-python$(echo "$version" | sed 's/\.//')
+      docker compose build ${DOCKER_SERVICE}
+      docker compose run ${DOCKER_SERVICE}
+
+  - name: Upload wheel artifact for Python ${{ inputs.python-version }}
+    uses: actions/upload-artifact@v4.4.0
+    with:
+      name: ml-metadata-wheel-py${{ inputs.python-version }}
+      path: dist/*.whl
+
+  - name: Install built wheel
+    shell: bash
+    run: pip install dist/*.whl

.github/workflows/build.yml

@@ -0,0 +1,53 @@
+name: Build ml-metadata
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+  workflow_dispatch:
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.9", "3.10", "3.11"]
+
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v4
+
+    - name: Build ml-metadata
+      id: build-ml-metadata
+      uses: ./.github/reusable-build
+      with:
+        python-version: ${{ matrix.python-version }}
+
+  upload_to_pypi:
+    name: Upload to PyPI
+    runs-on: ubuntu-latest
+    if: (github.event_name == 'release' && startsWith(github.ref, 'refs/tags')) || (github.event_name == 'workflow_dispatch')
+    needs: [build]
+    environment:
+      name: pypi
+      url: https://pypi.org/p/ml-metadata/
+    permissions:
+      id-token: write
+    steps:
+      - name: Retrieve wheels
+        uses: actions/download-artifact@v4.1.8
+        with:
+          merge-multiple: true
+          path: wheels
+
+      - name: List the build artifacts
+        run: |
+          ls -lAs wheels/
+
+      - name: Upload to PyPI
+        uses: pypa/gh-action-pypi-publish@release/v1.9
+        with:
+          packages_dir: wheels/

.github/workflows/test.yml

@@ -0,0 +1,34 @@
+name: Test ml-metadata
+
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+    branches:
+      - master
+  workflow_dispatch:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.9", "3.10", "3.11"]
+
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v4
+
+    - name: Build ml-metadata
+      id: build-ml-metadata
+      uses: ./.github/reusable-build
+      with:
+        python-version: ${{ matrix.python-version }}
+
+    - name: Test
+      run: |
+        # cleanup (interferes with tests)
+        rm -rf bazel-*
+        # run tests
+        pytest -vv

README.md

@@ -53,8 +53,8 @@ Then, run the following at the project root:
 
 ```bash
 DOCKER_SERVICE=manylinux-python${PY_VERSION}
-sudo docker-compose build ${DOCKER_SERVICE}
-sudo docker-compose run ${DOCKER_SERVICE}
+sudo docker compose build ${DOCKER_SERVICE}
+sudo docker compose run ${DOCKER_SERVICE}
 ```
 
 where `PY_VERSION` is one of `{39, 310, 311}`.

ml_metadata/metadata_store/conftest.py

@@ -0,0 +1,13 @@
+def pytest_addoption(parser):
+    parser.addoption(
+        '--use_grpc_backend', action='store_true', default=False,
+        help='Set this to true to use gRPC instead of sqlLite backend.'
+    )
+    parser.addoption(
+        '--grpc_host', type=str, default=None,
+        help="The gRPC host name to use when use_grpc_backed is set to 'True'"
+    )
+    parser.addoption(
+        '--grpc_port', type=int, default=0,
+        help="The gRPC port number to use when use_grpc_backed is set to 'True'"
+    )