[Fleet] Do not uninstall package on reinstall error

[nchaulet]

Summary

We currently uninstalling package on reinstall error, it seems to me it's a dangerous behaviour, reinstall can come from an automation like upgrading kibana, and I think uninstalling the package in this case can make things worse, and we probably should keep the package and mark it as install_failed, (this is what happening when a rollback after an upgrade fail for example)

Also we are going to show install_failed in installed integrations (Related issue here) we will be able to show those errors to the user.

[elasticmachine]

Pinging @elastic/fleet (Team:Fleet)

[apmmachine]

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

• /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
• /oblt-deploy-serverless : Deploy a serverless Kibana instance using the Observability test environments.
• run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

[nchaulet]

@elasticmachine merge upstream

[kpollich]

LGTM 🚀

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 042dd9a

Metrics [docs]

✅ unchanged

History

• 💛 Build #178639 was flaky dee3a5b
• 💚 Build #178606 succeeded 36f76eb

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