Remove PuChuangSiDa Certificate Transparency Log |
|||||||||
Issue descriptionFor the reasons documented on https://groups.google.com/a/chromium.org/forum/#!topic/ct-policy/M-rhwDQ1h9E
,
Jun 12 2017
,
Jul 3 2017
Already have approval to merge two log_list.json changes that happened after this (crbug.com/692782, crbug.com/703700 ), but one of them conflicts with this change. Can I get approval to merge this change as well, to simplify the process?
,
Jul 3 2017
Pending LGTM from rsleevi@/awhalley@ of course.
,
Jul 3 2017
This bug requires manual review: M60 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 3 2017
This is an opportunistic merge of a data-only change with no user impact.
,
Jul 5 2017
Good for M60
,
Jul 5 2017
Please add appropriate OSs. Thanks.
,
Jul 5 2017
,
Jul 6 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/317ac81885617d1f3d0d3f18175fbfec65c2ee9f commit 317ac81885617d1f3d0d3f18175fbfec65c2ee9f Author: Eran Messeri <eranm@google.com> Date: Thu Jul 06 10:57:32 2017 Remove PuChuangSiDa as a known Certificate Transparency Log As noted on https://groups.google.com/a/chromium.org/forum/#!topic/ct-policy/M-rhwDQ1h9E , the log has failed to abide by the uptime requirement. BUG= 731836 TBR=eroman@chromium.org Review-Url: https://codereview.chromium.org/2932093002 Cr-Original-Commit-Position: refs/heads/master@{#478639} Review-Url: https://codereview.chromium.org/2966373003 . Cr-Commit-Position: refs/branch-heads/3112@{#527} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} [modify] https://crrev.com/317ac81885617d1f3d0d3f18175fbfec65c2ee9f/net/data/ssl/certificate_transparency/log_list.json
,
Jul 7 2017
,
Jul 7 2017
Can we please make sure to not merge until Merge-Approved-60 label is applied? We need to ensure that Release Owner has provided approval. Marking bug as Merge-Approved-60.
,
Jul 7 2017
My bad, sorry, I was under the impression it was approved, as it's similar to a few other merge requests mentioned above.
,
Jul 7 2017
Sorry about that, I too saw "Good for M60" and missed the fact that the approval hadn't actually been granted yet.
,
Jul 7 2017
My fault, sorry - should have been more explicit. |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by bugdroid1@chromium.org
, Jun 12 2017