Skip to content

[chore] [receiver/sqlserver] Streamline obfuscate usage#41214

Merged
dmitryax merged 1 commit into
open-telemetry:mainfrom
dmitryax:simplify-mysql-obfuscate
Jul 14, 2025
Merged

[chore] [receiver/sqlserver] Streamline obfuscate usage#41214
dmitryax merged 1 commit into
open-telemetry:mainfrom
dmitryax:simplify-mysql-obfuscate

Conversation

@dmitryax

@dmitryax dmitryax commented Jul 9, 2025

Copy link
Copy Markdown
Member

Don't use sync.Once, maintain the instance explicitly as part of the scraper

@dmitryax dmitryax requested review from a team and crobert-1 as code owners July 9, 2025 18:55
@github-actions github-actions Bot added receiver/sqlserver Run Windows Enable running windows test on a PR labels Jul 9, 2025
@github-actions github-actions Bot requested review from StefanKurek and sincejune July 9, 2025 18:55
@dmitryax dmitryax changed the title [receiver/sqlserver] Stramline obfuscate usage [chore] [receiver/sqlserver] Stramline obfuscate usage Jul 9, 2025
@dmitryax dmitryax added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jul 9, 2025
Don't use sync.Once, maintain the instance explicitly as part of the scraper
@dmitryax dmitryax force-pushed the simplify-mysql-obfuscate branch from e5124c9 to a4e317f Compare July 9, 2025 19:32
@dmitryax dmitryax changed the title [chore] [receiver/sqlserver] Stramline obfuscate usage [chore] [receiver/sqlserver] Streamline obfuscate usage Jul 10, 2025
@dmitryax dmitryax merged commit 5c3822a into open-telemetry:main Jul 14, 2025
211 checks passed
@github-actions github-actions Bot added this to the next release milestone Jul 14, 2025
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
…ry#41214)

Don't use sync.Once, maintain the instance explicitly as part of the
scraper
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

receiver/sqlserver Run Windows Enable running windows test on a PR Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants