Skip to content

Conversation

@jakelandis
Copy link
Contributor

@jakelandis jakelandis commented Oct 14, 2022

Compatible REST tests (like normal REST tests) allow copying the tests from either rest-api-spec/src/yamlRestTest/resources/rest-api-spec/test (includeCore) or x-pack/plugin/src/yamlRestTest/resources/rest-api-spec/test (includeXpack). Both are currently broken. #90851 fixes the former , and this commit fixes the later. The fix here is to ensure the correct path to the x-pack rest tests when pulling from the elder branch for compatible testing. This only impacts modules that have the compatibility plugin enabled and are configured to pull from the main x-pack set of tests. Currently, this should only impact :x-pack:plugin:enrich:qa:rest but will fix it for future projects with that configuration.

@jakelandis jakelandis changed the title Fx path to x-pack tests Fx path to compatible x-pack tests Oct 14, 2022
@jakelandis
Copy link
Contributor Author

Semi related: #90929 . Here we are fixing the sourcing tests from x-pack/plugin/src/yamlRestTest/resources/rest-api-spec/test, but more generally, we should be abit smarter from where we can pull these tests from. :x-pack:plugin:watcher:qa:with-security also has the compatibility plugin + includeXpack defined...but in that case the tests live in a completely different module and we don't support sourcing tests from there.

@jakelandis jakelandis marked this pull request as ready for review October 14, 2022 19:09
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Oct 14, 2022
@jakelandis jakelandis added >non-issue and removed needs:triage Requires assignment of a team area label labels Oct 14, 2022
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Oct 14, 2022
Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

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

LGTM

@jakelandis jakelandis added :Delivery/Build Build or test infrastructure and removed needs:triage Requires assignment of a team area label labels Oct 14, 2022
@elasticsearchmachine elasticsearchmachine added the Team:Delivery Meta label for Delivery team label Oct 14, 2022
@elasticsearchmachine
Copy link
Collaborator

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

@nik9000 nik9000 changed the title Fx path to compatible x-pack tests Fix path to compatible x-pack tests Oct 17, 2022
@jakelandis jakelandis merged commit c3918cd into elastic:main Oct 17, 2022
@jakelandis jakelandis deleted the fix_xpack_compat_test_path branch October 17, 2022 16:41
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 >non-issue Team:Delivery Meta label for Delivery team v8.6.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants