Skip to content
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

Oracledb discovery #3633

Merged

Conversation

greatestusername
Copy link
Contributor

Description:
Adds discovery mode tests and configs for oracledb receiver

Link to Splunk idea:
No idea link. Internal contribution

Testing:

  • Added new integration test TestOracledbDockerObserver
  • verified that TestOracleDBIntegration works as intended

Documentation:
No new documentation added yet

@greatestusername greatestusername marked this pull request as ready for review September 14, 2023 21:44
@greatestusername greatestusername requested review from a team as code owners September 14, 2023 21:44
@rmfitzpatrick
Copy link
Contributor

Is this compatible with windows? If so it should be added to https://github.com/signalfx/splunk-otel-collector/blob/main/internal/confmapprovider/discovery/bundle/bundle_windows.go#L30

@rmfitzpatrick
Copy link
Contributor

Do you mind adding this receiver to the bundled section in https://github.com/signalfx/splunk-otel-collector/blob/main/internal/confmapprovider/discovery/README.md#discovery-mode?

@greatestusername

This comment was marked as outdated.

@greatestusername
Copy link
Contributor Author

not sure what to make of that lint check failure's output. Definitely lint'd and doing it again doesn't show any changes

@rmfitzpatrick
Copy link
Contributor

not sure what to make of that lint check failure's output. Definitely lint'd and doing it again doesn't show any changes

If you rebase/pull in main it should be resolved w/ #3681

Copy link
Contributor

@rmfitzpatrick rmfitzpatrick left a comment

Choose a reason for hiding this comment

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

Thanks!

@greatestusername
Copy link
Contributor Author

Thanks!

Quite welcome and happy to help! Will see about getting other integration discovery going for our other targets.

@rmfitzpatrick rmfitzpatrick merged commit 6225582 into signalfx:main Sep 25, 2023
127 checks passed
@aunshc
Copy link
Contributor

aunshc commented Sep 25, 2023

Thanks @greatestusername and @rmfitzpatrick!

samiura pushed a commit to samiura/splunk-otel-collector that referenced this pull request Sep 25, 2023
* update bundled test and configs for oraclediscovery

* update bundled test and configs for oraclediscovery

* running bundled_test now evaluate metrics matching

* add working bundled_test

* update fail and partial messages for receiver config variables

* order of bundled configs matters

* make fmt and add more time

* lower durations, add in args for endpoint and service

* reduce time on image pull and add to windows bundle

* add to bundle_windows_test

* lower times on tests

* address feedback on yaml template and test using env var

* gotta make generate-metrics

* re-arrange bundled test

* add oracledb to the discovery readme

* make generate-metrics fmt

* address feedback

* Update internal/confmapprovider/discovery/README.md

Co-authored-by: Ryan Fitzpatrick <[email protected]>

* fix to re-check  and other attribute fields

* add new test data for bundled test

* add generated yaml

---------

Co-authored-by: greatestusername-splunk <[email protected]>
Co-authored-by: Ryan Fitzpatrick <[email protected]>
@greatestusername greatestusername deleted the oracledb-discovery branch September 26, 2023 16:48
@github-actions github-actions bot mentioned this pull request Dec 2, 2024
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.

5 participants