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

[UA] ML anomaly job indices special handling #211407

Merged
merged 10 commits into from
Feb 18, 2025

Conversation

jloleysens
Copy link
Contributor

@jloleysens jloleysens commented Feb 17, 2025

@jloleysens jloleysens added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc release_note:skip Skip the PR/issue when compiling release notes Feature:Upgrade Assistant backport:version Backport to applied version labels v8.18.0 v8.19.0 labels Feb 17, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@jloleysens jloleysens enabled auto-merge (squash) February 17, 2025 13:45
Comment on lines +82 to +95
let showEsTransformsGuidance = false;
let showMlAnomalyReindexingGuidance = false;
let showReadOnlyGuidance = false;
let showDefaultGuidance = false;

if (isESTransformTarget) {
showEsTransformsGuidance = true;
} else if (meta.isReadonly) {
showReadOnlyGuidance = true;
} else if (isMLAnomalyIndex) {
showMlAnomalyReindexingGuidance = true;
} else {
showDefaultGuidance = true;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Attempted to clarify this logic... We should only be showing one of these instances of guidance at a time.

@jloleysens jloleysens merged commit 5d3fb74 into elastic:8.x Feb 18, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
upgradeAssistant 192 193 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
aiAssistantManagementSelection 96.1KB 96.2KB +125.0B
lists 140.1KB 140.2KB +125.0B
upgradeAssistant 202.6KB 205.1KB +2.6KB
total +2.8KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 440.5KB 440.6KB +125.0B
upgradeAssistant 23.8KB 23.8KB -2.0B
total +123.0B

History

@jloleysens
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.18

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

Questions ?

Please refer to the Backport tool documentation

jloleysens added a commit to jloleysens/kibana that referenced this pull request Feb 19, 2025
jloleysens added a commit that referenced this pull request Feb 19, 2025
# Backport

This will backport the following commits from `8.x` to `8.18`:
- [[UA] ML anomaly job indices special handling
(#211407)](#211407)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Jean-Louis
Leysens","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-18T11:40:27Z","message":"[UA]
ML anomaly job indices special handling (#211407)\n\nClose
https://github.com/elastic/kibana-team/issues/1501\r\n\r\n\r\n<img
width=\"535\" alt=\"Screenshot 2025-02-17 at 11 43
19\"\r\nsrc=\"https://github.com/user-attachments/assets/648cc007-aef8-4959-add8-6aec943e9e41\"\r\n/>","sha":"5d3fb74647d24decd1adca486004244809b79957","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Core","release_note:skip","Feature:Upgrade
Assistant","backport:version","v8.18.0","v8.19.0"],"title":"[UA] ML
anomaly job indices special
handling","number":211407,"url":"https://github.com/elastic/kibana/pull/211407","mergeCommit":{"message":"[UA]
ML anomaly job indices special handling (#211407)\n\nClose
https://github.com/elastic/kibana-team/issues/1501\r\n\r\n\r\n<img
width=\"535\" alt=\"Screenshot 2025-02-17 at 11 43
19\"\r\nsrc=\"https://github.com/user-attachments/assets/648cc007-aef8-4959-add8-6aec943e9e41\"\r\n/>","sha":"5d3fb74647d24decd1adca486004244809b79957"}},"sourceBranch":"8.x","suggestedTargetBranches":["8.18"],"targetPullRequestStates":[{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels Feature:Upgrade Assistant release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v8.18.0 v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants