at first sight I do not like much keep plumbing things down, can we ignore the metric without having to keep passing parameters just for that?

at first sight I do not like much keep plumbing things down, can we ignore the metric without having to keep passing parameters just for that?

@aojea this metric label is required to ignore metrics generated from other pods. I have modified the approach and now returning an extra variable instead of passing it down. Could you please take a look and suggest an approach on how we can improve this?

@michaelasp, @aojea I hope the changes and the rationale make sense now. I'll proceed with merging assuming you were fine with the current state. If there's additional changes, we can follow up separately.

@michaelasp, @aojea I hope the changes and the rationale make sense now. I'll proceed with merging assuming you were fine with the current state. If there's additional changes, we can follow up separately.

it is ok, the logic is not encapsulated into the methods, and we are not spreading the metrics details