Skip to content

Conversation

@mhkarimi1383
Copy link
Member

Partially fixes #234

@mhkarimi1383 mhkarimi1383 requested a review from a team as a code owner November 9, 2025 19:33
@mergify
Copy link
Contributor

mergify bot commented Nov 9, 2025

🧪 CI Insights

Here's what we observed from your CI run for 379b615.

🟢 All jobs passed!

But CI Insights is watching 👀

@mhkarimi1383 mhkarimi1383 force-pushed the metadata-dir-seperation branch 3 times, most recently from 78e43e8 to 2a20b29 Compare November 9, 2025 20:06
@mhkarimi1383
Copy link
Member Author

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Nov 13, 2025

rebase

✅ Branch has been successfully rebased

@mhkarimi1383 mhkarimi1383 force-pushed the metadata-dir-seperation branch from 60d3b47 to 88b31aa Compare November 13, 2025 15:38
@mhkarimi1383
Copy link
Member Author

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Nov 14, 2025

rebase

✅ Branch has been successfully rebased

@mhkarimi1383 mhkarimi1383 force-pushed the metadata-dir-seperation branch from 88b31aa to cada651 Compare November 14, 2025 12:17
Copy link
Member

@tiagolobocastro tiagolobocastro left a comment

Choose a reason for hiding this comment

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

Also how do you envision adding the storage pools to the configuration?

@mhkarimi1383
Copy link
Member Author

Also how do you envision adding the storage pools to the configuration?

a new config option called storagePools

It will be an array consist of

Name that should be unique
Path
isDefault (only one default pool is allowed)
capacityOverride
reservedCapacity

In the chart we will add the current data dir as a default pool

And a new storageclass parameter called storagePool

Copy link
Member

@tiagolobocastro tiagolobocastro left a comment

Choose a reason for hiding this comment

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

btw typo on commit and PR -> s/Seperating/Separating

@mhkarimi1383 mhkarimi1383 changed the title ♻️ Seperating metadata dir and data dir ♻️ Separating metadata dir and data dir Nov 19, 2025
@mhkarimi1383
Copy link
Member Author

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Nov 19, 2025

rebase

✅ Branch has been successfully rebased

@mhkarimi1383 mhkarimi1383 force-pushed the metadata-dir-seperation branch from cea5b45 to 21c1f70 Compare November 19, 2025 13:51
@mhkarimi1383
Copy link
Member Author

@mergify squash

@mergify
Copy link
Contributor

mergify bot commented Nov 19, 2025

squash

✅ Pull request squashed successfully

mhkarimi1383 added a commit to mhkarimi1383/rawfile-localpv that referenced this pull request Nov 19, 2025
* ♻️ Seperating metadata dir and data dir

Partially fixes openebs#234

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* 🔀 Add metadata path migration

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* ♻️ Migrate tasks store file to metadata directory

Signed-off-by: Muhammed Hussain Karimi <[email protected]>
@mhkarimi1383 mhkarimi1383 force-pushed the metadata-dir-seperation branch from 21c1f70 to 275676e Compare November 19, 2025 13:52
@mhkarimi1383
Copy link
Member Author

@mergify squash

@mergify
Copy link
Contributor

mergify bot commented Nov 19, 2025

squash

✅ Pull request squashed successfully

mhkarimi1383 added a commit to mhkarimi1383/rawfile-localpv that referenced this pull request Nov 19, 2025
* ♻️ Seperating metadata dir and data dir

Partially fixes openebs#234

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* 🔀 Add metadata path migration

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* ♻️ Migrate tasks store file to metadata directory

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* ♻️ Separating metadata dir and data dir (openebs#247)

* ♻️ Seperating metadata dir and data dir

Partially fixes openebs#234

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* 🔀 Add metadata path migration

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* ♻️ Migrate tasks store file to metadata directory

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* 🚚 Change default metadata dir

Signed-off-by: Muhammed Hussain Karimi <[email protected]>
@mhkarimi1383 mhkarimi1383 force-pushed the metadata-dir-seperation branch from ad67137 to ed6cc08 Compare November 19, 2025 16:38
* ♻️ Separating metadata dir and data dir

Partially fixes openebs#234

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* 🔀 Add metadata path migration

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* ♻️ Migrate tasks store file to metadata directory

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* ♻️ Separating metadata dir and data dir (openebs#247)

* ♻️ Separating metadata dir and data dir

Partially fixes openebs#234

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* 🔀 Add metadata path migration

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* ♻️ Migrate tasks store file to metadata directory

Signed-off-by: Muhammed Hussain Karimi <[email protected]>

* 🚚 Change default metadata dir

Signed-off-by: Muhammed Hussain Karimi <[email protected]>
@mhkarimi1383 mhkarimi1383 force-pushed the metadata-dir-seperation branch from ed6cc08 to 379b615 Compare November 19, 2025 16:40
@mhkarimi1383
Copy link
Member Author

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Nov 19, 2025

queue

✅ The pull request has been merged automatically

Details

The pull request has been merged automatically at ee49023

@mergify mergify bot added the queued label Nov 19, 2025
@mergify mergify bot merged commit ee49023 into openebs:develop Nov 19, 2025
14 checks passed
@mergify mergify bot removed the queued label Nov 19, 2025
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