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

[Backport 2.x] #3530 enhance sanity testing framework #1185

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport e1e0716 from #958

@kavilla
Copy link
Member

kavilla commented Apr 2, 2024

I'll defer to another maintainer to merge this in. The original PR doesn't do clean up and its from a while ago but I think it should be address in a follow up PR for someone who has the chance.

@SuZhou-Joe
Copy link
Member

SuZhou-Joe commented Apr 2, 2024

the sanity_tests.spec.js still fails, I'd prefer we fixing the failed cases in this PR if no following PR already raised.

@SuZhou-Joe
Copy link
Member

@kavilla The new added test cases failed, could you please take a look here?

* initial skeleton

Signed-off-by: Sam <samuel.costa@eliatra.com>

* first two sections

Signed-off-by: Sam <samuel.costa@eliatra.com>

* first two sections

Signed-off-by: Sam <samuel.costa@eliatra.com>

* index pattern creation

Signed-off-by: Sam <samuel.costa@eliatra.com>

* index pattern creation

Signed-off-by: Sam <samuel.costa@eliatra.com>

* Bug Fixes

Signed-off-by: Sam <samuel.costa@eliatra.com>

* Cleanup

Signed-off-by: Sam <samuel.costa@eliatra.com>

* Cleanup

Signed-off-by: Sam <samuel.costa@eliatra.com>

* Cleanup

Signed-off-by: Sam <samuel.costa@eliatra.com>

* Refactor

Signed-off-by: Sam <samuel.costa@eliatra.com>

* Comment/ Extra line removal

Signed-off-by: Sam <samuel.costa@eliatra.com>

* Using loadSampleData command instead of custom method.
Check if pattern was acually created

Signed-off-by: Sam <samuel.costa@eliatra.com>

---------

Signed-off-by: Sam <samuel.costa@eliatra.com>
(cherry picked from commit e1e0716)
@SuZhou-Joe
Copy link
Member

No update for 3 months and rerun the CI still failed, close the PR. Please feel free to open if any update.

@SuZhou-Joe SuZhou-Joe closed this Sep 3, 2024
@SuZhou-Joe SuZhou-Joe removed the v2.17.0 label Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants