Skip to content

Add securityContext with fsGroup to GeoServer Helm chart - #89

Open
Bskr-P wants to merge 1 commit into
developfrom
geoserver-chart-with-securityContext
Open

Add securityContext with fsGroup to GeoServer Helm chart#89
Bskr-P wants to merge 1 commit into
developfrom
geoserver-chart-with-securityContext

Conversation

@Bskr-P

@Bskr-P Bskr-P commented Jul 18, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@Bskr-P
Bskr-P requested review from lgkgh and tharanathkartoza and removed request for lgkgh July 18, 2024 19:24
@dunn

dunn commented Sep 12, 2024

Copy link
Copy Markdown

i mentioned this last time but the UID and GID are configurable with environment variables, so this shouldn't be hardcoded to a value that might be overridden:

https://github.com/kartoza/docker-geoserver/blob/57b998ca564d59d0c089ff25db9e91b692f7c669/.env#L14-L15

@NyakudyaA

Copy link
Copy Markdown
Contributor

i mentioned this last time but the UID and GID are configurable with environment variables, so this shouldn't be hardcoded to a value that might be overridden:

https://github.com/kartoza/docker-geoserver/blob/57b998ca564d59d0c089ff25db9e91b692f7c669/.env#L14-L15

@dunn Please either raise a PR or make the suggested changes in a review and they will get resolved

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