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

metrics: observed address mgr #2203

Closed
Tracked by #1356
p-shahi opened this issue Mar 20, 2023 · 2 comments · Fixed by #2246
Closed
Tracked by #1356

metrics: observed address mgr #2203

p-shahi opened this issue Mar 20, 2023 · 2 comments · Fixed by #2246
Assignees

Comments

@p-shahi
Copy link
Member

p-shahi commented Mar 20, 2023

Number of activated addresses: IP Version + Transport
Nat Type: (Unknown, Symmetric, Cone)

@sukunrt
Copy link
Member

sukunrt commented Mar 27, 2023

I'm tracking these from holepunch metrics. We are only interested in activated addresses. Tracking at observedAddressManager leads to a lot of book keeping like wasActivatedBefore isActivatedNow everywhere.

At the moment we are only interested in public address counts by ipversion and transport for holepunches, so my approach feels okay.

@sukunrt
Copy link
Member

sukunrt commented May 9, 2023

I'm tracking these from holepunch metrics. We are only interested in activated addresses. Tracking at observedAddressManager leads to a lot of book keeping like wasActivatedBefore isActivatedNow everywhere.

I have changed this strategy. It is better to track these from observed address manager. I don't remember what my reasoning was here, but I think there's a way to do this that's hopefully not too complicated.

@p-shahi p-shahi changed the title observed address mgr metrics: observed address mgr May 11, 2023
@marten-seemann marten-seemann linked a pull request Jun 5, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants