Added more logging

Added more logging

BUG=160709
R=phajdan.jr@chromium.org

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=274227

[Paweł Hajdan Jr., 2014-06-02 13:45:47]

LGTM with comments.

https://codereview.chromium.org/309013004/diff/20001/scripts/master/try_job_r...
File scripts/master/try_job_rietveld.py (right):

https://codereview.chromium.org/309013004/diff/20001/scripts/master/try_job_r...
scripts/master/try_job_rietveld.py:329: if len(new_processed_keys):
nit: No need to use len in the condition.

Actually it may be better to always log something here.

https://codereview.chromium.org/309013004/diff/20001/scripts/master/try_job_r...
scripts/master/try_job_rietveld.py:338: if num_removed:
Same here, why not make the log statement unconditional?

[Sergiy Byelozyorov, 2014-06-02 13:52:27]

https://codereview.chromium.org/309013004/diff/20001/scripts/master/try_job_r...
File scripts/master/try_job_rietveld.py (right):

https://codereview.chromium.org/309013004/diff/20001/scripts/master/try_job_r...
scripts/master/try_job_rietveld.py:329: if len(new_processed_keys):
On 2014/06/02 13:45:47, Paweł Hajdan Jr. wrote:
> nit: No need to use len in the condition.
> 
> Actually it may be better to always log something here.

Done.

https://codereview.chromium.org/309013004/diff/20001/scripts/master/try_job_r...
scripts/master/try_job_rietveld.py:338: if num_removed:
On 2014/06/02 13:45:47, Paweł Hajdan Jr. wrote:
> Same here, why not make the log statement unconditional?

Done.

[Sergiy Byelozyorov, 2014-06-02 13:52:42]

The CQ bit was checked by sergiyb@chromium.org

[commit-bot: I haz the power, 2014-06-02 13:52:52]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sergiyb@chromium.org/309013004/40001

[commit-bot: I haz the power, 2014-06-02 13:53:49]

Change committed as 274227