New issue
Advanced search Search tips

Issue 760219 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 764045
Owner: ----
Closed: Sep 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----


Previous locations:
gerrit:7116


Sign in to add a comment

Removed CC-ed emails are added back when a second patchset is uploaded

Project Member Reported by xunji...@chromium.org, Aug 29 2017

Issue description

Affected Version:

What steps will reproduce the problem?
1.Upload a CL that will auto cc some people
2. Remove those people for cc-ed list
3. Upload a new PatchSet to that CL. The people who were cc-ed will get added back

What is the expected output?
When I modify cc list in Rietveld, that edit will persist across patchsets. I expect PolyGerrit to do the same.


What do you see instead?

The edit of cc list doesn't persist across patchsets. When a new PS is uploaded, people will get cc-ed.


Please provide any additional information below.

 

Comment 1 by logan@google.com, Aug 29 2017

Components: -PolyGerrit
Labels: Proj-Gerrit-Migration
git_cl.py may need to read from Gerrit to determine which CCs to exclude because they've been removed before, but technically it's possible. Or do we need a way for Gerrit to apply this logic itself when a CC is requested to be added?

Comment 2 by logan@google.com, Aug 29 2017

Project: chromium
Moved issue gerrit:7116 to now be  issue chromium:760219 .

Comment 3 by aga...@chromium.org, Sep 11 2017

Components:
Mergedinto: 764045
Status: Duplicate (was: New)

Sign in to add a comment