-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Response Ops][Task Manager] Onboard 12.5% of ECH clusters to use mget
task claiming
#196317
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR looks good so far!
mget
task claiming
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM, tested locally and verified on ECH and serverless.
@elasticmachine merge upstream |
Pinging @elastic/response-ops (Team:ResponseOps) |
@elasticmachine merge upstream |
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Public APIs missing comments
History
cc @ymao1 |
Starting backport for target branches: 8.x |
…et` task claiming (elastic#196317) Resolves elastic/response-ops-team#239 ## Summary Deployed to cloud: deployment ID was `ab4e88d139f93d43024837d96144e7d4`. Since the deployment ID starts with an `a`, this should start with `mget` and I can see in the logs with the latest push that this is true <img width="2190" alt="Screenshot 2024-10-15 at 2 59 20 PM" src="https://github.com/user-attachments/assets/079bc4d8-365e-4ba6-b7a9-59fe506283d9"> Deployed to serverless: project ID was `d33d22a94ce246d091220eace2c4e4bb`. See in the logs: `Using claim strategy mget as configured for deployment d33d22a94ce246d091220eace2c4e4bb` Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> (cherry picked from commit 267efdf)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…se `mget` task claiming (#196317) (#196460) # Backport This will backport the following commits from `main` to `8.x`: - [[Response Ops][Task Manager] Onboard 12.5% of ECH clusters to use `mget` task claiming (#196317)](#196317) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ying Mao","email":"ying.mao@elastic.co"},"sourceCommit":{"committedDate":"2024-10-16T00:24:52Z","message":"[Response Ops][Task Manager] Onboard 12.5% of ECH clusters to use `mget` task claiming (#196317)\n\nResolves https://github.com/elastic/response-ops-team/issues/239\r\n\r\n## Summary\r\n\r\nDeployed to cloud: deployment ID was `ab4e88d139f93d43024837d96144e7d4`.\r\nSince the deployment ID starts with an `a`, this should start with\r\n`mget` and I can see in the logs with the latest push that this is true\r\n\r\n<img width=\"2190\" alt=\"Screenshot 2024-10-15 at 2 59 20 PM\"\r\nsrc=\"https://github.com/user-attachments/assets/079bc4d8-365e-4ba6-b7a9-59fe506283d9\">\r\n\r\n\r\nDeployed to serverless: project ID was\r\n`d33d22a94ce246d091220eace2c4e4bb`. See in the logs: `Using claim\r\nstrategy mget as configured for deployment\r\nd33d22a94ce246d091220eace2c4e4bb`\r\n\r\nCo-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>","sha":"267efdf31fe9ae314b0bed99bc23db5452a2aaa3","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Feature:Task Manager","Team:ResponseOps","v9.0.0","backport:prev-minor","ci:cloud-deploy","ci:project-deploy-elasticsearch","v8.16.0"],"title":"[Response Ops][Task Manager] Onboard 12.5% of ECH clusters to use `mget` task claiming","number":196317,"url":"https://github.com/elastic/kibana/pull/196317","mergeCommit":{"message":"[Response Ops][Task Manager] Onboard 12.5% of ECH clusters to use `mget` task claiming (#196317)\n\nResolves https://github.com/elastic/response-ops-team/issues/239\r\n\r\n## Summary\r\n\r\nDeployed to cloud: deployment ID was `ab4e88d139f93d43024837d96144e7d4`.\r\nSince the deployment ID starts with an `a`, this should start with\r\n`mget` and I can see in the logs with the latest push that this is true\r\n\r\n<img width=\"2190\" alt=\"Screenshot 2024-10-15 at 2 59 20 PM\"\r\nsrc=\"https://github.com/user-attachments/assets/079bc4d8-365e-4ba6-b7a9-59fe506283d9\">\r\n\r\n\r\nDeployed to serverless: project ID was\r\n`d33d22a94ce246d091220eace2c4e4bb`. See in the logs: `Using claim\r\nstrategy mget as configured for deployment\r\nd33d22a94ce246d091220eace2c4e4bb`\r\n\r\nCo-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>","sha":"267efdf31fe9ae314b0bed99bc23db5452a2aaa3"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196317","number":196317,"mergeCommit":{"message":"[Response Ops][Task Manager] Onboard 12.5% of ECH clusters to use `mget` task claiming (#196317)\n\nResolves https://github.com/elastic/response-ops-team/issues/239\r\n\r\n## Summary\r\n\r\nDeployed to cloud: deployment ID was `ab4e88d139f93d43024837d96144e7d4`.\r\nSince the deployment ID starts with an `a`, this should start with\r\n`mget` and I can see in the logs with the latest push that this is true\r\n\r\n<img width=\"2190\" alt=\"Screenshot 2024-10-15 at 2 59 20 PM\"\r\nsrc=\"https://github.com/user-attachments/assets/079bc4d8-365e-4ba6-b7a9-59fe506283d9\">\r\n\r\n\r\nDeployed to serverless: project ID was\r\n`d33d22a94ce246d091220eace2c4e4bb`. See in the logs: `Using claim\r\nstrategy mget as configured for deployment\r\nd33d22a94ce246d091220eace2c4e4bb`\r\n\r\nCo-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>","sha":"267efdf31fe9ae314b0bed99bc23db5452a2aaa3"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Ying Mao <ying.mao@elastic.co>
Similar to #196317 In this PR, I'm flipping the mget feature flag to on for all elastic owned ECH clusters. Elastic owned clusters are determined by looking at `plugins.cloud?.isElasticStaffOwned`. ## To verify Observe the PR deployment which doesn't start with `a` or `b` yet is using the mget claim strategy by logging `Using claim strategy mget` on startup.
…ic#196757) Similar to elastic#196317 In this PR, I'm flipping the mget feature flag to on for all elastic owned ECH clusters. Elastic owned clusters are determined by looking at `plugins.cloud?.isElasticStaffOwned`. ## To verify Observe the PR deployment which doesn't start with `a` or `b` yet is using the mget claim strategy by logging `Using claim strategy mget` on startup. (cherry picked from commit 97f2a90)
…ic#196757) Similar to elastic#196317 In this PR, I'm flipping the mget feature flag to on for all elastic owned ECH clusters. Elastic owned clusters are determined by looking at `plugins.cloud?.isElasticStaffOwned`. ## To verify Observe the PR deployment which doesn't start with `a` or `b` yet is using the mget claim strategy by logging `Using claim strategy mget` on startup. (cherry picked from commit 97f2a90)
… use `mget` task claiming (elastic#196317)" This reverts commit 267efdf.
Resolves https://github.com/elastic/response-ops-team/issues/239
Summary
Deployed to cloud: deployment ID was
ab4e88d139f93d43024837d96144e7d4
. Since the deployment ID starts with ana
, this should start withmget
and I can see in the logs with the latest push that this is trueDeployed to serverless: project ID was
d33d22a94ce246d091220eace2c4e4bb
. See in the logs:Using claim strategy mget as configured for deployment d33d22a94ce246d091220eace2c4e4bb