DPDK patches and discussions
 help / color / mirror / Atom feed
From: Bruce Richardson <bruce.richardson@intel.com>
To: Jay Rolette <rolette@infinite.io>
Cc: "techboard@dpdk.org" <techboard@dpdk.org>, "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] Minutes of technical board meeting 2017-12-06
Date: Wed, 13 Dec 2017 09:38:22 +0000	[thread overview]
Message-ID: <20171213093822.GA18344@bricha3-MOBL3.ger.corp.intel.com> (raw)
In-Reply-To: <CADNuJVod-=YZc8GGxCFBuY0d1uewN4ga-s7d5tYu4GjQaKEzfQ@mail.gmail.com>

On Tue, Dec 12, 2017 at 12:30:11PM -0600, Jay Rolette wrote:
> On Tue, Dec 12, 2017 at 11:26 AM, Richardson, Bruce <
> bruce.richardson@intel.com> wrote:
> 
> 
> > Topic: Management of old patches in patchwork * Unanimous agreement
> > that old patches should be rejected in patchwork after a reasonable
> > period, set initially at 3 releases (9 months).  * After a release,
> > all patches dated older than 3 releases previously, e.g. after
> > 17.11, any patches submitted before the release of 17.02, will be
> > marked as rejected and a mail reply sent to the appropriate mailing
> > list thread informing people of the same.  * To have patches
> > reconsidered for future inclusion, a new version of the patches
> > should be submitted to the mailing list.
> >
> 
> Does this mean there is a commitment to act on submitted patches in a
> timely manner? Maybe this is better now, but at least in the past,
> even small patches to fix bugs would sit around with no action on them
> for 6+ months.
> 
> It's been a while since I submitted any patches, so if this isn't an
> issue now, then nevermind :-)
> 
Being honest, I don't think we could ever say it's not a problem, and I
also don't believe we could ever make a committment to always respond to
patches in a timely manner. Each maintainer is responsible for reviewing
and "managing" patches in their area of responsibility and some
maintainers will be faster to respond than others. We have, however, a
documented procedure for having patches merged once acked by the
maintainer, so that should have improved things. Up to getting
maintainer ack, it is still up to the submitter to ping the maintainer
to review if no response is forthcoming on the patch submission.

/Bruce

      reply	other threads:[~2017-12-13  9:38 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-12 17:26 Richardson, Bruce
2017-12-12 18:30 ` Jay Rolette
2017-12-13  9:38   ` Bruce Richardson [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171213093822.GA18344@bricha3-MOBL3.ger.corp.intel.com \
    --to=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    --cc=rolette@infinite.io \
    --cc=techboard@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).