Skip to content

Fix TSDBSyntheticIdUpgradeIT exception assertion#142849

Merged
burqen merged 1 commit intoelastic:mainfrom
burqen:ap/2026.02.23.synthetic-id-fix-SyntheticIdUpgradeIT
Feb 23, 2026
Merged

Fix TSDBSyntheticIdUpgradeIT exception assertion#142849
burqen merged 1 commit intoelastic:mainfrom
burqen:ap/2026.02.23.synthetic-id-fix-SyntheticIdUpgradeIT

Conversation

@burqen
Copy link
Copy Markdown
Contributor

@burqen burqen commented Feb 23, 2026

When creating an index with ESRestTestCase.createIndex a random xContentType will be used. If the type is YAML, the exception message will be split into multiple lines if the lines are long. This will break the format of the expected string and the test fails.

This is fixed by getting the actual Response object from the exception and evaluate error.reason which will give the original exception message without the line breaks.

Fix #141734, #141749

When creating an index with ESRestTestCase.createIndex a random
xContentType will be used. If the type is YAML, the exception message
will be split into multiple lines if the lines are long. This will break
the format of the expected string and the test fails.

This is fixed by getting the actual Response object from the exception
and evaluate error.reason which will give the original exception message
without the line breaks.
@burqen burqen added >test Issues or PRs that are addressing/adding tests :StorageEngine/TSDB You know, for Metrics v9.4.0 labels Feb 23, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

Copy link
Copy Markdown
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

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

LGTM

@burqen burqen merged commit a4313a8 into elastic:main Feb 23, 2026
35 checks passed
sidosera pushed a commit to sidosera/elasticsearch that referenced this pull request Feb 24, 2026
When creating an index with ESRestTestCase.createIndex a random
xContentType will be used. If the type is YAML, the exception message
will be split into multiple lines if the lines are long. This will break
the format of the expected string and the test fails.

This is fixed by getting the actual Response object from the exception
and evaluate error.reason which will give the original exception message
without the line breaks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:StorageEngine/TSDB You know, for Metrics Team:StorageEngine >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] TSDBSyntheticIdUpgradeIT testRollingUpgrade {upgradedNodes=2} failing

3 participants