From 91162f2b6c922f8915ff6956692a6c3410fb22f7 Mon Sep 17 00:00:00 2001 From: Aseem Bansal Date: Wed, 28 Aug 2024 17:21:44 +0530 Subject: [PATCH 1/3] ci(build): update outdated action --- .github/workflows/dagster-plugin.yml | 2 +- .github/workflows/docker-unified.yml | 2 +- .github/workflows/gx-plugin.yml | 2 +- metadata-ingestion/docs/sources/looker/lookml_pre.md | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/dagster-plugin.yml b/.github/workflows/dagster-plugin.yml index 48f1b24196c9e0..0a4c446c3771f9 100644 --- a/.github/workflows/dagster-plugin.yml +++ b/.github/workflows/dagster-plugin.yml @@ -44,7 +44,7 @@ jobs: with: distribution: "zulu" java-version: 17 - - uses: actions/checkout@v3 + - uses: actions/checkout@v4 - uses: actions/setup-python@v4 with: python-version: ${{ matrix.python-version }} diff --git a/.github/workflows/docker-unified.yml b/.github/workflows/docker-unified.yml index 32e68a76a88f5a..19a635aeda2a36 100644 --- a/.github/workflows/docker-unified.yml +++ b/.github/workflows/docker-unified.yml @@ -433,7 +433,7 @@ jobs: actions: read # only required for a private repository by github/codeql-action/upload-sarif to get the Action run status steps: - name: Checkout # adding checkout step just to make trivy upload happy - uses: actions/checkout@v3 + uses: actions/checkout@v4 - name: Download image uses: ishworkh/docker-image-artifact-download@v1 if: ${{ needs.setup.outputs.publish != 'true' && needs.setup.outputs.pr-publish != 'true' }} diff --git a/.github/workflows/gx-plugin.yml b/.github/workflows/gx-plugin.yml index 84ba2e0559be1b..48f7664dd4c2ae 100644 --- a/.github/workflows/gx-plugin.yml +++ b/.github/workflows/gx-plugin.yml @@ -46,7 +46,7 @@ jobs: with: distribution: "zulu" java-version: 17 - - uses: actions/checkout@v3 + - uses: actions/checkout@v4 - uses: actions/setup-python@v4 with: python-version: ${{ matrix.python-version }} diff --git a/metadata-ingestion/docs/sources/looker/lookml_pre.md b/metadata-ingestion/docs/sources/looker/lookml_pre.md index 68a4828a5ce2ae..c34a3bd7aaf5c6 100644 --- a/metadata-ingestion/docs/sources/looker/lookml_pre.md +++ b/metadata-ingestion/docs/sources/looker/lookml_pre.md @@ -95,7 +95,7 @@ jobs: lookml-metadata-upload: runs-on: ubuntu-latest steps: - - uses: actions/checkout@v3 + - uses: actions/checkout@v4 - uses: actions/setup-python@v4 with: python-version: "3.10" From a800fb60c6544bfaeded3fc23b28de1abcebc1a7 Mon Sep 17 00:00:00 2001 From: Aseem Bansal Date: Wed, 28 Aug 2024 18:17:40 +0530 Subject: [PATCH 2/3] pin deepdiff --- metadata-ingestion-modules/airflow-plugin/setup.py | 2 +- metadata-ingestion-modules/dagster-plugin/setup.py | 2 +- metadata-ingestion-modules/gx-plugin/setup.py | 2 +- metadata-ingestion/setup.py | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/metadata-ingestion-modules/airflow-plugin/setup.py b/metadata-ingestion-modules/airflow-plugin/setup.py index 80d2efd3ed164e..cc8e9e3b34122e 100644 --- a/metadata-ingestion-modules/airflow-plugin/setup.py +++ b/metadata-ingestion-modules/airflow-plugin/setup.py @@ -83,7 +83,7 @@ def get_long_description(): "pytest-cov>=2.8.1", "tox", "tox-uv", - "deepdiff", + "deepdiff==7.0.1", "tenacity", "build", "twine", diff --git a/metadata-ingestion-modules/dagster-plugin/setup.py b/metadata-ingestion-modules/dagster-plugin/setup.py index ed489af8501c7d..3896aa049eeaaa 100644 --- a/metadata-ingestion-modules/dagster-plugin/setup.py +++ b/metadata-ingestion-modules/dagster-plugin/setup.py @@ -77,7 +77,7 @@ def get_long_description(): "pytest-asyncio>=0.16.0", "pytest-cov>=2.8.1", "tox", - "deepdiff", + "deepdiff==7.0.1", "requests-mock", "freezegun", "jsonpickle", diff --git a/metadata-ingestion-modules/gx-plugin/setup.py b/metadata-ingestion-modules/gx-plugin/setup.py index 1584111f820f59..fd9f8603bb81aa 100644 --- a/metadata-ingestion-modules/gx-plugin/setup.py +++ b/metadata-ingestion-modules/gx-plugin/setup.py @@ -82,7 +82,7 @@ def get_long_description(): "pytest-asyncio>=0.16.0", "pytest-cov>=2.8.1", "tox", - "deepdiff", + "deepdiff==7.0.1", "requests-mock", "freezegun", "jsonpickle", diff --git a/metadata-ingestion/setup.py b/metadata-ingestion/setup.py index 301e571c96accc..2486eea226e59f 100644 --- a/metadata-ingestion/setup.py +++ b/metadata-ingestion/setup.py @@ -544,7 +544,7 @@ test_api_requirements = { "pytest>=6.2.2", - "deepdiff", + "deepdiff==7.0.1", "PyYAML", "pytest-docker>=1.1.0", } From 528d784ac338943f68a93c0b9cd278e712348789 Mon Sep 17 00:00:00 2001 From: Aseem Bansal Date: Wed, 28 Aug 2024 18:26:50 +0530 Subject: [PATCH 3/3] update pin for deepdiff --- metadata-ingestion-modules/airflow-plugin/setup.py | 3 ++- metadata-ingestion-modules/dagster-plugin/setup.py | 3 ++- metadata-ingestion-modules/gx-plugin/setup.py | 3 ++- metadata-ingestion/setup.py | 3 ++- smoke-test/requirements.txt | 3 ++- 5 files changed, 10 insertions(+), 5 deletions(-) diff --git a/metadata-ingestion-modules/airflow-plugin/setup.py b/metadata-ingestion-modules/airflow-plugin/setup.py index cc8e9e3b34122e..1587b4793fbf1d 100644 --- a/metadata-ingestion-modules/airflow-plugin/setup.py +++ b/metadata-ingestion-modules/airflow-plugin/setup.py @@ -83,7 +83,8 @@ def get_long_description(): "pytest-cov>=2.8.1", "tox", "tox-uv", - "deepdiff==7.0.1", + # Missing numpy requirement in 8.0.0 + "deepdiff!=8.0.0", "tenacity", "build", "twine", diff --git a/metadata-ingestion-modules/dagster-plugin/setup.py b/metadata-ingestion-modules/dagster-plugin/setup.py index 3896aa049eeaaa..50450ddd5917af 100644 --- a/metadata-ingestion-modules/dagster-plugin/setup.py +++ b/metadata-ingestion-modules/dagster-plugin/setup.py @@ -77,7 +77,8 @@ def get_long_description(): "pytest-asyncio>=0.16.0", "pytest-cov>=2.8.1", "tox", - "deepdiff==7.0.1", + # Missing numpy requirement in 8.0.0 + "deepdiff!=8.0.0", "requests-mock", "freezegun", "jsonpickle", diff --git a/metadata-ingestion-modules/gx-plugin/setup.py b/metadata-ingestion-modules/gx-plugin/setup.py index fd9f8603bb81aa..48b9d2d170d8cc 100644 --- a/metadata-ingestion-modules/gx-plugin/setup.py +++ b/metadata-ingestion-modules/gx-plugin/setup.py @@ -82,7 +82,8 @@ def get_long_description(): "pytest-asyncio>=0.16.0", "pytest-cov>=2.8.1", "tox", - "deepdiff==7.0.1", + # Missing numpy requirement in 8.0.0 + "deepdiff!=8.0.0", "requests-mock", "freezegun", "jsonpickle", diff --git a/metadata-ingestion/setup.py b/metadata-ingestion/setup.py index 2486eea226e59f..22ff8025aa0a06 100644 --- a/metadata-ingestion/setup.py +++ b/metadata-ingestion/setup.py @@ -544,7 +544,8 @@ test_api_requirements = { "pytest>=6.2.2", - "deepdiff==7.0.1", + # Missing numpy requirement in 8.0.0 + "deepdiff!=8.0.0", "PyYAML", "pytest-docker>=1.1.0", } diff --git a/smoke-test/requirements.txt b/smoke-test/requirements.txt index 952e8ed355d052..c5294d95177cb9 100644 --- a/smoke-test/requirements.txt +++ b/smoke-test/requirements.txt @@ -18,4 +18,5 @@ types-requests>=2.28.11.6,<=2.31.0.3 types-PyYAML # https://github.com/docker/docker-py/issues/3256 requests<=2.31.0 -deepdiff \ No newline at end of file +# Missing numpy requirement in 8.0.0 +deepdiff!=8.0.0 \ No newline at end of file