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

pcaseiro · 2024-10-01T16:37:11+02:00

pcaseiro commented

2024-10-01 16:37:11 +02:00

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

feat(postgres): adding pgbouncer support for cnpg component 8eddcc8fc7

Mandatory for large scale deployements

pcaseiro added 1 commit 2024-10-04 10:02:01 +02:00

Remove pgBouncer for hydra cc78d7afb7

pcaseiro added 1 commit 2024-10-04 10:05:06 +02:00

Remove pgBouncer for hydra 7473bbe603

pcaseiro added 1 commit 2024-10-04 10:13:25 +02:00

Remove pgBouncer for hydra 1a6a489d5e

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

lgourvenec commented

2024-10-07 15:40:43 +02:00

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.

👍 1

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

pcaseiro commented

2024-10-09 09:46:51 +02:00

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 referenced this issue from a commit

2024-10-09 10:00:04 +02:00

Merge pull request 'feat(postgres): adding max_conns hydra parameter support for cnpg component' (#47) from feat-hydra-pooler into unstable

vfebvre deleted branch feat-hydra-pooler

2024-10-09 10:00:06 +02:00

Sign in to join this conversation.

No reviewers