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

refactor(csi-driver): replace MountIters with SafeMountIters #749

Merged
merged 2 commits into from
Feb 20, 2024

Conversation

niladrih
Copy link
Member

@niladrih niladrih commented Feb 18, 2024

@niladrih
Copy link
Member Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Feb 18, 2024
@bors-openebs-mayastor
Copy link

try

Build succeeded:

@niladrih niladrih marked this pull request as draft February 18, 2024 13:54
@niladrih niladrih marked this pull request as ready for review February 19, 2024 15:56
@niladrih niladrih force-pushed the refactor-mountiter branch 2 times, most recently from 2784d94 to 98ac45c Compare February 19, 2024 15:58
@niladrih
Copy link
Member Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Feb 19, 2024
@bors-openebs-mayastor
Copy link

try

Build succeeded:

@niladrih
Copy link
Member Author

bors merge

@bors-openebs-mayastor
Copy link

Build succeeded:

@bors-openebs-mayastor bors-openebs-mayastor bot merged commit efdc30e into develop Feb 20, 2024
4 checks passed
@bors-openebs-mayastor bors-openebs-mayastor bot deleted the refactor-mountiter branch February 20, 2024 17:06
bors-openebs-mayastor bot pushed a commit to openebs/mayastor that referenced this pull request Feb 21, 2024
1591: refactor(io-engine): use SafeMountIter instead of proc_mounts::MountIter r=niladrih a=niladrih

Changes:
- Removes proc_mounts as a dependency.
- Adds utils/dependencies/devinfo as a depdency
- Uses devinfo::mountinfo::SafeMountIter in place of proc_mounts::MountIter

Ref:
- openebs/mayastor-dependencies#65
- openebs/mayastor-control-plane#749

Co-authored-by: Niladri Halder <[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.

2 participants