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

You should now be able to push the app to cloud foundry #1

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

Conversation

moficodes
Copy link

Few of the problems were

  1. Not specifying a port. Flask chooses 5000 on run by default. Cloud foundry has a env variable to do that
  2. Requirements.txt not having all the dependencies. Things like numpy and urllib are available to your python env but not to cloudfoundry. For future rule of thumb use virtual env, that way you dont pollute your python environment.
  3. I added a Procfile that sends a command python server.py. Not sure if that was 100% necessary. but its working so not bothering with it.
  4. added a runtime.txt that defines the python version
  5. added buildpack to manifest.yml to select python. I think it was doing that automatically but just made it easier for cf
  6. Changed some version of the botcore, doubt that has to do anything though. I was just tackling the error messages.
  7. Oh and finally added a gitignore. like one should.

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.

1 participant