All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ken Dreyer <kdreyer@redhat.com>
To: Sage Weil <sage@newdream.net>
Cc: Andrew Schoen <aschoen@redhat.com>,
	Nathan Cutler <ncutler@suse.cz>,
	"Robin H. Johnson" <robbat2@gentoo.org>,
	Alfredo Deza <adeza@redhat.com>, Abhishek <abhishek@suse.com>,
	ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: Better anticipation for minor releases
Date: Mon, 5 Feb 2018 09:22:52 -0700	[thread overview]
Message-ID: <CALqRxCwD199oo-=TGsnNgy5vLAaFyD2sqqX2YSjtJx9AVZtLQg@mail.gmail.com> (raw)
In-Reply-To: <alpine.DEB.2.11.1802051529210.24931@piezo.novalocal>

On Mon, Feb 5, 2018 at 8:50 AM, Sage Weil <sage@newdream.net> wrote:
> Building from a tag seems like the wrong approach.  What if the build
> fails for some reason (e.g., minor error in cmake files that affects
> release build but not test build)?

Do you have a specific example of this type of failure?

In general I want our test builds to be as close to our release builds
as possible. Some of the reasons for the differences are mainly
historical rather than technical. If there are particular differences
that could cause build failures, let's identify those and fix them!

> Or, perhaps for important, what if we
> get to the point where we do some testing on the final build
> artifact before finalizing/blessing the release?

This is an interesting idea. How would you picture someone doing this
testing, and who would that person be?

What happens when the testing fails and people have already merged
more racing PRs to luminous? We would have to back those out right?

v10.2.5 and v10.2.9 were quick releases to fix problems in the
immediate prior releases. If we merged a bunch of PRs at the same time
there, those quick versions would not have been as quick/safe.

> I don't understand why racing updates is a problem.  git branching/merge
> solves exactly this problem:

Again just my opinion, I think this makes the history harder to read.
For example, it's already a challenge to remember to run "git
describe" on the merge commit rather than the code change commit.

  reply	other threads:[~2018-02-05 16:22 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-31 17:46 Better anticipation for minor releases Alfredo Deza
2018-01-31 19:25 ` Abhishek
2018-01-31 19:35   ` Ken Dreyer
2018-02-02 12:05   ` Alfredo Deza
2018-02-02 12:28     ` Nathan Cutler
2018-02-02 17:15       ` Ken Dreyer
2018-02-02 19:55         ` Robin H. Johnson
2018-02-02 20:15           ` Vasu Kulkarni
2018-02-02 21:35           ` Nathan Cutler
2018-02-02 21:52             ` Robin H. Johnson
2018-02-05 15:28             ` Andrew Schoen
2018-02-05 15:50               ` Sage Weil
2018-02-05 16:22                 ` Ken Dreyer [this message]
2018-02-05 16:30                   ` Sage Weil
2018-02-05 21:20                     ` Nathan Cutler
2018-02-05 21:32                       ` Yuri Weinstein
2018-02-05 21:45                       ` Vasu Kulkarni

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='CALqRxCwD199oo-=TGsnNgy5vLAaFyD2sqqX2YSjtJx9AVZtLQg@mail.gmail.com' \
    --to=kdreyer@redhat.com \
    --cc=abhishek@suse.com \
    --cc=adeza@redhat.com \
    --cc=aschoen@redhat.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=ncutler@suse.cz \
    --cc=robbat2@gentoo.org \
    --cc=sage@newdream.net \
    /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.