Skip to content

Otel testing improvements 5.8#4436

Merged
dsessler7 merged 2 commits intoCrunchyData:REL_5_8from
dsessler7:otel-testing-improvements-5.8
Feb 18, 2026
Merged

Otel testing improvements 5.8#4436
dsessler7 merged 2 commits intoCrunchyData:REL_5_8from
dsessler7:otel-testing-improvements-5.8

Conversation

@dsessler7
Copy link
Collaborator

Checklist:

  • Have you added an explanation of what your changes do and why you'd like them to be included?
  • Have you updated or added documentation for the change, as applicable?
  • Have you tested your changes on all related environments with successful results, as applicable?
    • Have you added automated tests?

Type of Changes:

  • New feature
  • Bug fix
  • Documentation
  • Testing enhancement
  • Other

What is the current behavior (link to any open issues here)?

No test for collector.AddToPod. Missing test cases for batches or detectors with attributes in TestEnablePostgresLogging.

What is the new behavior (if this is a feature change)?

  • Breaking change (fix or feature that would cause existing functionality to change)

This PR adds a Go test for collector.AddToPod function which gives us more collector package test coverage and ensures the Instrumentation configuration fields environmentVariables and files get tested. It also adds tests that ensure settings for batches and detectors with attributes get set properly in the collector configuration.

Other Information:

…ctor package TestEnablePostgresLogging test.
@dsessler7 dsessler7 force-pushed the otel-testing-improvements-5.8 branch from 8e3c845 to e245d47 Compare February 18, 2026 18:41
@dsessler7 dsessler7 enabled auto-merge (rebase) February 18, 2026 18:45
@dsessler7 dsessler7 merged commit dd7c6ff into CrunchyData:REL_5_8 Feb 18, 2026
18 checks passed
@dsessler7 dsessler7 deleted the otel-testing-improvements-5.8 branch February 18, 2026 19:10
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.

2 participants

Comments