Skip to content

[8.15] [Automatic Import] Fix deploy error after editing the pipeline (#194203)#194216

Merged
kibanamachine merged 1 commit intoelastic:8.15from
kibanamachine:backport/8.15/pr-194203
Sep 26, 2024
Merged

[8.15] [Automatic Import] Fix deploy error after editing the pipeline (#194203)#194216
kibanamachine merged 1 commit intoelastic:8.15from
kibanamachine:backport/8.15/pr-194203

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.15:

Questions ?

Please refer to the Backport tool documentation

…ic#194203)

## Release Note

Fixes a bug that is causing the deploy step to fail after a pipeline
edit/save.

## Summary

elastic#190407 introduced a bug that deployment fails when a pipeline is edited
and saved in the review step.

The issue is that after the edit pipeline flow is executed the review
step's result is overridden and `samplesFormat` is removed which if not
present [the `useEffect` in Deploy
step](https://github.com/elastic/kibana/blob/main/x-pack/plugins/integration_assistant/public/components/create_integration/create_integration_assistant/steps/deploy_step/use_deploy_integration.ts#L41)
fails.

This PR fixes the issue by saving the `samplesFormat` that is present in
the original result before the edit pipeline flow is executed there by
having samplesFormat in the result.

(cherry picked from commit 6366dc3)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Sep 26, 2024
@kibanamachine kibanamachine enabled auto-merge (squash) September 26, 2024 19:14
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
integrationAssistant 947.9KB 947.9KB +36.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @bhapas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants