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

osd::devices allow working on dmcrypt block devices. #15

Closed
wants to merge 7 commits into from

Conversation

Akendo
Copy link

@Akendo Akendo commented Jul 2, 2013

Allowing to create a partition on top of a dmcrypt block devices. As well you can enable gpt or not.

ceph::osd::device { "/dev/mapper/osd-${id}":
partition_table => false,
partition_table_type => none,
}

This will apply a osd to the dmcrypt devices without a gpt layout. When you want to use
a gpt table on the dmcrypt set this:

ceph::osd::device { "/dev/mapper/osd-${id}":
partition_table => true,
partition_table_type => gpt,
dmcrypt_device
}

This will create a gpt on the dmcrypt block devices. By default a gpt partition table will create.

define ceph::osd::device (
$partition_table = true,
$partition_table_type = gpt,
$dmcrypt_device = false,
)

This is my solution for Unable to create filesystem on dmcrypt devices - puppet ceph issue #13. By default nothing changes for the call of the osd::device.

Only when required when having a dmcrypt device. More things will come later.

Note: How the dmcrypt will be unblock isn't take care of.

so far
4k3nd0

Akendo and others added 7 commits June 11, 2013 17:33
Current workaround to apply osds to dmcrypt devices. Short overview:
 - disbaled creation of partition table and first partition.
 - removed the hardcoded first partition from creation of fs and more(templates etc.).
 - regex to remove mapper from incoming variables.

Now need some clean up, some work on the handling of the device with the right partiton id.
Maybe a check and a parameter option for creation of a partition.
Used the wrong variable inside the unless statment to check for existing xfs filesystem.
@dalgaaf dalgaaf deleted the dmcrypt_parameter branch November 7, 2013 08:50
@EmilienM
Copy link
Member

This pull-request is one year and a half old. I close it, but feel free to re-open it and work on it.

@EmilienM EmilienM closed this Jan 22, 2015
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