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

Update index.rst #289

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
16 changes: 16 additions & 0 deletions install/advanced/core/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -846,6 +846,22 @@ Serving {“geonode”, “geoserver”} via NGINX
cheaper-busyness-backlog-alert = 16 ; Spawn emergency workers if more than this many requests are waiting in the queue
cheaper-busyness-backlog-step = 2 ; How many emergency workers to create if there are too many requests in the queue

Note
...........................

Since this is a local environment, Geoserver and Geonode should be able to reach the default MEDIA_ROOT. We need to change the FILE_UPLOAD_DIRECTORY_PERMISSIONS and FILE_UPLOAD_PERMISSIONS in settings.py to make the folder accessible to both.
Copy link
Contributor

Choose a reason for hiding this comment

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

HI @Raji-Ahmad
I would keep it more hipotetical, since everyone can have a different environment.
What do you think about something like this?

Geoserver and Geonode should be able to reach the MEDIA_ROOT. If needed, is possible to change the FILE_UPLOAD_DIRECTORY_PERMISSIONS and FILE_UPLOAD_PERMISSIONS in settings.py to make the folder and the file accessible to both.

Copy link
Author

Choose a reason for hiding this comment

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

Yes, that's better. Only change permission when necessary.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can you please update the pr? So I can approve and merge it to master


.. code-block:: shell

cd /opt/geonode

sudo vim geonode/settings.py

# append this to the file

FILE_UPLOAD_DIRECTORY_PERMISSIONS = 0o777
FILE_UPLOAD_PERMISSIONS = 0o777

.. code-block:: shell

# Enable the GeoNode UWSGI config
Expand Down