Skip to content

[18.09] Remove unused health check func - #1406

Merged
silvin-lubecki merged 1 commit into
docker:18.09from
dhiltgen:revamp_18.09
Oct 2, 2018
Merged

[18.09] Remove unused health check func#1406
silvin-lubecki merged 1 commit into
docker:18.09from
dhiltgen:revamp_18.09

Conversation

@dhiltgen

Copy link
Copy Markdown
Contributor

Follow up from #1388

During the refactoring for 18.09 the activate/update flows no longer
restart the engine explicitly but let the user do that when they're ready,
so the health check logic is no longer required.

Signed-off-by: Daniel Hiltgen daniel.hiltgen@docker.com

During the refactoring for 18.09 the activate/update flows no longer
restart the engine explicitly but let the user do that when they're ready,
so the health check logic is no longer required.
Signed-off-by: Daniel Hiltgen <daniel.hiltgen@docker.com>
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #1406 into 18.09 will increase coverage by 0.02%.
The diff coverage is 60%.

@@ Coverage Diff @@## 18.09 #1406 +/- ##
==========================================
+ Coverage 54.09% 54.12% +0.02% 
==========================================
Files 290 290 Lines 19313 19303 -10 ==========================================
- Hits 10448 10447 -1 + Misses 8201 8192 -9 
Partials 664 664

@vdemeestervdemeester left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@silvin-lubeckisilvin-lubecki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@silvin-lubecki
silvin-lubecki merged commit 3e1a0bd into docker:18.09Oct 2, 2018
@thaJeztahthaJeztah changed the title Remove unused helath check func[18.09] Remove unused health check funcOct 2, 2018
@thaJeztahthaJeztah added this to the 18.09.0 milestone Oct 2, 2018
@thaJeztah

Copy link
Copy Markdown
Member

Thanks!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@dhiltgen@codecov-io@thaJeztah@vdemeester@silvin-lubecki@GordonTheTurtle