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 test: APM Deployment Agnostic - serverless Transaction groups alerts when data is loaded with avg transaction duration alerts returns the correct number of alert counts #201531

Closed
rmyz opened this issue Nov 25, 2024 · 1 comment · Fixed by #201755
Closed

Failing test: APM Deployment Agnostic - serverless Transaction groups alerts when data is loaded with avg transaction duration alerts returns the correct number of alert counts #201531

rmyz opened this issue Nov 25, 2024 · 1 comment · Fixed by #201755
Assignees
@rmyz
Labels
failed-test A test failure on a tracked branch, potentially flaky-test Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team

Comments

@rmyz
Copy link
Contributor

rmyz commented Nov 25, 2024 •
edited
Loading

The following test Transaction groups alerts when data is loaded with avg transaction duration alerts returns the correct number of alert counts under x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts is flaky and it fails on MKI.

Failure details

Transaction groups alerts when data is loaded with avg transaction duration alerts returns the correct number of alert counts

Error message

Error: expected [ { name: 'GET /api/failed/request', alertsCount: 0 },
  { name: 'GET /api/request/p95', alertsCount: 0 },
  { name: 'GET /api/request/p99', alertsCount: 0 },
  { name: 'GET /api/product/list', alertsCount: 1 },
  { name: 'GET /api/request/avg', alertsCount: 1 } ] to sort of equal [ { name: 'GET /api/failed/request', alertsCount: 0 },
  { name: 'GET /api/request/p95', alertsCount: 0 },
  { name: 'GET /api/request/p99', alertsCount: 0 },
  { name: 'GET /api/request/avg', alertsCount: 1 } ]
    at Assertion.assert (expect.js:100:11)
    at Assertion.eql (expect.js:244:8)
    at Context. (transactions_groups_alerts.spec.ts:222:31)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at Object.apply (wrap_function.js:74:16) {
  actual: '[\n' +
    '  {\n' +
    '    "alertsCount": 0\n' +
    '    "name": "GET /api/failed/request"\n' +
    '  }\n' +
    '  {\n' +
    '    "alertsCount": 0\n' +
    '    "name": "GET /api/request/p95"\n' +
    '  }\n' +
    '  {\n' +
    '    "alertsCount": 0\n' +
    '    "name": "GET /api/request/p99"\n' +
    '  }\n' +
    '  {\n' +
    '    "alertsCount": 1\n' +
    '    "name": "GET /api/product/list"\n' +
    '  }\n' +
    '  {\n' +
    '    "alertsCount": 1\n' +
    '    "name": "GET /api/request/avg"\n' +
    '  }\n' +
    ']',
  expected: '[\n' +
    '  {\n' +
    '    "alertsCount": 0\n' +
    '    "name": "GET /api/failed/request"\n' +
    '  }\n' +
    '  {\n' +
    '    "alertsCount": 0\n' +
    '    "name": "GET /api/request/p95"\n' +
    '  }\n' +
    '  {\n' +
    '    "alertsCount": 0\n' +
    '    "name": "GET /api/request/p99"\n' +
    '  }\n' +
    '  {\n' +
    '    "alertsCount": 1\n' +
    '    "name": "GET /api/request/avg"\n' +
    '  }\n' +
    ']',
  showDiff: true
}

A proper fix will be needed in order to close this ticket.
@rmyz rmyz added failed-test A test failure on a tracked branch, potentially flaky-test Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Nov 25, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@rmyz rmyz mentioned this issue Nov 25, 2024
Merged
rmyz added a commit that referenced this issue Nov 25, 2024
@rmyz
Skip Serverless Transaction groups alerts when data is loaded with av…
a76084f 
…g transaction duration alerts returns the correct number of alert counts (#201533)

## Summary
Related to #201531

This PR skips `Transaction groups alerts when data is loaded with avg
transaction duration alerts returns the correct number of alert counts`,
which is inside
`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`
file, as it's failing on MKI.
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 25, 2024
@rmyz
Skip Serverless Transaction groups alerts when data is loaded with av…
c4cb622 
…g transaction duration alerts returns the correct number of alert counts (elastic#201533)

## Summary
Related to elastic#201531

This PR skips `Transaction groups alerts when data is loaded with avg
transaction duration alerts returns the correct number of alert counts`,
which is inside
`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`
file, as it's failing on MKI.

(cherry picked from commit a76084f)
@rmyz rmyz changed the title failsOnMKI: serverless Transaction groups alerts when data is loaded with avg transaction duration alerts returns the correct number of alert counts Failing test: APM Deployment Agnostic - serverless Transaction groups alerts when data is loaded with avg transaction duration alerts returns the correct number of alert counts Nov 25, 2024
@rmyz rmyz self-assigned this Nov 26, 2024
@rmyz rmyz mentioned this issue Nov 26, 2024
Merged
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this issue Nov 26, 2024
@rmyz @paulinashakirova
Skip Serverless Transaction groups alerts when data is loaded with av…
e3167b4 
…g transaction duration alerts returns the correct number of alert counts (elastic#201533)

## Summary
Related to elastic#201531

This PR skips `Transaction groups alerts when data is loaded with avg
transaction duration alerts returns the correct number of alert counts`,
which is inside
`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`
file, as it's failing on MKI.
rmyz added a commit that referenced this issue Nov 27, 2024
@rmyz
[APM] Fix transactions test (#201755)
6620529 
## Summary

Closes #201531

This PR fixes the skipped test for MKI `Transaction groups alerts when
data is loaded with avg transaction duration alerts returns the correct
number of alert counts` in
`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 27, 2024
@rmyz
[APM] Fix transactions test (elastic#201755)
3d1e208 
## Summary

Closes elastic#201531

This PR fixes the skipped test for MKI `Transaction groups alerts when
data is loaded with avg transaction duration alerts returns the correct
number of alert counts` in
`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`

(cherry picked from commit 6620529)
@crespocarlos crespocarlos mentioned this issue Nov 27, 2024
Closed
kibanamachine added a commit that referenced this issue Nov 27, 2024
@kibanamachine @rmyz
[8.x] [APM] Fix transactions test (#201755) (#202017)
1c2c65f 
# Backport

This will backport the following commits from `main` to `8.x`:
- [[APM] Fix transactions test
(#201755)](#201755)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Sergi
Romeu","email":"sergi.romeu@elastic.co"},"sourceCommit":{"committedDate":"2024-11-27T15:51:23Z","message":"[APM]
Fix transactions test (#201755)\n\n## Summary\n\nCloses #201531\n\nThis
PR fixes the skipped test for MKI `Transaction groups alerts when\ndata
is loaded with avg transaction duration alerts returns the
correct\nnumber of alert counts`
in\n`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`","sha":"662052919c8663d76a93cce8fe0df687fff92a15","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","Team:obs-ux-infra_services"],"title":"[APM]
Fix transactions
test","number":201755,"url":"https://github.com/elastic/kibana/pull/201755","mergeCommit":{"message":"[APM]
Fix transactions test (#201755)\n\n## Summary\n\nCloses #201531\n\nThis
PR fixes the skipped test for MKI `Transaction groups alerts when\ndata
is loaded with avg transaction duration alerts returns the
correct\nnumber of alert counts`
in\n`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`","sha":"662052919c8663d76a93cce8fe0df687fff92a15"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201755","number":201755,"mergeCommit":{"message":"[APM]
Fix transactions test (#201755)\n\n## Summary\n\nCloses #201531\n\nThis
PR fixes the skipped test for MKI `Transaction groups alerts when\ndata
is loaded with avg transaction duration alerts returns the
correct\nnumber of alert counts`
in\n`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`","sha":"662052919c8663d76a93cce8fe0df687fff92a15"}}]}]
BACKPORT-->

Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
@crespocarlos crespocarlos mentioned this issue Nov 28, 2024
Closed
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Dec 12, 2024
@rmyz @CAWilson94
Skip Serverless Transaction groups alerts when data is loaded with av…
0fa9ecd 
…g transaction duration alerts returns the correct number of alert counts (elastic#201533)

## Summary
Related to elastic#201531

This PR skips `Transaction groups alerts when data is loaded with avg
transaction duration alerts returns the correct number of alert counts`,
which is inside
`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`
file, as it's failing on MKI.
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Dec 12, 2024
@rmyz @CAWilson94
[APM] Fix transactions test (elastic#201755)
c6bda45 
## Summary

Closes elastic#201531

This PR fixes the skipped test for MKI `Transaction groups alerts when
data is loaded with avg transaction duration alerts returns the correct
number of alert counts` in
`x-pack/test/api_integration/deployment_agnostic/apis/observability/apm/transactions/transactions_groups_alerts.spec.ts`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rmyz rmyz

Labels
failed-test A test failure on a tracked branch, potentially flaky-test Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[APM] Fix transactions test rmyz/kibana
2 participants
@elasticmachine @rmyz