Skip to content

Conversation

@spong
Copy link
Member

@spong spong commented Jan 6, 2021

Backports the following commits to 7.x:

…e date range filter (elastic#86921)

## Summary

Ref: elastic#86874



replaces `should` with a single lower bounded and upper bounded date range filter.

### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
@spong spong added the backport This PR is a backport of another PR label Jan 6, 2021
@kibanamachine
Copy link
Contributor

💛 Build succeeded, but was flaky


Test Failures

"before all" hook for "We show one primary callout".Detections > Callouts indicating read-only access to resources On Detections home page "before all" hook for "We show one primary callout"

Link to Jenkins

Stack Trace

Failed Tests Reporter:
  - Test has failed 2 times on tracked branches: https://github.com/elastic/kibana/issues/86981

Error: Timed out while retrying, last result was: {false}

Because this error occurred during a `before all` hook we are skipping the remaining tests in the current suite: `Detections > Callouts indic...`

Although you have test retries enabled, we do not retry tests when `before all` or `after all` hooks fail
    at Context.completeOrRetry (http://localhost:6111/__cypress/tests?p=cypress/support/index.js:1414:13)

Metrics [docs]

Distributable file count

id before after diff
default 47599 48359 +760

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@peluja1012 peluja1012 merged commit da64375 into elastic:7.x Jan 6, 2021
@spong spong deleted the backport/7.x/pr-86921 branch April 12, 2022 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants