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

[BUG] PR fail at integ-test #47

Open
jordarlu opened this issue Feb 17, 2023 · 2 comments
Open

[BUG] PR fail at integ-test #47

jordarlu opened this issue Feb 17, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@jordarlu
Copy link

What is the bug?
I raised a PR to update expired pem file as #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:
4. See error at https://github.com/opensearch-project/logstash-input-opensearch/actions/runs/4169404574/jobs/7217326593

What is the expected behavior?
integ-test Pass

@jordarlu jordarlu added bug Something isn't working untriaged labels Feb 17, 2023
@dlvenable
Copy link
Member

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
Copy link
Author

jordarlu commented Mar 7, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants