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

Allow to configure PBKDF #163

Merged
merged 10 commits into from
Jan 22, 2021

Conversation

felixfontein
Copy link
Contributor

SUMMARY

Fixes #152 (and implements some more).

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

luks_device

@felixfontein
Copy link
Contributor Author

The luks_device tests now run in < 50 seconds (https://app.shippable.com/github/ansible-collections/community.crypto/runs/843/7/console), as opposed to ~3 minutes (see #152).

@felixfontein felixfontein mentioned this pull request Jan 1, 2021
6 tasks
@felixfontein
Copy link
Contributor Author

@mulatinho if you have some time, please also take a look at this luks_device PR :-)

Copy link
Contributor

@Andersson007 Andersson007 left a comment

Choose a reason for hiding this comment

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

couldn't we cover all the suboptions in CI?

plugins/modules/luks_device.py Outdated Show resolved Hide resolved
@felixfontein
Copy link
Contributor Author

@Andersson007 the main reason is that not every version of cryptsetup supports all of them. I found a way to make it still work though :)

plugins/modules/luks_device.py Outdated Show resolved Hide resolved
plugins/modules/luks_device.py Outdated Show resolved Hide resolved
@felixfontein felixfontein merged commit d921ff1 into ansible-collections:main Jan 22, 2021
@felixfontein felixfontein deleted the luks-iterations branch January 22, 2021 12:21
@felixfontein
Copy link
Contributor Author

@Andersson007 thanks a lot for reviewing this!

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.

luks_device: allow to configure number of iterations
2 participants