potiuk commented on code in PR #27829:
URL: https://github.com/apache/airflow/pull/27829#discussion_r1030696177
##########
.github/workflows/ci.yml:
##########
@@ -737,6 +737,42 @@ jobs:
run: breeze ci fix-ownership
if: always()
+ test-airflow-release-commands:
+ timeout-minutes: 80
+ name: "Test Airflow release commands"
+ runs-on: "${{needs.build-info.outputs.runs-on}}"
+ needs: [build-info, wait-for-ci-images]
+ env:
+ RUNS_ON: "${{needs.build-info.outputs.runs-on}}"
+ PYTHON_MAJOR_MINOR_VERSION:
"${{needs.build-info.outputs.default-python-version}}"
+ steps:
+ - name: Cleanup repo
+ run: docker run -v "${GITHUB_WORKSPACE}:/workspace" -u 0:0 bash -c "rm
-rf /workspace/*"
+ - name: "Checkout ${{ github.ref }} ( ${{ github.sha }} )"
+ uses: actions/checkout@v3
+ with:
+ persist-credentials: false
+ - name: >
+ Prepare breeze & CI image:
${{needs.build-info.outputs.default-python-version}}:${{env.IMAGE_TAG}}
+ uses: ./.github/actions/prepare_breeze_and_image
+ - name: "Cleanup dist files"
+ run: rm -fv ./dist/*
+ - name: Install subversion
+ shell: bash
+ run: sudo apt-get install libsvn-dev libapache2-mod-svn
subversion-tools subversion
Review Comment:
You can try now and it should be fine. Our self-hosted images should have
subversion installed.
But you should add a step with this condition:
```
if: needs.build-info.outputs.runs-in == 'self-hosted'
````
Only self-hosted has SVN installed, and I cannot imagine a non-committer
changing it, so this should be ok.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
