All of lore.kernel.org
 help / color / mirror / Atom feed
From: Loic Dachary <loic@dachary.org>
To: Sage Weil <sweil@redhat.com>, ceph-devel@vger.kernel.org
Subject: Re: giant and hammer dates
Date: Wed, 30 Jul 2014 10:59:43 +0600	[thread overview]
Message-ID: <53D87BBF.2020408@dachary.org> (raw)
In-Reply-To: <alpine.DEB.2.00.1407291558060.26221@cobra.newdream.net>

[-- Attachment #1: Type: text/plain, Size: 3362 bytes --]

Hi Sage,

From my (biased) point of view, the upside is that it will give me more time to complete the locally repairable code for Giant ;-). The downside is that it puts a little less pressure to improve the tools and methods that make a rapid release cycles possible (i.e. unit tests, bug tracking, patch acceptance workflow, package building/gitbuilder, teuthology, pulpito, upgrades testing, ...). In a perfect world Ceph could sustain a three month release cycle without inconveniencing anyone. A longer release cycle (five or six months) would encourage even more complex / bigger changes within a release cycle. It would also probably encourage Ceph developers to forget about the release process tools during two or three months and not improve them as they should be.

IMHO the test cycle is significantly slowing down the release process and a faster, more comprehensive test cycle would help a lot. 

Each commit should be unit / functional tested within seconds, locally (see https://github.com/ceph/ceph/blob/master/src/test/osd/types.cc#L1295 for instance). It is usually more difficult to diagnose / fix a border case when it is discovered during integration tests (i.e. teuthology) rather than with a unit / functional test designed for it. Creating unit tests is often problematic because some of the code base cannot be easily isolated. With a continuous effort to re-arrange parts of the code to be more test friendly, this can eventually be resolved.

Every commit proposed to master should be run against the relevant teuthology suite to help the reviewer. The problem here is that it requires more resources than what Ceph currently has. Harvesting more machines, making it possible for people and organizations amicable to Ceph to easily donate virtual machines could probably help.

This deserves a separate discussions but I wanted to expand on what I meant by "test cycle" and its impact on the release cycle. 

Cheers

On 30/07/2014 05:11, Sage Weil wrote:
> We've talked a bit about moving to a ~4 month (instead of 3 month) 
> cadence.  I'm still inclined in this direction because it means fewer 
> stable releases that we will be maintaining and a longer and (hopefully) 
> more productive interval to do real work in between.
> 
> The other key point is that we don't want a repeat of the firefly delay.  
> I think we should stay as close to a train model as we can.  If something 
> isn't ready by freeze, let it wait for the next cycle.  We shouldn't be 
> cramming things in at the end, especially big things.  As a general rule, 
> big things should be merged early in the cycle so that we have lots of 
> time to shake out the issues that only come out of lots of testing and 
> aren't obvious from code review.
> 
> Anyway, how about:
> 
>           Freeze         Approx Release
>   Giant   Mon Sep  1     Mon Sep 29
>   Hammer  Mon Jan  4     Mon Feb  2
> 
> That gives us another month for Giant, then September to shake out 
> anything issues.  And then three full months before the Hammer freeze.
> 
> What say ye?
> sage
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

-- 
Loïc Dachary, Artisan Logiciel Libre


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 263 bytes --]

  parent reply	other threads:[~2014-07-30  5:00 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-29 23:11 giant and hammer dates Sage Weil
2014-07-29 23:20 ` Mark Nelson
2014-07-30  4:59 ` Loic Dachary [this message]
2014-07-30 14:22   ` Sage Weil
2014-07-30 16:05     ` Loic Dachary
2014-07-30 14:22 ` Gregory Farnum
2014-07-30 16:15   ` Sage Weil
2014-08-02  8:17     ` Justin Erenkrantz
2014-07-30 16:52   ` Mark Nelson
2014-07-30 20:26     ` Eric Eastman

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=53D87BBF.2020408@dachary.org \
    --to=loic@dachary.org \
    --cc=ceph-devel@vger.kernel.org \
    --cc=sweil@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.