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 2021/01/12 09:26:05 UTC
[airflow] branch master updated: Add submodules in added workflow
job (#13631)
This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/master by this push:
new eb40eea Add submodules in added workflow job (#13631)
eb40eea is described below
commit eb40eea81be95ecd0e71807145797b6d82375885
Author: Jarek Potiuk <ja...@polidea.com>
AuthorDate: Tue Jan 12 10:25:52 2021 +0100
Add submodules in added workflow job (#13631)
The most recent submodule change for actions #13514 was done in
parallel to Optimising worklfows in #13562 and the job added in
the #13562 still uses non-submodule version of check action.
Also few checkout steps missed:
'submodules: recursive' input
This PR fixes that and all 3rd-party actions now are used
from submodule.
---
.github/workflows/build-images-workflow-run.yml | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/.github/workflows/build-images-workflow-run.yml b/.github/workflows/build-images-workflow-run.yml
index 03b5835..29deda4 100644
--- a/.github/workflows/build-images-workflow-run.yml
+++ b/.github/workflows/build-images-workflow-run.yml
@@ -205,6 +205,7 @@ jobs:
uses: actions/checkout@v2
with:
persist-credentials: false
+ submodules: recursive
- name: >
Event: ${{ needs.cancel-workflow-runs.outputs.sourceEvent }}
Repo: ${{ needs.cancel-workflow-runs.outputs.sourceHeadRepo }}
@@ -223,12 +224,14 @@ jobs:
ref: ${{ needs.cancel-workflow-runs.outputs.targetCommitSha }}
fetch-depth: 2
persist-credentials: false
+ submodules: recursive
if: needs.cancel-workflow-runs.outputs.sourceEvent == 'pull_request'
# checkout the master version again, to use the right script in master workflow
- name: "Checkout ${{ github.ref }} ( ${{ github.sha }} )"
uses: actions/checkout@v2
with:
persist-credentials: false
+ submodules: recursive
- name: Selective checks
id: selective-checks
env:
@@ -281,6 +284,7 @@ jobs:
with:
ref: ${{ needs.cancel-workflow-runs.outputs.targetCommitSha }}
persist-credentials: false
+ submodules: recursive
- name: "Retrieve DEFAULTS from the _initialization.sh"
# We cannot "source" the script here because that would be a security problem (we cannot run
# any code that comes from the sources coming from the PR. Therefore we extract the
@@ -353,7 +357,7 @@ jobs:
run: ./scripts/ci/images/ci_push_ci_images.sh
if: steps.defaults.outputs.proceed == 'true'
- name: Update GitHub Checks for Building image with status
- uses: apache/airflow-checks-action@9f02872da71b6f558c6a6f190f925dde5e4d8798 # v1.1.0
+ uses: ./main-airflow/.github/actions/checks-action
if: always() && steps.defaults.outputs.proceed == 'true'
with:
token: ${{ secrets.GITHUB_TOKEN }}
@@ -405,6 +409,7 @@ jobs:
with:
ref: ${{ needs.cancel-workflow-runs.outputs.targetCommitSha }}
persist-credentials: false
+ submodules: recursive
- name: "Retrieve DEFAULTS from the _initialization.sh"
# We cannot "source" the script here because that would be a security problem (we cannot run
# any code that comes from the sources coming from the PR. Therefore we extract the
@@ -426,8 +431,8 @@ jobs:
else
echo "::set-output name=proceed::true"
fi
- - name: Initiate GitHub Checks for Building image
- uses: apache/airflow-checks-action@9f02872da71b6f558c6a6f190f925dde5e4d8798 # v1.1.0
+ - name: Initiate GitHub Checks for Building image with status
+ uses: ./main-airflow/.github/actions/checks-action
id: build-image-check
with:
token: ${{ secrets.GITHUB_TOKEN }}
@@ -449,6 +454,7 @@ jobs:
path: "main-airflow"
ref: "${{ needs.cancel-workflow-runs.outputs.targetBranch }}"
persist-credentials: false
+ submodules: recursive
if: steps.defaults.outputs.proceed == 'true'
- name: "Setup python"
uses: actions/setup-python@v2