Skip to content

Conversation

kkrik-es
Copy link
Contributor

@kkrik-es kkrik-es commented Sep 26, 2024

The skip test entries were added in #113584, no longer needed after backporting it.

@kkrik-es kkrik-es added >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:StorageEngine :StorageEngine/Mapping The storage related side of mappings labels Sep 26, 2024
@kkrik-es kkrik-es self-assigned this Sep 26, 2024
@kkrik-es kkrik-es marked this pull request as ready for review September 26, 2024 11:02
@kkrik-es kkrik-es requested a review from a team as a code owner September 26, 2024 11:02
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@kkrik-es kkrik-es changed the title Rest skipped tests after backporting Remove skipped tests after backporting Sep 26, 2024
Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@elasticsearchmachine elasticsearchmachine merged commit b344493 into elastic:main Sep 26, 2024
15 checks passed
@kkrik-es kkrik-es deleted the fix/remove-skip-tests branch September 26, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :StorageEngine/Mapping The storage related side of mappings Team:StorageEngine >test Issues or PRs that are addressing/adding tests v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants