public inbox for fstests@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox