Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-12-13 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review110144
---


Patch looks great!

Reviews applied: [36816]

Passed command: export OS=ubuntu:14.04;export CONFIGURATION="--verbose";export 
COMPILER=gcc; ./support/docker_build.sh

- Mesos ReviewBot


On Dec. 12, 2015, 4:08 p.m., haosdent huang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36816/
> ---
> 
> (Updated Dec. 12, 2015, 4:08 p.m.)
> 
> 
> Review request for mesos, Adam B, Michael Park, and Timothy Chen.
> 
> 
> Bugs: MESOS-2533
> https://issues.apache.org/jira/browse/MESOS-2533
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Support HTTP checks in Mesos health check program
> 
> 
> Diffs
> -
> 
>   include/mesos/mesos.proto 8ca213062c480f0266ffc51a621eb4a118140c77 
>   src/docker/executor.cpp 4042cec0acbe03d937ea3c53ffde745cbba552d2 
>   src/health-check/main.cpp 0beaed575ec865d81e6e3d83d8a0c894613acba4 
>   src/tests/health_check_tests.cpp 0fbccc373204d3b9431c614bdd6d046cc07e4566 
> 
> Diff: https://reviews.apache.org/r/36816/diff/
> 
> 
> Testing
> ---
> 
> * Add unit test cases: HealthCheckTest.HealthyTaskThroughHttp and 
> HealthCheckTest.HealthyTaskNotMatchHttpStatuses
> make check
> 
> 
> Thanks,
> 
> haosdent huang
> 
>

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-12-12 Thread haosdent huang 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/
---

(Updated Dec. 12, 2015, 4:08 p.m.)


Review request for mesos, Adam B, Michael Park, and Timothy Chen.


Bugs: MESOS-2533
https://issues.apache.org/jira/browse/MESOS-2533


Repository: mesos


Description
---

Support HTTP checks in Mesos health check program


Diffs
-

  include/mesos/mesos.proto 8ca213062c480f0266ffc51a621eb4a118140c77 
  src/docker/executor.cpp 4042cec0acbe03d937ea3c53ffde745cbba552d2 
  src/health-check/main.cpp 0beaed575ec865d81e6e3d83d8a0c894613acba4 
  src/tests/health_check_tests.cpp 0fbccc373204d3b9431c614bdd6d046cc07e4566 

Diff: https://reviews.apache.org/r/36816/diff/


Testing (updated)
---

* Add unit test cases: HealthCheckTest.HealthyTaskThroughHttp and 
HealthCheckTest.HealthyTaskNotMatchHttpStatuses
make check


Thanks,

haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-12-12 Thread haosdent huang 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/
---

(Updated Dec. 12, 2015, 4:08 p.m.)


Review request for mesos, Adam B, Michael Park, and Timothy Chen.


Changes
---

Address comments


Bugs: MESOS-2533
https://issues.apache.org/jira/browse/MESOS-2533


Repository: mesos


Description
---

Support HTTP checks in Mesos health check program


Diffs (updated)
-

  include/mesos/mesos.proto 8ca213062c480f0266ffc51a621eb4a118140c77 
  src/docker/executor.cpp 4042cec0acbe03d937ea3c53ffde745cbba552d2 
  src/health-check/main.cpp 0beaed575ec865d81e6e3d83d8a0c894613acba4 
  src/tests/health_check_tests.cpp 0fbccc373204d3b9431c614bdd6d046cc07e4566 

Diff: https://reviews.apache.org/r/36816/diff/


Testing
---

* Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
make check


Thanks,

haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-11-17 Thread Timothy Chen 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review106892
---



src/health-check/main.cpp (line 238)


Seems weird to perform a http check with not statuses to check. Instead of 
returning silently I think we should send an error instead since the user might 
as well don't perform this check right?



src/health-check/main.cpp (line 243)


Please log VLOG(2) the http url we're trying to health check



src/health-check/main.cpp (line 263)


Response should also have a code uint32_t field now, you can check that 
directly without checking strings.



src/health-check/main.cpp (line 295)


This formatting problem somehow slipped in, we need to align the << in both 
lines.



src/tests/health_check_tests.cpp (line 930)


Can you also test that it fails the health check correctly when status 
doesn't match?


- Timothy Chen


On Oct. 3, 2015, 6:01 p.m., haosdent huang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36816/
> ---
> 
> (Updated Oct. 3, 2015, 6:01 p.m.)
> 
> 
> Review request for mesos, Adam B, Michael Park, and Timothy Chen.
> 
> 
> Bugs: MESOS-2533
> https://issues.apache.org/jira/browse/MESOS-2533
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Support HTTP checks in Mesos health check program
> 
> 
> Diffs
> -
> 
>   include/mesos/mesos.proto 4a16be1f570769f3ce42a50a9da9f4fb1c227999 
>   src/docker/executor.cpp 1e4901335854c49e46cd7b132e79ccb11cd72ade 
>   src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
>   src/tests/health_check_tests.cpp ff6275b19206b49eacb6761f3aeb58dd87651ade 
> 
> Diff: https://reviews.apache.org/r/36816/diff/
> 
> 
> Testing
> ---
> 
> * Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
> make check
> 
> 
> Thanks,
> 
> haosdent huang
> 
>

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-10-03 Thread haosdent huang 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/
---

(Updated Oct. 3, 2015, 5:49 p.m.)


Review request for mesos, Adam B, Michael Park, and Timothy Chen.


Changes
---

Rebase


Bugs: MESOS-2533
https://issues.apache.org/jira/browse/MESOS-2533


Repository: mesos


Description
---

Support HTTP checks in Mesos health check program


Diffs (updated)
-

  include/mesos/mesos.proto 4a16be1f570769f3ce42a50a9da9f4fb1c227999 
  src/docker/executor.cpp 1e4901335854c49e46cd7b132e79ccb11cd72ade 
  src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
  src/tests/health_check_tests.cpp ff6275b19206b49eacb6761f3aeb58dd87651ade 

Diff: https://reviews.apache.org/r/36816/diff/


Testing
---

* Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
make check


Thanks,

haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-10-03 Thread haosdent huang 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/
---

(Updated Oct. 3, 2015, 6:01 p.m.)


Review request for mesos, Adam B, Michael Park, and Timothy Chen.


Changes
---

Rebase


Bugs: MESOS-2533
https://issues.apache.org/jira/browse/MESOS-2533


Repository: mesos


Description
---

Support HTTP checks in Mesos health check program


Diffs (updated)
-

  include/mesos/mesos.proto 4a16be1f570769f3ce42a50a9da9f4fb1c227999 
  src/docker/executor.cpp 1e4901335854c49e46cd7b132e79ccb11cd72ade 
  src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
  src/tests/health_check_tests.cpp ff6275b19206b49eacb6761f3aeb58dd87651ade 

Diff: https://reviews.apache.org/r/36816/diff/


Testing
---

* Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
make check


Thanks,

haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-10-03 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review101426
---


Patch looks great!

Reviews applied: [36816]

All tests passed.

- Mesos ReviewBot


On Oct. 3, 2015, 6:01 p.m., haosdent huang wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/36816/
> ---
> 
> (Updated Oct. 3, 2015, 6:01 p.m.)
> 
> 
> Review request for mesos, Adam B, Michael Park, and Timothy Chen.
> 
> 
> Bugs: MESOS-2533
> https://issues.apache.org/jira/browse/MESOS-2533
> 
> 
> Repository: mesos
> 
> 
> Description
> ---
> 
> Support HTTP checks in Mesos health check program
> 
> 
> Diffs
> -
> 
>   include/mesos/mesos.proto 4a16be1f570769f3ce42a50a9da9f4fb1c227999 
>   src/docker/executor.cpp 1e4901335854c49e46cd7b132e79ccb11cd72ade 
>   src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
>   src/tests/health_check_tests.cpp ff6275b19206b49eacb6761f3aeb58dd87651ade 
> 
> Diff: https://reviews.apache.org/r/36816/diff/
> 
> 
> Testing
> ---
> 
> * Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
> make check
> 
> 
> Thanks,
> 
> haosdent huang
> 
>

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-08-07 Thread haosdent huang 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/
---

(Updated Aug. 7, 2015, 3:35 p.m.)


Review request for mesos, Adam B and Michael Park.


Bugs: MESOS-2533
https://issues.apache.org/jira/browse/MESOS-2533


Repository: mesos


Description
---

Support HTTP checks in Mesos health check program


Diffs (updated)
-

  include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
  src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
  src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 

Diff: https://reviews.apache.org/r/36816/diff/


Testing
---

* Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
make check


Thanks,

haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-08-07 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review94569
---


Patch looks great!

Reviews applied: [36816]

All tests passed.

- Mesos ReviewBot


On Aug. 7, 2015, 3:35 p.m., haosdent huang wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36816/
 ---
 
 (Updated Aug. 7, 2015, 3:35 p.m.)
 
 
 Review request for mesos, Adam B and Michael Park.
 
 
 Bugs: MESOS-2533
 https://issues.apache.org/jira/browse/MESOS-2533
 
 
 Repository: mesos
 
 
 Description
 ---
 
 Support HTTP checks in Mesos health check program
 
 
 Diffs
 -
 
   include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
   src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
   src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 
 
 Diff: https://reviews.apache.org/r/36816/diff/
 
 
 Testing
 ---
 
 * Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
 make check
 
 
 Thanks,
 
 haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-08-04 Thread Adam B 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review93562
---



src/health-check/main.cpp (line 242)
https://reviews.apache.org/r/36816/#comment147942

Why is this comment relevant here? You aren't even mentioning https here. 
I'd remove it.



include/mesos/mesos.proto (line 207)
https://reviews.apache.org/r/36816/#comment147943

s/compile/is compiled/


- Adam B


On Aug. 4, 2015, 2:28 a.m., haosdent huang wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36816/
 ---
 
 (Updated Aug. 4, 2015, 2:28 a.m.)
 
 
 Review request for mesos, Adam B and Michael Park.
 
 
 Bugs: MESOS-2533
 https://issues.apache.org/jira/browse/MESOS-2533
 
 
 Repository: mesos
 
 
 Description
 ---
 
 Support HTTP checks in Mesos health check program
 
 
 Diffs
 -
 
   include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
   src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
   src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 
 
 Diff: https://reviews.apache.org/r/36816/diff/
 
 
 Testing
 ---
 
 * Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
 make check
 
 
 Thanks,
 
 haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-07-29 Thread haosdent huang 


 On July 28, 2015, 8:05 a.m., Adam B wrote:
  src/health-check/main.cpp, line 243
  https://reviews.apache.org/r/36816/diff/1/?file=1021956#file1021956line243
 
  Maybe we should add `http.protocol()` in case the user wants https? Or 
  `http.ssl` like BenH suggested. Would we ever want anything besides 
  http/https?
  We can always add that in a later patch, so feel free to ignore for now.

Could add https now.


- haosdent


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review93254
---


On July 25, 2015, 6:57 p.m., haosdent huang wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36816/
 ---
 
 (Updated July 25, 2015, 6:57 p.m.)
 
 
 Review request for mesos and Adam B.
 
 
 Bugs: MESOS-2533
 https://issues.apache.org/jira/browse/MESOS-2533
 
 
 Repository: mesos
 
 
 Description
 ---
 
 Support HTTP checks in Mesos health check program
 
 
 Diffs
 -
 
   include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
   src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
   src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 
 
 Diff: https://reviews.apache.org/r/36816/diff/
 
 
 Testing
 ---
 
 * Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
 make check
 
 
 Thanks,
 
 haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-07-29 Thread haosdent huang 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/
---

(Updated July 29, 2015, 6:08 p.m.)


Review request for mesos and Adam B.


Bugs: MESOS-2533
https://issues.apache.org/jira/browse/MESOS-2533


Repository: mesos


Description
---

Support HTTP checks in Mesos health check program


Diffs (updated)
-

  include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
  src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
  src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 

Diff: https://reviews.apache.org/r/36816/diff/


Testing
---

* Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
make check


Thanks,

haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-07-29 Thread haosdent huang 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/
---

(Updated July 30, 2015, 2:47 a.m.)


Review request for mesos and Adam B.


Bugs: MESOS-2533
https://issues.apache.org/jira/browse/MESOS-2533


Repository: mesos


Description
---

Support HTTP checks in Mesos health check program


Diffs (updated)
-

  include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
  src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
  src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 

Diff: https://reviews.apache.org/r/36816/diff/


Testing
---

* Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
make check


Thanks,

haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-07-29 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review93550
---


Patch looks great!

Reviews applied: [36816]

All tests passed.

- Mesos ReviewBot


On July 30, 2015, 2:47 a.m., haosdent huang wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36816/
 ---
 
 (Updated July 30, 2015, 2:47 a.m.)
 
 
 Review request for mesos and Adam B.
 
 
 Bugs: MESOS-2533
 https://issues.apache.org/jira/browse/MESOS-2533
 
 
 Repository: mesos
 
 
 Description
 ---
 
 Support HTTP checks in Mesos health check program
 
 
 Diffs
 -
 
   include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
   src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
   src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 
 
 Diff: https://reviews.apache.org/r/36816/diff/
 
 
 Testing
 ---
 
 * Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
 make check
 
 
 Thanks,
 
 haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-07-28 Thread Adam B 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review93254
---


First pass over non-test code. Looks good. I'll give it a closer look after the 
next revision.


include/mesos/mesos.proto (lines 199 - 200)
https://reviews.apache.org/r/36816/#comment147613

Update comment



include/mesos/mesos.proto (lines 212 - 213)
https://reviews.apache.org/r/36816/#comment147614

The second sentence isn't true in your current implementation. Add a check 
for an empty statuses list and return success.
Please add a test for this as well.



include/mesos/mesos.proto (line 223)
https://reviews.apache.org/r/36816/#comment147609

Please update this comment.



src/health-check/main.cpp (line 233)
https://reviews.apache.org/r/36816/#comment147608

Maybe we should add `http.protocol()` in case the user wants https? Or 
`http.ssl` like BenH suggested. Would we ever want anything besides http/https?
We can always add that in a later patch, so feel free to ignore for now.



src/health-check/main.cpp (line 241)
https://reviews.apache.org/r/36816/#comment147610

failed with error is redundant. Please remove and just add query.failure()



src/health-check/main.cpp (line 254)
https://reviews.apache.org/r/36816/#comment147611

s/auto/uint32/
Re: `auto`: The main goal is to increase code readability. This is safely 
the case if the exact same type omitted on the left is already fully stated on 
the right. - 
http://mesos.apache.org/documentation/latest/mesos-c++-style-guide/



src/health-check/main.cpp (line 260)
https://reviews.apache.org/r/36816/#comment147612

s/Health http check return/HTTP health check returned/


- Adam B


On July 25, 2015, 11:57 a.m., haosdent huang wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36816/
 ---
 
 (Updated July 25, 2015, 11:57 a.m.)
 
 
 Review request for mesos and Adam B.
 
 
 Bugs: MESOS-2533
 https://issues.apache.org/jira/browse/MESOS-2533
 
 
 Repository: mesos
 
 
 Description
 ---
 
 Support HTTP checks in Mesos health check program
 
 
 Diffs
 -
 
   include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
   src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
   src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 
 
 Diff: https://reviews.apache.org/r/36816/diff/
 
 
 Testing
 ---
 
 * Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
 make check
 
 
 Thanks,
 
 haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-07-25 Thread Mesos ReviewBot 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/#review93036
---


Patch looks great!

Reviews applied: [36816]

All tests passed.

- Mesos ReviewBot


On July 25, 2015, 6:57 p.m., haosdent huang wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/36816/
 ---
 
 (Updated July 25, 2015, 6:57 p.m.)
 
 
 Review request for mesos and Adam B.
 
 
 Bugs: MESOS-2533
 https://issues.apache.org/jira/browse/MESOS-2533
 
 
 Repository: mesos
 
 
 Description
 ---
 
 Support HTTP checks in Mesos health check program
 
 
 Diffs
 -
 
   include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
   src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
   src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 
 
 Diff: https://reviews.apache.org/r/36816/diff/
 
 
 Testing
 ---
 
 * Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
 make check
 
 
 Thanks,
 
 haosdent huang

Re: Review Request 36816: Support HTTP checks in Mesos health check program

2015-07-25 Thread haosdent huang 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36816/
---

(Updated July 25, 2015, 6:57 p.m.)


Review request for mesos and Adam B.


Bugs: MESOS-2533
https://issues.apache.org/jira/browse/MESOS-2533


Repository: mesos


Description
---

Support HTTP checks in Mesos health check program


Diffs
-

  include/mesos/mesos.proto a6748d1cd82238f005c6a49c70d22d095462f1ba 
  src/health-check/main.cpp 97b25716335ec5719c1100bd73d06b7fc98036c9 
  src/tests/health_check_tests.cpp 157a56aa06677d8b7a2cef53b29ed05cb4b5d8ea 

Diff: https://reviews.apache.org/r/36816/diff/


Testing (updated)
---

* Add a new unit test: HealthCheckTest.HealthyTaskThroughHttp
make check


Thanks,

haosdent huang