Skip to content
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

Add etcd_server_range_duration_seconds metrics #16866

Closed
lavacat opened this issue Nov 5, 2023 · 11 comments
Closed

Add etcd_server_range_duration_seconds metrics #16866

lavacat opened this issue Nov 5, 2023 · 11 comments

Comments

@lavacat
Copy link

lavacat commented Nov 5, 2023

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 lavacat added type/feature contribfest Issues for Contribfest workshop happening during KubeCon labels Nov 5, 2023
@RaviHari
Copy link
Contributor

RaviHari commented Nov 7, 2023

I would like to work on this

Copy link

stale bot commented Mar 17, 2024

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 stale bot added the stale label Mar 17, 2024
@jmhbnz jmhbnz added help wanted good first issue area/observability and removed stale contribfest Issues for Contribfest workshop happening during KubeCon labels May 9, 2024
@jmhbnz
Copy link
Member

jmhbnz commented May 9, 2024

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.

@thedtripp
Copy link
Member

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

@jmhbnz
Copy link
Member

jmhbnz commented May 10, 2024

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

@thedtripp
Copy link
Member

@jmhbnz
I made a PR: #17983

@thedtripp
Copy link
Member

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

@jmhbnz
Copy link
Member

jmhbnz commented May 18, 2024

@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.

@thedtripp
Copy link
Member

@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
Copy link
Member

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

@ivanvc
Copy link
Member

ivanvc commented Jun 27, 2024

I'm closing this, as it was completed by #17983. Thanks, @thedtripp. Please re-open it @ahrtr / @jmhbnz if you think we need to backport this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

5 participants