etcd icon indicating copy to clipboard operation
etcd copied to clipboard

Add etcd_server_range_duration_seconds metrics

Open lavacat opened this issue 2 years ago • 6 comments

What would you like to be added?

We already have etcd_server_apply_duration_seconds metrics.

But Range doesn't go through Applier. The only metrics we have about Range timing is grpc_server_handled_total{grpc_method="Range"} provided by grpc_middleware. Problem is that this grpc metric also includes timing of linearizableReadNotify.

Please add etcd_server_range_duration_seconds in txn.Range

Why is this needed?

better observability

lavacat avatar Nov 05 '23 09:11 lavacat

I would like to work on this

RaviHari avatar Nov 07 '23 22:11 RaviHari

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Mar 17 '24 12:03 stale[bot]

Discussed during sig-etcd triage meeting. This would be a good candidate for a new contributor. The original pr #16902 was abandoned by the previous contributor so we need to recreate this, and add a test.

jmhbnz avatar May 09 '24 18:05 jmhbnz

Hello @jmhbnz. I'd like to take a crack at this one. I'm a new contributor looking for mentorship.

thedtripp avatar May 10 '24 00:05 thedtripp

Hello @jmhbnz. I'd like to take a crack at this one. I'm a new contributor looking for mentorship.

Thanks for volunteering @thedtripp! Assigned to you. Please start by recreating the earlier proposed changes from #16902. Then we need a second commit to add a basic test verifying the new metric works. Let us know how you go and if you have any questions 🙏🏻

/assign @thedtripp

jmhbnz avatar May 10 '24 08:05 jmhbnz

@jmhbnz I made a PR: https://github.com/etcd-io/etcd/pull/17983

thedtripp avatar May 11 '24 01:05 thedtripp

@jmhbnz @ivanvc @ahrtr Thank you for guiding me through that first PR. Any suggestions on where I should focus for my next contribution?

thedtripp avatar May 18 '24 22:05 thedtripp

@jmhbnz @ivanvc @ahrtr Thank you for guiding me through that first PR. Any suggestions on where I should focus for my next contribution?

Most welcome! For a much quicker next contribution we need to update our website community page https://github.com/etcd-io/website/issues/850. If you're interested please feel free to leave a comment and I will assign it to you. Alternatively please take a look at our help wanted issues.

jmhbnz avatar May 18 '24 22:05 jmhbnz

@jmhbnz @ivanvc @ahrtr Thank you for guiding me through that first PR. Any suggestions on where I should focus for my next contribution?

Most welcome! For a much quicker next contribution we need to update our website community page etcd-io/website#850. If you're interested please feel free to leave a comment and I will assign it to you. Alternatively please take a look at our help wanted issues.

Sure! I'm happy to work on this.

thedtripp avatar May 18 '24 22:05 thedtripp

Hey Team, Is this issue ready to be closed or does it need further action?

thedtripp avatar Jun 27 '24 22:06 thedtripp

I'm closing this, as it was completed by https://github.com/etcd-io/etcd/pull/17983. Thanks, @thedtripp. Please re-open it @ahrtr / @jmhbnz if you think we need to backport this.

ivanvc avatar Jun 27 '24 22:06 ivanvc