Re: Review Request 46486: Add logging to check_host.py

2016-04-21 Thread Vitalyi Brodetskyi

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


Ship it!




Ship It!

- Vitalyi Brodetskyi


On April 21, 2016, 11:52 a.m., Andrew Onischuk wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46486/
> ---
> 
> (Updated April 21, 2016, 11:52 a.m.)
> 
> 
> Review request for Ambari and Vitalyi Brodetskyi.
> 
> 
> Bugs: AMBARI-16006
> https://issues.apache.org/jira/browse/AMBARI-16006
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Sometimes host check after host registration lasted too long (more than 720
> sec.). I's affect some system tests. I can't reproduce this, so it would be
> good to add logging to this operation.
> 
> Currently, all the host check logs available are
> 
> 
> 
> 
> [root@c6404 data]# cat /var/lib/ambari-agent/data/output-3.txt
> Last Agent Env check started.
> Last Agent Env check completed successfully.
> 
> 
> Diffs
> -
> 
>   ambari-server/src/main/resources/custom_actions/scripts/check_host.py 
> 85ccef1 
>   ambari-server/src/test/python/custom_actions/TestCheckHost.py 8bc1774 
> 
> Diff: https://reviews.apache.org/r/46486/diff/
> 
> 
> Testing
> ---
> 
> mvn clean test
> 
> 
> Thanks,
> 
> Andrew Onischuk
> 
>



Re: Review Request 46486: Add logging to check_host.py

2016-04-21 Thread Andrew Onischuk

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

(Updated April 21, 2016, 11:52 a.m.)


Review request for Ambari and Vitalyi Brodetskyi.


Bugs: AMBARI-16006
https://issues.apache.org/jira/browse/AMBARI-16006


Repository: ambari


Description
---

Sometimes host check after host registration lasted too long (more than 720
sec.). I's affect some system tests. I can't reproduce this, so it would be
good to add logging to this operation.

Currently, all the host check logs available are




[root@c6404 data]# cat /var/lib/ambari-agent/data/output-3.txt
Last Agent Env check started.
Last Agent Env check completed successfully.


Diffs (updated)
-

  ambari-server/src/main/resources/custom_actions/scripts/check_host.py 85ccef1 
  ambari-server/src/test/python/custom_actions/TestCheckHost.py 8bc1774 

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


Testing
---

mvn clean test


Thanks,

Andrew Onischuk