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

[8.x] Fixes Failing test: Jest Integration Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests - capacity based claiming should claim tasks to full capacity (#201681) #209922

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

ymao1
Copy link
Contributor

@ymao1 ymao1 commented Feb 5, 2025

…ared/task_manager/server/integration_tests - capacity based claiming should claim tasks to full capacity (elastic#201681)

Resolves elastic#205949,
elastic#191117

## Summary

Trying to fix flaky integration test by performing a bulk create for the
test tasks instead of creating one by one. After making this change, was
able to run the integration test ~100 times without failure.

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 7f28ae6)

# Conflicts:
#	x-pack/platform/plugins/shared/task_manager/server/integration_tests/task_manager_capacity_based_claiming.test.ts
@ymao1 ymao1 merged commit e8c2612 into elastic:8.x Feb 5, 2025
11 checks passed
ymao1 added a commit that referenced this pull request Feb 7, 2025
…gins/shared/task_manager/server/integration_tests - capacity based claiming should claim tasks to full capacity (#201681) (#210232)

# Backport

This will backport the following commits from `main` to `8.16`:
- [Fixes Failing test: Jest Integration
Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests
- capacity based claiming should claim tasks to full capacity
(#201681)](#201681)

<!--- Backport version: 9.6.4 -->

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

<!--BACKPORT [{"author":{"name":"Ying
Mao","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-05T21:20:38Z","message":"Fixes
Failing test: Jest Integration
Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests
- capacity based claiming should claim tasks to full capacity
(#201681)\n\nResolves
https://github.com/elastic/kibana/issues/205949,\r\nhttps://github.com/elastic/kibana/issues/191117\r\n\r\n##
Summary\r\n\r\nTrying to fix flaky integration test by performing a bulk
create for the\r\ntest tasks instead of creating one by one. After
making this change, was\r\nable to run the integration test ~100 times
without failure.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"7f28ae63e36d73ec471df7109909b1249f7edafd","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Feature:Task
Manager","Team:ResponseOps","v9.0.0","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"Fixes
Failing test: Jest Integration
Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests
- capacity based claiming should claim tasks to full
capacity","number":201681,"url":"https://github.com/elastic/kibana/pull/201681","mergeCommit":{"message":"Fixes
Failing test: Jest Integration
Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests
- capacity based claiming should claim tasks to full capacity
(#201681)\n\nResolves
https://github.com/elastic/kibana/issues/205949,\r\nhttps://github.com/elastic/kibana/issues/191117\r\n\r\n##
Summary\r\n\r\nTrying to fix flaky integration test by performing a bulk
create for the\r\ntest tasks instead of creating one by one. After
making this change, was\r\nable to run the integration test ~100 times
without failure.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"7f28ae63e36d73ec471df7109909b1249f7edafd"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/209920","number":209920,"state":"MERGED","mergeCommit":{"sha":"90f844b2255f7463614f029c329b80cc741cf151","message":"[9.0]
Fixes Failing test: Jest Integration
Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests
- capacity based claiming should claim tasks to full capacity (#201681)
(#209920)\n\n# Backport\n\nThis will backport the following commits from
`main` to `9.0`:\n- [Fixes Failing test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)](https://github.com/elastic/kibana/pull/201681)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Ying\nMao\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2025-02-05T21:20:38Z\",\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\",\"branchLabelMapping\":{\"^v9.1.0$\":\"main\",\"^v8.19.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:skip\",\"Feature:Task\nManager\",\"Team:ResponseOps\",\"v9.0.0\",\"backport:version\",\"v8.18.0\",\"v9.1.0\",\"v8.19.0\"],\"title\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to
full\ncapacity\",\"number\":201681,\"url\":\"https://github.com/elastic/kibana/pull/201681\",\"mergeCommit\":{\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[\"9.0\",\"8.18\",\"8.x\"],\"targetPullRequestStates\":[{\"branch\":\"9.0\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"},{\"branch\":\"8.18\",\"label\":\"v8.18.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"},{\"branch\":\"main\",\"label\":\"v9.1.0\",\"branchLabelMappingKey\":\"^v9.1.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/201681\",\"number\":201681,\"mergeCommit\":{\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\"}},{\"branch\":\"8.x\",\"label\":\"v8.19.0\",\"branchLabelMappingKey\":\"^v8.19.0$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"}]}]\nBACKPORT-->\n\nCo-authored-by:
Ying Mao
<[email protected]>"}},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/209923","number":209923,"state":"MERGED","mergeCommit":{"sha":"afb1bd58a86cd7d43ec3dfcc7473f9b1a896ad4f","message":"[8.18]
Fixes Failing test: Jest Integration
Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests
- capacity based claiming should claim tasks to full capacity (#201681)
(#209923)\n\n# Backport\n\nThis will backport the following commits from
`main` to `8.18`:\n- [Fixes Failing test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)](https://github.com/elastic/kibana/pull/201681)\n\n<!---
Backport version: 9.6.4 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Ying\nMao\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2025-02-05T21:20:38Z\",\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\",\"branchLabelMapping\":{\"^v9.1.0$\":\"main\",\"^v8.19.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:skip\",\"Feature:Task\nManager\",\"Team:ResponseOps\",\"v9.0.0\",\"backport:version\",\"v8.18.0\",\"v9.1.0\",\"v8.19.0\"],\"title\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to
full\ncapacity\",\"number\":201681,\"url\":\"https://github.com/elastic/kibana/pull/201681\",\"mergeCommit\":{\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[\"8.18\",\"8.x\"],\"targetPullRequestStates\":[{\"branch\":\"9.0\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"url\":\"https://github.com/elastic/kibana/pull/209920\",\"number\":209920,\"state\":\"OPEN\"},{\"branch\":\"8.18\",\"label\":\"v8.18.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"},{\"branch\":\"main\",\"label\":\"v9.1.0\",\"branchLabelMappingKey\":\"^v9.1.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/201681\",\"number\":201681,\"mergeCommit\":{\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\"}},{\"branch\":\"8.x\",\"label\":\"v8.19.0\",\"branchLabelMappingKey\":\"^v8.19.0$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"}]}]\nBACKPORT-->"}},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201681","number":201681,"mergeCommit":{"message":"Fixes
Failing test: Jest Integration
Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests
- capacity based claiming should claim tasks to full capacity
(#201681)\n\nResolves
https://github.com/elastic/kibana/issues/205949,\r\nhttps://github.com/elastic/kibana/issues/191117\r\n\r\n##
Summary\r\n\r\nTrying to fix flaky integration test by performing a bulk
create for the\r\ntest tasks instead of creating one by one. After
making this change, was\r\nable to run the integration test ~100 times
without failure.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"7f28ae63e36d73ec471df7109909b1249f7edafd"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/209922","number":209922,"state":"MERGED","mergeCommit":{"sha":"e8c26120d3d58beec3a233f6ee92556d96b20835","message":"[8.x]
Fixes Failing test: Jest Integration
Tests.x-pack/platform/plugins/shared/task_manager/server/integration_tests
- capacity based claiming should claim tasks to full capacity (#201681)
(#209922)\n\n# Backport\n\nThis will backport the following commits from
`main` to `8.x`:\n- [Fixes Failing test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)](https://github.com/elastic/kibana/pull/201681)\n\n<!---
Backport version: 9.6.4 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Ying\nMao\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2025-02-05T21:20:38Z\",\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\",\"branchLabelMapping\":{\"^v9.1.0$\":\"main\",\"^v8.19.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:skip\",\"Feature:Task\nManager\",\"Team:ResponseOps\",\"v9.0.0\",\"backport:version\",\"v8.18.0\",\"v9.1.0\",\"v8.19.0\"],\"title\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to
full\ncapacity\",\"number\":201681,\"url\":\"https://github.com/elastic/kibana/pull/201681\",\"mergeCommit\":{\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[\"8.18\",\"8.x\"],\"targetPullRequestStates\":[{\"branch\":\"9.0\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"url\":\"https://github.com/elastic/kibana/pull/209920\",\"number\":209920,\"state\":\"OPEN\"},{\"branch\":\"8.18\",\"label\":\"v8.18.0\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"},{\"branch\":\"main\",\"label\":\"v9.1.0\",\"branchLabelMappingKey\":\"^v9.1.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/201681\",\"number\":201681,\"mergeCommit\":{\"message\":\"Fixes\nFailing
test: Jest
Integration\nTests.x-pack/platform/plugins/shared/task_manager/server/integration_tests\n-
capacity based claiming should claim tasks to full
capacity\n(#201681)\\n\\nResolves\nhttps://github.com//issues/205949,\\r\\nhttps://github.com//issues/191117\\r\\n\\r\\n##\nSummary\\r\\n\\r\\nTrying
to fix flaky integration test by performing a bulk\ncreate for
the\\r\\ntest tasks instead of creating one by one. After\nmaking this
change, was\\r\\nable to run the integration test ~100 times\nwithout
failure.\\r\\n\\r\\n---------\\r\\n\\r\\nCo-authored-by: Elastic
Machine\n<[email protected]>\",\"sha\":\"7f28ae63e36d73ec471df7109909b1249f7edafd\"}},{\"branch\":\"8.x\",\"label\":\"v8.19.0\",\"branchLabelMappingKey\":\"^v8.19.0$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"}]}]\nBACKPORT-->"}}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants