All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan van der Ster <dan@vanderster.com>
To: Sage Weil <sage@newdream.net>
Cc: "ceph-devel@vger.kernel.org" <ceph-devel@vger.kernel.org>,
	Herve Rousseau <herve.rousseau@cern.ch>
Subject: Re: scrub randomization and load threshold
Date: Mon, 16 Nov 2015 16:32:11 +0100	[thread overview]
Message-ID: <CABZ+qqnezk0=D1rArOogctrjpsAU7n_wOdLRPJ0HkhwzRbQSpQ@mail.gmail.com> (raw)
In-Reply-To: <alpine.DEB.2.00.1511160717510.7964@cobra.newdream.net>

On Mon, Nov 16, 2015 at 4:20 PM, Sage Weil <sage@newdream.net> wrote:
> On Mon, 16 Nov 2015, Dan van der Ster wrote:
>> Instead of keeping a 24hr loadavg, how about we allow scrubs whenever
>> the loadavg is decreasing (or below the threshold)? As long as the
>> 1min loadavg is less than the 15min loadavg, we should be ok to allow
>> new scrubs. If you agree I'll add the patch below to my PR.
>
> I like the simplicity of that, I'm afraid its going to just trigger a
> feedback loop and oscillations on the host.  I.e., as soo as we see *any*
> decrease, all osds on the host will start to scrub, which will push the
> load up.  Once that round of PGs finish, the load will start to drop
> again, triggering another round.  This'll happen regardless of whether
> we're in the peak hours or not, and the high-level goal (IMO at least) is
> to do scrubbing in non-peak hours.

We checked our OSDs' 24hr loadavg plots today and found that the
original idea of 0.8 * 24hr loadavg wouldn't leave many chances for
scrubs to run. So maybe if we used 0.9 or 1.0 it would be doable.

BTW, I realized there was a silly error in that earlier patch, and we
anyway need an upper bound, say # cpus. So until your response came I
was working with this idea:
https://stikked.web.cern.ch/stikked/view/raw/5586a912

-- dan

  reply	other threads:[~2015-11-16 15:32 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-11-12  9:24 scrub randomization and load threshold Dan van der Ster
2015-11-12 13:29 ` Sage Weil
2015-11-12 14:36   ` Dan van der Ster
2015-11-12 15:10     ` Sage Weil
2015-11-12 15:34       ` Dan van der Ster
2015-11-16 14:25         ` Dan van der Ster
2015-11-16 15:20           ` Sage Weil
2015-11-16 15:32             ` Dan van der Ster [this message]
2015-11-16 15:58               ` Dan van der Ster
2015-11-16 17:06                 ` Dan van der Ster
2015-11-16 17:13                   ` Sage Weil
2015-11-16 17:30                     ` Dan van der Ster

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='CABZ+qqnezk0=D1rArOogctrjpsAU7n_wOdLRPJ0HkhwzRbQSpQ@mail.gmail.com' \
    --to=dan@vanderster.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=herve.rousseau@cern.ch \
    --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.