Skip to content

Commit

Permalink
Merge pull request #258 from openvinotoolkit/enable-pre-production-pa…
Browse files Browse the repository at this point in the history
…ckages-in-test-builds

Enable using pre-production dependencies in test builds
  • Loading branch information
ljcornel authored and atwinand committed Jul 31, 2023
1 parent 5aedf0e commit 902e3d9
Show file tree
Hide file tree
Showing 10 changed files with 29 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-docs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:

- name: Install package with dev, notebooks and docs requirements
run: |
pip install ".[dev,notebooks]"
pip install ".[dev,notebooks]" --pre
pip install -r requirements/requirements-docs.txt
- name: Build sphinx documentation
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/build-package-internal-pypi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ on:
jobs:
build:
uses: ./.github/workflows/build-package.yaml
with:
enable_dev_dependencies: 1
secrets:
PYPI_USER: ${{ secrets.SCLAB_PYPI_USERNAME }}
PYPI_PASSWORD: ${{ secrets.SCLAB_PYPI_PASSWORD }}
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/build-package-pypi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,8 @@ on:
jobs:
build:
uses: ./.github/workflows/build-package.yaml
with:
enable_dev_dependencies: 0
secrets:
PYPI_USER: ${{ secrets.PYPI_USERNAME }}
PYPI_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/build-package-test-pypi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ on:
jobs:
build:
uses: ./.github/workflows/build-package.yaml
with:
enable_dev_dependencies: 1
secrets:
PYPI_USER: ${{ secrets.TEST_PYPI_USERNAME }}
PYPI_PASSWORD: ${{ secrets.TEST_PYPI_PASSWORD }}
Expand Down
10 changes: 10 additions & 0 deletions .github/workflows/build-package.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@ name: Build and publish package

on:
workflow_call: # Make workflow reusable
inputs:
enable_dev_dependencies:
required: false
type: string
secrets:
PYPI_USER:
required: true
Expand All @@ -17,6 +21,7 @@ env:
PYPI_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
PYPI_HOST: ${{ secrets.PYPI_HOST }}
PYPI_PORT: ${{ secrets.PYPI_PORT }}
ENABLE_DEV_DEPENDENCIES: ${{ inputs.enable_dev_dependencies }}

jobs:
build_sdk:
Expand All @@ -33,7 +38,12 @@ jobs:
- name: Install package with dev and notebook requirements
run: |
python -m pip install --upgrade pip
if [[ $ENABLE_DEV_DEPENDENCIES = 1 ]]; then
pip install ".[dev,notebooks]" --pre
else
pip install ".[dev,notebooks]"
fi
- name: Install build tools
run: pip install build twine
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/nightly-tests-geti-develop.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ jobs:
uses: ./.github/workflows/nightly-tests.yaml
with:
learning_parameter_settings: reduced_mem
enable_dev_dependencies: 1
secrets:
GETI_HOST: ${{ secrets.GETI_DEVELOP_HOST }}
GETI_USERNAME: ${{ secrets.GETI_USERNAME }}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/nightly-tests-geti-latest.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ jobs:
uses: ./.github/workflows/nightly-tests.yaml
with:
learning_parameter_settings: default
enable_dev_dependencies: 0
secrets:
GETI_HOST: ${{ secrets.GETI_HOST }}
GETI_USERNAME: ${{ secrets.GETI_USERNAME }}
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/nightly-tests-geti-mvp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ jobs:
uses: ./.github/workflows/nightly-tests.yaml
with:
learning_parameter_settings: default
enable_dev_dependencies: 0
secrets:
GETI_HOST: ${{ secrets.GETI_MVP_HOST }}
GETI_USERNAME: ${{ secrets.GETI_USERNAME }}
Expand Down
8 changes: 8 additions & 0 deletions .github/workflows/nightly-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ on:
learning_parameter_settings:
required: false
type: string
enable_dev_dependencies:
required: false
type: string
secrets:
GETI_HOST:
required: true
Expand All @@ -25,6 +28,7 @@ env:
TEST_MODE: "ONLINE"
CLEAR_EXISTING_TEST_PROJECTS: 1
LEARNING_PARAMETER_SETTINGS: ${{ inputs.learning_parameter_settings }}
ENABLE_DEV_DEPENDENCIES: ${{ inputs.enable_dev_dependencies }}
# Artifacts and test reports
ARTIFACT_DIRECTORY: artifacts
REPORT_DIRECTORY: reports
Expand Down Expand Up @@ -68,7 +72,11 @@ jobs:
python -m pip install --upgrade pip
if [[ $ENABLE_DEV_DEPENDENCIES = 1 ]]; then
pip install ".[dev]" --pre
else;
pip install ".[dev]"
fi
- name: Create artifact directories
# Creates the temporary directories that are used to store the test
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/security-scan.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
# Install dependencies
run: |
python -m pip install --upgrade pip
pip install ".[dev]"
pip install ".[dev]" --pre
- name: Create report directory
# Creates the temporary directory used to store the test reports
Expand Down

0 comments on commit 902e3d9

Please sign in to comment.