Skip to content

fix(client): sonde /healthz au lieu de /health - #220

Merged
BryanFRD merged 1 commit into
mainfrom
fix/probe-healthz
Aug 9, 2026
Merged

fix(client): sonde /healthz au lieu de /health#220
BryanFRD merged 1 commit into
mainfrom
fix/probe-healthz

Conversation

@BryanFRD

@BryanFRD BryanFRD commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Toute FerrVaultConnection pointant vers une API FerrVault >= 5.9 est bloquée en Unreachable: health check returned Not Found.

Le Probe sondait /health, une route qui n'existe pas : l'API expose /healthz (api/src/lib.rs:117). Vérifié contre l'instance dev (5.9.2) :

/health     404
/healthz    200
/v1/health  404

La PR #209 avait migré /v1/operator/secrets/reveal mais laissé le Probe de côté. Le test ne vérifiait pas le chemin appelé, d'où le passage inaperçu.

Changes

  • internal/ferrvault/client.goProbe sonde /healthz ; commentaire mis à jour.
  • internal/controller/ferrvaultconnection_controller.go — message de condition Reachable aligné.
  • internal/ferrvault/client_test.goTestProbe_RetriesOn5xx assert désormais le chemin, ce qui aurait attrapé la régression.

Testing

Les tests n'ont pas été exécutés : Go n'est pas disponible sur la machine où le correctif a été écrit. La CI est seule juge.

Le diagnostic, lui, est vérifié en direct : les codes HTTP ci-dessus viennent d'un appel réel à ferrvault-api dans ferrvault-dev.

Déploiement

L'opérateur déployé est en 5.0.6 alors que main est en 5.2.0 : le correctif /v1 de #209 n'est pas en production. Il faudra une release puis un bump d'image, sinon les connections resteront cassées.

Débloque ferrtrack-dev, et aussi FerrFleet / FerrGrowth / FerrGames / FerrLabs en dev, plus ferrvault-ghcr.

Copilot AI lite review requested due to automatic review settings August 9, 2026 09:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@BryanFRD
BryanFRD merged commit 6a300ef into main Aug 9, 2026
18 checks passed
@BryanFRD
BryanFRD deleted the fix/probe-healthz branch August 9, 2026 09:40
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.

2 participants