Skip to content
This repository has been archived by the owner on Jul 29, 2022. It is now read-only.

Changes to make this run in OpenShift 3 #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

gregswift
Copy link

@gregswift gregswift commented Feb 7, 2018

For the most part I've got this running in OpenShift 3 using python 3.5 s2i.

  • icanhaz.py was not executable, so I updated that here
  • Upon trying to run icanhaz.py dependencies (flask) were missing and it will process a requirements.txt so I added one
  • Figure out how to configure bind using --host=0.0.0.0 See next item:
  • Decided to let gunicorn handle the serving and binding for openshift, so added to requirements.
    ** Had to set APP_MODULE environment variable in Deployment config to icanhaz:app (not represented here)
  • Add support for X-Fordwarded-For headers for running behind lb/proxy

Because icanhaz* requires system level packages, to get to fully featured it would likely require a custom Docker image. I didn't actually try running the existing version of this image.

@gregswift
Copy link
Author

@major would it help if I made the proxy fronting support a feature flag? I can definitely see why you wouldnt want it on your implementation

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

Successfully merging this pull request may close these issues.

1 participant