| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: ci: Generate crashlogs on Windows |
| Date: | 2026-06-10 13:49:00 |
| Message-ID: | dlhbmj35oc2blnzyrcuhapruifix7c7abv6iqyazww4dbwnwuj@p2ms3d5ucbmp |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On 2026-06-10 15:59:20 +0300, Nazir Bilal Yavuz wrote:
> cdb.exe was available on GHA Windows images but it wasn't configured
> to generate crashlogs. Attached patch configures cdb.exe to generate
> crashlogs to "<github.workspace>\crashlogs\" in
> "crashlog-<pid-in-hex>.txt" format. Current upload logs step already
> collects these logs so no change is needed there.
>
> Most of the logic is copied from where (previous) Postgres CI Windows
> images were generated [1]. When I use the '!adplusext.adpprocname'
> debugger flag, the CI job hangs. It needs 'adplusext.dll' to run but
> it isn't available on GHA Windows images.
>
> Example GHA run which crashlogs are generated and collected:
> https://github.com/nbyavuz/postgres/actions/runs/27275147059
Thanks for working on this!
> +++ b/.github/actions/windows-setup-debugger/action.yml
> @@ -0,0 +1,82 @@
> +name: Setup Windows debugger
> +description: >-
> + Setup Windows debugger to log all crashes to
> + <workspace>\crashlogs\crashlog-<pid-in-hex>.txt
> +
> +runs:
> + using: composite
> + steps:
> + - name: Setup Windows debugger
> + shell: pwsh
> + run: |
> + $crashdir = "${{ github.workspace }}/crashlogs"
> + New-Item -ItemType Directory -Force -Path $crashdir
Any reason to not just put this into src/tools/ci/<something>.ps1? Given that
we're just talking about a single step, I don't really see the benefit of a
composite action here.
Greetings,
Andres Freund
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2026-06-10 13:51:03 | Re: Fix unqualified catalog references in psql describe queries |
| Previous Message | Peter Eisentraut | 2026-06-10 13:44:45 | Re: Fix missing semicolon in pl_gram.y for option_value rule |