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

Set PostgreSQL password in CircleCI build #2596

Merged
merged 1 commit into from
Feb 17, 2020

Conversation

reupen
Copy link
Contributor

@reupen reupen commented Feb 17, 2020

Description of change

The postgres Docker image now requires a password to be set. This change sets a password in the CircleCI build to meet that requirement.

See docker-library/postgres#681 for more information.

Checklist

  • Has a new newsfragment been created? Check changelog/README.md for instructions
  • Do any added or updated endpoints appear in the API documentation? See docs/Maintaining the API documentation.md for more details
  • Have any relevant search models been updated?
  • Have any relevant fixtures (fixtures/test_data.yaml) been updated?
  • Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
  • Has the admin site been updated (for new models, fields etc.)?
  • Has the README been updated (if needed)?

@reupen reupen force-pushed the bugfix/circleci-postgres-pw branch 5 times, most recently from 3f886a0 to fabb830 Compare February 17, 2020 10:23
@reupen reupen marked this pull request as ready for review February 17, 2020 10:26
@alixedi alixedi requested review from elcct and alixedi and removed request for sekharpanja, web-bert and alixedi February 17, 2020 10:32
The `postgres` Docker image now requires a password to be set. This change sets a password in the CircleCI build to meet that requirement.

See docker-library/postgres#681 for more information.
@reupen reupen merged commit 0e9d69c into develop Feb 17, 2020
@reupen reupen deleted the bugfix/circleci-postgres-pw branch February 17, 2020 11:10
@codecov
Copy link

codecov bot commented Feb 17, 2020

Codecov Report

Merging #2596 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2596   +/-   ##
========================================
  Coverage    97.83%   97.83%           
========================================
  Files          345      345           
  Lines        10663    10663           
  Branches      1040     1040           
========================================
  Hits         10432    10432           
  Misses         156      156           
  Partials        75       75

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b69d7dd...f35b407. Read the comment docs.

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.

4 participants