-
Notifications
You must be signed in to change notification settings - Fork 9
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
Feature: refector nginx deployment #177
Comments
i will postpone this issue to later release |
I'm about open an PR on optionally, based on ingress enabled or disabled, excluding What keeps me from always enabling |
current nginx deployed by this helm chart includes several tasks, which can also be implemented using the kubernetes build in ingress functionallity. These are artifacts comming from the migration process from the original docker-compose.yaml file in geonode/geonode repository towards geonode-k8s. The current tasks of the nginx deployment include:
pvc-{{ .Release.Name }}-geonode
)I suggest to implement proxies to geoserver and pycsw using the ingress functionality of kubernetes. further it reduces complexity of the nginx configuration file and gives more flexibility for the different endpoints within the installation. Anyway i would keep the default as it is.
Further the nginx deployment could get independent from the geonode initialization process by having a exclusiv volume for the static content. This the can be build by a init container in the nginx deployment only running
manage.py makestatics
. This change would be the first step away from the single RWX volume towards multiple RWO volumes.I think this could be implemented without having breaking changes with current installations
The text was updated successfully, but these errors were encountered: