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

Unable to run subset of bwc yaml tests #52383

Closed
benwtrent opened this issue Feb 14, 2020 · 3 comments · Fixed by #52785
Closed

Unable to run subset of bwc yaml tests #52383

benwtrent opened this issue Feb 14, 2020 · 3 comments · Fixed by #52785
Labels
:Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team >test Issues or PRs that are addressing/adding tests

Comments

@benwtrent
Copy link
Member

For a while the way to run multiple the full BWC suite against a subsection of the yaml tests was to use a command like the following:

./gradlew :x-pack:qa:rolling-upgrade:v7.7.0#bwcTest -Dtests.class=org.elasticsearch.upgrades.UpgradeClusterClientYamlTestSuiteIT -Dtests.method="test {p0=*/40_ml_datafeed_crud/*}"

This no longer works.

The tests fail in the initial oldCluster runner with

org.elasticsearch.upgrades.TransformSurvivesUpgradeIT.initializationError
...
org.elasticsearch.upgrades.WatcherRestartIT.initializationError

Each failing with

org.junit.runner.manipulation.NoTestsRemainException

The typical new way of running tests with --tests "org.elasticsearch.upgrades.UpgradeClusterClientYamlTestSuiteIT" only works with a specific runner selected.

This is not an option with BWC tests.

@benwtrent benwtrent added >test Issues or PRs that are addressing/adding tests :Delivery/Build Build or test infrastructure labels Feb 14, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Build)

@williamrandolph
Copy link
Contributor

I ran into something like this as well recently. -Dtests.method="test {p0=*/40_ml_datafeed_crud/*}" was working, and you could see whether your test passed or failed, but the output was junked up with the NoTestsRemain exceptions and stack traces.

@mark-vieira
Copy link
Contributor

This turns out to be an issue introduced in version 2.7.5 of the randomized testing library. I've opened an issue with them as well as #52785 on our end to revert back to a known good version in the meantime.

jakelandis added a commit that referenced this issue Feb 27, 2020
Update documentation for:
* restResources config (related #52114)
* call out YAML vs. Java based Rest tests
* update example to use newer syntax 
* update example to target a test that is not skipped 
* provide example for bwcRest test (related #52383)
jakelandis added a commit to jakelandis/elasticsearch that referenced this issue Feb 27, 2020
Update documentation for:
* restResources config (related elastic#52114)
* call out YAML vs. Java based Rest tests
* update example to use newer syntax 
* update example to target a test that is not skipped 
* provide example for bwcRest test (related elastic#52383)
jakelandis added a commit to jakelandis/elasticsearch that referenced this issue Feb 27, 2020
Update documentation for:
* restResources config (related elastic#52114)
* call out YAML vs. Java based Rest tests
* update example to use newer syntax 
* update example to target a test that is not skipped 
* provide example for bwcRest test (related elastic#52383)
jakelandis added a commit that referenced this issue Feb 27, 2020
Update documentation for:
* restResources config (related #52114)
* call out YAML vs. Java based Rest tests
* update example to use newer syntax 
* update example to target a test that is not skipped 
* provide example for bwcRest test (related #52383)
jakelandis added a commit that referenced this issue Feb 27, 2020
Update documentation for:
* restResources config (related #52114)
* call out YAML vs. Java based Rest tests
* update example to use newer syntax 
* update example to target a test that is not skipped 
* provide example for bwcRest test (related #52383)
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants