Skip to content

Add datasource, aka "connection string", option to sqlserverreceiver #39235

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

pjanotti
Copy link
Contributor

@pjanotti pjanotti commented Apr 8, 2025

Description

Adds an option, datasource, that allows full control of the connection string used when connecting to the DB. There are many possible variations that are useful that can be specified via this option, e.g.: one could enable Windows authentication and have the collector run with an account configured to access the SQL server.

Testing

Covered the new option in an extended validation test and used it to connect to a SQL server using Windows credentials.

Documentation

Added a note about the new option.

@pjanotti pjanotti requested review from crobert-1 and a team as code owners April 8, 2025 03:39
@github-actions github-actions bot added receiver/sqlserver Run Windows Enable running windows test on a PR labels Apr 8, 2025
@atoulme atoulme merged commit a0f2f5d into open-telemetry:main Apr 9, 2025
202 of 204 checks passed
@github-actions github-actions bot added this to the next release milestone Apr 9, 2025
LucianoGiannotti pushed a commit to LucianoGiannotti/opentelemetry-collector-contrib that referenced this pull request Apr 9, 2025
…er` (open-telemetry#39235)

#### Description
Adds an option, `datasource`, that allows full control of the connection
string used when connecting to the DB. There are many possible
variations that are useful that can be specified via this option, e.g.:
one could enable Windows authentication and have the collector run with
an account configured to access the SQL server.

#### Testing
Covered the new option in an extended validation test and used it to
connect to a SQL server using Windows credentials.

#### Documentation
Added a note about the new option.

---------

Co-authored-by: Antoine Toulme <[email protected]>
@pjanotti pjanotti deleted the add-datasource-option-to-sqlserverreceiver branch April 21, 2025 19:03
Fiery-Fenix pushed a commit to Fiery-Fenix/opentelemetry-collector-contrib that referenced this pull request Apr 24, 2025
…er` (open-telemetry#39235)

#### Description
Adds an option, `datasource`, that allows full control of the connection
string used when connecting to the DB. There are many possible
variations that are useful that can be specified via this option, e.g.:
one could enable Windows authentication and have the collector run with
an account configured to access the SQL server.

#### Testing
Covered the new option in an extended validation test and used it to
connect to a SQL server using Windows credentials.

#### Documentation
Added a note about the new option.

---------

Co-authored-by: Antoine Toulme <[email protected]>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants