feature/cm-metrics #4

Merged
Ghost merged 5 commits from feature/cm-metrics into rc 2025-02-27 14:13:46 +03:00
Collaborator

▶ самоопределение типа для Value (я научил его понимать тегированный ли он массив или нет )
▶ отрефакторил, убрал ненужное, оставил/добавил нужное сейчас и в будущем
▶ написал механизм регистрации метрики с ее пересохранением если уже была

ea0a9ba5ff31e9664acdd93916df472a9ace42b9e4870a4422e6f8d329901274f48127657f9af173a6bbc82130c56c19

▶ самоопределение типа для Value (я научил его понимать тегированный ли он массив или нет ) ▶ отрефакторил, убрал ненужное, оставил/добавил нужное сейчас и в будущем ▶ написал механизм регистрации метрики с ее пересохранением если уже была ea0a9ba5ff31e9664acdd93916df472a9ace42b9e4870a4422e6f8d329901274f48127657f9af173a6bbc82130c56c19
DmitriyA was assigned by VladislavD 2025-02-27 13:32:11 +03:00
Ghost was assigned by VladislavD 2025-02-27 13:32:11 +03:00
VladislavD added 5 commits 2025-02-27 13:32:12 +03:00
VladislavD requested review from Ghost 2025-02-27 13:35:41 +03:00
VladislavD requested review from DmitriyA 2025-02-27 13:35:41 +03:00
DmitriyA approved these changes 2025-02-27 13:35:43 +03:00
Ghost approved these changes 2025-02-27 14:10:09 +03:00
@ -8,2 +7,2 @@
use prometheus::{Encoder, Registry, TextEncoder, Gauge};
use std::sync::{Arc, MutexGuard};
use prometheus::{ Encoder, Gauge, Registry, TextEncoder};
use std::sync::{ Arc, MutexGuard };

Почему крейты идут через через \n? разве импорты не должны идти одним рядом друг за другом?

Почему крейты идут через через \n? разве импорты не должны идти одним рядом друг за другом?
Poster
Collaborator

правила оформления кода - универсальные для всех программ написанных на расте, эти меры вшиты в во все бьютифаеры и рефакторящие код утилиты

правила оформления кода - универсальные для всех программ написанных на расте, эти меры вшиты в во все бьютифаеры и рефакторящие код утилиты
VladislavD marked this conversation as resolved
Ghost reviewed 2025-02-27 14:10:45 +03:00
@ -19,3 +17,3 @@
info!("post on /update");
// let resp = Response::new("body");
// println!("{:?}", request);
// debug!("{:?}", request);

Что значит debug! ? Для чего это необходимо?

Что значит debug! ? Для чего это необходимо?
Poster
Collaborator

трасировка кода, не больше

трасировка кода, не больше
Ghost reviewed 2025-02-27 14:11:58 +03:00
@ -18,7 +16,7 @@ pub async fn update_metrics(
) -> impl IntoResponse {
info!("post on /update");
// let resp = Response::new("body");

По-хорошему не закидывать такие комментарии на ветку

По-хорошему не закидывать такие комментарии на ветку
Ghost reviewed 2025-02-27 14:12:41 +03:00
@ -97,0 +115,4 @@
))
},
}
// use prometheus::opts;

По хорошему эти комментарии тоже не добавлять

По хорошему эти комментарии тоже не добавлять
Ghost reviewed 2025-02-27 14:13:21 +03:00
@ -97,3 +149,4 @@
},
}
Ok(())
// registry.gather()

То же что и выше, все комментарии лучше убрать, которые не относятся к документации

То же что и выше, все комментарии лучше убрать, которые не относятся к документации
Ghost approved these changes 2025-02-27 14:13:26 +03:00
Ghost merged commit 442c937e06 into rc 2025-02-27 14:13:46 +03:00
Sign in to join this conversation.
No reviewers
No Label
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: deployer3000/prometheus-exporter#4
There is no content yet.