Skip to content

Make YAML Test Case work with API protections#98371

Closed
tvernum wants to merge 1 commit intoelastic:mainfrom
tvernum:yaml-test-with-api-protections
Closed

Make YAML Test Case work with API protections#98371
tvernum wants to merge 1 commit intoelastic:mainfrom
tvernum:yaml-test-with-api-protections

Conversation

@tvernum
Copy link
Contributor

@tvernum tvernum commented Aug 11, 2023

Modifies ESClientYamlSuiteTestCase to skip loading the global template if legacy templates are not supported.

Relates: #98250

Modifies `ESClientYamlSuiteTestCase` to ship loading the global
template if legacy templates are not supported.

Relates: elastic#98250
@tvernum tvernum added >test Issues or PRs that are addressing/adding tests :Delivery/Build Build or test infrastructure v8.10.0 labels Aug 11, 2023
@tvernum tvernum requested a review from ywangd August 11, 2023 04:55
@elasticsearchmachine elasticsearchmachine added the Team:Delivery Meta label for Delivery team label Aug 11, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@tvernum
Copy link
Contributor Author

tvernum commented Aug 11, 2023

Closing. I've reverted #98250 and I have a better solution coming for this

@tvernum tvernum closed this Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team >test Issues or PRs that are addressing/adding tests v8.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants