Skip to content

Commit fa54dfe

Browse files
committed
[CI] dms: workflow artifact uploads + workflow_dispatch + checklog tolerance
Fork-only — improves test-failure observability without modifying upstream-bound surface area. Strip before opening any upstream OCA PR (oca-addons-repo-template owns the .github workflow templates). CI artifact pipeline (test-failure observability) - "Upload screenshots from JS tests" step captures /tmp/odoo_tests/<DB> on failure. Forward-port from the cookiecutter; companion template PR at ledoent/oca-addons-repo-template#1. - "Upload odoo server log on failure" step captures /var/log/odoo/*.log + workspace + runner-temp. - workflow_dispatch: so we can manually re-trigger when GitHub silently throttles fork-PR runs after a push burst. checklog-odoo.cfg - Ignore "Killing chrome descendants" WARN (benign browser_js cleanup; OCA_ENABLE_CHECKLOG_ODOO=1 was treating it as a build error).
1 parent 5089b85 commit fa54dfe

2 files changed

Lines changed: 22 additions & 0 deletions

File tree

.github/workflows/test.yml

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,23 @@ jobs:
6969
run: oca_init_test_database
7070
- name: Run tests
7171
run: oca_run_tests
72+
- name: Upload screenshots from JS tests
73+
uses: actions/upload-artifact@v4
74+
if: ${{ failure() }}
75+
with:
76+
name: Screenshots of failed JS tests - ${{ matrix.name }}
77+
path: /tmp/odoo_tests/${{ env.PGDATABASE }}
78+
if-no-files-found: ignore
79+
- name: Upload odoo server log on failure
80+
uses: actions/upload-artifact@v4
81+
if: ${{ failure() }}
82+
with:
83+
name: Odoo server log - ${{ matrix.name }}
84+
path: |
85+
/var/log/odoo/*.log
86+
${{ github.workspace }}/odoo.log
87+
${{ runner.temp }}/odoo*.log
88+
if-no-files-found: ignore
7289
- uses: codecov/codecov-action@v4
7390
with:
7491
token: ${{ secrets.CODECOV_TOKEN }}

checklog-odoo.cfg

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
11
[checklog-odoo]
22
ignore=
33
WARNING.* 0 failed, 0 error\(s\).*
4+
# browser_js cleanup logs a warning when killing lingering chrome
5+
# children — benign by design (cleanup IS doing its job), but fails
6+
# CHECKLOG. Only matches the exact cleanup phrase to keep the filter
7+
# narrow.
8+
WARNING.*Killing chrome descendants-or-self.*

0 commit comments

Comments
 (0)