Skip to content

only transform legend if present#213814

Merged
TinaHeiligers merged 2 commits intoelastic:mainfrom
TinaHeiligers:handle-optional-legend
Mar 10, 2025
Merged

only transform legend if present#213814
TinaHeiligers merged 2 commits intoelastic:mainfrom
TinaHeiligers:handle-optional-legend

Conversation

@TinaHeiligers
Copy link
Copy Markdown
Contributor

@TinaHeiligers TinaHeiligers commented Mar 10, 2025

The inventory_view saved object allows the legend attribute to be optional and limits and/or sets the number of steps during an upgrade between model version 1 and 2. The transform function needs to handle cases where legend is not set to prevent migration failures.

Related to #207007

@TinaHeiligers TinaHeiligers added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 backport:version Backport to applied version labels v8.18.0 v8.19.0 labels Mar 10, 2025
@TinaHeiligers TinaHeiligers requested a review from a team March 10, 2025 18:33
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

@TinaHeiligers TinaHeiligers merged commit 85baab2 into elastic:main Mar 10, 2025
9 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.18, 8.x, 9.0

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 10, 2025
The inventory_view saved object allows the legend attribute to be
optional and limits and/or sets the number of steps during an upgrade
between model version 1 and 2. The transform function needs to handle
cases where legend is not set to prevent migration failures.

Related to elastic#207007

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 85baab2)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 10, 2025
The inventory_view saved object allows the legend attribute to be
optional and limits and/or sets the number of steps during an upgrade
between model version 1 and 2. The transform function needs to handle
cases where legend is not set to prevent migration failures.

Related to elastic#207007

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 85baab2)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 10, 2025
The inventory_view saved object allows the legend attribute to be
optional and limits and/or sets the number of steps during an upgrade
between model version 1 and 2. The transform function needs to handle
cases where legend is not set to prevent migration failures.

Related to elastic#207007

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 85baab2)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.18
8.x
9.0

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Mar 10, 2025
# Backport

This will backport the following commits from `main` to `9.0`:
- [only transform legend if present
(#213814)](#213814)

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

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

<!--BACKPORT [{"author":{"name":"Christiane (Tina)
Heiligers","email":"christiane.heiligers@elastic.co"},"sourceCommit":{"committedDate":"2025-03-10T22:03:56Z","message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"only
transform legend if
present","number":213814,"url":"https://github.com/elastic/kibana/pull/213814","mergeCommit":{"message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f"}},"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/213814","number":213814,"mergeCommit":{"message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Christiane (Tina) Heiligers <christiane.heiligers@elastic.co>
kibanamachine added a commit that referenced this pull request Mar 10, 2025
# Backport

This will backport the following commits from `main` to `8.x`:
- [only transform legend if present
(#213814)](#213814)

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

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

<!--BACKPORT [{"author":{"name":"Christiane (Tina)
Heiligers","email":"christiane.heiligers@elastic.co"},"sourceCommit":{"committedDate":"2025-03-10T22:03:56Z","message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"only
transform legend if
present","number":213814,"url":"https://github.com/elastic/kibana/pull/213814","mergeCommit":{"message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f"}},"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/213814","number":213814,"mergeCommit":{"message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Christiane (Tina) Heiligers <christiane.heiligers@elastic.co>
kibanamachine added a commit that referenced this pull request Mar 10, 2025
# Backport

This will backport the following commits from `main` to `8.18`:
- [only transform legend if present
(#213814)](#213814)

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

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

<!--BACKPORT [{"author":{"name":"Christiane (Tina)
Heiligers","email":"christiane.heiligers@elastic.co"},"sourceCommit":{"committedDate":"2025-03-10T22:03:56Z","message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"only
transform legend if
present","number":213814,"url":"https://github.com/elastic/kibana/pull/213814","mergeCommit":{"message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f"}},"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/213814","number":213814,"mergeCommit":{"message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Christiane (Tina) Heiligers <christiane.heiligers@elastic.co>
@TinaHeiligers TinaHeiligers added backport:8.17 v8.17.0 backport:version Backport to applied version labels and removed backport:version Backport to applied version labels labels Mar 13, 2025
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.17, 8.18, 8.x, 9.0

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 13, 2025
The inventory_view saved object allows the legend attribute to be
optional and limits and/or sets the number of steps during an upgrade
between model version 1 and 2. The transform function needs to handle
cases where legend is not set to prevent migration failures.

Related to elastic#207007

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 85baab2)
@kibanamachine
Copy link
Copy Markdown
Contributor

💔 Some backports could not be created

Status Branch Result
8.17
8.18 Cherrypick failed because the selected commit (85baab2) is empty. It looks like the commit was already backported in #213838
8.x Cherrypick failed because the selected commit (85baab2) is empty. It looks like the commit was already backported in #213839
9.0 Cherrypick failed because the selected commit (85baab2) is empty. It looks like the commit was already backported in #213840

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 213814

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Mar 13, 2025
# Backport

This will backport the following commits from `main` to `8.17`:
- [only transform legend if present
(#213814)](#213814)

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

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

<!--BACKPORT [{"author":{"name":"Christiane (Tina)
Heiligers","email":"christiane.heiligers@elastic.co"},"sourceCommit":{"committedDate":"2025-03-10T22:03:56Z","message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:version","v8.17.0","v8.18.0","backport:8.17","v9.1.0","v8.19.0"],"title":"only
transform legend if
present","number":213814,"url":"https://github.com/elastic/kibana/pull/213814","mergeCommit":{"message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f"}},"sourceBranch":"main","suggestedTargetBranches":["8.17"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/213840","number":213840,"state":"MERGED","mergeCommit":{"sha":"755e23ac6765e64ce9f68ccfdad5ef4eb394467f","message":"[9.0]
only transform legend if present (#213814) (#213840)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`9.0`:\n- [only transform legend if
present\n(#213814)](https://github.com/elastic/kibana/pull/213814)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Christiane (Tina) Heiligers
<christiane.heiligers@elastic.co>"}},{"branch":"8.17","label":"v8.17.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,"url":"https://github.com/elastic/kibana/pull/213838","number":213838,"state":"MERGED","mergeCommit":{"sha":"fb7e387a319929eeeafc8b394355c76aad168329","message":"[8.18]
only transform legend if present (#213814) (#213838)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.18`:\n- [only transform legend if
present\n(#213814)](https://github.com/elastic/kibana/pull/213814)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Christiane (Tina) Heiligers
<christiane.heiligers@elastic.co>"}},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/213814","number":213814,"mergeCommit":{"message":"only
transform legend if present (#213814)\n\nThe inventory_view saved object
allows the legend attribute to be\noptional and limits and/or sets the
number of steps during an upgrade\nbetween model version 1 and 2. The
transform function needs to handle\ncases where legend is not set to
prevent migration failures.\n\nRelated to
https://github.com/elastic/kibana/pull/207007\n\n- [x] [Unit or
functional\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\nwere
updated or added to match the most common
scenarios","sha":"85baab2431bea918a68d752ec912b792ee93e00f"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/213839","number":213839,"state":"MERGED","mergeCommit":{"sha":"d7675f0b32ecfca0a01b10a9779ae492f1e652db","message":"[8.x]
only transform legend if present (#213814) (#213839)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.x`:\n- [only transform legend if
present\n(#213814)](https://github.com/elastic/kibana/pull/213814)\n\n\n\n###
Questions ?\nPlease refer to the [Backport
tool\ndocumentation](https://github.com/sorenlouv/backport)\n\n\n\nCo-authored-by:
Christiane (Tina) Heiligers <christiane.heiligers@elastic.co>"}}]}]
BACKPORT-->

Co-authored-by: Christiane (Tina) Heiligers <christiane.heiligers@elastic.co>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Mar 22, 2025
The inventory_view saved object allows the legend attribute to be
optional and limits and/or sets the number of steps during an upgrade
between model version 1 and 2. The transform function needs to handle
cases where legend is not set to prevent migration failures.

Related to elastic#207007

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
@Danouchka
Copy link
Copy Markdown

Danouchka commented Mar 25, 2025

Hi @CAWilson94 , is the fix available on ECH 8.17.4 that is released today ?

@rudolf rudolf added the Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. label Mar 25, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@rudolf rudolf added the bug Fixes for quality problems that affect the customer experience label Mar 25, 2025
@Danouchka
Copy link
Copy Markdown

Hi @CAWilson94 @rudolf @bryce-b , is the fix available on ECH 8.17.4 that was released yesterday ?

@rudolf
Copy link
Copy Markdown
Contributor

rudolf commented Mar 26, 2025

Yes, this fix was included in 8.17.4

@rudolf rudolf deleted the handle-optional-legend branch March 26, 2025 15:16
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 bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. v8.17.0 v8.17.4 v8.18.0 v8.19.0 v9.0.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants