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

[Fleet] Use new package install state machine in installCustomPackage #189347

Closed
2 tasks
criamico opened this issue Jul 29, 2024 · 1 comment · Fixed by #192438
Closed
2 tasks

[Fleet] Use new package install state machine in installCustomPackage #189347

criamico opened this issue Jul 29, 2024 · 1 comment · Fixed by #192438
Assignees
Labels
Team:Fleet Team label for Observability Data Collection Fleet team

Comments

@criamico
Copy link
Contributor

criamico commented Jul 29, 2024

Follow up of #175592.

With 8.14.0 a new state machine functionality to install packages was introduced, but it was only enabled in install by registry and put under feature flag enablePackagesStateMachine.

  • Introduce the functionality also in installCustomPackage, similar to what we did in installPackageWitStateMachine
  • Use a new feature flag to trigger this functionality - enablePackagesStateMachine is currently enabled in production se it can be called enablePackagesStateMachineV2
@criamico criamico added the Team:Fleet Team label for Observability Data Collection Fleet team label Jul 29, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@criamico criamico changed the title [Fleet] Use new package install state machine in installCustomPackage [Fleet] Use new package install state machine in installCustomPackage Jul 29, 2024
@criamico criamico self-assigned this Sep 10, 2024
gergoabraham pushed a commit to gergoabraham/kibana that referenced this issue Sep 13, 2024
…ate machine and clean up (elastic#192438)

Closes elastic#189346
Closes elastic#189347

## Summary
Final PR for part I of package lifecycle improvements. 

- Migrate both `installPackageByUpload` and `installCustomPackage` to
`installPackageWithStateMachine` and clean up the old code. As the state
machine is been working in production for two releases, I am confident
in using it for the custom and upload packages without a feature flag.

### Testing

- Try installing a custom package
- Try installing a package via upload

They should work as usual

### Checklist
- [ ]
[Documentation](https://www.elastic.co/guide/en/kibana/master/development-documentation.html)
was added for features that require explanation or tutorials
- [ ] [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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Fleet Team label for Observability Data Collection Fleet team
Projects
None yet
2 participants