New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 792250 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 0
Type: Bug



Sign in to add a comment

[Findit] BadRequestError: The value of property "signals" is longer than 1048487 bytes.

Project Member Reported by wylieb@chromium.org, Dec 5 2017

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Dec 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/70548b555fcda813a2565f45de0fb1c38f301330

commit 70548b555fcda813a2565f45de0fb1c38f301330
Author: Shuotao Gao <stgao@chromium.org>
Date: Wed Dec 06 00:28:04 2017

[Findit] Workaround for too large signals.

Bug:  792250 
Change-Id: I117959cea106c3726a4004957a596a8b786ec6fd
Reviewed-on: https://chromium-review.googlesource.com/810022
Reviewed-by: Jeffrey Li <lijeffrey@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>
Commit-Queue: Shuotao Gao <stgao@chromium.org>

[modify] https://crrev.com/70548b555fcda813a2565f45de0fb1c38f301330/appengine/findit/model/wf_analysis.py

Comment 2 by st...@chromium.org, Dec 6 2017

Cc: chanli@chromium.org
Issue 792256 has been merged into this issue.

Comment 3 by st...@chromium.org, Dec 6 2017

Owner: st...@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 5 by bugdroid1@chromium.org, Dec 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/c0c5156302832e931b4270894d035a0f41bf44e4

commit c0c5156302832e931b4270894d035a0f41bf44e4
Author: Shuotao Gao <stgao@chromium.org>
Date: Wed Dec 06 01:01:24 2017

[Findit] Not save extracted signals for test failures.

Unlike compile failures, signals for test failures are not used for try-jobs, so
there is no need to save it in datastore.

Bug:  792250 
Change-Id: I3b6e3aaed63335bbdfb378ad67dda60bffcf6830
Reviewed-on: https://chromium-review.googlesource.com/809348
Reviewed-by: Sharu Jiang <katesonia@chromium.org>
Reviewed-by: Brandon Wylie <wylieb@chromium.org>
Commit-Queue: Shuotao Gao <stgao@chromium.org>

[modify] https://crrev.com/c0c5156302832e931b4270894d035a0f41bf44e4/appengine/findit/services/test_failure/extract_test_signal.py
[modify] https://crrev.com/c0c5156302832e931b4270894d035a0f41bf44e4/appengine/findit/services/test_failure/test/extract_test_signal_test.py

Comment 6 by st...@chromium.org, Dec 6 2017

Status: Fixed (was: Started)

Sign in to add a comment