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

[Integration Test] Add integration test for telemetry plugin #3961

Open
Gaganjuneja opened this issue Jan 19, 2024 · 3 comments
Open

[Integration Test] Add integration test for telemetry plugin #3961

Gaganjuneja opened this issue Jan 19, 2024 · 3 comments
Labels
enhancement New feature or request help wanted Community contributions are especially encouraged for these issues. triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.

Comments

@Gaganjuneja
Copy link
Contributor

Is your feature request related to a problem?
We have seen a compatibility issues between telemetry plugin and the security plugin. So we should have a integration test in the security plugin repo to make sure there is no conflict.

What solution would you like?
Write an integration test.

What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.

Do you have any additional context?
We can use this integ test as a reference that might provide some guidance on cluster configuration and a basic setup.
https://github.com/opensearch-project/security/blob/main/src/integrationTest/java/org/opensearch/security/http/OnBehalfOfJwtAuthenticationTest.java

@Gaganjuneja Gaganjuneja added enhancement New feature or request untriaged Require the attention of the repository maintainers and may need to be prioritized labels Jan 19, 2024
@Gaganjuneja
Copy link
Contributor Author

@atharvasharma61, Can you please take this up? Please let me know, if you need any help.

@stephen-crawford
Copy link
Contributor

[Triage] Hi @Gaganjuneja thanks for filing this issue. It seems like there is a circular dependency issue here, but for the meantime, we can go forward with looking for a way to add this test. Going to mark as triaged and help wanted since it seems like this is going to be handled by someone from the telemetry plugin-side.

@stephen-crawford stephen-crawford added help wanted Community contributions are especially encouraged for these issues. triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable. and removed untriaged Require the attention of the repository maintainers and may need to be prioritized labels Jan 22, 2024
@Gaganjuneja
Copy link
Contributor Author

@scrawfor99, This is a pending item from another PR. @atharvasharma61 will be working on this, if needed I will collaborate with him. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Community contributions are especially encouraged for these issues. triaged Issues labeled as 'Triaged' have been reviewed and are deemed actionable.
Projects
None yet
Development

No branches or pull requests

2 participants