Add commit / rollback SQLA on commands which edit the database on harddisk. See https://github.com/google/vizier/issues/1187. Originally executions only occurred in memory but weren't saved to harddisk. #3398
Workflow file for this run
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: ci | |
| on: | |
| pull_request: | |
| branches: | |
| - main | |
| push: | |
| branches: | |
| - main | |
| jobs: | |
| test-ubuntu: | |
| name: "pytest_${{ matrix.suffix }} on ${{ matrix.python-version }} on ubuntu-latest" | |
| runs-on: ubuntu-latest | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| python-version: ["3.10"] # 3.x disabled b/c of 3.11 grpcio-tools build fail. | |
| suffix: ["core", "benchmarks", "algorithms", "clients", "pyglove", "raytune"] | |
| include: | |
| - suffix: "clients" | |
| python-version: "3.8" | |
| - suffix: "clients" | |
| python-version: "3.9" | |
| - suffix: "clients" | |
| python-version: "3.10" | |
| steps: | |
| - name: Clone repo | |
| uses: actions/[email protected] | |
| - name: Set up Python ${{ matrix.python-version }} | |
| uses: actions/setup-python@v4 | |
| with: | |
| # Cache pip packages. | |
| python-version: ${{ matrix.python-version }} | |
| cache: 'pip' | |
| cache-dependency-path: '**/requirements.txt' | |
| - name: Cache dependencies (for faster installation) | |
| # This was taken from: https://github.com/actions/setup-python/issues/330#issuecomment-1500667578 | |
| uses: actions/[email protected] | |
| id: cache | |
| with: | |
| path: ${{ env.pythonLocation }} | |
| key: >- | |
| ${{ env.pythonLocation }} | |
| ${{ hashFiles('**/requirements*.txt') }} | |
| - name: Install essential dependencies | |
| run: | | |
| sudo apt-get install -y libprotobuf-dev | |
| python -m pip install --upgrade pip setuptools | |
| pip install wheel | |
| pip install grpcio-tools==1.48.2 | |
| pip install pytest pytest-xdist | |
| pip wheel -e . | |
| pip install -e . | |
| pip install -r requirements-test.txt | |
| - name: Print installed dependencies | |
| run: pip list | |
| - name: Install test-specific dependencies and run test | |
| # TODO: Split install and run. | |
| run: bash run_tests.sh ${{ matrix.suffix }} |