New issue
Advanced search Search tips

Issue 910379 link

Starred by 1 user
Status: Fixed
Owner:
ehmaldonado@chromium.org
Closed: Dec 10
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

appengine/monorail presubmit tests flaky

Project Member Reported by akes...@chromium.org, Nov 29 
Presubmit tests flaking with

AssertionError: 'editissue' unexpectedly found in frozenset(['addissuecomment', 'viewquota', 'createissue', 'verdictspam', 'setstar', 'viewcontributorlist', 'deleteown', 'viewinboundmessages', 'commit', 'flagspam', 'editissue', 'view'])

See https://logs.chromium.org/logs/infra/buildbucket/cr-buildbucket.appspot.com/8928483603150335024/+/steps/python_tests/0/stdout for example, on innocent CL https://chromium-review.googlesource.com/c/infra/infra/+/1343601

Possibly culprit https://chromium-review.googlesource.com/c/1354190
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 30 

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

commit 0fcbb2369b3e9cd015fdabaae02ec036ade6c697
Author: Edward Lemur <ehmaldonado@chromium.org>
Date: Fri Nov 30 23:52:40 2018

monorail: Fix flaky UpdateIssuePermissions tests.

Bug: monorail:4610,  chromium:910379 
Change-Id: If6441ffbc6dbc6e3d8d54df5547f573c455cdf13
Reviewed-on: https://chromium-review.googlesource.com/c/1355907
Reviewed-by: Jason Robbins <jrobbins@chromium.org>
Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Cr-Commit-Position: refs/heads/master@{#19281}
[modify] https://crrev.com/0fcbb2369b3e9cd015fdabaae02ec036ade6c697/appengine/monorail/framework/test/permissions_test.py

Comment 2 by prasadv@google.com, Dec 10

Components: -Infra Infra>Platform
Chrome infra bug triage here:
ehmaldonado@ Is  this fixed? I think this issue is related to monorail and I'm not sure whether Infra is the correct component for this and I don't see a component related to monorail.
I'm chaning this to Infra>Platforms, please feel free to change to other components if this is not the right one. 

Thanks.

Comment 3 by ehmaldonado@chromium.org, Dec 10

Status: Fixed (was: Assigned)
Yes I think this is fixed now.
►

Sign in to add a comment