Skip to content

graphite: multiple servers support#540

Merged
patapenka-alexey merged 1 commit intomasterfrom
patapenka-alexey/tntp-6584-graphite-export
Mar 24, 2026
Merged

graphite: multiple servers support#540
patapenka-alexey merged 1 commit intomasterfrom
patapenka-alexey/tntp-6584-graphite-export

Conversation

@patapenka-alexey
Copy link
Contributor

@patapenka-alexey patapenka-alexey commented Mar 17, 2026

This patch adds the ability to send metrics to the multiple servers.

Before this patch, graphite.init() would find and kill any fibers named metrics_graphite_worker before creating a new fiber with the same name. This prevented the creation of multiple fibers for sending metrics.

Part of #TNTP-6584

@patapenka-alexey patapenka-alexey marked this pull request as ready for review March 17, 2026 20:03
@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-6584-graphite-export branch 4 times, most recently from c19b6cc to ce27ae1 Compare March 19, 2026 08:37
@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-6584-graphite-export branch 3 times, most recently from d655d13 to 377025d Compare March 19, 2026 12:45
@patapenka-alexey patapenka-alexey requested review from bigbes and oleg-jukovec and removed request for oleg-jukovec March 19, 2026 12:46
@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-6584-graphite-export branch 5 times, most recently from f6d93fc to 16041e8 Compare March 20, 2026 12:28
Copy link
Contributor

@olegrok olegrok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your patch! See my comments below.

@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-6584-graphite-export branch from 49efc48 to 7463a5a Compare March 23, 2026 12:04
@patapenka-alexey patapenka-alexey marked this pull request as draft March 23, 2026 12:51
@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-6584-graphite-export branch from a656b12 to 5139b43 Compare March 23, 2026 12:58
@patapenka-alexey patapenka-alexey marked this pull request as ready for review March 23, 2026 13:06
@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-6584-graphite-export branch from 5139b43 to 1f0e0b6 Compare March 23, 2026 17:10
This patch adds the ability to send metrics to the multiple servers.

Part of #TNTP-6584
@patapenka-alexey patapenka-alexey force-pushed the patapenka-alexey/tntp-6584-graphite-export branch from 1f0e0b6 to 09b586e Compare March 23, 2026 17:44
@patapenka-alexey patapenka-alexey merged commit de1de78 into master Mar 24, 2026
16 checks passed
@patapenka-alexey patapenka-alexey deleted the patapenka-alexey/tntp-6584-graphite-export branch March 24, 2026 05:23
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 this pull request may close these issues.

4 participants