feat/ceph_validate_health #121

Merged
wjro merged 3 commits from feat/ceph_validate_health into master 2025-08-25 19:32:43 +00:00
Owner
No description provided.
wjro added 2 commits 2025-08-25 17:32:01 +00:00
mod.rs
Some checks failed
Run Check Script / check (pull_request) Failing after 20s
65cc9befeb
wjro added 1 commit 2025-08-25 17:33:41 +00:00
fix: cargo fmt
All checks were successful
Run Check Script / check (pull_request) Successful in 1m0s
a9fe4ab267
johnride approved these changes 2025-08-25 18:46:33 +00:00
johnride left a comment
Owner

LGTM. Ca prendra des tests pour ces features la plus tot que tard mais on n'a pas encore toute l'infra mise en place malheureusement.

LGTM. Ca prendra des tests pour ces features la plus tot que tard mais on n'a pas encore toute l'infra mise en place malheureusement.
@ -0,0 +43,4 @@
topology: &T,
) -> Result<Outcome, InterpretError> {
let client = topology.k8s_client().await.unwrap();
self.verify_ceph_toolbox_exists(client.clone()).await?;
Owner

Ca me fait penser, et d'ailleurs ça intéressera @letian :

Je pense qu'on peut set du contexte dans le thread d'exécution Tokio. Ça pourrait être utile pour l'instrumentation.

Pourquoi j'y pense ici c'est qu'on pourrait transformer verify_ceph_toolbox en ensure_ceph_toolbox et lancer le score qui s'occupe de la lever. Cote UX actuellement on voit des logs apparaitre, et cote code nous on n'a pas la trace de qui a parti ca pourquoi. Mais en injectant du "metadata" dans le thread on pourrait faire en sorte qu'un score qui s'execute par une nouvelle task qui a un execution context qui a l'info suffisante pour de la bonne instrumentation (genre, c'est qui mon parent).

Just food for thought.

Ca me fait penser, et d'ailleurs ça intéressera @letian : Je pense qu'on peut set du contexte dans le thread d'exécution Tokio. Ça pourrait être utile pour l'instrumentation. Pourquoi j'y pense ici c'est qu'on pourrait transformer `verify_ceph_toolbox` en `ensure_ceph_toolbox` et lancer le score qui s'occupe de la lever. Cote UX actuellement on voit des logs apparaitre, et cote code nous on n'a pas la trace de qui a parti ca pourquoi. Mais en injectant du "metadata" dans le thread on pourrait faire en sorte qu'un score qui s'execute par une nouvelle task qui a un execution context qui a l'info suffisante pour de la bonne instrumentation (genre, c'est qui mon parent). Just food for thought.
wjro merged commit 3ca31179d0 into master 2025-08-25 19:32:43 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 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: NationTech/harmony#121
No description provided.