Mysterious flush of underutilised chunks 1hr after ingester rollout

[awh]

On the 15th of June:

On the 19th of June:

Both of these happened approximately one hour after an ingester upgrade in which chunks were successfully transferred from terminating ingesters. Chunk max idle is 1h, possibly related?

CC @tomwilkie any thoughts?

[tomwilkie]

Do you know if these are spurious? Or did 800k/3 timeseries stop being written to?

…

On Mon, Jun 19, 2017 at 5:34 PM Adam Harrison ***@***.***> wrote: On the 15th of June: [image: screenshot from 2017-06-19 17-23-20] <https://user-images.githubusercontent.com/1504438/27295498-13ea51d0-5515-11e7-8d8c-2de8b93238b2.png> On the 19th of June: [image: screenshot from 2017-06-19 17-23-33] <https://user-images.githubusercontent.com/1504438/27295513-1ff314b2-5515-11e7-8fcd-2e25cbcdf091.png> Both of these happened approximately one hour after an ingester upgrades in which chunks were successfully transferred from terminating ingesters. Chunk max idle is 1h, possibly related? CC @tomwilkie <https://github.com/tomwilkie> any thoughts? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#467>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAbGhRfyCvo82TNSkCcmdJGS3qxnMfk2ks5sFqMvgaJpZM4N-e40> .

[awh]

Oh - could those be series from metrics scraped from the ingesters which have now terminated?

[awh]

Querying for {job="cortex/ingester"} returns only 1406 time series.

[tomwilkie]

I assume this was part of a rolling upgrade of all of cortex? What does count({job=~"cortex/.*"}) return?

[awh]

9862, and no, in both cases it was just the ingester that was rolled out (in the first instance we had upgraded all of cortex but delayed the ingester whilst we understood #458 so it was eventually done in isolation, and in the second instance we only needed to do the ingester to deploy #460)

[awh]

It's almost like the state reconstructed from transferred chunks in the new ingesters is not identical to that in the old, and so they are making different decisions about what should be flushed...

[awh]

This is also happening on rolling reboots.

[leth]

As @awh said, we just had a single ingester replaced, due to a node reboot, and the queue peaked at 600K

[leth]

Oops

[leth]

I had a brainwave last night; during ingester handover the old and new ingesters are no longer in ACTIVE state, so the distributor will pick an extra ingester, which was not selected before the handover started.
After the handover has finished, it will return to selecting ingester covering the original part of the ring (the new ingester), and the extra ingester will no longer be selected.
After $idleChunkTime the extra ingester will flush these idle chunks.

[leth]

To prevent the ingester picker from picking a temporary node (and allow N concurrent upgrades), we'd need to ensure replication - quorum - N > 0, and allow the picker to select fewer nodes during a handover (quorum < X < replication).

[tomwilkie]

Good catch! We can work round this by including leaving ingesters in the N, but not writing to them, as you suggest. Should be an easy fix.

…

On Thu, 3 Aug 2017 at 12:21, Marcus Cobden ***@***.***> wrote: To prevent the ingester picker from picking a temporary node (and allow N concurrent upgrades), we'd need to ensure replication - quorum - N > 1, and allow the picker to select fewer nodes during a handover (quorum < X < replication). — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#467 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAbGhfUg4ZQygo1EX_9poQbrKIZNs3Juks5sUa1WgaJpZM4N-e40> .

[awh]

Aye, good spot 👏

[stale]

This issue has been automatically marked as stale because it has not had any activity in the past 30 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.