From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mx.groups.io with SMTP id smtpd.web12.2610.1607393081404902408 for ; Mon, 07 Dec 2020 18:04:41 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=NZtgKuVd; spf=pass (domain: redhat.com, ip: 216.205.24.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1607393080; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=3NyJ6JHvCB+pqHq8ktKhjLjkV5kUC1CPF2jVLihIsxg=; b=NZtgKuVdJiZq2hb3kAtywqOehs9QBAVBa01/oxCuEGGhIH30vhaxU/jWmq54MGOA3r/dXB ypvH3TLIiE6cXbiBUIJXHNI8+gGgk1ShG4aVbhHUgPCY7BCADRgPLA+NN6U0XYkZ6C5hMa jSftm/5WpicYiQhxlEw0x9owU8uSO/c= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-227-m2ZBy7LVNtiZp0yo077ovw-1; Mon, 07 Dec 2020 21:04:38 -0500 X-MC-Unique: m2ZBy7LVNtiZp0yo077ovw-1 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 81ADE800D55; Tue, 8 Dec 2020 02:04:37 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-30.ams2.redhat.com [10.36.112.30]) by smtp.corp.redhat.com (Postfix) with ESMTP id E36C610023AD; Tue, 8 Dec 2020 02:04:35 +0000 (UTC) To: Michael Kinney , Sean Brogan , "Ard Biesheuvel (ARM address)" , "Leif Lindholm (Nuvia address)" , Andrew Fish Cc: edk2-devel-groups-io From: "Laszlo Ersek" Subject: please let's disable the ECC plugin in CI, in its current form Message-ID: <5c535e4c-a703-6dab-0067-48d7a3f0c3dd@redhat.com> Date: Tue, 8 Dec 2020 03:04:34 +0100 MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Hi All, in my opinion, the ECC plugin in CI has not been productive or helpful. The errors it reports are not convincing, and exceptions are difficult to add, or even express. I request that we disable ECC globally for edk2, or at least make it controllable through a github PR flag (not through CI metafile changes in edk2). (Metafile changes could be an acceptable way of controlling ECC if (a) they didn't have to go through review, (b) if technically such changes would take effect in CI even if they were included in the patch series subject to the CI run. Then a maintainer could evaluate and *immediately* suppress such ECC issues by adding the exceptions as a prepended patch, and force-pushing the updated branch to the same open PR. But this would remain much inferior to simply disabling ECC, or controlling it through a PR label.) Thanks Laszlo