Skip to content

Increment connect_to_node_attempts_total during dial attempts#36633

Merged
rosstimothy merged 1 commit intomasterfrom
tross/node_metrics
Jan 16, 2024
Merged

Increment connect_to_node_attempts_total during dial attempts#36633
rosstimothy merged 1 commit intomasterfrom
tross/node_metrics

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

@rosstimothy rosstimothy commented Jan 12, 2024

During refactoring performed in #18370 the counter was moved but never used outside of registration. This restores incrementing the counter for dial attempts and improves error reporting in traces in the router.

changelog: Ensure connect_to_node_attempts_total is always incremented when dialing hosts

During refactoring performed in #18370 the counter was moved but
never used outside of registration. This restores incrementing the
counter for dial attempts and improves error reporting in traces
in the router.
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from rudream January 15, 2024 10:06
@rosstimothy rosstimothy added this pull request to the merge queue Jan 16, 2024
Merged via the queue into master with commit ab9fe0e Jan 16, 2024
@rosstimothy rosstimothy deleted the tross/node_metrics branch January 16, 2024 14:30
@public-teleport-github-review-bot
Copy link
Copy Markdown

@rosstimothy See the table below for backport results.

Branch Result
branch/v12 Create PR
branch/v13 Create PR
branch/v14 Create PR
branch/v15 Create PR

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants