Merge lp:~rbalint/livecd-rootfs/vagrant-interactive-xenial into lp:~ubuntu-core-dev/livecd-rootfs/xenial-proposed

Proposed by Balint Reczey
Status: Merged
Merged at revision: 1452
Proposed branch: lp:~rbalint/livecd-rootfs/vagrant-interactive-xenial
Merge into: lp:~ubuntu-core-dev/livecd-rootfs/xenial-proposed
Diff against target: 29 lines (+10/-1)
2 files modified
debian/changelog (+7/-0)
live-build/ubuntu-cpc/hooks/042-vagrant.binary (+3/-1)
To merge this branch: bzr merge lp:~rbalint/livecd-rootfs/vagrant-interactive-xenial
Reviewer Review Type Date Requested Status
Chris Glass (community) Approve
Review via email: mp+335508@code.launchpad.net

Description of the change

Fix running vagrant image build interactively.

To post a comment you must log in.
Revision history for this message
Balint Reczey (rbalint) wrote :

Without the fix the build asks for password for the new user when running lb in interactive shell and this blocks the build.

1452. By Balint Reczey

Don't ask for password and GECOS while creating vagrant user

1453. By Balint Reczey

Update changelog

Revision history for this message
Chris Glass (tribaal) wrote :

Thanks for adding the comment!
+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2017-12-18 09:11:26 +0000
3+++ debian/changelog 2017-12-21 08:51:27 +0000
4@@ -1,3 +1,10 @@
5+livecd-rootfs (2.408.27) xenial; urgency=medium
6+
7+ * Don't ask for password and GECOS while creating vagrant user
8+ (LP: #1569237)
9+
10+ -- Balint Reczey <rbalint@ubuntu.com> Thu, 21 Dec 2017 09:20:32 +0100
11+
12 livecd-rootfs (2.408.26) xenial; urgency=medium
13
14 * Added a "vagrant" user to the vagrant image in addition to the "ubuntu"
15
16=== modified file 'live-build/ubuntu-cpc/hooks/042-vagrant.binary'
17--- live-build/ubuntu-cpc/hooks/042-vagrant.binary 2017-12-07 13:09:43 +0000
18+++ live-build/ubuntu-cpc/hooks/042-vagrant.binary 2017-12-21 08:51:27 +0000
19@@ -63,7 +63,9 @@
20 # Vagrant users expect a "vagrant" user with a "vagrant" username.
21 # See https://www.vagrantup.com/docs/boxes/base.html
22 # Note: We decided NOT to allow root login with a default password.
23-chroot ${mount_d} adduser vagrant
24+# --disabled-password and --gecos need to be passed to avoid adduser
25+# asking questions when running in an interactive shell
26+chroot ${mount_d} adduser vagrant --disabled-password --gecos ""
27 echo "vagrant:vagrant" | chroot ${mount_d} chpasswd
28
29 # The vagrant user should have passwordless sudo.

Subscribers

People subscribed via source and target branches