charts icon indicating copy to clipboard operation
charts copied to clipboard

add support to add shared_preload_libraries list in chart

Open kirankumar-grootan opened this issue 1 year ago • 4 comments

add support to add shared library instead of hardcoded time timescaledb lib alone https://github.com/cloudnative-pg/charts/blob/main/charts/cluster/templates/cluster.yaml#L51

kirankumar-grootan avatar Jun 12 '24 10:06 kirankumar-grootan

   shared_preload_libraries:
      {{- if eq .Values.type "timescaledb" }}
      - timescaledb
      {{- end }}
      {{- with .Values.cluster.postgresql.shared_preload_libraries }}
      {{ . | indent 6}}
      {{- end }}
    parameters: {{- toYaml .Values.cluster.postgresql.parameters | nindent 6 }}

we can change something like this

kirankumar-grootan avatar Jun 12 '24 10:06 kirankumar-grootan

Duplicate of #278, which I closed as I'm not sure I want to do this.

That being said, what is your use case? Change my mind.

itay-grudev avatar Jun 13 '24 08:06 itay-grudev

Hi @itay-grudev

Thank you for looking into this ticket. I'm closing this issue. We previously deployed PostgreSQL using the default PostgreSQL image from Docker Hub, which required manual installation of the pg_stat_statements extension to retrieve query time. However, with the CloudNative PG image, this functionality is included by default, allowing us to obtain query time information without any additional steps.

kirankumar-grootan avatar Jun 25 '24 09:06 kirankumar-grootan

Hi @itay-grudev , sorry to comment on a closed issue but I would like to push for this feature to be added to the chart. My use case is simple : deploy a CNPG Cluster for immich. It depends on the pgvectors extension which fortunately has its own CNPG docker image. However, I cannot use it with this chart as I have to modify the shared_preload_libraries parameter. Being able to change this value would be beneficial for this chart and would encourage the development of these kinds of Docker images.

cterence avatar Jul 22 '24 22:07 cterence

Yeah, just stumbled upon the same issue. Would be great to have this covered by the charts.

11qu1d avatar Aug 17 '24 13:08 11qu1d