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

workaround for #7 - "oc new-app" build failure on postgresql-client (… #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

based3
Copy link

@based3 based3 commented Apr 20, 2021

@based3
Copy link
Author

based3 commented Apr 20, 2021

the real fix working is requirements.txt/psycopg2-binary==2.8.6

you can use:
git clone https://github.com/based3/blog-django-py

@SudoBrendan
Copy link

@based3 - I ran into this and made that change as well (psycopg2-binary==2.8.6), and now have a working build! Would you mind pushing that change here so it can be merged? Thanks for your work debugging this.

@based3
Copy link
Author

based3 commented May 7, 2021

@SudoBrendan I am not the owner of the project, you should ask to @GrahamDumpleton https://github.com/GrahamDumpleton

@GrahamDumpleton
Copy link
Contributor

You will need to ask RH. I don't work at RH anymore. I am not even quite sure who is still there to ping. Maybe @ryanj can help.

Copy link
Member

@ryanj ryanj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ryanj
Copy link
Member

ryanj commented May 17, 2021

I don't have write access on the repo. I'll see if I can find someone to merge it

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