charts icon indicating copy to clipboard operation
charts copied to clipboard

[common] Allow custom postgresGID on cnpg definition.

Open nlaille opened this issue 1 year ago • 4 comments

Is your feature request related to a problem?

I need to be able to set postgresGID in cluster spec for CloudNative PG clusters, otherwise CloudNative PG clusters' pods will inherit the default value (26). I am using NFS (synology) as storage provider, and the GID 26 doesn't exists and cannot be created.

Describe the solution you'd like

Allow custom postgresGID on cnpg definition.

ClusterSpec for CNPG : https://cloudnative-pg.io/documentation/1.15/api_reference/#ClusterSpec

Describe alternatives you've considered

Create CNPG cluster outside of charts, however some charts (authelia) are relying on cnpg values and cannot be overriden.

Additional context

No response

I've read and agree with the following

  • [X] I've checked all open and closed issues and my request is not there.
  • [X] I've checked all open and closed pull requests and my request is not there.

nlaille avatar Jul 10 '24 08:07 nlaille

Actually, preferably we have it follow the .Values.securityContext setting unless overridden. So it shouldn't be using 26 in the first place.

Though I would HEAVILY advice running postgresql on a NFS backend.

PrivatePuffin avatar Jul 10 '24 09:07 PrivatePuffin

@Ornias1993 , I may have missed something, but even with securityContext set, the cnpg clusters' pods won't inherit it. It seems that the only way to change the securityContext of cnpg clusters' pods is with postgresUID and postgresGID.

nlaille avatar Jul 10 '24 10:07 nlaille

@Ornias1993 , I may have missed something, but even with securityContext set, the cnpg clusters' pods won't inherit it. It seems that the only way to change the securityContext of cnpg clusters' pods is with postgresUID and postgresGID.

That's why I was saying we should implement that instead.

PrivatePuffin avatar Jul 10 '24 11:07 PrivatePuffin

Locked as no use input is required.

PrivatePuffin avatar Jul 10 '24 11:07 PrivatePuffin

This issue is locked to prevent necro-posting on closed issues. Please create a new issue or contact staff on discord of the problem persists

truecharts-admin avatar Feb 09 '25 00:02 truecharts-admin