Skip to content

Compose modernization and added ref to Stash pHash compat img. #591

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Compose modernization and added ref to Stash pHash compat img. #591

wants to merge 2 commits into from

Conversation

Leopere
Copy link
Collaborator

@Leopere Leopere commented Feb 1, 2023

  • removed redundant container_name:
  • removed deprecated link:
  • added image: stashapp/postgres:latest to postgres image.

@Leopere
Modernization and added ref to Stash pHash compat img.
a8b4c0b 
- removed redundant `container_name:`
- removed deprecated `link:`
- added image: stashapp/postgres:latest to postgres image.
@Leopere Leopere requested a review from InfiniteStash February 1, 2023 22:32
@Leopere Leopere changed the title Modernization and added ref to Stash pHash compat img. Compose modernization and added ref to Stash pHash compat img. Feb 1, 2023
@Leopere
Copy link
Collaborator Author

Leopere commented Feb 1, 2023

Going forward with these tweaks you could run docker compose build postgres && docker compose push postgres so long as you have push perms on the official repo until we get some gitops setup for it.

@Leopere
Following documentation on disabling sha crypto
d59a31e 
Adding an envvar that's supported by the upstream postgres:14.2 container to disable the sha crypto as per the README.md recommendation. Inevitably prior to a public release this is probably smart to disable as the additional crypto can't hurt to have enabled by default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@InfiniteStash InfiniteStash Awaiting requested review from InfiniteStash

1 more reviewer

@treyt02 treyt02 treyt02 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Leopere @treyt02