search

opendistro-for-elasticsearch / opendistro-build

🧰 Open Distro Build Scripts
https://opendistro.github.io/
Apache License 2.0
344 stars 175 forks source link

Asynchronous Search IT On-boarding #673

Open Bukhtawar opened 3 years ago

Bukhtawar commented 3 years ago

Automated IT executions for asynchronous search

gaiksaya commented 3 years ago

Hi @Bukhtawar ,

Can you provide details for the tests for with and without security enabled clusters? Eg: Without security With security Please let me know if you need more info.

Thanks!

bbarani commented 3 years ago

@Bukhtawar @rramachand21 @eirsep Can you please provide the above details?

gaiksaya commented 3 years ago

Hi @Bukhtawar ,

For Async plugin integration tests with no security, tests are passing for most of the distributions and architecture. However, docker seems to be an issue. Here is the test workflow run that might help you to debug.

Also, one more thing we observed. I misconfigured something in one of the workflow due to which ES didn’t start at all. And it showed me below error 75 tests completed, 18 failed However, for docker with ES actively running it showed us the same error. Reference: Docker with ES running: https://github.com/opendistro-for-elasticsearch/opendistro-build/runs/2081562455?check_suite_focus=true#step:7:1940 Tarball where ES didn't start at all: https://github.com/opendistro-for-elasticsearch/opendistro-build/runs/2081167860?check_suite_focus=true#step:7:2187

Can you please check at your end what the issue might be? Is this the expected behavior?

Thanks!

eirsep commented 3 years ago

@gaiksaya The integration tests are failing because of WarningFailureException

This arises when there are warnings (not errors) in code execution.

gaiksaya commented 3 years ago

Hi @eirsep , How can we override that? also the integration tests are only failing for docker. For all other distributions we are good Thanks!

eirsep commented 3 years ago

https://github.com/opendistro-for-elasticsearch/asynchronous-search/pull/39/files

This should fix the issue... we removed deprecated api calls made in the tests

gaiksaya commented 3 years ago

Thank you for fixing the tests @eirsep . The test are passing now. Created above PR to add the IT into our workflows. Thanks!

gaiksaya commented 3 years ago

Hi @Bukhtawar and @eirsep , The integration test for async plugin with no security are now added in our workflows. Please let us know when you have IT for security enabled clusters. Thanks!