Skip to content

Docker healthcheck (cont. of #3493)#3666

Merged
danielnelson merged 4 commits intoinfluxdata:masterfrom
jmccann:docker_healthcheck_passing_tests
Jan 13, 2018
Merged

Docker healthcheck (cont. of #3493)#3666
danielnelson merged 4 commits intoinfluxdata:masterfrom
jmccann:docker_healthcheck_passing_tests

Conversation

@jmccann
Copy link
Contributor

@jmccann jmccann commented Jan 12, 2018

This is a continuation of #3493

I fixed the tests to pass by adding mock data to the container inspect data that is being referenced.

Let me know if anything else is required.

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

@jmccann jmccann changed the title Docker healthcheck passing tests Docker healthcheck (cont. of #3493) Jan 12, 2018
@danielnelson danielnelson added this to the 1.6.0 milestone Jan 12, 2018
@danielnelson danielnelson added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Jan 12, 2018
@danielnelson danielnelson merged commit 7ab0d50 into influxdata:master Jan 13, 2018
@jmccann jmccann deleted the docker_healthcheck_passing_tests branch January 13, 2018 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants