-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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] Implement state machine behavior for package install #178657
Merged
criamico
merged 51 commits into
elastic:main
from
criamico:175592_package_lifecycle_state_machine
Apr 8, 2024
Merged
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
d4710a3
[Fleet] Implement state machine behavior for package install
criamico 94aaa4d
fix types and test
criamico 7234abd
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine b89756e
Improve naming
criamico 7e027ec
Add context and start writing new function
criamico 6a54196
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine f68cdcd
Add post transition function
criamico 28d4de0
Save data between iterations
criamico 59cd767
Improvements to state machine
criamico fffba51
Remove wrapper function and return updatedContext
criamico 6267ec6
Implement step functions
criamico 8c00a9b
remove changes to _install_packages
criamico addab4d
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine 04f0e2d
Naming changes and tests
criamico 25c5ea5
Add feature flag and plug new functions
criamico d92c638
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine c3d5537
Fix error in header functions and add logging
criamico c6f080f
Fix errors in install process and modify logging
criamico 8074328
Fix unit tests and add feature flag check
criamico 16ac037
Save latest executed state in saved object and improve unit tests
criamico 5f09e58
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine a82bd68
[CI] Auto-commit changed files from 'node scripts/check_mappings_upda…
kibanamachine cefff9b
Save error in saved object
criamico fa85aa6
Add unit tests
criamico 5f850b6
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine 18ded6f
Add openapi changes and fix wrong parameter name
criamico 12a0e33
[CI] Auto-commit changed files from 'node scripts/jest_integration -u…
kibanamachine 6059c93
Fix type error
criamico 5746610
Fixes to the logic, add tests
criamico f88752d
Split up steps in smaller functions and improve files organization
criamico e9ee148
Enable feature flag for integration tests
criamico 0f81778
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine 28edd3b
Fix filename
criamico 4a16aeb
Fix missing assets return
criamico beeaefd
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine 741065f
Fix postTransition call
criamico 6f7809c
Address code review comments
criamico 1beb1bb
remove console log
criamico dc95c45
fix integration test and add unit test
criamico f503c1d
Fix tests and enable feature flag
criamico 9d969e6
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine 44f1924
Unit test for create_restart_installation
criamico 9a09893
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine d49a317
Fix unit test
criamico 783ca85
Add more unit tests and try fixing mapping change
criamico 3a26fb4
Add unit test for stepDeletePreviousPipelines
criamico ef9a9fa
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine 179db75
More unit tests for install steps
criamico 11b1432
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine 4a96479
Add unit test for stepSaveSystemObject
criamico 117feaf
Merge branch 'main' into 175592_package_lifecycle_state_machine
kibanamachine File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Linter noticed that this parameter wasn't used anymore, I removed it