You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by po...@apache.org on 2024/02/06 21:13:48 UTC
(airflow) branch main updated: Upgrade arifacts upload action to v4 (#37199)
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 3ffface2ae Upgrade arifacts upload action to v4 (#37199)
3ffface2ae is described below
commit 3ffface2ae801073855ff158d4b4ba72865e0706
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Tue Feb 6 22:13:41 2024 +0100
Upgrade arifacts upload action to v4 (#37199)
V3 version uses old version of Node and we should update it to get
rid of warnings in CI.
---
.github/actions/build-ci-images/action.yml | 2 +-
.github/actions/build-prod-images/action.yml | 4 ++--
.github/actions/post_tests_failure/action.yml | 6 +++---
.github/actions/post_tests_success/action.yml | 2 +-
.github/workflows/ci.yml | 24 +++++++++++-----------
.../amazon/aws/hooks/test_hooks_signature.py | 1 +
6 files changed, 20 insertions(+), 19 deletions(-)
diff --git a/.github/actions/build-ci-images/action.yml b/.github/actions/build-ci-images/action.yml
index 820c858366..72a140604e 100644
--- a/.github/actions/build-ci-images/action.yml
+++ b/.github/actions/build-ci-images/action.yml
@@ -38,7 +38,7 @@ runs:
breeze release-management generate-constraints --run-in-parallel
--airflow-constraints-mode constraints-source-providers
- name: "Upload constraint artifacts"
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: source-constraints
path: ./files/constraints-*/constraints-*.txt
diff --git a/.github/actions/build-prod-images/action.yml b/.github/actions/build-prod-images/action.yml
index b58aa3a2ba..141a2cf9b1 100644
--- a/.github/actions/build-prod-images/action.yml
+++ b/.github/actions/build-prod-images/action.yml
@@ -62,13 +62,13 @@ runs:
shell: bash
run: cp -v --no-preserve=mode,ownership ./dist/*.whl ./docker-context-files
- name: "Download constraints from the CI build"
- uses: actions/download-artifact@v3
+ uses: actions/download-artifact@v4
with:
name: source-constraints
path: ./docker-context-files
if: ${{ inputs.build-provider-packages == 'true' }}
- name: "Download constraints from the Generate & Verify build"
- uses: actions/download-artifact@v3
+ uses: actions/download-artifact@v4
with:
name: constraints
path: ./docker-context-files
diff --git a/.github/actions/post_tests_failure/action.yml b/.github/actions/post_tests_failure/action.yml
index 5b51db97ed..34d35ed47d 100644
--- a/.github/actions/post_tests_failure/action.yml
+++ b/.github/actions/post_tests_failure/action.yml
@@ -22,19 +22,19 @@ runs:
using: "composite"
steps:
- name: "Upload airflow logs"
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: airflow-logs-${{env.JOB_ID}}
path: './files/airflow_logs*'
retention-days: 7
- name: "Upload container logs"
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: container-logs-${{env.JOB_ID}}
path: "./files/container_logs*"
retention-days: 7
- name: "Upload other logs"
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: container-logs-${{env.JOB_ID}}
path: "./files/other_logs*"
diff --git a/.github/actions/post_tests_success/action.yml b/.github/actions/post_tests_success/action.yml
index ac1e4fb8d2..214f731989 100644
--- a/.github/actions/post_tests_success/action.yml
+++ b/.github/actions/post_tests_success/action.yml
@@ -22,7 +22,7 @@ runs:
using: "composite"
steps:
- name: "Upload artifact for warnings"
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: test-warnings-${{env.JOB_ID}}
path: ./files/warnings-*.txt
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 50a84543aa..153b00fed1 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -420,7 +420,7 @@ jobs:
run: killall python || true # just in case there is a webserver running in the background
if: always()
- name: "Upload python client packages"
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: python-client-packages
path: ./dist/apache_airflow_client-*
@@ -537,7 +537,7 @@ jobs:
cat "files/constraints-${PYTHON_VERSION}"/*.md >> $GITHUB_STEP_SUMMARY || true
done
- name: "Upload constraint artifacts"
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: constraints
path: ./files/constraints-*/constraints-*.txt
@@ -1111,7 +1111,7 @@ jobs:
env:
SIGN_WITH: dev@airflow.apache.org
- name: "Upload Helm artifacts"
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
with:
name: Helm artifacts
path: ./dist/airflow-*
@@ -1497,7 +1497,7 @@ jobs:
PYTHON_MAJOR_MINOR_VERSION: "${{needs.build-info.outputs.default-python-version}}"
POSTGRES_VERSION: "${{needs.build-info.outputs.default-postgres-version}}"
BACKEND_VERSION: "${{needs.build-info.outputs.default-postgres-version}}"
- JOB_ID: "integration"
+ JOB_ID: "integration-postgres"
SKIP_PROVIDER_TESTS: "${{needs.build-info.outputs.skip-provider-tests}}"
if: needs.build-info.outputs.run-tests == 'true'
steps:
@@ -1569,7 +1569,7 @@ jobs:
PYTHON_MAJOR_MINOR_VERSION: "${{needs.build-info.outputs.default-python-version}}"
MYSQL_VERSION: "${{needs.build-info.outputs.default-mysql-version}}"
BACKEND_VERSION: "${{needs.build-info.outputs.default-mysql-version}}"
- JOB_ID: "integration"
+ JOB_ID: "integration-mysql"
SKIP_PROVIDER_TESTS: "${{needs.build-info.outputs.skip-provider-tests}}"
if: needs.build-info.outputs.run-tests == 'true'
steps:
@@ -1666,9 +1666,9 @@ jobs:
env:
RUNS_ON: "${{needs.build-info.outputs.runs-on}}"
PR_LABELS: "${{needs.build-info.outputs.pull-request-labels}}"
- PYTHON_MAJOR_MINOR_VERSION: "${{needs.build-info.outputs.default-python-version}}"
+ PYTHON_MAJOR_MINOR_VERSION: "${{matrix.python-version}}"
DEBUG_RESOURCES: "${{needs.build-info.outputs.debug-resources}}"
- JOB_ID: "quarantined-${{needs.build-info.outputs.default-python-version}}"
+ JOB_ID: "no-db-${{matrix.python-version}}"
ENABLE_COVERAGE: "${{needs.build-info.outputs.run-coverage}}"
if: needs.build-info.outputs.run-tests == 'true'
steps:
@@ -1705,7 +1705,7 @@ jobs:
PR_LABELS: "${{needs.build-info.outputs.pull-request-labels}}"
PYTHON_MAJOR_MINOR_VERSION: "${{needs.build-info.outputs.default-python-version}}"
DEBUG_RESOURCES: "${{needs.build-info.outputs.debug-resources}}"
- JOB_ID: "quarantined-${{needs.build-info.outputs.default-python-version}}"
+ JOB_ID: "no-db-pendulum-2-${{needs.build-info.outputs.default-python-version}}"
ENABLE_COVERAGE: "${{needs.build-info.outputs.run-coverage}}"
DOWNGRADE_PENDULUM: "true"
if: needs.build-info.outputs.run-tests == 'true'
@@ -1754,7 +1754,7 @@ jobs:
with:
persist-credentials: false
- name: "Download all artifacts from the current build"
- uses: actions/download-artifact@v3
+ uses: actions/download-artifact@v4
with:
path: ./artifacts
- name: "Summarize all warnings"
@@ -2243,7 +2243,7 @@ jobs:
USE_STANDARD_NAMING: ${{matrix.use-standard-naming}}
VERBOSE: "false"
- name: Upload KinD logs on failure ${{needs.build-info.outputs.kubernetes-combos-list-as-string}}
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
if: failure() || cancelled()
with:
name: kind-logs-${{matrix.executor}}-${{matrix.use-standard-naming}}
@@ -2251,7 +2251,7 @@ jobs:
retention-days: 7
- name: >
Upload test resource logs on failure ${{needs.build-info.outputs.kubernetes-combos-list-as-string}}
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
if: failure() || cancelled()
with:
name: k8s-test-resources-${{matrix.executor}}-${{matrix.use-standard-naming}}
@@ -2307,7 +2307,7 @@ jobs:
persist-credentials: true
fetch-depth: 0
- name: "Download constraints from the constraints generated by build CI image"
- uses: actions/download-artifact@v3
+ uses: actions/download-artifact@v4
with:
name: constraints
path: ./files
diff --git a/tests/providers/amazon/aws/hooks/test_hooks_signature.py b/tests/providers/amazon/aws/hooks/test_hooks_signature.py
index b0ab654be2..34a1d011fa 100644
--- a/tests/providers/amazon/aws/hooks/test_hooks_signature.py
+++ b/tests/providers/amazon/aws/hooks/test_hooks_signature.py
@@ -31,6 +31,7 @@ ALLOWED_THICK_HOOKS_PARAMETERS: dict[str, set[str]] = {
# This list should only be reduced not extended with new parameters,
# unless there is an exceptional reason.
"AthenaHook": {"sleep_time", "log_query"},
+ "AthenaSQLHook": {"athena_conn_id"},
"BatchClientHook": {"status_retries", "max_retries"},
"BatchWaitersHook": {"waiter_config"},
"DataSyncHook": {"wait_interval_seconds"},