-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Derive macro to register struct of metrics #218
base: master
Are you sure you want to change the base?
Changes from all commits
4eec75d
537eb62
7105446
9ec4c61
2cea529
bc6a172
7ff4367
fc92deb
9bf89a0
4b1c002
f4c4a1a
b29615b
b2d3516
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
[package] | ||
name = "prometheus-client-derive-register" | ||
version = "0.1.0" | ||
edition = "2021" | ||
|
||
[dependencies] | ||
darling = "0.20.10" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would like to keep For users:
For maintainers:
It is not only How much work do you think would it be to implement this feature without There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Probably only an hour or two? It doesn't seem too difficult There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Great. Would you mind removing |
||
proc-macro2 = "1" | ||
quote = "1" | ||
syn = "2" | ||
|
||
[dev-dependencies] | ||
prometheus-client = { path = "../" } | ||
|
||
[lib] | ||
proc-macro = true | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,114 @@ | ||
use darling::{ast, util::Flag, FromDeriveInput, FromField}; | ||
use proc_macro::TokenStream; | ||
use quote::quote; | ||
use syn::{DeriveInput, Expr, Generics, Ident, Lit, Meta, Type}; | ||
|
||
#[derive(Debug, FromDeriveInput)] | ||
#[darling(attributes(register), supports(struct_named))] | ||
struct Register { | ||
ident: Ident, | ||
generics: Generics, | ||
data: ast::Data<(), RegisterField>, | ||
} | ||
|
||
#[derive(Debug, FromField)] | ||
#[darling(attributes(register), forward_attrs(doc))] | ||
struct RegisterField { | ||
ident: Option<Ident>, | ||
ty: Type, | ||
attrs: Vec<syn::Attribute>, | ||
skip: Flag, | ||
unit: Option<String>, | ||
name: Option<String>, | ||
help: Option<String>, | ||
} | ||
|
||
#[proc_macro_derive(Register, attributes(register))] | ||
pub fn derive_register(input: TokenStream) -> TokenStream { | ||
let ast: DeriveInput = syn::parse(input).unwrap(); | ||
let info = Register::from_derive_input(&ast).unwrap(); | ||
|
||
let name = info.ident; | ||
let (impl_generics, ty_generics, where_clause) = info.generics.split_for_impl(); | ||
|
||
let field_register = info | ||
.data | ||
.take_struct() | ||
.unwrap() | ||
.into_iter() | ||
.filter(|x| !x.skip.is_present()) | ||
.map(|field| { | ||
let mut help = String::new(); | ||
for attr in field.attrs { | ||
let path = attr.path(); | ||
if path.is_ident("doc") && help.is_empty() { | ||
if let Some(doc) = extract_doc_comment(&attr.meta) { | ||
help = doc.trim().to_string(); | ||
} | ||
} | ||
} | ||
|
||
if let Some(custom_help) = field.help { | ||
help = custom_help; | ||
} | ||
|
||
let ident = field.ident.unwrap(); | ||
let ty = field.ty; | ||
let name = if let Some(name) = field.name { | ||
name | ||
} else { | ||
ident.to_string() | ||
}; | ||
|
||
let unit = if let Some(unit) = field.unit { | ||
quote!(Some(::prometheus_client::registry::Unit::Other(#unit.to_string()))) | ||
} else { | ||
quote!(None) | ||
}; | ||
|
||
quote! { | ||
<#ty as ::prometheus_client::registry::RegisterField>::register_field( | ||
&self.#ident, | ||
#name, | ||
#help, | ||
#unit, | ||
registry, | ||
) | ||
} | ||
}); | ||
|
||
quote! { | ||
impl #impl_generics ::prometheus_client::registry::Register for #name #ty_generics #where_clause { | ||
fn register(&self, registry: &mut ::prometheus_client::registry::Registry) { | ||
#(#field_register);* | ||
} | ||
} | ||
|
||
impl #impl_generics ::prometheus_client::registry::RegisterField for #name #ty_generics #where_clause { | ||
fn register_field<N: ::std::convert::Into<::std::string::String>, H: ::std::convert::Into<::std::string::String>>( | ||
&self, | ||
name: N, | ||
help: H, | ||
unit: Option<::prometheus_client::registry::Unit>, | ||
registry: &mut ::prometheus_client::registry::Registry) | ||
{ | ||
let name = name.into(); | ||
let mut registry = registry.sub_registry_with_prefix(name); | ||
<Self as ::prometheus_client::registry::Register>::register(&self, &mut registry); | ||
} | ||
} | ||
}.into() | ||
} | ||
|
||
fn extract_doc_comment(meta: &Meta) -> Option<String> { | ||
let Meta::NameValue(nv) = meta else { | ||
return None; | ||
}; | ||
let Expr::Lit(lit) = &nv.value else { | ||
return None; | ||
}; | ||
let Lit::Str(lit_str) = &lit.lit else { | ||
return None; | ||
}; | ||
Some(lit_str.value()) | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,60 @@ | ||
use prometheus_client::{ | ||
encoding::text::encode, | ||
metrics::{counter::Counter, gauge::Gauge}, | ||
registry::{Register, RegisterDefault, Registry}, | ||
}; | ||
|
||
#[derive(Register, Default)] | ||
struct Metrics { | ||
/// This is my counter | ||
my_counter: Counter, | ||
nested: NestedMetrics, | ||
#[register(skip)] | ||
skipped: Counter, | ||
#[register(unit = "bytes")] | ||
custom_unit: Counter, | ||
#[register(name = "my_custom_name")] | ||
custom_name: Counter, | ||
/// This will get ignored | ||
#[register(help = "my custom help")] | ||
custom_help: Counter, | ||
} | ||
|
||
#[derive(Register, Default)] | ||
struct NestedMetrics { | ||
/// This is my gauge | ||
my_gauge: Gauge, | ||
} | ||
|
||
#[test] | ||
fn basic_flow() { | ||
let mut registry = Registry::default(); | ||
|
||
let metrics = Metrics::register_default(&mut registry); | ||
|
||
metrics.my_counter.inc(); | ||
metrics.nested.my_gauge.set(23); | ||
|
||
// Encode all metrics in the registry in the text format. | ||
let mut buffer = String::new(); | ||
encode(&mut buffer, ®istry).unwrap(); | ||
|
||
let expected = "# HELP my_counter This is my counter.\n".to_owned() | ||
+ "# TYPE my_counter counter\n" | ||
+ "my_counter_total 1\n" | ||
+ "# HELP custom_unit_bytes .\n" | ||
+ "# TYPE custom_unit_bytes counter\n" | ||
+ "# UNIT custom_unit_bytes bytes\n" | ||
+ "custom_unit_bytes_total 0\n" | ||
+ "# HELP my_custom_name .\n" | ||
+ "# TYPE my_custom_name counter\n" | ||
+ "my_custom_name_total 0\n" | ||
+ "# HELP custom_help my custom help.\n" | ||
+ "# TYPE custom_help counter\n" | ||
+ "custom_help_total 0\n" | ||
+ "# HELP nested_my_gauge This is my gauge.\n" | ||
+ "# TYPE nested_my_gauge gauge\n" | ||
+ "nested_my_gauge 23\n" | ||
+ "# EOF\n"; | ||
assert_eq!(expected, buffer); | ||
} |
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -2,6 +2,8 @@ | |||||
//! | ||||||
//! See [`Registry`] for details. | ||||||
|
||||||
pub use prometheus_client_derive_register::*; | ||||||
|
||||||
use std::borrow::Cow; | ||||||
|
||||||
use crate::collector::Collector; | ||||||
|
@@ -389,3 +391,51 @@ | |||||
|
||||||
impl<T> Metric for T where T: crate::encoding::EncodeMetric + Send + Sync + std::fmt::Debug + 'static | ||||||
{} | ||||||
|
||||||
pub trait Register { | ||||||
Check failure on line 395 in src/registry.rs GitHub Actions / Check
Check failure on line 395 in src/registry.rs GitHub Actions / Check rustdoc intra-doc links
Check failure on line 395 in src/registry.rs GitHub Actions / Clippy
|
||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
I guess something to be registered would be a registrant. What do you think? Is that more or less intuitive? I am not a native speaker. |
||||||
fn register(&self, registry: &mut Registry); | ||||||
Check failure on line 396 in src/registry.rs GitHub Actions / Check
Check failure on line 396 in src/registry.rs GitHub Actions / Check rustdoc intra-doc links
Check failure on line 396 in src/registry.rs GitHub Actions / Clippy
|
||||||
} | ||||||
|
||||||
pub trait RegisterDefault { | ||||||
Check failure on line 399 in src/registry.rs GitHub Actions / Check
Check failure on line 399 in src/registry.rs GitHub Actions / Check rustdoc intra-doc links
Check failure on line 399 in src/registry.rs GitHub Actions / Clippy
|
||||||
fn register_default(registry: &mut Registry) -> Self; | ||||||
Check failure on line 400 in src/registry.rs GitHub Actions / Check
Check failure on line 400 in src/registry.rs GitHub Actions / Check rustdoc intra-doc links
Check failure on line 400 in src/registry.rs GitHub Actions / Clippy
|
||||||
} | ||||||
|
||||||
impl<T> RegisterDefault for T | ||||||
where | ||||||
T: Register + Default, | ||||||
{ | ||||||
fn register_default(registry: &mut Registry) -> Self { | ||||||
let this = Self::default(); | ||||||
this.register(registry); | ||||||
this | ||||||
} | ||||||
} | ||||||
Comment on lines
+399
to
+412
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Neat! |
||||||
|
||||||
pub trait RegisterField { | ||||||
Check failure on line 414 in src/registry.rs GitHub Actions / Check
Check failure on line 414 in src/registry.rs GitHub Actions / Check rustdoc intra-doc links
Check failure on line 414 in src/registry.rs GitHub Actions / Clippy
|
||||||
fn register_field<N: Into<String>, H: Into<String>>( | ||||||
Check failure on line 415 in src/registry.rs GitHub Actions / Check
Check failure on line 415 in src/registry.rs GitHub Actions / Check rustdoc intra-doc links
Check failure on line 415 in src/registry.rs GitHub Actions / Clippy
|
||||||
&self, | ||||||
name: N, | ||||||
help: H, | ||||||
unit: Option<Unit>, | ||||||
registry: &mut Registry, | ||||||
); | ||||||
} | ||||||
|
||||||
impl<T> RegisterField for T | ||||||
where | ||||||
T: Metric + Clone, | ||||||
{ | ||||||
fn register_field<N: Into<String>, H: Into<String>>( | ||||||
&self, | ||||||
name: N, | ||||||
help: H, | ||||||
unit: Option<Unit>, | ||||||
registry: &mut Registry, | ||||||
) { | ||||||
if let Some(unit) = unit { | ||||||
registry.register_with_unit(name, help, unit, self.clone()) | ||||||
} else { | ||||||
registry.register(name, help, self.clone()) | ||||||
} | ||||||
} | ||||||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we could as well do both encode and register in one derive crate? I am undecided. What is your rational for two crates?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just went off of the existing naming. I would be fine with one crate, but then I think it should be named just
prometheus-client-derive
. Also, should this be behind a feature so people don't have to compile syn if they don't need to?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prometheus-client-derive
sounds good.prometheus-client-derive
behind a feature sounds good to me.