Merge ~dbungert/curtin:cryptsetup-runtime into curtin:master

Proposed by Dan Bungert
Status: Merged
Approved by: Michael Hudson-Doyle
Approved revision: f5f580cf066e9813cb75778af29904c4b820c166
Merge reported by: Server Team CI bot
Merged at revision: not available
Proposed branch: ~dbungert/curtin:cryptsetup-runtime
Merge into: curtin:master
Diff against target: 20 lines (+2/-0)
1 file modified
curtin/block/deps.py (+2/-0)
Reviewer Review Type Date Requested Status
Michael Hudson-Doyle Approve
Server Team CI bot continuous-integration Approve
Review via email: mp+429517@code.launchpad.net

Commit message

block: install cryptsetup if dm_crypt

Co-authored-by: J-P Nurmi <email address hidden>

To post a comment you must log in.
Revision history for this message
Dan Bungert (dbungert) wrote :

vmtested for server, curtin does report wanting to install cryptsetup. Not really needed on server since ubuntu-server-minimal already has it.

Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/curtin/block/deps.py b/curtin/block/deps.py
2index db449d8..5cb23e0 100644
3--- a/curtin/block/deps.py
4+++ b/curtin/block/deps.py
5@@ -59,6 +59,7 @@ def detect_required_packages_mapping(osfamily=DISTROS.debian):
6 DISTROS.debian: {
7 'bcache': ['bcache-tools'],
8 'btrfs': ['^btrfs-(progs|tools)$'],
9+ 'dm_crypt': ['cryptsetup'],
10 'ext2': ['e2fsprogs'],
11 'ext3': ['e2fsprogs'],
12 'ext4': ['e2fsprogs'],
13@@ -77,6 +78,7 @@ def detect_required_packages_mapping(osfamily=DISTROS.debian):
14 DISTROS.redhat: {
15 'bcache': [],
16 'btrfs': ['btrfs-progs'],
17+ 'dm_crypt': ['cryptsetup'],
18 'ext2': ['e2fsprogs'],
19 'ext3': ['e2fsprogs'],
20 'ext4': ['e2fsprogs'],

Subscribers

People subscribed via source and target branches