Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

document startup procedure #1186

Merged
merged 4 commits into from
Dec 27, 2018
Merged

document startup procedure #1186

merged 4 commits into from
Dec 27, 2018

Conversation

Dieterbe
Copy link
Contributor

FYI to on call people.

docs/startup.md Outdated
| start input plugin(s) | starts backfill (kafka) or listening (carbon, prometheus) and maintain priority based on input lag | if backfilling: above-normal CPU and RAM usage |
| mark ready state | immediately (primary) or after warmup period (secondary) (combined with priority for clustering) | no |

The CPU increase during the kafka backfilling is very significant. typically 10x compared to normal usage.
Copy link
Contributor

@replay replay Dec 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think typically 10x is a statement which includes many assumptions. In our case it might be 10x but in other cases that number could be completely different. I'd rather articulate that in a way which does not make the reader assume that in their own environment it will probably also be 10x. Something like

It can easily be 10x of the normal usage.

Copy link
Contributor Author

@Dieterbe Dieterbe Dec 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

our sizing recommendations (admittedly, not well documented) account for being able to replay a +- 7hr backlog in 30minutes, so 10x to 15x is expected for a properly set up ("typical") cluster. could be higher if the cluster is overprovisioned or lightly loaded, maybe i can clarify that.

docs/startup.md Outdated
| create Input(s) | open connections (kafka) or listening sockets (carbon, prometheus) | no |
| start cluster | starts gossip, joins cluster | no |
| create Index | creates instance and starts write queues | minor RAM increase ~ queue size |
| start API server | opens listening socket and starts handling requests | no |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a little confusing because it sounds like MT will already handle queries, but it loads a lot of data after this step. Maybe it should be mentioned that even though the sockets are listening, it will reject queries at this point until it is ready.

Copy link
Contributor

@replay replay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 with two comments

@Dieterbe Dieterbe merged commit 8ddbfd2 into master Dec 27, 2018
@Dieterbe Dieterbe mentioned this pull request Dec 27, 2018
@Dieterbe Dieterbe added this to the vnext milestone Feb 11, 2019
@Dieterbe Dieterbe deleted the document-startup-produce branch March 27, 2019 21:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants