Skip to content

Conversation

@aleksmaus
Copy link
Contributor

What is the problem this PR solves?

Adds integration test coverage for the case where the index is not found.
This is related to the defect that was not covered by the integration test:
#876

How does this PR solve the problem?

Add integration test coverage for actions cleanup with non-existing index

How to test this PR locally

make test-int 

or if the instance of Elasticsearch is already running:

make test-int-set

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

Related issues

@aleksmaus aleksmaus added cleanup v8.0.0 backport-v8.0.0 Automated backport with mergify labels Nov 16, 2021
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-11-16T10:39:56.403+0000

  • Duration: 9 min 45 sec

  • Commit: eac969a

Test stats 🧪

Test Results
Failed 0
Passed 212
Skipped 0
Total 212

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@aleksmaus aleksmaus merged commit 3f02142 into elastic:master Nov 17, 2021
mergify bot pushed a commit that referenced this pull request Nov 17, 2021
mergify bot added a commit that referenced this pull request Nov 17, 2021
…ndex (#882) (#889)

(cherry picked from commit 3f02142)

Co-authored-by: Aleksandr Maus <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v8.0.0 Automated backport with mergify cleanup v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants