Skip to content

Conversation

@mhkarimi1383
Copy link
Member

@mhkarimi1383 mhkarimi1383 commented Nov 23, 2025

resolves #234

@mergify
Copy link
Contributor

mergify bot commented Nov 23, 2025

🧪 CI Insights

Here's what we observed from your CI run for 013931c.

🟢 All jobs passed!

But CI Insights is watching 👀

@mhkarimi1383
Copy link
Member Author

@tiagolobocastro

Can you review the structure of the configuration?

default=None,
description="Overrides total capacity of storage pool",
)
default_fs: FileSystemName = Field(
Copy link
Member

Choose a reason for hiding this comment

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

I see this more as a "frontend" rather than a "backend storage-pool" configuration?

Copy link
Member Author

Choose a reason for hiding this comment

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

This will be used to make how users are configuring storage pools

@mhkarimi1383
Copy link
Member Author

@mergify rebase

Signed-off-by: Muhammed Hussain Karimi <[email protected]>
Signed-off-by: Muhammed Hussain Karimi <[email protected]>
Signed-off-by: Muhammed Hussain Karimi <[email protected]>
@mergify
Copy link
Contributor

mergify bot commented Dec 7, 2025

rebase

✅ Branch has been successfully rebased

@mhkarimi1383 mhkarimi1383 force-pushed the storage-pool branch 6 times, most recently from 88d8b18 to a6000ef Compare December 7, 2025 14:33
Signed-off-by: Muhammed Hussain Karimi <[email protected]>
Signed-off-by: Muhammed Hussain Karimi <[email protected]>
@mhkarimi1383 mhkarimi1383 force-pushed the storage-pool branch 2 times, most recently from c0722a1 to 6f002de Compare December 7, 2025 15:14
Signed-off-by: Muhammed Hussain Karimi <[email protected]>
Signed-off-by: Muhammed Hussain Karimi <[email protected]>
Signed-off-by: Muhammed Hussain Karimi <[email protected]>
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.

Feature: Multiple StorageClasses with separate dataDirPaths

2 participants