healthcheck support in docker-compose configuration

[StasShymov]

• Docker-compose healthcheck configuration support
• Docker container wait timeout default value made configurable using startContainerWaitTimeout configuration option

[rohanKanojia]

@StasShymov : Hello, do you think changes in this pull request collide with #1736 (it's a draft PR that was created some time ago but author didn't get time to fix it) ?

[StasShymov]

@rohanKanojia yes, partially
Both are adding support for healthcheck configuration in docker compose.
But mine doesn't touch any existing <run> definitions, since there is already <wait>:

Add <healthCheck> to <run> as known from <build>

[codecov]

Codecov Report

Attention: Patch coverage is 84.84848% with 15 lines in your changes missing coverage. Please review.

Project coverage is 66.24%. Comparing base (150970d) to head (ca38e86).
Report is 21 commits behind head on master.

Files with missing lines	Patch %	Lines
.../maven/docker/access/util/ComposeDurationUtil.java	82.14%	3 Missing and 2 partials ⚠️
...g/handler/compose/DockerComposeServiceWrapper.java	86.11%	1 Missing and 4 partials ⚠️
...c/main/java/io/fabric8/maven/docker/StartMojo.java	0.00%	2 Missing ⚠️
...a/io/fabric8/maven/docker/service/WaitService.java	0.00%	2 Missing ⚠️
...ic8/maven/docker/access/ContainerCreateConfig.java	96.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1825      +/-   ##
============================================
+ Coverage     65.87%   66.24%   +0.37%     
- Complexity     2306     2346      +40     
============================================
  Files           172      174       +2     
  Lines         10194    10342     +148     
  Branches       1408     1434      +26     
============================================
+ Hits           6715     6851     +136     
- Misses         2926     2928       +2     
- Partials        553      563      +10     

Files with missing lines	Coverage Δ
...ic8/maven/docker/config/RunImageConfiguration.java	91.25% <100.00%> (+0.14%)	⬆️
...ig/handler/compose/DockerComposeConfigHandler.java	73.45% <100.00%> (+0.23%)	⬆️
...va/io/fabric8/maven/docker/service/RunService.java	68.10% <100.00%> (+0.10%)	⬆️
...ic8/maven/docker/access/ContainerCreateConfig.java	88.07% <96.00%> (+2.19%)	⬆️
...c/main/java/io/fabric8/maven/docker/StartMojo.java	0.00% <0.00%> (ø)
...a/io/fabric8/maven/docker/service/WaitService.java	4.16% <0.00%> (-0.08%)	⬇️
.../maven/docker/access/util/ComposeDurationUtil.java	82.14% <82.14%> (ø)
...g/handler/compose/DockerComposeServiceWrapper.java	68.53% <86.11%> (+2.53%)	⬆️

... and 3 files with indirect coverage changes

[StasShymov]

@rohanKanojia is there anything I can do to help this patch go through?

[rohanKanojia]

@StasShymov : Let me try to get this merged this weekend.

[rohanKanojia]

Could you please revert these wildcard import changes?

[rohanKanojia]

@StasShymov : Is it possible to write an integration test for this functionality? I see there are integration tests for docker-compose in it/ folder .

[StasShymov]

@rohanKanojia I updated the one that already exists, does it look ok to you?

[sonarcloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
87.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[StasShymov]

I confirmed that it/docker-compose-dependon is still green after my changes

[rohanKanojia]

@StasShymov : Thanks a lot!