[PG] Specifying --cc during upload and adding emails to CC_LIST does not seem to be sending emails |
||||||||
Issue descriptionI tested it in the Skia repo by doing: git cl upload --cc=benjaminwagner@google.com --bypass-hooks The refspec looked like this: refs/for/refs/heads/master%notify=NONE,cc=reviews@skia.org,cc=benjaminwagner@google.com but Ben (who has an active account in skia-review) did not get any emails (neither did reviews@skia.org).
,
Sep 23 2016
,
Sep 23 2016
Andrii, mind taking a look?
,
Sep 23 2016
Ah, I got it: notify=NONE! Have you tried git cl upload with -s flag to actually send email? :)
,
Sep 23 2016
I've uploaded a new issue as a test. If you got email, then it works: To https://chromium.googlesource.com/infra/infra.git * [new branch] 1fd432a2a593d48b2d65e73446b0f149cc866907 -> refs/for/refs/heads/master%notify=ALL,cc=chromium-reviews@chromium.org,cc=infra-reviews+infra@chromium.org,cc=sergiyb@chromium.org,r=rmistry@chromium.org
,
Sep 23 2016
Yes I got the email. But I still do not follow- For things in the CC_LIST how do I send out an email without specifying a reviewer on the commandline? If I do "git cl upload -s" it says "Must specify reviewers to send email." I get the same with -cc and no --reviewers. For Rietveld, the CC_LIST is added to the issue during upload and stays there for all notifications. Since the CC_LIST is not added to the Gerrit issue all future review comments and notifications done from the UI do not update the CC_LIST like it does in Rietveld.
,
Sep 26 2016
Upping the priority of this to P1 because we had to pause the Skia dogfood of Gerrit until reviews@skia.org starts receiving emails.
,
Sep 26 2016
Also, I am OOO 9/26 and 9/27 so please coordinate with mtklein@ if you need him to test out something for Skia.
,
Sep 26 2016
TBH, I don't have a good comprehensive solution here. Project-wide CC list should probably be moved to Gerrit's project-wide config, hereby making skia-reviews@ receive emails from Gerrit about changes in its own way, as opposed to whenever user hits git cl upload with --send-email.
,
Sep 26 2016
Oh, how do I set project-wide CC list? Is that something I can set here: https://skia-review.googlesource.com/#/admin/projects/All-Projects,access ?
,
Sep 26 2016
https://gerrit-review.googlesource.com/Documentation/user-notify.html says to do something like: [notify "team"] email = team-address@example.com email = paranoid-manager@example.com
,
Sep 26 2016
Made this change: https://skia.googlesource.com/All-Projects/+/8c8003a7a91a29dc9630f6a0d97f22ffc25c3722%5E%21/#F0 Let's see if that works around our CC_LIST problem.
,
Sep 27 2016
Did the configuration change solve the issue?
,
Sep 27 2016
Kind of. It sends emails for everything now, even when a change is first uploaded and not ready for review. Also seems to send emails for every new patchset and dry runs. It is a good workaround for now, but IMO the CC_LIST should be added to the Change by calling the 'Add Reviewer' endpoint from Gerrit's API. Doing that will ensure that the existing functionality of CC_LIST in the various codereview.settings is maintained. I can look into this when I am back from vacation if somebody does not beat me to it.
,
Sep 27 2016
Ravi get away from your computer :). Go back to vacation, please.
,
Sep 27 2016
,
Sep 28 2016
Sent out for review https://codereview.chromium.org/2378703002/
,
Sep 28 2016
Adding below since bugdroid does not seem to want to update this bug. Submitted https://codereview.chromium.org/2378703002/ : Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld. The CC_LIST appears to be now correctly added Eg: https://skia-review.googlesource.com/c/2741/ The problem of too frequent updates still exists though. Sending out an email to the cc list whenever the desc is updated and dry runs are triggered is spammy especially to Rietveld users who will not expect this. Is there an existing bug to address this? should I file one?
,
Sep 28 2016
wyatta and heisel are looking to improve email product excellence type of bugs. I would file one against polygerrit with your desired flow. I'm too far removed with how emails work in Rietveld (or critique) to give a comprehensive overview of desired behavior.
,
Sep 28 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/3ca6e8a40b853337432916ce446b00e72b145381 commit 3ca6e8a40b853337432916ce446b00e72b145381 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Sep 28 15:34:21 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) depot_tools: https://crrev.com/3574740929abd37c45db1d2f8a2c3799bdfe77c5 Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (rmistry@google.com) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2375963004 [modify] https://crrev.com/3ca6e8a40b853337432916ce446b00e72b145381/infra/config/recipes.cfg
,
Sep 28 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/3ca6e8a40b853337432916ce446b00e72b145381 commit 3ca6e8a40b853337432916ce446b00e72b145381 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Sep 28 15:34:21 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) depot_tools: https://crrev.com/3574740929abd37c45db1d2f8a2c3799bdfe77c5 Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (rmistry@google.com) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2375963004 [modify] https://crrev.com/3ca6e8a40b853337432916ce446b00e72b145381/infra/config/recipes.cfg
,
Sep 28 2016
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/c51a945bb90e3e100d417984cd9098929626f6bd commit c51a945bb90e3e100d417984cd9098929626f6bd Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Sep 28 15:44:53 2016
,
Sep 28 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/3ca6e8a40b853337432916ce446b00e72b145381 commit 3ca6e8a40b853337432916ce446b00e72b145381 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Sep 28 15:34:21 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) depot_tools: https://crrev.com/3574740929abd37c45db1d2f8a2c3799bdfe77c5 Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (rmistry@google.com) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2375963004 [modify] https://crrev.com/3ca6e8a40b853337432916ce446b00e72b145381/infra/config/recipes.cfg
,
Sep 28 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/1eca99b81c0f3ab16712c38ea766d4316fcb80d7 commit 1eca99b81c0f3ab16712c38ea766d4316fcb80d7 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Sep 28 15:55:23 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/3ca6e8a40b853337432916ce446b00e72b145381 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/3574740929abd37c45db1d2f8a2c3799bdfe77c5 Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (rmistry@google.com) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2381563002 Cr-Commit-Position: refs/heads/master@{#421530} [modify] https://crrev.com/1eca99b81c0f3ab16712c38ea766d4316fcb80d7/infra/config/recipes.cfg
,
Sep 28 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/f100cada231af0b1a0c70845f8d8c06fd9726086 commit f100cada231af0b1a0c70845f8d8c06fd9726086 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Sep 28 18:35:17 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/ba86a553518c0321687aff9497624d84f6a17be0 WebRTC: Update tests after renaming AppRTCDemo -> AppRTCMobile. (kjellander@chromium.org) https://crrev.com/597c27e7cd712b9a07dd091a44f22af5b957c54a WebRTC: Add android_junit_tests. (ehmaldonado@chromium.org) https://crrev.com/22636ece1d2c8507ca8dbebd6b434b0fe9ba6ef1 Revert of WebRTC: Add android_junit_tests. (patchset #8 id:200001 of https://codereview.chromium.org/2354363006/ ) (ehmaldonado@chromium.org) https://crrev.com/567475a8f817b58253b6e305bf5eb5c980f04bc6 WebRTC: Add android_junit_tests to FYI bots. (ehmaldonado@chromium.org) https://crrev.com/3ca6e8a40b853337432916ce446b00e72b145381 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/3574740929abd37c45db1d2f8a2c3799bdfe77c5 Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (rmistry@google.com) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= 647390 , chromium:649660 , webrtc:6359 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2377953004 [modify] https://crrev.com/f100cada231af0b1a0c70845f8d8c06fd9726086/infra/config/recipes.cfg
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+/31d165a5c15c7311e0eed1cf731a5585ac9c7f57 commit 31d165a5c15c7311e0eed1cf731a5585ac9c7f57 Author: tandrii <tandrii@chromium.org> Date: Thu Sep 29 08:40:46 2016 Revert of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (patchset #4 id:60001 of https://codereview.chromium.org/2378703002/ ) Reason for revert: Actually, it doesn't break uploads, add cc-ed emails post-upload fails with exception and confuses users, and I couldn't find workaround. Original issue's description: > Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld > > BUG= chromium:649660 > > Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/3574740929abd37c45db1d2f8a2c3799bdfe77c5 TBR=rmistry@google.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= chromium:649660 Review-Url: https://codereview.chromium.org/2375393002 [modify] https://crrev.com/31d165a5c15c7311e0eed1cf731a5585ac9c7f57/gerrit_util.py [modify] https://crrev.com/31d165a5c15c7311e0eed1cf731a5585ac9c7f57/git_cl.py [modify] https://crrev.com/31d165a5c15c7311e0eed1cf731a5585ac9c7f57/tests/git_cl_test.py
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/f9511903eb2d5e04717055a3d7b38df53d01b68b commit f9511903eb2d5e04717055a3d7b38df53d01b68b Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 08:54:37 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) depot_tools: https://crrev.com/31d165a5c15c7311e0eed1cf731a5585ac9c7f57 Revert of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (patchset #4 id:60001 of https://codereview.chromium.org/2378703002/ ) (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2382673002 [modify] https://crrev.com/f9511903eb2d5e04717055a3d7b38df53d01b68b/infra/config/recipes.cfg
,
Sep 29 2016
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/9b1e33a297e846e6b102b90a8844b6d192c35b19 commit 9b1e33a297e846e6b102b90a8844b6d192c35b19 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 09:05:22 2016
,
Sep 29 2016
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/9b1e33a297e846e6b102b90a8844b6d192c35b19 commit 9b1e33a297e846e6b102b90a8844b6d192c35b19 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 09:05:22 2016
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/92909c7eef49ebc6617025fa59f90f16f2a796c9 commit 92909c7eef49ebc6617025fa59f90f16f2a796c9 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 09:09:57 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/f9511903eb2d5e04717055a3d7b38df53d01b68b Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/31d165a5c15c7311e0eed1cf731a5585ac9c7f57 Revert of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (patchset #4 id:60001 of https://codereview.chromium.org/2378703002/ ) (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2376053003 Cr-Commit-Position: refs/heads/master@{#421784} [modify] https://crrev.com/92909c7eef49ebc6617025fa59f90f16f2a796c9/infra/config/recipes.cfg
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/2e5d6d5e1943ac51b1dc862e6143977c1293a94f commit 2e5d6d5e1943ac51b1dc862e6143977c1293a94f Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 09:15:00 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/f9511903eb2d5e04717055a3d7b38df53d01b68b Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/31d165a5c15c7311e0eed1cf731a5585ac9c7f57 Revert of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (patchset #4 id:60001 of https://codereview.chromium.org/2378703002/ ) (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2378283003 [modify] https://crrev.com/2e5d6d5e1943ac51b1dc862e6143977c1293a94f/infra/config/recipes.cfg
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/depot_tools.git/+/8818977467d88125743535010c5b37521694df0a commit 8818977467d88125743535010c5b37521694df0a Author: tandrii <tandrii@chromium.org> Date: Thu Sep 29 11:29:57 2016 Reland of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld Originally landed as issue 2378703002 at patchset 60001 (http://crrev.com/2378703002#ps60001). BUG= chromium:649660 TBR=rmistry@chromium.org Review-Url: https://codereview.chromium.org/2376323002 [modify] https://crrev.com/8818977467d88125743535010c5b37521694df0a/gerrit_util.py [modify] https://crrev.com/8818977467d88125743535010c5b37521694df0a/git_cl.py [modify] https://crrev.com/8818977467d88125743535010c5b37521694df0a/tests/git_cl_test.py
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/e89dd33e1ad112c84ce73ceb58321eb2b2bb3030 commit e89dd33e1ad112c84ce73ceb58321eb2b2bb3030 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 11:34:18 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) depot_tools: https://crrev.com/8818977467d88125743535010c5b37521694df0a Reland of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2379653003 [modify] https://crrev.com/e89dd33e1ad112c84ce73ceb58321eb2b2bb3030/infra/config/recipes.cfg
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/e89dd33e1ad112c84ce73ceb58321eb2b2bb3030 commit e89dd33e1ad112c84ce73ceb58321eb2b2bb3030 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 11:34:18 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) depot_tools: https://crrev.com/8818977467d88125743535010c5b37521694df0a Reland of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2379653003 [modify] https://crrev.com/e89dd33e1ad112c84ce73ceb58321eb2b2bb3030/infra/config/recipes.cfg
,
Sep 29 2016
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/5adf612a2bd69619f94da5c81bc8921c23312453 commit 5adf612a2bd69619f94da5c81bc8921c23312453 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 11:45:12 2016
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/62059a1ce785f0eb404ca27f11cd0ca7b04c4882 commit 62059a1ce785f0eb404ca27f11cd0ca7b04c4882 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 11:50:18 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/e89dd33e1ad112c84ce73ceb58321eb2b2bb3030 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/8818977467d88125743535010c5b37521694df0a Reland of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2378623004 Cr-Commit-Position: refs/heads/master@{#421801} [modify] https://crrev.com/62059a1ce785f0eb404ca27f11cd0ca7b04c4882/infra/config/recipes.cfg
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/9ddc55f51e60feb0399513ffb5d85510188b7544 commit 9ddc55f51e60feb0399513ffb5d85510188b7544 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 11:54:53 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/e89dd33e1ad112c84ce73ceb58321eb2b2bb3030 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) depot_tools: https://crrev.com/8818977467d88125743535010c5b37521694df0a Reland of Add CC_LIST and --cc to Gerrit issues via API to be similar to CCs in Rietveld (tandrii@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:649660 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2376333002 [modify] https://crrev.com/9ddc55f51e60feb0399513ffb5d85510188b7544/infra/config/recipes.cfg
,
Sep 29 2016
Filed https://bugs.chromium.org/p/gerrit/issues/detail?id=4673 to talk about the frequent emails from PG vs Rietveld. I think we can mark this one as fixed because CC_LIST and --cc should be correctly handled now. Please reopen if you disagree.
,
Oct 4 2016
Issue 651131 has been merged into this issue. |
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by rmis...@google.com
, Sep 23 2016