Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add rocksdb resharding playbook #74

Merged
merged 5 commits into from May 27, 2022
Merged

Conversation

guits
Copy link
Collaborator

@guits guits commented May 25, 2022

This adds a new playbook "rocksdb-resharding"

guits added 5 commits May 25, 2022 16:00
This fixes a typo in the documentation of the module ceph_orch_daemon.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
The module doesn't exit properly when the given osd id doesn't exist.
cephadm returns an empty array so we can't call `json.loads(out)[0]`.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
wip

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
This add the rocksdb-resharding.yml playbook to ease the workflow.

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2061501

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
This adds a functional test for rocksdb-resharding playbook.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
@guits
Copy link
Collaborator Author

guits commented May 25, 2022

@neha-ojha @vumrao

@vumrao
Copy link

vumrao commented May 25, 2022

@aclamk

@vumrao
Copy link

vumrao commented May 25, 2022

LGTM!

@guits guits merged commit 854f583 into devel May 27, 2022
6 checks passed
@guits guits deleted the add-rocksdb-resharding-playbook branch May 27, 2022 06:33
This was referenced May 27, 2022
guits added a commit that referenced this pull request May 27, 2022
guits added a commit that referenced this pull request May 27, 2022
@neha-ojha
Copy link
Member

@neha-ojha @vumrao

Thanks @guits!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants