Skip to content

Conversation

@bpintea
Copy link
Contributor

@bpintea bpintea commented Dec 13, 2024

Backports the following commits to 8.x:

Re-enable testing those tests too.

Fixes #118631, fixes #118632, fixes #118633, fixes #118638, fixes #118667.

This prevents two tests in `grok` and `dissect` suites - `overwriteName` and `overwriteNameWhere` and one in the `stats` suite - `byStringAndLongWithAlias` - to run against pre-8.13.0 versions. Reason being that coordinators prior to that version can generate invalid node plans, that'd fail (verification) on 8.18+ nodes.

(cherry picked from commit 0441555)
@bpintea bpintea added >test Issues or PRs that are addressing/adding tests backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Analytics/ES|QL AKA ESQL labels Dec 13, 2024
@bpintea bpintea changed the title ESQL: Disable grok.OverwriteName* on pre-8.13 BWC tests (#118655) ESQL: Disable tests planned incorrectly on pre-8.13 (failing BWC) (#118655) Dec 13, 2024
@elasticsearchmachine elasticsearchmachine merged commit 7846d8d into elastic:8.x Dec 16, 2024
15 checks passed
@bpintea bpintea deleted the port/118655 branch December 16, 2024 12:59
maxhniebergall pushed a commit to maxhniebergall/elasticsearch that referenced this pull request Dec 16, 2024
…astic#118655) (elastic#118685)

* ESQL: Disable grok.OverwriteName* on pre-8.13 BWC tests (elastic#118655)

This prevents two tests in `grok` and `dissect` suites - `overwriteName` and `overwriteNameWhere` and one in the `stats` suite - `byStringAndLongWithAlias` - to run against pre-8.13.0 versions. Reason being that coordinators prior to that version can generate invalid node plans, that'd fail (verification) on 8.18+ nodes.

(cherry picked from commit 0441555)

* re-enabled disabled tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >test Issues or PRs that are addressing/adding tests v8.18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants