security icon indicating copy to clipboard operation
security copied to clipboard

[Integration Test] Add integration test for telemetry plugin

Open Gaganjuneja opened this issue 1 year ago • 3 comments

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 avatar Jan 19 '24 07:01 Gaganjuneja

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

Gaganjuneja avatar Jan 19 '24 07:01 Gaganjuneja

[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 avatar Jan 22 '24 16:01 stephen-crawford

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

Gaganjuneja avatar Jan 23 '24 07:01 Gaganjuneja