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

oep(ndm): proposal for unique disk identification #2666

Merged
merged 11 commits into from
Sep 15, 2020

Conversation

akhilerm
Copy link
Member

@akhilerm akhilerm commented Jul 8, 2019

Enhancement proposal for NDM. This proposal put forward a solution to uniquely identify a disk in a cluster.

Signed-off-by: Akhil Mohan akhil.mohan@mayadata.io

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Copy link
Contributor

@gila gila left a comment

Choose a reason for hiding this comment

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

Thanks for this, some minor comments!

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Copy link
Member

@vishnuitta vishnuitta left a comment

Choose a reason for hiding this comment

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

Does this need changes to cstor? any limitations to cstor usage?
Any update upgrade things need to be taken care?

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
@github-actions
Copy link

github-actions bot commented Nov 2, 2019

PRs go stale after 30d of inactivity.

@akhilerm akhilerm added this to In progress in NDM via automation Nov 6, 2019
@github-actions github-actions bot closed this Nov 10, 2019
NDM automation moved this from In progress to Done Nov 10, 2019
@akhilerm akhilerm reopened this Nov 11, 2019
@akhilerm akhilerm moved this from Done to In progress in NDM Nov 11, 2019
@kmova kmova added this to the 1.7 milestone Jan 10, 2020
@kmova kmova added this to Pre-commits and Designs - Due: Dec 31st 2019 in 1.7 Release Tracker - Due Feb 15th. Jan 10, 2020
@kmova kmova moved this from To do to Pushed to Next release due to WIP in 1.6 Release Tracker - Due Jan 15th. Jan 10, 2020
@kmova kmova moved this from Pre-commits and Designs - Due: Jan 31st 2020 to To do in 1.7 Release Tracker - Due Feb 15th. Jan 10, 2020
@akhilerm akhilerm moved this from To do to Pre-commits and Designs - Due: Jan 31st 2020 in 1.7 Release Tracker - Due Feb 15th. Jan 17, 2020
@akhilerm akhilerm moved this from Pre-commits and Designs - Due: Jan 31st 2020 to RC1 - Due: Feb 5 2020 in 1.7 Release Tracker - Due Feb 15th. Jan 30, 2020
@kmova kmova moved this from RC1 - Due: Feb 5 2020 to Pushed to Next release due to WIP in 1.7 Release Tracker - Due Feb 15th. Feb 4, 2020
@kmova kmova modified the milestones: 1.7, 1.10 Apr 5, 2020
kmova pushed a commit to openebs-archive/node-disk-manager that referenced this pull request May 3, 2020
…386)

This PR fixes the issue of disk unique identification in NDM. It implements the algorithm mentioned in [OEP#2666](openebs/openebs#2666). This will make NDM uniquely identify disks without an identifier in a cluster. Disk movement from one node to another, creating block device resources for partitions is also supported.

This commit includes:
- add support for partitions in NDM
- implement the new UUID algorithm mentioned in [#2666](openebs/openebs#2666)
- changes made on the a claimed BlockDevice (like creating partitions) are ignored.
- if partitions are created on an unclaimed device, the device is marked inactive and resources will be created for the partitions.

The following manual tests were done with on this commit:
1. disk with WWN - single BlockDevice resource is created
2. disk with WWN and 2 partitions - 2 Block Device resources are created for each partition
3. disks of type 1) and 2) are connected and disconnected while NDM is running
4. disks of type 1) and 2) are removed from one node and attached to another node
5. cstor claiming and creating pool on a disk with WWN and a partitioned disk
6. formatting the device with ext4 after claiming and NDM pod is restarted. Filesystem details are not updated on the resource

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
@kmova kmova moved this from Under development to In Review in NDM May 5, 2020
@akhilerm akhilerm modified the milestones: 1.10, 1.11 Jun 2, 2020
@github-actions
Copy link

github-actions bot commented Sep 1, 2020

PRs go stale after 90d of inactivity.

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
@akhilerm
Copy link
Member Author

akhilerm commented Sep 1, 2020

@kmova . Updated the OEP with latest details regarding implementation and upgrade of blockdevices.

@github-actions github-actions bot closed this Sep 9, 2020
NDM automation moved this from In Review to Done Sep 9, 2020
@akhilerm akhilerm reopened this Sep 10, 2020
Copy link
Member

@kmova kmova left a comment

Choose a reason for hiding this comment

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

The design has been implemented in openebs 2.0.0 = with NDM 0.8.0.

@kmova kmova merged commit f221659 into openebs:master Sep 15, 2020
Akshay-Nagle pushed a commit to Akshay-Nagle/openebs that referenced this pull request Oct 7, 2020
* added workflow chart
* added WWN to flowchart
* added partition also into the workflow
* added 2 points in drawbacks section
* Added new configuration options for UID generation
* added QEMU_HARDDISK to list of models
* updates OEP with latest changes and implementation details

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Kung Fu Panda <akshay_1901cb06@iitp.ac.in>
Akshay-Nagle pushed a commit to Akshay-Nagle/openebs that referenced this pull request Oct 7, 2020
* added workflow chart
* added WWN to flowchart
* added partition also into the workflow
* added 2 points in drawbacks section
* Added new configuration options for UID generation
* added QEMU_HARDDISK to list of models
* updates OEP with latest changes and implementation details

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Kung Fu Panda <akshay_1901cb06@iitp.ac.in>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
NDM
  
Done
1.6 Release Tracker - Due Jan 15th.
  
Pushed to Next release due to WIP
1.7 Release Tracker - Due Feb 15th.
  
Pushed to Next release due to WIP
Development

Successfully merging this pull request may close these issues.

None yet

8 participants