potiuk commented on code in PR #49822:
URL: https://github.com/apache/airflow/pull/49822#discussion_r2061400207
##########
.github/workflows/k8s-tests.yml:
##########
@@ -103,27 +103,35 @@ jobs:
USE_STANDARD_NAMING: ${{ matrix.use-standard-naming }}
VERBOSE: "false"
- name: "\
- Upload KinD logs on failure ${{ matrix.executor }}-${{
matrix.kubernetes-combo }}-\
+ Dump logs on success ${{ matrix.executor }}-${{
env.PYTHON_MAJOR_MINOR_VERSION }}-\
+ ${{env.KUBERNETES_VERSION}}-${{ matrix.use-standard-naming }}"
+ run: breeze k8s logs
+ env:
+ EXECUTOR: ${{ matrix.executor }}
+ USE_STANDARD_NAMING: ${{ matrix.use-standard-naming }}
+ VERBOSE: "false"
+ if: success() && inputs.include-success-outputs == 'true'
+ - name: "\
+ Print logs ${{ matrix.executor }}-${{ matrix.kubernetes-combo }}-\
${{ matrix.use-standard-naming }}"
- uses: actions/upload-artifact@v4
- if: failure() || cancelled()
- with:
- name: "\
- kind-logs-${{ matrix.kubernetes-combo }}-${{ matrix.executor }}-\
- ${{ matrix.use-standard-naming }}"
- path: /tmp/kind_logs_*
- retention-days: '7'
+ run: |
+ for file in /tmp/kind_logs_*/*; do
+ echo "::group::${file}"
+ cat $file
+ echo "::endgroup::"
+ done
+ if: failure() || cancelled() || inputs.include-success-outputs ==
'true'
- name: "\
- Upload test resource logs on failure ${{ matrix.executor }}-${{
matrix.kubernetes-combo }}-\
+ Upload KinD logs ${{ matrix.executor }}-${{ matrix.kubernetes-combo
}}-\
${{ matrix.use-standard-naming }}"
uses: actions/upload-artifact@v4
- if: failure() || cancelled()
with:
name: "\
- k8s-test-resources-${{ matrix.kubernetes-combo }}-${{
matrix.executor }}-\
Review Comment:
This was a relic of a past.
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
