Skip to content

news/blog: Initial submission of RocksDB Tuning Deep Dive#413

Merged
Thingee merged 1 commit intoceph:mainfrom
markhpc:wip-rocksdb-tuning-deep-dive
Jul 25, 2022
Merged

news/blog: Initial submission of RocksDB Tuning Deep Dive#413
Thingee merged 1 commit intoceph:mainfrom
markhpc:wip-rocksdb-tuning-deep-dive

Conversation

@markhpc
Copy link
Member

@markhpc markhpc commented Jul 15, 2022

Initial submission of the RocksDB Tuning Deep Dive Article.

Signed-off-by: Mark Nelson mnelson@redhat.com

@markhpc markhpc changed the title blog/2022: Initial submission of RocksDB Tuning Deep Dive news/blog: Initial submission of RocksDB Tuning Deep Dive Jul 15, 2022
@markhpc
Copy link
Member Author

markhpc commented Jul 15, 2022

Looks like there's some issue with side-by-side images. Probably could clean up some other things to look a bit better too.

Current View:
https://wip-rocksdb-tuning-deep-dive.ceph.io/en/news/blog/2022/rocksdb-tuning-deep-dive/

@markhpc markhpc requested a review from Thingee July 15, 2022 13:48
@markhpc
Copy link
Member Author

markhpc commented Jul 15, 2022

@Thingee Should I just combine the side-by-side graphs into one image? Also, looks like I've got something kind of funky going on with some of the tables. Maybe the syntax being used or something. I'll take a look later. I think I'm going to unquote some of the setting names as well since it seems like it's getting too cluttered.

@markhpc markhpc added the do not merge Don't merge this PR when this label is assigned label Jul 15, 2022
@markhpc markhpc force-pushed the wip-rocksdb-tuning-deep-dive branch 2 times, most recently from 77ecd39 to e5ba647 Compare July 15, 2022 18:23
@@ -0,0 +1,301 @@
---
title: Ceph RocksDB Tuning Deep-Dive
Copy link
Member

@ljflores ljflores Jul 18, 2022

Choose a reason for hiding this comment

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

You could prefix this with "Quincy @ Scale" to match @pcuzner's #408.

Suggested change
title: Ceph RocksDB Tuning Deep-Dive
title: "Quincy @ Scale: Ceph RocksDB Tuning Deep-Dive"

Copy link
Member Author

Choose a reason for hiding this comment

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

I think I'd like to leave this as a standalone article this time. Most of the testing was on Pacific and the main thrust of the article is about the RocksDb settings which are mostly release-agnostic. It might make sense to have a separate Quincy performance article but this one probably isn't it.

---
title: Ceph RocksDB Tuning Deep-Dive
date: 2022-07-14
image: images/octopus.jpg
Copy link
Member

Choose a reason for hiding this comment

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

Love the image XD

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks!

Signed-off-by: Mark Nelson <mnelson@redhat.com>
@markhpc markhpc force-pushed the wip-rocksdb-tuning-deep-dive branch from e5ba647 to e370e29 Compare July 20, 2022 20:46
@markhpc markhpc removed the do not merge Don't merge this PR when this label is assigned label Jul 20, 2022
markhpc pushed a commit to markhpc/ceph that referenced this pull request Jul 21, 2022
@Thingee Thingee merged commit 704d073 into ceph:main Jul 25, 2022
neha-ojha pushed a commit to ceph/ceph-ci that referenced this pull request Jul 27, 2022
markhpc pushed a commit to markhpc/ceph that referenced this pull request Jul 28, 2022
Change RocksDB options based on research findings documented here:

ceph/ceph.io#413

Signed-off-by: Mark Nelson <mnelson@redhat.com>
aaSharma14 pushed a commit to rhcs-dashboard/ceph that referenced this pull request Feb 15, 2023
Change RocksDB options based on research findings documented here:

ceph/ceph.io#413

Signed-off-by: Mark Nelson <mnelson@redhat.com>
jinmyeonglee pushed a commit to jinmyeonglee/ceph that referenced this pull request Sep 26, 2023
Change RocksDB options based on research findings documented here:

ceph/ceph.io#413

Signed-off-by: Mark Nelson <mnelson@redhat.com>

Conflicts:
	src/common/options/global.yaml.in
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants