ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nate Cole" <nc...@hortonworks.com>
Subject Re: Review Request 28512: Ambari agent should catch all exceptions which raised during bootstrap agent process and traceback they only if it "-v" was setup
Date Fri, 05 Dec 2014 16:21:48 GMT

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

Ship it!


Ship It!

- Nate Cole


On Nov. 27, 2014, 10:57 a.m., Dmytro Sen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28512/
> -----------------------------------------------------------
> 
> (Updated Nov. 27, 2014, 10:57 a.m.)
> 
> 
> Review request for Ambari, Dmitro Lisnichenko, John Speidel, Mahadev Konar, Nate Cole,
and Jeff Sposetti.
> 
> 
> Bugs: AMBARI-8562
>     https://issues.apache.org/jira/browse/AMBARI-8562
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> Ambari agent should restart with -v option if previous ambari-agent process was run with
-v option. Accordinly this option we shoul show/hide traceback.
> How it will work:
> 1) ambari-agent always have been restarted even if ambari-agent didn't started before,
so we should restart ambari-agent with '-v' option if previous process of ambari-agent was
started with '-v'
> 2) If '-v' option is not defined in command line all exceptions with tracebacks, which
rised while setup of ambari-agent, will be showed in output, othewise only error messages
will be showed in output.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/python/os_check_type.py f890504 
>   ambari-server/src/main/python/setupAgent.py ede83c5 
>   ambari-server/src/test/python/TestOSCheck.py 20b8890 
>   ambari-server/src/test/python/TestSetupAgent.py 8293fd0 
> 
> Diff: https://reviews.apache.org/r/28512/diff/
> 
> 
> Testing
> -------
> 
> tests passed
> 
> 
> Thanks,
> 
> Dmytro Sen
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message