Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

Fix Postgres CI runs #3399

Closed
wants to merge 1 commit into from
Closed

Fix Postgres CI runs #3399

wants to merge 1 commit into from

Conversation

S7evinK
Copy link
Contributor

@S7evinK S7evinK commented Jul 28, 2024

For some reason Postgres wouldn't allow connections over TCP. Since the tests are run in isolation, this allows all "peer" authed connections without a password.

For CI to be fully happy, we need #3397

@S7evinK S7evinK requested a review from a team as a code owner July 28, 2024 17:33
Copy link

codecov bot commented Jul 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.50%. Comparing base (9897959) to head (2eff4f1).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3399      +/-   ##
==========================================
- Coverage   67.54%   67.50%   -0.04%     
==========================================
  Files         521      521              
  Lines       47381    47381              
==========================================
- Hits        32002    31984      -18     
- Misses      11394    11412      +18     
  Partials     3985     3985              
Flag Coverage Δ
unittests 52.79% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@S7evinK
Copy link
Contributor Author

S7evinK commented Aug 2, 2024

Fixed with #3360, as it was touching those files anyway.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant