Review Request 47297: Extend logging for ActionQueue's retry logic

2016-05-12 Thread Daniel Gergely

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

Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Oliver Szabo, 
Sandor Magyari, Sumit Mohanty, and Sebastian Toader.


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


Repository: ambari


Description
---

At info level, commands should be tracked if they are retried on failure or 
not, and if not, what is the reason for that.


Diffs
-

  ambari-agent/src/main/python/ambari_agent/ActionQueue.py 85389f9 
  ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 7d0efa2 

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


Testing
---

[INFO] 
[INFO] BUILD SUCCESS
[INFO] 
[INFO] Total time: 1:19.911s
[INFO] Finished at: Thu May 12 10:58:43 CEST 2016
[INFO] Final Memory: 17M/850M
[INFO] 


Thanks,

Daniel Gergely



Re: Review Request 47297: Extend logging for ActionQueue's retry logic

2016-05-12 Thread Sandor Magyari

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


Ship it!




Ship It!

- Sandor Magyari


On May 12, 2016, 9:05 a.m., Daniel Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47297/
> ---
> 
> (Updated May 12, 2016, 9:05 a.m.)
> 
> 
> Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Oliver Szabo, 
> Sandor Magyari, Sumit Mohanty, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-16330
> https://issues.apache.org/jira/browse/AMBARI-16330
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> At info level, commands should be tracked if they are retried on failure or 
> not, and if not, what is the reason for that.
> 
> 
> Diffs
> -
> 
>   ambari-agent/src/main/python/ambari_agent/ActionQueue.py 85389f9 
>   ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 7d0efa2 
> 
> Diff: https://reviews.apache.org/r/47297/diff/
> 
> 
> Testing
> ---
> 
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 1:19.911s
> [INFO] Finished at: Thu May 12 10:58:43 CEST 2016
> [INFO] Final Memory: 17M/850M
> [INFO] 
> 
> 
> 
> Thanks,
> 
> Daniel Gergely
> 
>



Re: Review Request 47297: Extend logging for ActionQueue's retry logic

2016-05-12 Thread Sebastian Toader

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


Ship it!




Ship It!

- Sebastian Toader


On May 12, 2016, 11:05 a.m., Daniel Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47297/
> ---
> 
> (Updated May 12, 2016, 11:05 a.m.)
> 
> 
> Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Oliver Szabo, 
> Sandor Magyari, Sumit Mohanty, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-16330
> https://issues.apache.org/jira/browse/AMBARI-16330
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> At info level, commands should be tracked if they are retried on failure or 
> not, and if not, what is the reason for that.
> 
> 
> Diffs
> -
> 
>   ambari-agent/src/main/python/ambari_agent/ActionQueue.py 85389f9 
>   ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 7d0efa2 
> 
> Diff: https://reviews.apache.org/r/47297/diff/
> 
> 
> Testing
> ---
> 
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 1:19.911s
> [INFO] Finished at: Thu May 12 10:58:43 CEST 2016
> [INFO] Final Memory: 17M/850M
> [INFO] 
> 
> 
> 
> Thanks,
> 
> Daniel Gergely
> 
>



Re: Review Request 47297: Extend logging for ActionQueue's retry logic

2016-05-12 Thread Oliver Szabo

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


Ship it!




Ship It!

- Oliver Szabo


On May 12, 2016, 9:05 a.m., Daniel Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47297/
> ---
> 
> (Updated May 12, 2016, 9:05 a.m.)
> 
> 
> Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Oliver Szabo, 
> Sandor Magyari, Sumit Mohanty, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-16330
> https://issues.apache.org/jira/browse/AMBARI-16330
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> At info level, commands should be tracked if they are retried on failure or 
> not, and if not, what is the reason for that.
> 
> 
> Diffs
> -
> 
>   ambari-agent/src/main/python/ambari_agent/ActionQueue.py 85389f9 
>   ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 7d0efa2 
> 
> Diff: https://reviews.apache.org/r/47297/diff/
> 
> 
> Testing
> ---
> 
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 1:19.911s
> [INFO] Finished at: Thu May 12 10:58:43 CEST 2016
> [INFO] Final Memory: 17M/850M
> [INFO] 
> 
> 
> 
> Thanks,
> 
> Daniel Gergely
> 
>



Re: Review Request 47297: Extend logging for ActionQueue's retry logic

2016-05-17 Thread Daniel Gergely

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

(Updated máj. 17, 2016, 7:31 de)


Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Oliver Szabo, 
Sandor Magyari, Sumit Mohanty, and Sebastian Toader.


Changes
---

TaskId is added to the log message


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


Repository: ambari


Description
---

At info level, commands should be tracked if they are retried on failure or 
not, and if not, what is the reason for that.


Diffs (updated)
-

  ambari-agent/src/main/python/ambari_agent/ActionQueue.py 85389f9 
  ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 7d0efa2 

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


Testing
---

[INFO] 
[INFO] BUILD SUCCESS
[INFO] 
[INFO] Total time: 1:19.911s
[INFO] Finished at: Thu May 12 10:58:43 CEST 2016
[INFO] Final Memory: 17M/850M
[INFO] 


Thanks,

Daniel Gergely



Re: Review Request 47297: Extend logging for ActionQueue's retry logic

2016-05-17 Thread Sebastian Toader

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


Ship it!




Ship It!

- Sebastian Toader


On May 17, 2016, 9:31 a.m., Daniel Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47297/
> ---
> 
> (Updated May 17, 2016, 9:31 a.m.)
> 
> 
> Review request for Ambari, Balázs Bence Sári, Laszlo Puskas, Oliver Szabo, 
> Sandor Magyari, Sumit Mohanty, and Sebastian Toader.
> 
> 
> Bugs: AMBARI-16630
> https://issues.apache.org/jira/browse/AMBARI-16630
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> At info level, commands should be tracked if they are retried on failure or 
> not, and if not, what is the reason for that.
> 
> 
> Diffs
> -
> 
>   ambari-agent/src/main/python/ambari_agent/ActionQueue.py 85389f9 
>   ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 7d0efa2 
> 
> Diff: https://reviews.apache.org/r/47297/diff/
> 
> 
> Testing
> ---
> 
> [INFO] 
> 
> [INFO] BUILD SUCCESS
> [INFO] 
> 
> [INFO] Total time: 1:19.911s
> [INFO] Finished at: Thu May 12 10:58:43 CEST 2016
> [INFO] Final Memory: 17M/850M
> [INFO] 
> 
> 
> 
> Thanks,
> 
> Daniel Gergely
> 
>