Skip to content

ci: remove postgres logs from big jobs - #1945

Merged
alxndrsn merged 1 commit into
getodk:masterfrom
alxndrsn:less-postgres-logs
Sep 21, 2026
Merged

alxndrsn merged 1 commit into
getodk:masterfrom
alxndrsn:less-postgres-logs

Conversation

@alxndrsn

@alxndrsn alxndrsn commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

These jobs generate huge PostgreSQL logs, which add ~1 minute to build times. While these logs might be helpful occasionally, it's probably not worth the overhead to have them always enabled.

This impact was noted in #1944.

Introduced in #1932, the original aim of that PR is still maintained, as PostgreSQL logs will still be available from other GitHub Actions workflows.

What has been done to verify that this works as intended?

  • ci

Why is this the best possible solution? Were any other approaches considered?

  • logs are still available to debug PostgreSQL startup issues, and issues with postgres-specific test suites
  • standard and oidc-integration suites will be faster

How does this change impact users? Describe intentional behavior changes from code updates. What are the regression risks?

No effect - just tests.

Does this change require updates to the API documentation? If so, please update docs/api.yaml as part of this PR.

No.

These jobs generate huge PostgreSQL logs, which add ~1 minute to build times.  While these logs might be helpful occasionally, it's probably not worth the overhead to have them _always_ enabled.

This impact was noted in getodk#1944.

Introduced in getodk#1932, the original aim of that PR is still maintained, as PostgreSQL logs will still be available from other GitHub Actions workflows.
@alxndrsn
alxndrsn marked this pull request as ready for review September 19, 2026 07:53
@alxndrsn
alxndrsn merged commit 02072e2 into getodk:master Sep 21, 2026
10 of 12 checks passed
@alxndrsn
alxndrsn deleted the less-postgres-logs branch September 21, 2026 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants