Skip to content

🥅 Validate volume readiness before mount#258

Merged
mergify[bot] merged 1 commit intoopenebs:developfrom
mhkarimi1383:validate-vol-readiness
Nov 30, 2025
Merged

🥅 Validate volume readiness before mount#258
mergify[bot] merged 1 commit intoopenebs:developfrom
mhkarimi1383:validate-vol-readiness

Conversation

@mhkarimi1383
Copy link
Copy Markdown
Member

No description provided.

@mhkarimi1383 mhkarimi1383 requested a review from a team as a code owner November 25, 2025 07:40
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Nov 25, 2025

🧪 CI Insights

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

🟢 All jobs passed!

But CI Insights is watching 👀

@mhkarimi1383 mhkarimi1383 force-pushed the validate-vol-readiness branch 2 times, most recently from eaf0207 to d9e2664 Compare November 25, 2025 09:25
Copy link
Copy Markdown
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.

This shouldn't be possible, how can a not ready volume be staged?
Are we completing the CreateVolume call before the volume is ready?

@mhkarimi1383
Copy link
Copy Markdown
Member Author

@tiagolobocastro I have added this for safety, These functions will not get called when volumes are not ready

@mhkarimi1383
Copy link
Copy Markdown
Member Author

@mergify rebase

Signed-off-by: Muhammed Hussain Karimi <info@karimi.dev>
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Nov 30, 2025

rebase

✅ Branch has been successfully rebased

@mhkarimi1383 mhkarimi1383 force-pushed the validate-vol-readiness branch from d9e2664 to f1b7fcb Compare November 30, 2025 19:09
@mhkarimi1383
Copy link
Copy Markdown
Member Author

@mergify queue

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Nov 30, 2025

queue

✅ The pull request has been merged automatically

Details

The pull request has been merged automatically at 024a44a

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Nov 30, 2025

Merge Queue Status Beta

✅ The pull request has been merged

This pull request spent 19 minutes 11 seconds in the queue, including 19 minutes 6 seconds waiting for CI.
The checks were run in-place.

Required conditions to merge

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

2 participants