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

doc for nginx and uwsgi #180

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

doc for nginx and uwsgi #180

wants to merge 3 commits into from

Conversation

lordzappo
Copy link

This doc looks great in dillinger.io but doesnt seem to work in my github fork...you may need to adjust the markdown further.

@coveralls
Copy link

coveralls commented Apr 7, 2017

Coverage Status

Coverage remained the same at 75.749% when pulling dd82193 on lordzappo:master into a98f466 on wooey:master.

Copy link
Member

@Chris7 Chris7 left a comment

Choose a reason for hiding this comment

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

Thanks for taking the time to write this up! You should add yourself to the contributors document as well!

uwsgi_pass unix:/PATH/TO/[socket_name from uwsgi conf];
uwsgi_read_timeout 300;
}
}
Copy link
Member

Choose a reason for hiding this comment

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

Extra bracket


1. add a domain name to ALLOWED_HOSTS in django_settings.py in the settings dir of your project
2. specify a static dir you like in STATIC_ROOT- default is OK
3. make sure the entire directory tree is chmodded and chgrouped appropriately for the webserver user..sqlite db needs to be writable by them, etc.
Copy link
Member

Choose a reason for hiding this comment

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

Can you specify if the user is using sqlite.db -- sqlite is not a good thing for a web server to be running off so I'd rather not encourage it.

----------------------------------------------------------

1. add a domain name to ALLOWED_HOSTS in django_settings.py in the settings dir of your project
2. specify a static dir you like in STATIC_ROOT- default is OK
Copy link
Member

Choose a reason for hiding this comment

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

Can you add that this is found under user_settings.py?

1. add a domain name to ALLOWED_HOSTS in django_settings.py in the settings dir of your project
2. specify a static dir you like in STATIC_ROOT- default is OK
3. make sure the entire directory tree is chmodded and chgrouped appropriately for the webserver user..sqlite db needs to be writable by them, etc.
4. add these config blocks to nginx, or use these as your files if only thing hosted
Copy link
Member

Choose a reason for hiding this comment

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

Can you nest the config blocks under each respective part?

Production installation of Wooey utilizing NGINX and uWSGI
----------------------------------------------------------

1. add a domain name to ALLOWED_HOSTS in django_settings.py in the settings dir of your project
Copy link
Member

Choose a reason for hiding this comment

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

This is actually settable in user_settings.py

@@ -48,3 +48,63 @@ Your server can then be run by the simple command:
honcho start

On Windows, the ``--beat`` option may not be supported.

Production installation of Wooey utilizing NGINX and uWSGI
Copy link
Member

Choose a reason for hiding this comment

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

Could you change this to Through NGINX and uWSGI to be in sync with the rest of this page. This also doesn't cover production things like turning on SSL/using a separate database so it's not accurate to claim 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.

3 participants