Skip to content

Comments

rw2sender-fix: moved to programatic approach#212

Open
bwplotka wants to merge 4 commits intofeature/rw2senderfixfrom
rw2sender-prog
Open

rw2sender-fix: moved to programatic approach#212
bwplotka wants to merge 4 commits intofeature/rw2senderfixfrom
rw2sender-prog

Conversation

@bwplotka
Copy link
Member

@bwplotka bwplotka commented Feb 20, 2026

Progresses #208

See an example use prometheus/prometheus#18121

This rewrites the framework so it can be used programmatically (go imported) and adds a sample test that shows how it's used.

Also:

  • Removed custom process target, too weird to use.
  • Simplified Makefile
  • Consolidated go.mod for receiver and sender for simplicity
  • Fixed HTML to not only say "receiver".
  • Tests are now defined in go files. (no test.go)
  • Removed looping for each sender. Now anyone can extend the test to do this.

cc @pipiland2612 @saswatamcode

Signed-off-by: bwplotka <bwplotka@gmail.com>
return labels.FormatOpenMetricsFloat(v)
}

func samplesTests() []Test {
Copy link
Member Author

Choose a reason for hiding this comment

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

FYI: Moved 1:1 from samples_test

Signed-off-by: bwplotka <bwplotka@gmail.com>
Signed-off-by: bwplotka <bwplotka@gmail.com>
Signed-off-by: bwplotka <bwplotka@gmail.com>
Copy link
Member

@saswatamcode saswatamcode left a comment

Choose a reason for hiding this comment

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

LGTM! This looks super clean!

Copy link
Contributor

@pipiland2612 pipiland2612 left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM

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.

3 participants