Skip to content

[7.17] Update YAML Rest tests to check for product header on all responses (#83290)#83996

Merged
jbaiera merged 4 commits intoelastic:7.17from
jbaiera:backport/7.17/pr-83290
Feb 16, 2022
Merged

[7.17] Update YAML Rest tests to check for product header on all responses (#83290)#83996
jbaiera merged 4 commits intoelastic:7.17from
jbaiera:backport/7.17/pr-83290

Conversation

@jbaiera
Copy link
Copy Markdown
Member

@jbaiera jbaiera commented Feb 15, 2022

Backports the following commits to 7.17:

…lastic#83290)

This PR adds assertions to YAML Rest tests to ensure that product headers are always
returned in rest responses. Additional work has been included to fix a number of misuses
of ThreadContext, mostly because of stashing listeners without their accompanying contexts.

BWC Rest tests have been disabled for a few cases while the fixes are backported.
# Conflicts:
#	rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/cat.snapshots/10_basic.yml
#	rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test/tsdb/30_snapshot.yml
#	test/framework/src/main/java/org/elasticsearch/test/rest/yaml/section/DoSection.java
#	x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/action/TransportXPackUsageAction.java
#	x-pack/plugin/watcher/src/main/java/org/elasticsearch/xpack/watcher/WatcherUsageTransportAction.java
@jbaiera jbaiera merged commit 92b3d0d into elastic:7.17 Feb 16, 2022
@jbaiera jbaiera deleted the backport/7.17/pr-83290 branch February 16, 2022 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants