[Findit] Process analysis requests of Waterfall failures concurrently.

[Findit] Process analysis requests of Waterfall failures concurrently.

This might not be the best fix, but it should work.
Integration through Pub/Sub might be a better solution to decouple alerts-dispatcher and findit: no direct http requests from alerts-dispatcher to findit.

BUG=655232
Committed: https://chromium.googlesource.com/infra/infra/+/545805bdd28931c2e13ca4e77ef623bf5e2df0aa

[stgao, 2016-10-15 00:27:21]

Description was changed from

==========
[Findit] Process analysis requests of Waterfall failures concurrently.

BUG=655232
==========

to

==========
[Findit] Process analysis requests of Waterfall failures concurrently.

BUG=655232
==========

[stgao, 2016-10-15 00:27:21]

stgao@chromium.org changed reviewers:
+ chanli@chromium.org, lijeffrey@chromium.org

[stgao, 2016-10-15 00:27:29]

ptal

[chanli, 2016-10-15 00:39:07]

On 2016/10/15 00:27:29, stgao wrote:
> ptal

lgtm with a nit

[chanli, 2016-10-15 00:39:47]

https://codereview.chromium.org/2425453002/diff/20001/appengine/findit/main.py
File appengine/findit/main.py (right):

https://codereview.chromium.org/2425453002/diff/20001/appengine/findit/main.p...
appengine/findit/main.py:22: from handlers import
process_failure_analysis_requests
Nit: import order

[stgao, 2016-10-15 00:41:11]

https://codereview.chromium.org/2425453002/diff/20001/appengine/findit/main.py
File appengine/findit/main.py (right):

https://codereview.chromium.org/2425453002/diff/20001/appengine/findit/main.p...
appengine/findit/main.py:22: from handlers import
process_failure_analysis_requests
On 2016/10/15 00:39:47, chanli wrote:
> Nit: import order

Done.

[stgao, 2016-10-15 00:41:16]

The CQ bit was checked by stgao@chromium.org

[stgao, 2016-10-15 00:41:16]

The patchset sent to the CQ was uploaded after l-g-t-m from chanli@chromium.org
Link to the patchset: https://codereview.chromium.org/2425453002/#ps40001
(title: "fix nit.")

[commit-bot: I haz the power, 2016-10-15 00:41:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[stgao, 2016-10-15 00:41:51]

Description was changed from

==========
[Findit] Process analysis requests of Waterfall failures concurrently.

BUG=655232
==========

to

==========
[Findit] Process analysis requests of Waterfall failures concurrently.

This might not be the best fix, but it should work.

BUG=655232
==========

[stgao, 2016-10-15 00:44:35]

Description was changed from

==========
[Findit] Process analysis requests of Waterfall failures concurrently.

This might not be the best fix, but it should work.

BUG=655232
==========

to

==========
[Findit] Process analysis requests of Waterfall failures concurrently.

This might not be the best fix, but it should work.
Integration through Pub/Sub might be a better solution to decouple
alerts-dispatcher and findit: no direct http requests from alerts-dispatcher to
findit.

BUG=655232
==========

[commit-bot: I haz the power, 2016-10-15 00:55:57]

Description was changed from

==========
[Findit] Process analysis requests of Waterfall failures concurrently.

This might not be the best fix, but it should work.
Integration through Pub/Sub might be a better solution to decouple
alerts-dispatcher and findit: no direct http requests from alerts-dispatcher to
findit.

BUG=655232
==========

to

==========
[Findit] Process analysis requests of Waterfall failures concurrently.

This might not be the best fix, but it should work.
Integration through Pub/Sub might be a better solution to decouple
alerts-dispatcher and findit: no direct http requests from alerts-dispatcher to
findit.

BUG=655232

Committed:
https://chromium.googlesource.com/infra/infra/+/545805bdd28931c2e13ca4e77ef62...
==========

[commit-bot: I haz the power, 2016-10-15 00:55:58]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/infra/infra/+/545805bdd28931c2e13ca4e77ef62...

[lijeffrey, 2016-10-15 01:25:18]

lgtm