opentelemetry-collector-contrib icon indicating copy to clipboard operation
opentelemetry-collector-contrib copied to clipboard
verified publisher icon open-telemetry

[exporter/tinybird] New exporter first PR

Open jordivilaseca opened this issue 6 months ago • 1 comments

Description

Add initial PR for Tinybird Exporter

Link to tracking issue

Related to #40475

Testing

Only config tests for now. No exporter tests because it doesn't have any logic yet.

Documentation

  • Basic README.md
  • chloggen

jordivilaseca avatar Jun 13 '25 13:06 jordivilaseca

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: jordivilaseca / name: Jordi Vilaseca (9a3f6a51ebd7daa295c2c97ef7cc145b59ff2156, b5ff2826add5514f22c4ece2f8ccddeaed6b49d3, 97d3818758e99f31a2b64a08202d478005b23b8e, 126d062340813ca34814591ef045a2f65b88a5f6, 833c74ec3d184bd3bcc3729f76d08c26718c06c6, 978a44eeb339c9f7b68f8d522c76b7bcd6ceccf7, 6e3cfd11b825064a74de39a9b43d8435adbb24b6, b5a572f0b17ce106d6bab83522e5644446bc58ee, ec0055ef72ecfe0db42f6cfcbae3d7a5a108f6d9, 2e919dea0fc5a32919d133fcb743a43b7fb72b3b, cb6994f378306ee087b26a2b0b479d68cc6d4b37, ff3710a626610a19af6ae2ff683fa16fc808a2f9, 3bde2bc28fc914d35b24a63c3cd65ce2a0310d99, 43a8a9bafd1ec062cc1b481a45f5256e4844d19c, 6af9ff56d7f0277ad1be62cbae57d0e8429590c8, 7839f797e304b632ddc1f4f70bc0edd28ec2fe9d, 1b5dc2b445ec18191d0f70cd7ed334d182e9867b, 980ffa0ec54982378adea9b252029278f0a8df9e, 0463c2aaa75e48f4b009c224b6c955f049e415d3, cf753f5a70cc19d4c8ab75659d0ced28494fd4ee, 7df20c500bfc3819ceb2370e0bbf511c6ab4f83b, eaaf3d58917f5c5dfc67ce0df022c66186e60d1a, b4c415753af5381adccc84a8f173492c1b303075, 70640625d8b689d16a43f068f81f896295c13b01)

linux-foundation-easycla[bot] avatar Jun 13 '25 13:06 linux-foundation-easycla[bot]

@jordivilaseca There are a bunch of CI failures related to one of our linters:

Error: exporter/tinybirdexporter/exporter.go:19:18: unused-parameter: parameter 'cfg' seems to be unused, consider removing or renaming it as _ (revive)
func newExporter(cfg component.Config, set exporter.Settings) (*tinybirdExporter, error) {
                 ^
Error: exporter/tinybirdexporter/exporter.go:23:34: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (e *tinybirdExporter) start(ctx context.Context, host component.Host) error {
                                 ^
Error: exporter/tinybirdexporter/exporter.go:27:39: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (e *tinybirdExporter) pushTraces(ctx context.Context, td ptrace.Traces) error {
                                      ^
Error: exporter/tinybirdexporter/exporter.go:31:40: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (e *tinybirdExporter) pushMetrics(ctx context.Context, md pmetric.Metrics) error {
                                       ^
Error: exporter/tinybirdexporter/exporter.go:35:37: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
func (e *tinybirdExporter) pushLogs(ctx context.Context, ld plog.Logs) error {
                                    ^

Can you address those? Since this is your first PR I need to manually approve CI each time so feel free to ping me if you need me to trigger CI

mx-psi avatar Jun 16 '25 11:06 mx-psi

Thanks! Can you add your Github usernames to https://github.com/open-telemetry/opentelemetry-collector-contrib/commits/main/cmd/githubgen/allowlist.txt ? I will put a reminder in one month to review this once you are GH members and remove you from this list

mx-psi avatar Jun 26 '25 20:06 mx-psi