From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com [209.85.128.42]) by mx.groups.io with SMTP id smtpd.web11.1652.1579292956336076752 for ; Fri, 17 Jan 2020 12:29:16 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=Ot2Kp5CK; spf=pass (domain: linaro.org, ip: 209.85.128.42, mailfrom: leif.lindholm@linaro.org) Received: by mail-wm1-f42.google.com with SMTP id p17so8920064wmb.0 for ; Fri, 17 Jan 2020 12:29:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=a3V4Qxg6WNdkpxZ9/86rhU4qZZVnm1GKWZt+nJC3CYM=; b=Ot2Kp5CKdF+/h+TlvYX6RS9PoicNm8FCeeiDDxNayeIUKE2IPa/SYpoLCpfBPyDw5T DaV/XTqhAwxJLLlmkSXSC5R/ikvSTHd+ogp80gkezYgldtFZuu1wL8cVHxTa97sG5k7i NWGSGnjMFWkeOOdEagqxD1TkQcQ9hNDIAJnxBAk+hDQPLg83vaZVP73uWqeguIvFKkNK gyb3L/u2D+45MbLuliyfYQA7TMO/W7ocoamxBgh8UlJE1PKBSNB8jAAGOswAt4X6Lu/e frNXisUv9hrzkX7JJA8hrjdArOnFGdyCpZqImMVCXQQkT2m7Jq401zmdi/L5syPTb/CC Qusg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=a3V4Qxg6WNdkpxZ9/86rhU4qZZVnm1GKWZt+nJC3CYM=; b=GFRfBEYvhJxu+uVy/4v1lV8dmGWYXX6rwjbdyH18diJ17uqSjVHeivfHFWY3eALxc7 j/3ClB8ec1EtpPn+CtIPgqWSTpxcsiXLu8aMw7tpcVjDWLJ7Pk3tcbyQzvj3b2+5cE+w Dg29IyHzbgP8EgLJ4wbibIQFlbQg1UIbvwLLSU8F3rlmxeeRJLfheClLbFYuzoDQTqYr s7mWyQXjWrqkyk82VbT4Vv7jmH7IssVJtYOPuxPBBRy8b0ILqR7NmFYSFNzLzI8YmtaA oNlJVSAkTFoH7Xzily3LaTJAk929Fvy6/9CmhBdvJppmJ6Ri7LlGwNG43i6/sMQs64ZE gjgA== X-Gm-Message-State: APjAAAVjg5/+aSsiZL6R5gxi0FZjypGxpjpSI9fQL33okOzl+bYgq051 ztgSxTTHSacV8ccMLaWuUqU8LA== X-Google-Smtp-Source: APXvYqyGJw5NB5R7v3Yaw/awFkZ6BlYOjrXFIagbLdFC+hZK3vp2xvs+QXsJpoU70XFMlu/6jHF5fQ== X-Received: by 2002:a1c:6707:: with SMTP id b7mr6489992wmc.54.1579292954894; Fri, 17 Jan 2020 12:29:14 -0800 (PST) Return-Path: Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id b137sm2419737wme.26.2020.01.17.12.29.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 17 Jan 2020 12:29:13 -0800 (PST) Date: Fri, 17 Jan 2020 20:29:12 +0000 From: Leif Lindholm To: "Kinney, Michael D" Cc: "devel@edk2.groups.io" Subject: Re: [edk2-devel] [Patch v5 21/22] .mergify: Add Mergify YML pull request rules configuration file Message-ID: <20200117202912.GO20629@bivouac.eciton.net> References: <20191111084107.5368-1-michael.d.kinney@intel.com> <20191111084107.5368-22-michael.d.kinney@intel.com> <20200117194418.GN20629@bivouac.eciton.net> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Mike, Many thanks for resolving! I'll try your other suggestions if this happens again in future. Best Regards, Leif On Fri, Jan 17, 2020 at 20:22:03 +0000, Kinney, Michael D wrote: > Hi Leif, > > I have not tried the mergify commands like that. I can investigate > further if we think we need them. > > I saw that some of the Azure pipelines checks did not run. That > Is what was blocking Mergify. Mergify was waiting for status. > > I did a Close and Re-open to re-trigger Azure Pipelines and that > worked. You changed have been committed. > > We have seen a very rare condition in communication between > Azure Pipelines and GitHub. This close/re-open is what we > can try first. The 2nd approach is to abandon the current > PR and create a new PR with the same content. That has only > had to be used once since enabling in early November. > > Mike > > > -----Original Message----- > > From: Leif Lindholm > > Sent: Friday, January 17, 2020 11:44 AM > > To: Kinney, Michael D > > Cc: devel@edk2.groups.io > > Subject: Re: [edk2-devel] [Patch v5 21/22] .mergify: > > Add Mergify YML pull request rules configuration file > > > > Hi Mike (on-list), > > > > Embarassing to admit, but I hadn't tried to do my first > > mergify commit > > until the Maintainers.txt right now, and that isn't > > working as I expected. > > > > I tried to force a retry on > > https://github.com/tianocore/edk2/pull/307 > > (since I added the 'push' tag *after* submitting the > > github pull > > request), but as you can see mergify says > > "@leiflindholm is not allowed > > to run commands". > > > > Am I missing something obvious? > > > > Best Regards, > > > > Leif > > > > On Mon, Nov 11, 2019 at 00:41:06 -0800, Michael D > > Kinney wrote: > > > Add directory for the Mergify YML configuration files > > that > > > provides rules and actions used to process a pull > > request. > > > > > > * Auto commit a PR from EDK II Maintainer with 'push' > > label > > > set and all CI checks pass > > > * Auto close a PR from any developers without 'push' > > label > > > set and all CI checks pass. > > > * Auto close a PR from a non EDK II Maintainer that > > has > > > the 'push' label set. > > > * Post a comment to a PR that has a merge conflict. > > > Submitter can resolved conflicts and reopen the PR. > > > * Post a comment to a PR that fails PatchCheck.py > > > Submitter can resolve PatchCheck.py issues and > > > reopen the PR. > > > > > > Cc: Liming Gao > > > Cc: Sean Brogan > > > Cc: Bret Barkelew > > > Signed-off-by: Michael D Kinney > > > > > Reviewed-by: Liming Gao > > > --- > > > .mergify/config.yml | 97 > > +++++++++++++++++++++++++++++++++++++++++++++ > > > 1 file changed, 97 insertions(+) > > > create mode 100644 .mergify/config.yml > > > > > > diff --git a/.mergify/config.yml > > b/.mergify/config.yml > > > new file mode 100644 > > > index 0000000000..26583de0f6 > > > --- /dev/null > > > +++ b/.mergify/config.yml > > > @@ -0,0 +1,97 @@ > > > +## @file > > > +# Mergify YML file that automatically merges a > > GitHub pull request against > > > +# edk2-ci if all of the GitHub branch protections > > have passed. It also > > > +# contains rules to: > > > +# * auto close branches that are not from an EDK II > > Maintainer > > > +# * post a comment on pull requests that have merge > > conflicts. > > > +# * post a comment on pull requests that have > > PatchCheck.py errors. > > > +# > > > +# Configuration Notes: > > > +# * Update the 'base=edk2-ci' statements with the > > name of the branch to merge > > > +# pull requests. > > > +# > > > +# * Update the 'status-failure' statement with the > > name of the name of the Azure > > > +# Pipelines Build that performs the EDK II > > Maintainer check. > > > +# > > > +# * This file must be checked into the 'default' > > branch of a repo. Copies > > > +# of this file on other branches of a repo are > > ignored by Mergify. > > > +# > > > +# Copyright (c) 2019, Intel Corporation. All rights > > reserved.
> > > +# SPDX-License-Identifier: BSD-2-Clause-Patent > > > +# > > > +# https://github.com/apps/mergify > > > +# https://doc.mergify.io/ > > > +# > > > +## > > > + > > > +pull_request_rules: > > > + > > > + - name: Automatically merge a PR when all required > > checks pass and 'push' label is present > > > + conditions: > > > + - base=master > > > + - label=push > > > + - author=@tianocore/edk-ii-maintainers > > > + - status-success=tianocore.PatchCheck > > > + - status-success=Ubuntu GCC5 PR > > > + - status-success=Windows VS2019 PR > > > + actions: > > > + merge: > > > + strict: true > > > + method: rebase > > > + > > > + - name: Automatically close a PR when all required > > checks pass and 'push' label is not present > > > + conditions: > > > + - base=master > > > + - -label=push > > > + - -closed > > > + - status-success=tianocore.PatchCheck > > > + - status-success=Ubuntu GCC5 PR > > > + - status-success=Windows VS2019 PR > > > + - status-success=Ubuntu GCC5 PR (FINISHED) > > > + - status-success=Windows VS2019 PR (FINISHED) > > > + actions: > > > + close: > > > + message: All checks passed. Auto close > > personal build. > > > + > > > + - name: Post a comment on a PR that can not be > > merged due to a merge conflict > > > + conditions: > > > + - base=master > > > + - conflict > > > + actions: > > > + comment: > > > + message: PR can not be merged due to > > conflict. Please rebase and resubmit > > > + > > > + - name: Automatically close a PR that fails the > > EDK II Maintainers membership check and 'push' label is > > present > > > + conditions: > > > + - base=master > > > + - label=push > > > + - -author=@tianocore/edk-ii-maintainers > > > + actions: > > > + close: > > > + message: PR submitter is not a member of the > > Tianocore EDK II Maintainers team > > > + > > > + - name: Post a comment on a PR if PatchCheck fails > > > + conditions: > > > + - base=master > > > + - status-failure=tianocore.PatchCheck > > > + actions: > > > + comment: > > > + message: PR can not be merged due to a > > PatchCheck failure. Please resolve and resubmit > > > + > > > + - name: Post a comment on a PR if Ubuntu GCC5 > > fails > > > + conditions: > > > + - base=master > > > + - status-failure=Ubuntu GCC5 PR > > > + - status-success=Ubuntu GCC5 PR (FAILED) > > > + actions: > > > + comment: > > > + message: PR can not be merged due to an > > Ubuntu GCC5 failure. Please resolve and resubmit > > > + > > > + - name: Post a comment on a PR if Windows VS2019 > > fails > > > + conditions: > > > + - base=master > > > + - status-failure=Windows VS2019 PR > > > + - status-success=Windows VS2019 PR (FAILED) > > > + actions: > > > + comment: > > > + message: PR can not be merged due to a > > Windows VS2019 failure. Please resolve and resubmit > > > -- > > > 2.21.0.windows.1 > > > > > > > > > > > >