feat(postgres): adding max_conns hydra parameter support for cnpg component #47

Merged
vfebvre merged 1 commits from feat-hydra-pooler into unstable 2024-10-09 10:00:03 +02:00
Owner

Mandatory for large scale deployements.

Using standard cnpg secret for passwords and users.

The standard secret is better, have more information and don't imply to store a password in the kustomization.

Mandatory for large scale deployements. Using standard cnpg secret for passwords and users. The standard secret is better, have more information and don't imply to store a password in the kustomization.
pcaseiro added 1 commit 2024-10-01 16:37:11 +02:00
pcaseiro added 1 commit 2024-10-04 10:02:01 +02:00
pcaseiro added 1 commit 2024-10-04 10:05:06 +02:00
pcaseiro added 1 commit 2024-10-04 10:13:25 +02:00
pcaseiro force-pushed feat-hydra-pooler from 1a6a489d5e to a5c9c733f6 2024-10-04 10:57:12 +02:00 Compare
pcaseiro changed title from feat(postgres): adding pgbouncer support for cnpg component to feat(postgres): adding max_conns hydra parameter support for cnpg component 2024-10-04 10:57:44 +02:00
Owner

Revue : OK
Tests : OK

Testé avec le mse; Pas de hydra-postgres-superuser car pas de enableSuperuserAccess ; un hydra-postgres-app est ajouté et si on supprime l'ancien secret hydra-postgres-user, tout fonctionne.
En revanche, le secret hydra-postgres-admin ne semble pas permettre de se connecter, il faudrait set "enableSuperuserAccess" à true dans le Cluster si on veut accéder au cluster et on peut supprimer cet ancien secret (qui n'a probablement jamais servi)
En ce qui concerne le MAX_CONNS, tout ce que je peux dire 'est qu'il est bien dans le DSN dans l'env d'hydra.

Revue : OK Tests : OK Testé avec le mse; Pas de hydra-postgres-superuser car pas de enableSuperuserAccess ; un hydra-postgres-app est ajouté et si on supprime l'ancien secret hydra-postgres-user, tout fonctionne. En revanche, le secret hydra-postgres-admin ne semble pas permettre de se connecter, il faudrait set "enableSuperuserAccess" à true dans le Cluster si on veut accéder au cluster et on peut supprimer cet ancien secret (qui n'a probablement jamais servi) En ce qui concerne le MAX_CONNS, tout ce que je peux dire 'est qu'il est bien dans le DSN dans l'env d'hydra.
lgourvenec requested review from lgourvenec 2024-10-07 15:40:58 +02:00
lgourvenec approved these changes 2024-10-07 15:42:11 +02:00
Author
Owner

Revue : OK
Tests : OK

Testé avec le mse; Pas de hydra-postgres-superuser car pas de enableSuperuserAccess ; un hydra-postgres-app est ajouté et si on supprime l'ancien secret hydra-postgres-user, tout fonctionne.
En revanche, le secret hydra-postgres-admin ne semble pas permettre de se connecter, il faudrait set "enableSuperuserAccess" à true dans le Cluster si on veut accéder au cluster et on peut supprimer cet ancien secret (qui n'a probablement jamais servi)
En ce qui concerne le MAX_CONNS, tout ce que je peux dire 'est qu'il est bien dans le DSN dans l'env d'hydra.

Le MAX_CONNS a été testé dans l'env "bench" et ça marche vraiment très bien.

> Revue : OK > Tests : OK > > Testé avec le mse; Pas de hydra-postgres-superuser car pas de enableSuperuserAccess ; un hydra-postgres-app est ajouté et si on supprime l'ancien secret hydra-postgres-user, tout fonctionne. > En revanche, le secret hydra-postgres-admin ne semble pas permettre de se connecter, il faudrait set "enableSuperuserAccess" à true dans le Cluster si on veut accéder au cluster et on peut supprimer cet ancien secret (qui n'a probablement jamais servi) > En ce qui concerne le MAX_CONNS, tout ce que je peux dire 'est qu'il est bien dans le DSN dans l'env d'hydra. Le MAX_CONNS a été testé dans l'env "bench" et ça marche vraiment très bien.
vfebvre approved these changes 2024-10-09 09:58:45 +02:00
vfebvre merged commit a0ff37edf6 into unstable 2024-10-09 10:00:03 +02:00
vfebvre deleted branch feat-hydra-pooler 2024-10-09 10:00:06 +02:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: CadolesKube/sso-kustom#47
No description provided.