Skip to content

Make task_cost_check test resilient to changes in order#204045

Merged
gsoldevila merged 1 commit intoelastic:mainfrom
gsoldevila:kbn-task-cost-check-update
Dec 12, 2024
Merged

Make task_cost_check test resilient to changes in order#204045
gsoldevila merged 1 commit intoelastic:mainfrom
gsoldevila:kbn-task-cost-check-update

Conversation

@gsoldevila
Copy link
Copy Markdown
Member

@gsoldevila gsoldevila commented Dec 12, 2024

Summary

Addresses failures such as https://buildkite.com/elastic/kibana-pull-request/builds/259739#0193bb07-c759-4749-965e-10e63ac0810a.

We believe the order in which task types are registered might have been affected by the relocation of the @kbn/observability-ai-assistant-app-plugin in the scope of Sustainable Kibana Architecture.

@gsoldevila gsoldevila added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// backport:prev-minor labels Dec 12, 2024
@gsoldevila gsoldevila requested a review from a team as a code owner December 12, 2024 14:59
@elasticmachine
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for fixing!

@gsoldevila gsoldevila enabled auto-merge (squash) December 12, 2024 15:10
@gsoldevila gsoldevila merged commit a4e4a60 into elastic:main Dec 12, 2024
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12300990470

@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #42 / discover/security/context_awareness security root profile cell renderers host.name DataView mode should open host.name flyout

Metrics [docs]

✅ unchanged

@kibanamachine
Copy link
Copy Markdown
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

You might need to backport the following PRs to 8.x:
- Disable Inference Connector experimental feature (#196036)

Manual backport

To create the backport manually run:

node scripts/backport --pr 204045

Questions ?

Please refer to the Backport tool documentation

@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 204045 locally

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Dec 13, 2024
gsoldevila added a commit to gsoldevila/kibana that referenced this pull request Dec 16, 2024
…4045)

## Summary

Addresses failures such as
https://buildkite.com/elastic/kibana-pull-request/builds/259739#0193bb07-c759-4749-965e-10e63ac0810a.

We believe the order in which task types are registered might have been
affected by the relocation of the
`@kbn/observability-ai-assistant-app-plugin` in the scope of
_Sustainable Kibana Architecture_.

(cherry picked from commit a4e4a60)

# Conflicts:
#	x-pack/plugins/task_manager/server/integration_tests/__snapshots__/task_cost_check.test.ts.snap
@gsoldevila
Copy link
Copy Markdown
Member Author

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

gsoldevila added a commit that referenced this pull request Dec 16, 2024
) (#204344)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Make `task_cost_check` test resilient to changes in order
(#204045)](#204045)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Gerard
Soldevila","email":"gerard.soldevila@elastic.co"},"sourceCommit":{"committedDate":"2024-12-12T16:50:03Z","message":"Make
`task_cost_check` test resilient to changes in order (#204045)\n\n##
Summary\r\n\r\nAddresses failures such
as\r\nhttps://buildkite.com/elastic/kibana-pull-request/builds/259739#0193bb07-c759-4749-965e-10e63ac0810a.\r\n\r\nWe
believe the order in which task types are registered might have
been\r\naffected by the relocation of
the\r\n`@kbn/observability-ai-assistant-app-plugin` in the scope
of\r\n_Sustainable Kibana
Architecture_.","sha":"a4e4a6061bdd6d75ae82dd005c507b2e4ed5f230","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:ResponseOps","backport
missing","v9.0.0","backport:prev-minor"],"number":204045,"url":"https://github.com/elastic/kibana/pull/204045","mergeCommit":{"message":"Make
`task_cost_check` test resilient to changes in order (#204045)\n\n##
Summary\r\n\r\nAddresses failures such
as\r\nhttps://buildkite.com/elastic/kibana-pull-request/builds/259739#0193bb07-c759-4749-965e-10e63ac0810a.\r\n\r\nWe
believe the order in which task types are registered might have
been\r\naffected by the relocation of
the\r\n`@kbn/observability-ai-assistant-app-plugin` in the scope
of\r\n_Sustainable Kibana
Architecture_.","sha":"a4e4a6061bdd6d75ae82dd005c507b2e4ed5f230"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/204045","number":204045,"mergeCommit":{"message":"Make
`task_cost_check` test resilient to changes in order (#204045)\n\n##
Summary\r\n\r\nAddresses failures such
as\r\nhttps://buildkite.com/elastic/kibana-pull-request/builds/259739#0193bb07-c759-4749-965e-10e63ac0810a.\r\n\r\nWe
believe the order in which task types are registered might have
been\r\naffected by the relocation of
the\r\n`@kbn/observability-ai-assistant-app-plugin` in the scope
of\r\n_Sustainable Kibana
Architecture_.","sha":"a4e4a6061bdd6d75ae82dd005c507b2e4ed5f230"}}]}]
BACKPORT-->
@kibanamachine kibanamachine added v8.18.0 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Platform ResponseOps team (formerly the Cases and Alerting teams) t// v8.18.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants