Skip to content

Commit

Permalink
comment out postgress
Browse files Browse the repository at this point in the history
 mn/#87-support-elastic-7
  • Loading branch information
codekiln committed Jul 7, 2023
1 parent ab7004d commit e480f1b
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 11 deletions.
5 changes: 3 additions & 2 deletions README_TESTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,11 @@

## Running Tests Locally with one version of python on mac

### install postgres - [deprecated / optional for now]
### [OPTIONAL / deprecated for now] install postgres -
* _This is deprecated because for now we are using sqlite for testing_
* _In time, we may add this back_
* we need `pg_config` to be installed in order to use python locally
* we need `pg_config` to be installed in order to install python locally,
(that's the only reason we need postgres installed; docker-compose serves postgres for testing)
* `brew install postgresql`
* you may need to do `brew tap homebrew/core` first

Expand Down
20 changes: 11 additions & 9 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,15 +23,17 @@ services:
interval: 10s
timeout: 10s
retries: 10
db:
image: postgres
restart: always
ports:
- "5432:5432"
environment:
POSTGRES_USER: "pguser"
POSTGRES_PASSWORD: "pgpass"
POSTGRES_DB: "pgdb"
# commented out because right now we're not using postgres for tests
# comment back in if we need it for multiprocessing tests
# db:
# image: postgres
# restart: always
# ports:
# - "5432:5432"
# environment:
# POSTGRES_USER: "pguser"
# POSTGRES_PASSWORD: "pgpass"
# POSTGRES_DB: "pgdb"

# uncomment to get kibana, which can be useful for analyzing index contents
# this is currently not a required part of testing infrastructure, just left here for convenience
Expand Down
2 changes: 2 additions & 0 deletions test_settings.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,8 @@ def root(*args):
if DEBUG:
ALLOWED_HOSTS = ['*']

# commented out because right now we're not using postgres for tests
# comment back in here and in in docker-compose if we need it for multiprocessing tests
# DATABASES = {
# 'default': {
# 'ENGINE': 'django.db.backends.postgresql',
Expand Down

0 comments on commit e480f1b

Please sign in to comment.