New issue
Advanced search Search tips

Issue 679814 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Dashboard - Don't auto-flag a bug as a duplicate of a bug which is already a duplicate of the former

Project Member Reported by gab@chromium.org, Jan 10 2017

Issue description

Issue 677034 was marked as a duplicate of  Issue 677036  which was itself already marked as a duplicate of 677034 (i.e. both bugs closed as dupes of each other...)
 
Components: -Tests>Telemetry
Labels: Performance-Dashboard
Owner: simonhatch@chromium.org
Simon, can you take a look?
Cc: simonhatch@chromium.org
Owner: ----
Summary: Dashboard - Don't auto-flag a bug as a duplicate of a bug which is already a duplicate of the former (was: [telemetry] Don't auto-flag a bug as a duplicate of a bug which is already a duplicate of the former)
Looks like this is done in dashboard/update_bug_with_results, can probably just add some logic to query about the issue to merge into before actually doing it.

I can take a look but probably not immediately, will un-assign for now and re-assign once I'm actually working on it.
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 19 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6ab584fe173d30622049bf8b0c230384b10b41c1

commit 6ab584fe173d30622049bf8b0c230384b10b41c1
Author: catapult-deps-roller <catapult-deps-roller@chromium.org>
Date: Thu Jan 19 15:36:50 2017

Roll src/third_party/catapult/ cfcae9b97..1e05d2f84 (15 commits).

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/cfcae9b972f2..1e05d2f8401c

$ git log cfcae9b97..1e05d2f84 --date=short --no-merges --format='%ad %ae %s'
2017-01-19 hjd Revert of [telemetry] Bullet proof scroll.js (patchset #4 id:60001 of https://codereview.chromium.org/2532443002/ )
2017-01-19 perezju [catapult] Remove Telemetry's android_command_line_backend
2017-01-18 dtu [pinpoint] Config changes to make /new work.
2017-01-18 dtu [pinpoint] Remove "blocked" from Attempt and Execution.
2017-01-18 benjhayden Translate Statistics methods to python.
2017-01-18 dtu [pinpoint] Centralize failure and completion handling in Executions.
2017-01-18 benjhayden Translate PercentToString to python.
2017-01-18 benjhayden Translate RunningStatistics to python.
2017-01-18 benjhayden Translate Range to python.
2017-01-18 jreck Make extract_thread_list slightly smarter
2017-01-18 jessimb Fixed alert count to match display.
2017-01-18 simonhatch This avoids getting into weird circular situations like A dupes on B which duped against A already. Kept it simple, if the other bug is already a duplicate, don't merge. Still cc the author and culprit though.
2017-01-18 perezju [telemetry] Provide new API to evaluate JavaScript code
2017-01-18 perezju [devil] Upgrade parser/serializer for command line flags
2017-01-18 hjd [telemetry] Bullet proof scroll.js

BUG= 664515 , 679814 , 664515 

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=catapult-sheriff@chromium.org

Review-Url: https://codereview.chromium.org/2642083002
Cr-Commit-Position: refs/heads/master@{#444740}

[modify] https://crrev.com/6ab584fe173d30622049bf8b0c230384b10b41c1/DEPS

Owner: simonhatch@chromium.org
Status: Fixed (was: Untriaged)
Components: Speed>Dashboard
Labels: -Performance-Dashboard

Sign in to add a comment