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

Issue 633837 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Aug 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

[Findit] Indexed JsonProperties used where values could exceed 1500 bytes

Project Member Reported by josiahk@google.com, Aug 3 2016

Issue description

What is the bug or feature:

WfFailureGroup.output_nodes and WfFailureGroup.failed_steps_and_tests both use ndb.JsonProperty(indexed=True), which (when indexed) has a max size of 1500 bytes.

WfFailureGroup.output_nodes and WfFailureGroup.failed_steps_and_tests could both exceed 1500 bytes, which would cause an error.

One solution is to use unindexed JsonProperties for WfFailureGroup.output_nodes and WfFailureGroup.failed_steps_and_tests.


 
Cc: chanli@chromium.org lijeffrey@chromium.org
Components: Tools>Test>FindIt
Status: Available (was: Unconfirmed)
Project Member

Comment 2 by sheriffbot@chromium.org, Aug 5 2016

Labels: Hotlist-Google
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 10 2016

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

commit d6897e11508a50d02e7da183a61445c8654ce38a
Author: josiahk <josiahk@google.com>
Date: Wed Aug 10 00:50:11 2016

[Findit] Fix 1500 byte cap on failed_steps_and_tests and output_nodes

Add created_time support for wf_failure_groups
Disable indexing on failed_steps_and_tests and output_nodes

BUG= 633837 

Review-Url: https://codereview.chromium.org/2227223002

[modify] https://crrev.com/d6897e11508a50d02e7da183a61445c8654ce38a/appengine/findit/model/wf_failure_group.py
[modify] https://crrev.com/d6897e11508a50d02e7da183a61445c8654ce38a/appengine/findit/waterfall/try_job_util.py

Comment 4 by st...@chromium.org, Aug 16 2016

Status: Fixed (was: Available)
Closed this bug as it seems fixed. Please reopen if not yet.

Sign in to add a comment