[Backport release-5.1.0] Add container start retry #3893
Merged
Conversation
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
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
* Added container start retry policy Signed-off-by: Maiero <[email protected]> * Started refactoring and callback Signed-off-by: Maiero <[email protected]> * Updated container start and stop logic. Started updating tests Signed-off-by: Maiero <[email protected]> * More tests updates Signed-off-by: Maiero <[email protected]> * More APIs changes. Splitted container stop and delete Signed-off-by: Maiero <[email protected]> * Moved apis to apis bundle. Other improvements Signed-off-by: Maiero <[email protected]> * Fixed dockerservice references Signed-off-by: Maiero <[email protected]> * Removed direct container state management Signed-off-by: Maiero <[email protected]> * Fixed wrong containerId reference Signed-off-by: Maiero <[email protected]> * ContainerDescriptor now is immutable Signed-off-by: Maiero <[email protected]> * Renamed ConfigurableGenericDockerService Signed-off-by: Maiero <[email protected]> * Removed unneeded methods with ContainerDescriptor Added javadoc Signed-off-by: Maiero <[email protected]> * Javadoc cleanup Signed-off-by: Maiero <[email protected]> * Image download timeout configurable Signed-off-by: Maiero <[email protected]> * Updated container instance container lifecycle Signed-off-by: Maiero <[email protected]> * Added ability to interrupt image pull Changed ContainerInstance lifecycle Removed container name from DockerService.registerListener Signed-off-by: Nicola Timeus <[email protected]> * Fixes in DockerServiceImpl Signed-off-by: Nicola Timeus <[email protected]> * Restored disabled tests Signed-off-by: Nicola Timeus <[email protected]> * Updated port parsing check Signed-off-by: Nicola Timeus <[email protected]> * Split ContainerDescriptor wip Signed-off-by: Nicola Timeus <[email protected]> * fixed and added more tests * Minor renamings in api Renewed equals and hash methods in API pojos Signed-off-by: Maiero <[email protected]> * Renamed dockerservice to ContainerOrchestrationService Signed-off-by: Maiero <[email protected]> * Fixed reference in tests Signed-off-by: Maiero <[email protected]> * More renamings Signed-off-by: Maiero <[email protected]> * Improved ContainerInstance behavior in startup with already existing container Signed-off-by: Nicola Timeus <[email protected]> * Small name change Signed-off-by: Maiero <[email protected]> * Moved existing container check to container orchestrator Signed-off-by: Nicola Timeus <[email protected]> Co-authored-by: Nicola Timeus <[email protected]> Co-authored-by: Gregory Ivo <[email protected]>
MMaiero
approved these changes
Mar 15, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Backport d032204 from #3869