search

opensearch-project / logstash-input-opensearch

Apache License 2.0
9 stars 14 forks source link

[BUG] PR fail at integ-test #47

Open jordarlu opened 1 year ago

jordarlu commented 1 year ago

What is the bug? I raised a PR to update expired pem file as https://github.com/opensearch-project/logstash-input-opensearch/pull/44, but the integ-fail with the following errors -

logstash_1 | Failures: logstash_1 | logstash_1 | 1) LogStash::Inputs::OpenSearch against a secured opensearch behaves like an opensearch index plugin should retrieve json event from opensearch logstash_1 | Failure/Error: opensearch.index(params) logstash_1 |
logstash_1 | ArgumentError: logstash_1 | URL parameter 'type' is not supported

How can one reproduce the bug? Steps to reproduce the behavior:

  1. See error at https://github.com/opensearch-project/logstash-input-opensearch/actions/runs/4169404574/jobs/7217326593

What is the expected behavior? integ-test Pass

dlvenable commented 1 year ago

I see the following logs in the tests.

integration_1  | [2023-02-14T01:11:21,183][INFO ][o.o.n.Node               ] [aba6720f09e5] version[2.5.0], pid[102], build[tar/b8a8b6c4d7fc7a7e32eb2cb68ecad8057a4636ad/2023-01-18T23:48:48.981786100Z], OS[Linux/5.15.0-1031-azure/amd64], JVM[Eclipse Adoptium/OpenJDK 64-Bit Server VM/17.0.5/17.0.5+8]

This corresponds to the following from OpenSearch:

https://github.com/opensearch-project/OpenSearch/blob/4dd0c5f00f6d21d3dbc29afcb2454d927ce7817f/server/src/main/java/org/opensearch/node/Node.java#L386-L389

So it appears that the current CI is using the latest version of OpenSearch and this plugin only supports 1.x. Resolving #36 will add support for 2.x. But, these tests should use consistent versions of OpenSearch rather than latest.

I believe the first fix will be to use a consistent version for the CI tests.

jordarlu commented 1 year ago

Hi, @VijayanB , is this something you could help with please ? Thanks!!