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

[updatecli] update elastic stack version for testing 8.18.0-c80fb744-SNAPSHOT #4184

Merged
merged 2 commits into from
Dec 13, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Dec 6, 2024

Bump elastic-stack to latest snapshot version

Update snapshot.yml

1 file(s) updated with "$1=8.18.0-c80fb744-SNAPSHOT": * dev-tools/integration/.env

GitHub Action workflow link
Updatecli logo

Created automatically by Updatecli

Options:

Most of Updatecli configuration is done via its manifest(s).

  • If you close this pull request, Updatecli will automatically reopen it, the next time it runs.
  • If you close this pull request and delete the base branch, Updatecli will automatically recreate it, erasing all previous commits made.

Feel free to report any issues at github.com/updatecli/updatecli.
If you find this tool useful, do not hesitate to star our GitHub repository as a sign of appreciation, and/or to tell us directly on our chat!

@github-actions github-actions bot added automation dependency backport-skip Skip notification from the automated backport with mergify labels Dec 6, 2024
@jlind23 jlind23 enabled auto-merge (squash) December 6, 2024 13:29
@jlind23 jlind23 added backport-8.x Automated backport to the 8.x branch with mergify and removed backport-8.x Automated backport to the 8.x branch with mergify labels Dec 6, 2024
jlind23
jlind23 previously approved these changes Dec 6, 2024
@jlind23
Copy link
Contributor

jlind23 commented Dec 6, 2024

FAIL: TestBulkCreate/Invalid_utf-8 (0.26s) this is the failing test.
@ycombinator IIRC there was a change on the ES that was changing the http code being returned right?
#4012 Could this be the problem here?

@ycombinator
Copy link
Contributor

ycombinator commented Dec 7, 2024

FAIL: TestBulkCreate/Invalid_utf-8 (0.26s) this is the failing test. @ycombinator IIRC there was a change on the ES that was changing the http code being returned right? #4012 Could this be the problem here?

It looks like it but I don't see why this would be the case. The fix in Elasticsearch should exist in 8.18, so this test should be passing.

@Tim-Brooks Is there some chance the fix in elastic/elasticsearch#114869 wouldn't exist in Elasticsearch 8.18.0? If it helps, the failing test is expecting 400: parse_exception but instead getting from Elasticsearch 400: x_content_parse_exception.

@github-actions github-actions bot force-pushed the updatecli_8.x_bump-elastic-stack-snapshot-8.x branch from 8fe963f to 0973d0b Compare December 9, 2024 15:07
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 8.18.0-86e6f629-SNAPSHOT [updatecli] update elastic stack version for testing 8.18.0-68454acb-SNAPSHOT Dec 9, 2024
@github-actions github-actions bot force-pushed the updatecli_8.x_bump-elastic-stack-snapshot-8.x branch from 0973d0b to 28b4bcf Compare December 10, 2024 15:07
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 8.18.0-68454acb-SNAPSHOT [updatecli] update elastic stack version for testing 8.18.0-07ca897b-SNAPSHOT Dec 10, 2024
@github-actions github-actions bot force-pushed the updatecli_8.x_bump-elastic-stack-snapshot-8.x branch from 28b4bcf to 457ea9e Compare December 11, 2024 15:08
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 8.18.0-07ca897b-SNAPSHOT [updatecli] update elastic stack version for testing 8.18.0-1631fdd1-SNAPSHOT Dec 11, 2024
@ycombinator ycombinator disabled auto-merge December 12, 2024 01:22
@github-actions github-actions bot force-pushed the updatecli_8.x_bump-elastic-stack-snapshot-8.x branch from 457ea9e to c906ade Compare December 12, 2024 15:07
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 8.18.0-1631fdd1-SNAPSHOT [updatecli] update elastic stack version for testing 8.18.0-711a652c-SNAPSHOT Dec 12, 2024
@Tim-Brooks
Copy link

If it helps, the failing test is expecting 400: parse_exception but instead getting from Elasticsearch 400: x_content_parse_exception.

This appears to have been a deliberate change: elastic/elasticsearch#114445.

@github-actions github-actions bot force-pushed the updatecli_8.x_bump-elastic-stack-snapshot-8.x branch from c906ade to 4026ff0 Compare December 13, 2024 15:07
@github-actions github-actions bot changed the title [updatecli] update elastic stack version for testing 8.18.0-711a652c-SNAPSHOT [updatecli] update elastic stack version for testing 8.18.0-c80fb744-SNAPSHOT Dec 13, 2024
@ycombinator ycombinator force-pushed the updatecli_8.x_bump-elastic-stack-snapshot-8.x branch from 4026ff0 to 2808f23 Compare December 13, 2024 19:08
@ycombinator ycombinator requested a review from a team as a code owner December 13, 2024 19:08
@ycombinator
Copy link
Contributor

Thanks @Tim-Brooks. Updated this PR with the new error type returned from ES in 2808f23.

@ycombinator ycombinator enabled auto-merge (squash) December 13, 2024 19:11
Copy link

@ycombinator ycombinator merged commit 58dc1ad into 8.x Dec 13, 2024
8 checks passed
@ycombinator ycombinator deleted the updatecli_8.x_bump-elastic-stack-snapshot-8.x branch December 13, 2024 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation backport-skip Skip notification from the automated backport with mergify dependency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants