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

Add new postgresql env vars (#1) #279

Closed
wants to merge 1 commit into from

Conversation

ksquaredkey
Copy link

  • Add support for additional PostgreSQL Container env variables.

Added interface for environment variables that the PostgreSQL container now supports:

  • POSTGRES_INITDB_ARGS
  • POSTGRES_INITDB_WALDIR
  • POSTGRES_HOST_AUTH_METHOD
  • PGDATA

As a bonus, added tests for POSTGRES_INITDB_ARGS and POSTGRES_HOST_AUTH_METHOD.

Issue #278

* Add support for additional PostgreSQL Container env variables.

Added interface for environment variables that the PostgreSQL container now
supports:

* POSTGRES_INITDB_ARGS
* POSTGRES_INITDB_WALDIR
* POSTGRES_HOST_AUTH_METHOD
* PGDATA

As a bonus, added tests for POSTGRES_INITDB_ARGS and POSTGRES_HOST_AUTH_METHOD.
@codecov-commenter
Copy link

codecov-commenter commented Dec 27, 2022

Codecov Report

Merging #279 (60b9eed) into master (29629e2) will increase coverage by 0.14%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #279      +/-   ##
==========================================
+ Coverage   86.73%   86.87%   +0.14%     
==========================================
  Files          29       29              
  Lines         814      823       +9     
  Branches       57       59       +2     
==========================================
+ Hits          706      715       +9     
  Misses         78       78              
  Partials       30       30              
Impacted Files Coverage Δ
testcontainers/postgres.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ksquaredkey
Copy link
Author

Hmm, is testcontaners-python an abandoned project?

@tillahoffmann
Copy link
Collaborator

Hey, sorry for the delay. I'll revisit this PR once #296 is merged.

@tillahoffmann
Copy link
Collaborator

#296 has been merged. Would you mind rebasing on main, and we can have a look at getting this merged?

@ksquaredkey ksquaredkey deleted the master branch May 19, 2023 21:59
@ksquaredkey
Copy link
Author

Uh. Not sure why the PR above changed. I renamed master to main in my Fork to match the upstream here about the time this says I deleted the master branch. From the look of #296, it looks like I'll have substantial refactoring to make to match current main, so maybe not a bad thing if I start with a fresh post-296 PR.

@ksquaredkey
Copy link
Author

If anyone stumbles across this old PR, I never got around to refactoring it and @santi posted a better, cleaner way to do what I wanted in the comments of #278.

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

Successfully merging this pull request may close these issues.

Support more environment variables in PostgreSQL container
3 participants