Installer stops after pressing Cancel on Select a language screen during OEM install

Thank you for taking the time to report this bug and helping to make Ubuntu better. It seems that your bug report is not filed about a specific source package though, rather it is just filed against Ubuntu in general. It is important that bug reports be filed about source packages so that people interested in the package can find the bugs about it. You can find some hints about determining what package your bug might be about at https://wiki.ubuntu.com/Bugs/FindRightPackage. You might also ask for help in the #ubuntu-bugs irc channel on Freenode.

To change the source package that this bug is filed about visit https://bugs.launchpad.net/ubuntu/+bug/1749289/+editstatus and add the package name in the text box next to the word Package.

[This is an automated message. I apologize if it reached you inappropriately; please just reply to this message indicating so.]

Status changed to 'Confirmed' because the bug affects multiple users.

Seems that oem-config-firstboot ignores the question that should be presented to the user on unfinished installation, because of the following line in /usr/bin/ubiquity:
125: fcntl.ioctl(0, termios.TIOCSCTTY, 1)
It steals the terminal from the calling script, so the question is not presented nor any response is read.

So the bottom line is:
1. handling 'wizard.do_reboot()' should not be called for debconf_ui frontend
2. the ioctl call above should not be made

Fixing these 2 things should ensure the flow is as designed in the code.

I tested the test package for Xenial in ppa:dgadomski/ubiquity-lp1749289 and it gives me a choice instead of failing now. After pressing Cancel on the first screen, I get:

Your system is not yet configured. Press 'a' to try again, 's' for a recovery shell, or 'r' to reboot.
[asr]

I did an OEM install and run these commands right before the oem-config-prepare step:

sudo add-apt-repository ppa:dgadomski/ubiquity-lp1749289
sudo apt-get update
sudo apt install ubiquity
sudo oem-config-prepare

Looks good to me.

Committed the fix in git branches, for xenial/bionic/cosmic.

xnox: sadly there was a misunderstanding - I didn't take any actions about it since the merge request was marked "on hold", but I hit some issues with the installer after removing fcntl.ioctl(0, termios.TIOCSCTTY, 1)
The terminal debconf tools were failing without it.

I came up with 2 other ways of mitigating this issue:
https://code.launchpad.net/~dgadomski/ubiquity/+git/ubiquity/+ref/lp1749289-a - preventing cancelling the first page of the installer

https://code.launchpad.net/~dgadomski/ubiquity/+git/ubiquity/+ref/lp1749289-b - implement do_reboot method in the debconf_ui wizard to prevent AttributeError caused by calling missing method.

Please consider using any of those instead (both branches include reverting the previous change).

I've taken your -b fix into trunk.

How is this an ubiquity issue? Shouldn't it be d-i or subiquity?

This bug was fixed in the package ubiquity - 18.10.7

---------------
ubiquity (18.10.7) cosmic; urgency=medium

  [ Dariusz Gadomski ]
  * Implement missing reboot and shutdown methods in debconf_ui (LP: #1749289)

  [ Michael Hudson-Doyle ]
  * Add systemd-resolved to oem-config.target's Wants (LP: #1777900)
  * Automatic update of included source packages: console-setup
    1.178ubuntu8, partman-jfs 53, partman-lvm 124.

 -- Michael Hudson-Doyle <email address hidden> Fri, 17 Aug 2018 10:23:14 +1200

This is an issue with oem-config basically, which comes from ubiquity.

This bug is missing some SRU information which is a bit unfortunate. But seeing that the test case is present and the fix itself seems to have a low regression potential, I will conditionally accept is as is. But please provide some regression potential analysis and fill in the missing information before performing fix validation. Thank you.

Hello David, or anyone else affected,

Accepted ubiquity into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubiquity/18.04.14.7 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

An upload of ubiquity to bionic-proposed has been rejected from the upload queue for the following reason: "should be re-uploaded with console-setup 1.178ubuntu2.7".

Hi,

Can someone update this bug description to follow the SRU template please?

Hello David, or anyone else affected,

Accepted ubiquity into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubiquity/18.04.14.8 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed.Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Hello Łukasz, I was not able to verify this on bionic. The boot stucks even before I had the chance to install the -proposed version of ubiquity.

Steps I took:
1. Use the alternative iso with legacy installer.
2. F4 -> set OEM mode
3. Go through the initial setup.
4. Reboot to perform the OEM configuration.
5. Boot process is stuck without any errors/messages.

Looks like the OEM installation with ubiquity is broken on Bionic. I will report a separate bug about that.

Reported bug #1789920.

Using the workaround of switching to tty2 to be able to log in as oem, clicking cancel on the select language screen now reboots the system and drops me into grub. Is that expected? (the test case doesn't really explain the expected behaviour). It seems vaguely reasonable to me...

Micheal: yes, that is expected after the patch has been applied. I just filled the gaps the installer was expecting. Before the config just died with AttributeError caused by calling a unimplemented method. Hence, my understanding was that this is the original intention.

Simply hiding the cancel button on the first page would be ideal, but I'm not sure if that's possible with the current approach.

So can we call this verification-done-xenial then?

Yes, definitely.

The verification of the Stable Release Update for ubiquity has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

This bug was fixed in the package ubiquity - 18.04.14.8

---------------
ubiquity (18.04.14.8) bionic; urgency=medium

  * Revert inclusion of package updates from cosmic.
  * Automatic update of included source packages: console-setup
    1.178ubuntu2.7.

ubiquity (18.04.14.7) bionic; urgency=medium

  [ Dariusz Gadomski ]
  * Implement missing reboot and shutdown methods in debconf_ui (LP: #1749289)

  [ Michael Hudson-Doyle ]
  * Add systemd-resolved to oem-config.target's Wants (LP: #1777900)

 -- Michael Hudson-Doyle <email address hidden> Sun, 26 Aug 2018 19:53:26 +1200

Michael, is there a release coming for Xenial that could include also a fix for this bug?

SRU proposal for Xenial.

uploaded to xenial queue, now waiting on SRU approver.

Hello David, or anyone else affected,

Accepted ubiquity into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubiquity/2.21.63.9 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-xenial to verification-done-xenial. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-xenial. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

I tested the new package ubiquity 2.21.63.9 from xenial-proposed with Ubuntu 16.04.3 and confirm the Cancel button on the select language screen now reboots the system. I tested multiple times and ended up doing a full real OEM install without any issues.

This bug was fixed in the package ubiquity - 2.21.63.9

---------------
ubiquity (2.21.63.9) xenial; urgency=medium

  * Revert the changes from 2.21.63.8 in favour of the following instead:

  [ Dariusz Gadomski ]
  * Implement missing reboot and shutdown methods in debconf_ui (LP: #1749289)

 -- Adam Conrad <email address hidden> Wed, 08 Aug 2018 20:08:18 -0600