-
Notifications
You must be signed in to change notification settings - Fork 202
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
Add developer guide/integration test section #4330
Add developer guide/integration test section #4330
Conversation
Aligned with Kafka example https://github.com/opensearch-project/data-prepper/blob/51ee0df595aaad1b921c888394bf3e110ffc74e9/data-prepper-plugins/kafka-plugins/README.md#developer-guide Signed-off-by: Jürgen Walter <juergen.walter@sap.com>
76b5555
to
d6aa7c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @juergen-walter !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@juergen-walter Thanks for filling in the gap. One minor inline comment
Version 2.12.0 onwards requires to use non-default admin password https://opensearch.org/blog/replacing-default-admin-credentials/ Signed-off-by: Jürgen Walter <juergen.walter@sap.com>
3e0e686
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thank you!
Description
Document how to perform integration tests for the opensearch-plugin.
This is only documentation
Aligned with Kafka example
https://github.com/opensearch-project/data-prepper/blob/51ee0df595aaad1b921c888394bf3e110ffc74e9/data-prepper-plugins/kafka-plugins/README.md#developer-guide
Issues Resolved
I am not aware of an issue, but it would have saved time for #3929 and future contributions
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.