All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eryu Guan <guan@eryu.me>
To: "Darrick J. Wong" <djwong@kernel.org>
Cc: Eryu Guan <guaneryu@gmail.com>,
	fstests@vger.kernel.org, Dave Chinner <david@fromorbit.com>,
	Eric Sandeen <sandeen@sandeen.net>,
	Amir Goldstein <amir73il@gmail.com>,
	zlang@redhat.com
Subject: Re: maintainership of fstests
Date: Mon, 11 Apr 2022 00:15:36 +0800	[thread overview]
Message-ID: <YlMCqJj3Mllobfj4@desktop> (raw)
In-Reply-To: <20220321221336.GM8241@magnolia>

On Mon, Mar 21, 2022 at 03:13:36PM -0700, Darrick J. Wong wrote:
> On Mon, Mar 21, 2022 at 01:38:23AM +0800, Eryu Guan wrote:
> > Hi folks,
> > 
> > It's almost 6 years since I've taken the maintainership of fstests, and
> > I tried my best to keep the pace of weekly update, or at least bi-weekly
> > update due to something like public holidays.
> > 
> > But it's been a month since last fstests update due to my personal
> > urgent issues, and some patches got no review for more than one month.
> > So I think it's time to re-consider the maintainership of fstests.
> > 
> > I will have less spare time in the foreseeable future, as I have to
> > spend more time on my family, so it's hard to keep the weekly update
> > pace. And six years is a long time, I think it's time to have a new
> > maintainer.
> 
> Thank you for your service for all that time! :)
> 
> > Or we could go to the group-maintain way? As Darrick mentioned before
> > (for xfs not fstests, if I recall correctly). Then we need a new primary
> > maintainer :)
> > 
> > What do you think?
> 
> Splitting responsibility for maintenance doesn't seem like a huge step
> to me -- for fs-specific tests, we need the developers for that fs to
> review test changes.  Review for tests/generic/ can be done by the
> fs{devel,*} community at large, much as it is done now.

Agreed.

> 
> The /difficult/ part, I think, is handling things like treewide
> reorganizations, and integration testing the test suite.  That, I think,
> is what really requires a primary maintainer who has broader visibility
> into what's going on.

Agreed.

> 
> That primary maintainer also has to have time to run a build-and-test
> farm of all the major fstests clients (ext*, xfs, btrfs, overlay, nfs,
> what else?) to make sure that new code doesn't break existing
> filesystems' ability to test themselves.

Yes, that's what I did when I was at RH, I ran regression tests on all
major filesystems and on all major arches (x86_64 ppc64(le) aarch64)
before pushing new patches to git repo to make sure the new release is
in good shape.

And that's the problem for me now, as I don't have such machine resource
anymore.. And that's what Zorro has now :)

> 
> Granted, (speaking only for XFS, probably btrfs, and maybe even ext*),
> we seem to notice regressions pretty fast when we download the weekly
> release, so I think the primary maintainer's focus probably ought to be
> more towards the non-mainstream Linux filesystems.
> 
> Also: over the past 6 years, I have /really/ enjoyed the fact that
> fstests has small releases every 1-2 weeks as opposed to larger
> infrequent drops.  It's a relief not to have to chase a merge window
> like I do for the kernel.  Thank you a bunch for keeping that going!

Thanks for all the help and reviews all these years!

Thanks,
Eryu

> 
> --D
> 
> FYI: It's Spring Break here in the US, so you might not get much of a
> response for a week or two.
> 
> > 
> > Thanks,
> > Eryu
> > 
> > P.S.
> > I'll keep the maintainer role and do the review & update as usual until
> > all things settle down.

      reply	other threads:[~2022-04-10 16:15 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-20 17:38 maintainership of fstests Eryu Guan
2022-03-21  5:51 ` Zorro Lang
2022-03-21 16:19 ` Zorro Lang
2022-04-04 15:36   ` Eric Sandeen
2022-04-04 16:15     ` Zorro Lang
2022-04-05  0:57       ` Darrick J. Wong
2022-04-05  9:04       ` Ritesh Harjani
2022-04-08  7:57       ` Boyang Xue
2022-04-10 16:24   ` Eryu Guan
2022-04-10 18:13     ` Darrick J. Wong
2022-03-21 22:13 ` Darrick J. Wong
2022-04-10 16:15   ` Eryu Guan [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=YlMCqJj3Mllobfj4@desktop \
    --to=guan@eryu.me \
    --cc=amir73il@gmail.com \
    --cc=david@fromorbit.com \
    --cc=djwong@kernel.org \
    --cc=fstests@vger.kernel.org \
    --cc=guaneryu@gmail.com \
    --cc=sandeen@sandeen.net \
    --cc=zlang@redhat.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.