FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

support multiple osd per node by exkson · Pull Request #1 · exkson/ceph-devstack · GitHub

Repository navigation

support multiple osd per node - #1

Draft
exkson wants to merge 8 commits into
mainfrom
deploy-multiple-osd-per-node
Draft

exkson wants to merge 8 commits into
mainfrom
deploy-multiple-osd-per-node

Conversation

exkson commented Jun 3, 2025 •
edited
Loading

Copy link
Copy Markdown
Owner

The goal of this PR is to support the deployment of multiple OSDs per testnode. To do this,

  • the number of loop devices per node is read from the configuration file
  • loop devices are created using the following naming loop.
  • the testnode container create_cmd is modified to dynamically include the created nodes.

With this implementation, I think each testnode will have enough devices available so that cephadm can deploy many OSDs to it if necessary.

"ceph-devstack",
"-p",
"5432:5432",
"5431:5432",

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This is a temporary change because exposing 5432 conflict with my local postgresql instance port. If there is no reason to expose specifically on this 5432 I think it can be a change to keep.

exkson force-pushed the deploy-multiple-osd-per-node branch from b9d8e22 to 7c23a0a Compare July 7, 2025 09:45
Repository owner deleted a comment from sourcery-ai Bot Aug 7, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL