Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the connection error for extension tests #10480

Merged
merged 5 commits into from
Jan 22, 2025

Conversation

a-masterov
Copy link
Contributor

@a-masterov a-masterov commented Jan 22, 2025

Problem

The trust connection to the compute required for pg_anon was removed.
However, the PGPASSWORD environment variable was not added to docker-compose.yml.
This caused connection errors, which were interpreted as success due to errors in the bash script.

Summary of changes

The environment variable was added, and the logic in the bash script was fixed.

@a-masterov a-masterov marked this pull request as ready for review January 22, 2025 14:09
@a-masterov a-masterov requested a review from a team January 22, 2025 14:15
Copy link

github-actions bot commented Jan 22, 2025

7396 tests run: 7011 passed, 0 failed, 385 skipped (full report)


Flaky tests (3)

Postgres 17

Postgres 14

Code coverage* (full report)

  • functions: 33.5% (8457 of 25279 functions)
  • lines: 49.2% (70935 of 144156 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
2f028e7 at 2025-01-22T16:44:00.209Z :recycle:

@a-masterov a-masterov added this pull request to the merge queue Jan 22, 2025
Merged via the queue into main with commit f1473dd Jan 22, 2025
88 of 89 checks passed
@a-masterov a-masterov deleted the amasterov/fix-extensin-tests-2201 branch January 22, 2025 16:35
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.

3 participants