Issue 763267 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	█ emso@chromium.org Last visit > 30 days ago
Closed:	Sep 2017
Cc:	qyears...@chromium.org █ emso@chromium.org mar...@chromium.org
Components:	Infra>CodeAnalysis
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	1
Type:	Bug
Tricium Milestone-Gerrit

Prevent posting of duplicate robot comments

Project Member Reported by emso@chromium.org, Sep 8 2017

Issue description

Example: https://chromium-review.googlesource.com/c/playground/gerrit-tricium/+/549343/40/README.md

Project Member

Comment 1 by bugdroid1@chromium.org, Sep 8 2017

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

commit aa15da1f7813da6f2bac54600836a7bb0bc321ae
Author: Emma <emso@chromium.org>
Date: Fri Sep 08 13:37:22 2017

Adds guard against processing of duplicate pubsub messages

Bug:763267
Change-Id: Ia9ee7507c09bcdefb4d73f98122ad5beb6c175e0
Reviewed-on: https://chromium-review.googlesource.com/657097
Commit-Queue: Emma Söderberg <emso@chromium.org>
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>

[modify] https://crrev.com/aa15da1f7813da6f2bac54600836a7bb0bc321ae/go/src/infra/tricium/appengine/driver/driver.infra_testing
[modify] https://crrev.com/aa15da1f7813da6f2bac54600836a7bb0bc321ae/go/src/infra/tricium/appengine/driver/handlers_test.go
[modify] https://crrev.com/aa15da1f7813da6f2bac54600836a7bb0bc321ae/go/src/infra/tricium/appengine/driver/handlers.go

Comment 2 by emso@chromium.org, Sep 8 2017

Status: Fixed (was: Started)

►

Issue 763267 link

Issue metadata

Prevent posting of duplicate robot comments

Issue description

Comment 1 by bugdroid1@chromium.org, Sep 8 2017

Comment 1 Deleted

Comment 2 by emso@chromium.org, Sep 8 2017

Comment 2 Deleted

Sign in to add a comment