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

Remove tests and branches that will never execute #38772

Merged
merged 6 commits into from
Feb 18, 2019

Conversation

alpar-t
Copy link
Contributor

@alpar-t alpar-t commented Feb 12, 2019

Some tests have assumeTrue that only makes
them run on versions prior to v7.0.0, but for current master at
v8.0.0 this will never happen.
Similarly remove branches that will never be hit on current master.

These tests have an version conditioned `assumeTrue` that only makes
them run on versions prior to `v7.0.0`, but for current master at
`v8.0.0` this will never happen.
@alpar-t alpar-t added >non-issue :Core/Infra/Settings Settings infrastructure and APIs :Core/Infra/Core Core issues without another label :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data v8.0.0 labels Feb 12, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo

Copy link
Contributor

@polyfractal polyfractal left a comment

Choose a reason for hiding this comment

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

Rollup removals LGTM. Also happy they are no longer relevant :)

@alpar-t alpar-t requested a review from jasontedor February 14, 2019 10:30
@alpar-t
Copy link
Contributor Author

alpar-t commented Feb 18, 2019

@jasontedor do the other changes LGTY ?

Copy link
Member

@danielmitterdorfer danielmitterdorfer left a comment

Choose a reason for hiding this comment

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

LGTM

@alpar-t alpar-t merged commit 6891475 into elastic:master Feb 18, 2019
@alpar-t alpar-t deleted the remove-non-relevant-tests branch February 18, 2019 15:37
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Feb 18, 2019
…ate-file

* elastic/master:
  Remove tests and branches that will never execute (elastic#38772)
  also check ccr stats api return empty response in ensureNoCcrTasks()
  Add overlapping, before, after filters to intervals query (elastic#38999)
  Mute test elastic#38949
  Add remote recovery to ShardFollowTaskReplicationTests (elastic#39007)
  [ML] More advanced post-test cleanup of ML indices (elastic#39049)
  wait for shard to be allocated before executing a resume follow api
  Update track-total-hits.asciidoc
  Force kill testcluster nodes (elastic#37353)
  Make pullFixture a task dependency of resolveAllDependencies (elastic#38956)
  set minimum supported version (elastic#39043)
  Enforce Completion Context Limit (elastic#38675)
  Mute test
  Don't close caches while there might still be in-flight requests. (elastic#38958)
  Fix elastic#38623 remove xpack namespace REST API (elastic#38625)
  Add data frame feature (elastic#38934)
  Test bi-directional index following during a rolling upgrade. (elastic#38962)
  Generate mvn pom for ssl-config library (elastic#39019)
  Mute testRetentionLeaseIsRenewedDuringRecovery
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Core Core issues without another label :Core/Infra/Settings Settings infrastructure and APIs >non-issue :StorageEngine/Rollup Turn fine-grained time-based data into coarser-grained data v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants