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.

Sign up for GitHub

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

Jump to bottom

Failing ES Promotion: X-Pack Alerting API Integration Tests - Actions x-pack/test/alerting_api_integration/spaces_only/tests/actions/max_queued_actions_circuit_breaker.ts #166770

Closed
jbudz opened this issue Sep 19, 2023 · 2 comments · Fixed by #166861
Closed

Failing ES Promotion: X-Pack Alerting API Integration Tests - Actions x-pack/test/alerting_api_integration/spaces_only/tests/actions/max_queued_actions_circuit_breaker.ts #166770

jbudz opened this issue Sep 19, 2023 · 2 comments · Fixed by #166861
Assignees
@doakalexi
Labels
failed-es-promotion Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@jbudz
Copy link
Member

jbudz commented Sep 19, 2023

Actions max queued actions circuit breaker completes execution and reports back whether it reached the limit

Error: expected 'failure' to sort of equal 'success'
    at Assertion.assert (expect.js:100:11)
    at Assertion.eql (expect.js:244:8)
    at Context.<anonymous> (max_queued_actions_circuit_breaker.ts:94:47)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Object.apply (wrap_function.js:73:16) {
  actual: 'failure',
  expected: 'success',
  showDiff: true
}
@jbudz jbudz added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) failed-es-promotion v8.11.0 labels Sep 19, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

jbudz added a commit that referenced this issue Sep 19, 2023
@jbudz
skip failing test suite (#166770)
dc34810
@jbudz
Copy link
Member Author

jbudz commented Sep 19, 2023

Skipped

main: dc34810

@doakalexi doakalexi moved this from Awaiting Triage to Todo in AppEx: ResponseOps - Execution & Connectors Sep 19, 2023
@doakalexi doakalexi self-assigned this Sep 19, 2023
@doakalexi doakalexi moved this from Todo to In Progress in AppEx: ResponseOps - Execution & Connectors Sep 20, 2023
@doakalexi doakalexi mentioned this issue Sep 20, 2023
Merged
1 task
@doakalexi doakalexi moved this from In Progress to In Review in AppEx: ResponseOps - Execution & Connectors Sep 20, 2023
doakalexi added a commit that referenced this issue Sep 21, 2023
@doakalexi
[ResponseOps][Alerting] Failing ES Promotion: max_queued_actions_circ…
bb4e6fc 
…uit_breaker.ts test (#166861)

Resolves #166770

## Summary

The test was failing for this reason: "illegal_argument_exception: can't
merge a non object mapping [ruleInfo.actions] with an object mapping". I
switched to es query rule so that we can use ESTestIndexTool that
defines a mapping for the es test index.


### Checklist

- [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


### To verify

Run the functional test runner with the following command and verify it
passes:
```
ES_SNAPSHOT_MANIFEST="https://storage.googleapis.com/kibana-ci-es-snapshots-daily/8.11.0/archives/20230919-155242_b9970223/manifest.json" node scripts/functional_tests_server.js --config x-pack/test/alerting_api_integration/spaces_only/tests/actions/config.ts
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@doakalexi doakalexi

Labels
failed-es-promotion Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
No open projects
AppEx: ResponseOps - Execution & Connectors
Status: Done
Milestone
No milestone
4 participants
@jbudz @elasticmachine @kibanamachine @doakalexi