Skip to content

Commit

Permalink
Merge pull request #281 from zillow/tz/AIP-8051-fix-aip-integration-t…
Browse files Browse the repository at this point in the history
…ests

AIP-8051 Fix AIP Integration tests
  • Loading branch information
talebzeghmi authored Jan 31, 2024
2 parents a1c451e + 0cbb9ac commit 9c0982a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ jobs:
- name: Install Python ${{ matrix.ver }} dependencies
run: |
python3 -m pip install --upgrade pip setuptools
python3 -m pip install tox numpy black
python3 -m pip install tox==3.25.1 numpy black==21.12b "click<8.1.0"
- name: Python Code Format Check
run: black --target-version py39 --diff --check ./metaflow/plugins/aip/*.py
4 changes: 2 additions & 2 deletions .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ variables:
# Run on the Internal cluster only on commit.
# We run the tests via the integration test framework for Non-Prod/Prod.
DEPLOY_INTERNAL: "true"
DEPLOY_NONPROD: "false"
DEPLOY_STAGE: "false"
DEPLOY_PROD: "false"
DEPLOY_SANDBOX: "false"
PUBLISH: "true"
Expand Down Expand Up @@ -172,7 +172,7 @@ test:nonprod:
METAFLOW_RUN_URL_PREFIX: "https://metaflow.int.stage-k8s.zg-aip.net"
METAFLOW_KUBERNETES_NAMESPACE: "metaflow-integration-testing-stage"
rules:
- if: '$DEPLOY_NONPROD == "true"'
- if: '$DEPLOY_STAGE == "true"'

test:prod:
extends:
Expand Down

0 comments on commit 9c0982a

Please sign in to comment.