Skip to content

[upgrade assistant] Create upgrade-assistant package, stub out storybook usage#218705

Merged
mattkime merged 18 commits intoelastic:mainfrom
mattkime:upgrade_assistant_storybook_stub
Apr 28, 2025
Merged

[upgrade assistant] Create upgrade-assistant package, stub out storybook usage#218705
mattkime merged 18 commits intoelastic:mainfrom
mattkime:upgrade_assistant_storybook_stub

Conversation

@mattkime
Copy link
Copy Markdown
Contributor

@mattkime mattkime commented Apr 21, 2025

Summary

This PR creates an upgrade-assistant package, moves a component to the package, and creates a storybook story for it. Run yarn storybook upgrade_assistant to see it.

The intention is to make it easier to work on upgrade assistant components in isolation from kibana and the state necessary to display them.

@mattkime mattkime changed the title stubbed out, need initial story [upgrade assistant] Create upgrade-assistant package, stub out storybook usage Apr 21, 2025
@mattkime mattkime self-assigned this Apr 21, 2025
@mattkime mattkime added Feature:Upgrade Assistant Team:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// labels Apr 21, 2025
@mattkime mattkime added backport:version Backport to applied version labels v8.19.0 v9.1.0 release_note:skip Skip the PR/issue when compiling release notes labels Apr 21, 2025
@mattkime mattkime requested a review from ElenaStoeva April 21, 2025 04:42
@mattkime mattkime marked this pull request as ready for review April 21, 2025 04:43
@mattkime mattkime requested a review from a team as a code owner April 21, 2025 04:43
@mattkime mattkime requested review from a team as code owners April 21, 2025 04:43
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-management (Team:Kibana Management)

Copy link
Copy Markdown
Contributor

@jeramysoucy jeramysoucy left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@ElenaStoeva ElenaStoeva left a comment

Choose a reason for hiding this comment

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

Changes lgtm with a few small suggestions.

…nts/deprecation_badge.stories.tsx

Co-authored-by: Elena Stoeva <59341489+ElenaStoeva@users.noreply.github.com>
@mattkime mattkime requested a review from kibanamachine as a code owner April 27, 2025 03:38
mattkime and others added 2 commits April 26, 2025 22:38
@mattkime mattkime merged commit 6d96c1a into elastic:main Apr 28, 2025
10 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.19

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

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
upgradeAssistant 190 192 +2

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/upgrade-assistant - 2 +2

Async chunks

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

id before after diff
upgradeAssistant 207.8KB 207.8KB -1.0B

Page load bundle

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

id before after diff
upgradeAssistant 23.9KB 23.9KB -1.0B
Unknown metric groups

API count

id before after diff
@kbn/upgrade-assistant - 2 +2

History

cc @mattkime

@kibanamachine
Copy link
Copy Markdown
Contributor

💔 All backports failed

Status Branch Result
8.19 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 218705

Questions ?

Please refer to the Backport tool documentation

@mattkime
Copy link
Copy Markdown
Contributor Author

💚 All backports created successfully

Status Branch Result
8.19

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

Questions ?

Please refer to the Backport tool documentation

mattkime added a commit to mattkime/kibana that referenced this pull request Apr 28, 2025
…ook usage (elastic#218705)

## Summary

This PR creates an `upgrade-assistant` package, moves a component to the
package, and creates a storybook story for it. Run `yarn storybook
upgrade_assistant` to see it.

The intention is to make it easier to work on upgrade assistant
components in isolation from kibana and the state necessary to display
them.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Elena Stoeva <59341489+ElenaStoeva@users.noreply.github.com>
(cherry picked from commit 6d96c1a)

# Conflicts:
#	.github/CODEOWNERS
#	src/dev/storybook/aliases.ts
#	x-pack/.i18nrc.json
#	x-pack/platform/plugins/private/upgrade_assistant/tsconfig.json
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Apr 30, 2025
@kibanamachine
Copy link
Copy Markdown
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @mattkime

@kibanamachine
Copy link
Copy Markdown
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @mattkime

@kibanamachine
Copy link
Copy Markdown
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @mattkime

1 similar comment
@kibanamachine
Copy link
Copy Markdown
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @mattkime

@kibanamachine
Copy link
Copy Markdown
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @mattkime

@kibanamachine
Copy link
Copy Markdown
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @mattkime

@kibanamachine
Copy link
Copy Markdown
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.
cc: @mattkime

mattkime added a commit that referenced this pull request May 9, 2025
… storybook usage (#218705) (#219453)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[upgrade assistant] Create upgrade-assistant package, stub out
storybook usage
(#218705)](#218705)

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

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

<!--BACKPORT [{"author":{"name":"Matthew
Kime","email":"matt@mattki.me"},"sourceCommit":{"committedDate":"2025-04-28T18:45:43Z","message":"[upgrade
assistant] Create upgrade-assistant package, stub out storybook usage
(#218705)\n\n## Summary\n\nThis PR creates an `upgrade-assistant`
package, moves a component to the\npackage, and creates a storybook
story for it. Run `yarn storybook\nupgrade_assistant` to see it.\n\nThe
intention is to make it easier to work on upgrade assistant\ncomponents
in isolation from kibana and the state necessary to
display\nthem.\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\nCo-authored-by: Elena
Stoeva
<59341489+ElenaStoeva@users.noreply.github.com>","sha":"6d96c1a3df964b8df2374f07f424756aa6f40b4e","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Kibana
Management","release_note:skip","Feature:Upgrade
Assistant","backport:version","v9.1.0","v8.19.0"],"title":"[upgrade
assistant] Create upgrade-assistant package, stub out storybook
usage","number":218705,"url":"https://github.com/elastic/kibana/pull/218705","mergeCommit":{"message":"[upgrade
assistant] Create upgrade-assistant package, stub out storybook usage
(#218705)\n\n## Summary\n\nThis PR creates an `upgrade-assistant`
package, moves a component to the\npackage, and creates a storybook
story for it. Run `yarn storybook\nupgrade_assistant` to see it.\n\nThe
intention is to make it easier to work on upgrade assistant\ncomponents
in isolation from kibana and the state necessary to
display\nthem.\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\nCo-authored-by: Elena
Stoeva
<59341489+ElenaStoeva@users.noreply.github.com>","sha":"6d96c1a3df964b8df2374f07f424756aa6f40b4e"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/218705","number":218705,"mergeCommit":{"message":"[upgrade
assistant] Create upgrade-assistant package, stub out storybook usage
(#218705)\n\n## Summary\n\nThis PR creates an `upgrade-assistant`
package, moves a component to the\npackage, and creates a storybook
story for it. Run `yarn storybook\nupgrade_assistant` to see it.\n\nThe
intention is to make it easier to work on upgrade assistant\ncomponents
in isolation from kibana and the state necessary to
display\nthem.\n\n---------\n\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>\nCo-authored-by: Elena
Stoeva
<59341489+ElenaStoeva@users.noreply.github.com>","sha":"6d96c1a3df964b8df2374f07f424756aa6f40b4e"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 9, 2025
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request May 29, 2025
…ook usage (elastic#218705)

## Summary

This PR creates an `upgrade-assistant` package, moves a component to the
package, and creates a storybook story for it. Run `yarn storybook
upgrade_assistant` to see it.

The intention is to make it easier to work on upgrade assistant
components in isolation from kibana and the state necessary to display
them.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Elena Stoeva <59341489+ElenaStoeva@users.noreply.github.com>
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:Kibana Management Dev Tools, Index Management, Upgrade Assistant, ILM, Ingest Node Pipelines, and more t// v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants