Activity log for bug #1402042

Date Who What changed Old value New value Message
2014-12-12 20:19:45 Scott Moser bug added bug
2014-12-12 20:59:54 Scott Moser nominated for series maas/1.7
2014-12-12 20:59:54 Scott Moser bug task added maas/1.7
2014-12-12 20:59:54 Scott Moser nominated for series maas/trunk
2014-12-12 20:59:54 Scott Moser bug task added maas/trunk
2014-12-12 21:20:04 Blake Rouse maas/1.7: status New Triaged
2014-12-12 21:20:06 Blake Rouse maas/trunk: status New Triaged
2014-12-12 21:20:11 Blake Rouse maas/1.7: importance Undecided High
2014-12-12 21:20:13 Blake Rouse maas/trunk: importance Undecided High
2014-12-12 21:20:16 Blake Rouse maas/1.7: milestone 1.7.2
2015-03-03 16:34:33 Andres Rodriguez maas/1.7: milestone 1.7.2 1.7.3
2015-03-04 17:43:57 Blake Rouse maas/trunk: milestone next
2015-04-09 16:32:45 Scott Moser bug watch added http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=764675
2015-04-09 16:32:45 Scott Moser bug task added debian
2015-04-09 18:35:51 Bug Watch Updater debian: status Unknown Fix Released
2015-07-06 21:13:32 Launchpad Janitor branch linked lp:~smoser/curtin/trunk.lp1402042
2015-07-07 15:44:53 Scott Moser description Kernel documentation at https://www.kernel.org/doc/Documentation/kernel-parameters.txt says: | The kernel parses parameters from the kernel command line up to "--"; | if it doesn't recognize a parameter and it doesn't contain a '.', the | parameter gets passed to init: parameters with '=' go into init's | environment, others are passed as command line arguments to init. | Everything after "--" is passed as an argument to init. In my experience that was never true, and I can reproduce the kernel paying attention to a command line like: -- root=LABEL=cloudimg-rootfs ro console=ttyS0 and the system booting fine, and with data ton ttyS0 on a 3.13 (trusty) kernel. The fun begins a 3.16 (utopic/hwe-u) where the kernel documentation quoted above seems to be valid. Currently maas is takes global command line parameters (as set via 'maas admin set-config name=kernel_opts value="console=ttyS0"') and places them after a '--' in during commissioning and curtin install (and i believe d-i install). The end result is that setting console=ttyS0 as kernel parameters does not work with 3.16 in the commissioning and install environment. Curtin (in following with convention I believe to be true in 'd-i' ) will copy parameters after a '--' to the installed system. So, the installed system works fine, but the install and commissioning is 'blind'. There are 2 solutions here: a.) put kernel flags twice (both before and after a '--') this could in the future make systemd unhappy as those would then turn into flags to systemd. b.) put kernel flags only before and find another way to pass to curtin or d-i Joy. === Begin SRU Information === [Description] In kernel version 3.15, the linux kernel began ignoring command line options that come after a '--' on the kernel command line. Kernel documentation at https://www.kernel.org/doc/Documentation/kernel-parameters.txt says: | The kernel parses parameters from the kernel command line up to "--"; | if it doesn't recognize a parameter and it doesn't contain a '.', the | parameter gets passed to init: parameters with '=' go into init's | environment, others are passed as command line arguments to init. | Everything after "--" is passed as an argument to init. This change of behavior collided with Ubuntu installer's usage (both d-i and curtin). The installers would copy arguments after '--' to the installed system. For example, booting an installer with: kernelarg1 arg2 arg3=bar -- console=ttyS1,115200 would result in the installed system having 'console=ttyS1,115200' on the command line. The problem is then that if you're booting a kernel > 3.15 for your installer (which is the case for trusty-hwe-u or utopic onward) the 'console=' flag will not be paid attention to in the install environment. The solution used by d-i was to modify its token to be '---' or '--'. This solution has been implmented in curtin also. MAAS has used '--' in the past, but will now need to use '---'. However, maas can't simply use '---' as that would result in un-patched installers not copying the parameters over to the installed system. The install environment would get them, but not the installed environment. [Impact] The impact is that when booting an installer with a command line like: kernelarg1 arg2 arg3=bar -- console=ttyS1,115200 the output does not go to the console. Other kernel command line parameters are also ignored, console= is just the most immediately obvious. [Regression Potential] The fix in curtin and in d-i honor both '--' or '---', so chance of regression is low. The fix in MAAS is relatively ignorant as it does not make its decision to use '---' or '--' based on what it is installing. The decision would be hard to make as it would require knowing the version of debian-installer-utils was to be used. With the change in place in MAAS, the failure case would show itself as arguments not being copied to the installed system. [Other Info] To do this properly, we really need to hold the change in MAAS until all expected installers have the fix. Since MAAS uses curtin as the default installer in current trunk and 1.8 branches, we've made curtin expose the information of which token to use to MAAS. That does mean that d-i would broken until all SRUs are done. === End SRU Information === === Original Bug Report === Kernel documentation at https://www.kernel.org/doc/Documentation/kernel-parameters.txt says:  | The kernel parses parameters from the kernel command line up to "--";  | if it doesn't recognize a parameter and it doesn't contain a '.', the  | parameter gets passed to init: parameters with '=' go into init's  | environment, others are passed as command line arguments to init.  | Everything after "--" is passed as an argument to init. In my experience that was never true, and I can reproduce the kernel paying attention to a command line like:   -- root=LABEL=cloudimg-rootfs ro console=ttyS0 and the system booting fine, and with data ton ttyS0 on a 3.13 (trusty) kernel. The fun begins a 3.16 (utopic/hwe-u) where the kernel documentation quoted above seems to be valid. Currently maas is takes global command line parameters (as set via 'maas admin set-config name=kernel_opts value="console=ttyS0"') and places them after a '--' in during commissioning and curtin install (and i believe d-i install). The end result is that setting console=ttyS0 as kernel parameters does not work with 3.16 in the commissioning and install environment. Curtin (in following with convention I believe to be true in 'd-i' ) will copy parameters after a '--' to the installed system. So, the installed system works fine, but the install and commissioning is 'blind'. There are 2 solutions here:  a.) put kernel flags twice (both before and after a '--')     this could in the future make systemd unhappy as those would then turn into flags to systemd.  b.) put kernel flags only before and find another way to pass to curtin or d-i Joy.
2015-07-07 15:46:27 Scott Moser bug task added debian-installer-utils (Ubuntu)
2015-07-07 15:47:18 Scott Moser debian-installer-utils (Ubuntu): status New Fix Released
2015-07-07 15:48:25 Scott Moser nominated for series Ubuntu Utopic
2015-07-07 15:48:25 Scott Moser bug task added debian-installer-utils (Ubuntu Utopic)
2015-07-07 15:48:25 Scott Moser nominated for series Ubuntu Trusty
2015-07-07 15:48:25 Scott Moser bug task added debian-installer-utils (Ubuntu Trusty)
2015-07-07 15:48:25 Scott Moser nominated for series Ubuntu Wily
2015-07-07 15:48:25 Scott Moser bug task added debian-installer-utils (Ubuntu Wily)
2015-07-07 15:48:25 Scott Moser nominated for series Ubuntu Precise
2015-07-07 15:48:25 Scott Moser bug task added debian-installer-utils (Ubuntu Precise)
2015-07-07 15:48:25 Scott Moser nominated for series Ubuntu Vivid
2015-07-07 15:48:25 Scott Moser bug task added debian-installer-utils (Ubuntu Vivid)
2015-07-07 15:48:58 Scott Moser bug task added maas (Ubuntu)
2015-07-07 15:49:36 Scott Moser bug task added curtin
2015-07-07 15:49:54 Scott Moser curtin: importance Undecided Medium
2015-07-07 15:49:54 Scott Moser curtin: status New Confirmed
2015-07-07 15:50:05 Scott Moser bug task added curtin (Ubuntu)
2015-07-07 15:50:52 Scott Moser curtin (Ubuntu Precise): status New Won't Fix
2015-07-07 15:51:08 Scott Moser bug task deleted curtin (Ubuntu Precise)
2015-07-07 15:51:34 Scott Moser curtin (Ubuntu Utopic): importance Undecided Medium
2015-07-07 15:51:34 Scott Moser curtin (Ubuntu Utopic): status New Won't Fix
2015-07-07 15:51:51 Scott Moser curtin (Ubuntu Vivid): importance Undecided Medium
2015-07-07 15:51:51 Scott Moser curtin (Ubuntu Vivid): status New Confirmed
2015-07-07 15:52:07 Scott Moser curtin (Ubuntu Wily): importance Undecided High
2015-07-07 15:52:07 Scott Moser curtin (Ubuntu Wily): status New Confirmed
2015-07-07 15:52:24 Scott Moser curtin (Ubuntu Trusty): importance Undecided High
2015-07-07 15:52:24 Scott Moser curtin (Ubuntu Trusty): status New Confirmed
2015-07-07 16:07:47 Launchpad Janitor branch linked lp:~smoser/maas/lp1402042
2015-07-08 18:00:59 Mathieu Trudel-Lapierre debian-installer-utils (Ubuntu Vivid): status New Fix Released
2015-07-08 18:01:10 Mathieu Trudel-Lapierre debian-installer-utils (Ubuntu Utopic): status New In Progress
2015-07-08 18:01:24 Mathieu Trudel-Lapierre debian-installer-utils (Ubuntu Trusty): status New In Progress
2015-07-08 18:01:24 Mathieu Trudel-Lapierre debian-installer-utils (Ubuntu Trusty): assignee Mathieu Trudel-Lapierre (mathieu-tl)
2015-07-08 18:01:36 Mathieu Trudel-Lapierre debian-installer-utils (Ubuntu Precise): status New In Progress
2015-07-08 18:01:36 Mathieu Trudel-Lapierre debian-installer-utils (Ubuntu Precise): assignee Mathieu Trudel-Lapierre (mathieu-tl)
2015-07-08 18:01:45 Mathieu Trudel-Lapierre debian-installer-utils (Ubuntu Utopic): assignee Mathieu Trudel-Lapierre (mathieu-tl)
2015-07-09 14:11:31 Launchpad Janitor branch linked lp:curtin
2015-07-09 20:10:50 Scott Moser curtin: status Confirmed Fix Committed
2015-07-09 20:17:41 Andres Rodriguez nominated for series maas/1.8
2015-07-09 20:17:41 Andres Rodriguez bug task added maas/1.8
2015-07-09 20:18:03 Andres Rodriguez maas/1.8: milestone 1.8.1
2015-07-09 20:24:16 MAAS Lander maas/trunk: status Triaged Fix Committed
2015-07-09 20:33:53 Launchpad Janitor branch linked lp:ubuntu/wily-proposed/curtin
2015-07-09 21:12:36 Launchpad Janitor curtin (Ubuntu Wily): status Confirmed Fix Released
2015-07-10 13:19:22 Scott Moser maas (Ubuntu Precise): importance Undecided Medium
2015-07-10 13:19:22 Scott Moser maas (Ubuntu Precise): status New Confirmed
2015-07-15 13:30:37 Launchpad Janitor maas (Ubuntu): status New Confirmed
2015-07-15 13:30:37 Launchpad Janitor maas (Ubuntu Trusty): status New Confirmed
2015-07-15 13:30:37 Launchpad Janitor maas (Ubuntu Utopic): status New Confirmed
2015-07-15 13:30:37 Launchpad Janitor maas (Ubuntu Vivid): status New Confirmed
2015-07-15 19:02:41 Chris J Arges debian-installer-utils (Ubuntu Precise): status In Progress Fix Committed
2015-07-15 19:02:49 Chris J Arges bug added subscriber Ubuntu Stable Release Updates Team
2015-07-15 19:02:51 Chris J Arges bug added subscriber SRU Verification
2015-07-15 19:02:58 Chris J Arges tags verification-needed
2015-07-15 19:03:44 Chris J Arges debian-installer-utils (Ubuntu Trusty): status In Progress Fix Committed
2015-07-15 19:04:32 Chris J Arges debian-installer-utils (Ubuntu Utopic): status In Progress Fix Committed
2015-07-15 19:34:58 Launchpad Janitor branch linked lp:ubuntu/precise-proposed/debian-installer-utils
2015-07-15 19:35:00 Launchpad Janitor branch linked lp:ubuntu/trusty-proposed/debian-installer-utils
2015-07-15 19:35:04 Launchpad Janitor branch linked lp:ubuntu/utopic-proposed/debian-installer-utils
2015-07-17 14:22:16 Adam Conrad bug task added debian-installer (Ubuntu)
2015-07-17 15:07:45 Launchpad Janitor branch linked lp:~ubuntu-core-dev/debian-installer/precise-proposed
2015-07-17 15:12:45 Launchpad Janitor branch linked lp:~ubuntu-core-dev/debian-installer/utopic-proposed
2015-07-17 15:21:40 Adam Conrad debian-installer (Ubuntu Utopic): status New Fix Committed
2015-07-17 15:22:46 Adam Conrad debian-installer (Ubuntu Vivid): status New Fix Released
2015-07-17 15:23:01 Adam Conrad debian-installer (Ubuntu Wily): status New Fix Released
2015-07-17 20:09:45 Launchpad Janitor branch linked lp:~ubuntu-core-dev/debian-installer/trusty-proposed
2015-07-17 20:12:07 Adam Conrad debian-installer (Ubuntu Trusty): status New Fix Committed
2015-07-20 19:02:52 Scott Moser attachment added log of running ./run-boottest proposed-di.txt https://bugs.launchpad.net/maas/+bug/1402042/+attachment/4431686/+files/output.log
2015-07-20 19:05:33 Scott Moser tags verification-needed verification-done
2015-07-21 21:27:42 Launchpad Janitor debian-installer-utils (Ubuntu Precise): status Fix Committed Fix Released
2015-07-21 21:27:54 Adam Conrad removed subscriber Ubuntu Stable Release Updates Team
2015-07-21 21:28:13 Launchpad Janitor debian-installer-utils (Ubuntu Trusty): status Fix Committed Fix Released
2015-07-21 21:28:30 Launchpad Janitor debian-installer-utils (Ubuntu Utopic): status Fix Committed Fix Released
2015-07-21 21:30:59 Launchpad Janitor debian-installer (Ubuntu Trusty): status Fix Committed Fix Released
2015-07-21 21:31:17 Launchpad Janitor debian-installer (Ubuntu Utopic): status Fix Committed Fix Released
2015-07-24 09:22:36 Andres Rodriguez maas/1.8: status New Fix Committed
2015-08-10 10:08:16 Andres Rodriguez maas/1.8: status Fix Committed Fix Released
2015-09-17 18:45:58 Launchpad Janitor maas (Ubuntu Wily): status Confirmed Fix Released
2015-09-23 16:11:11 Chris J Arges curtin (Ubuntu Trusty): status Confirmed Fix Committed
2015-09-23 16:11:21 Chris J Arges bug added subscriber Ubuntu Stable Release Updates Team
2015-09-23 16:11:26 Chris J Arges tags verification-done
2015-09-23 16:11:27 Chris J Arges tags verification-needed
2015-09-23 16:13:05 Chris J Arges curtin (Ubuntu Vivid): status Confirmed Fix Committed
2015-09-23 16:33:14 Launchpad Janitor branch linked lp:ubuntu/vivid-proposed/curtin
2015-09-23 16:33:16 Launchpad Janitor branch linked lp:ubuntu/trusty-proposed/curtin
2015-12-23 20:30:08 Ubuntu Foundations Team Bug Bot tags verification-needed removal-candidate verification-needed
2015-12-23 20:54:37 Blake Rouse tags removal-candidate verification-needed verification-done
2016-01-07 00:38:32 Mathew Hodson affects debian debian-installer (Debian)
2016-01-07 00:42:55 Mathew Hodson maas (Ubuntu Utopic): importance Undecided Medium
2016-01-07 00:42:55 Mathew Hodson maas (Ubuntu Utopic): status Confirmed Won't Fix
2016-01-07 00:44:52 Mathew Hodson maas (Ubuntu): importance Undecided Medium
2016-01-07 00:45:06 Mathew Hodson maas (Ubuntu Wily): importance Undecided Medium
2016-01-07 00:45:19 Mathew Hodson maas (Ubuntu Vivid): importance Undecided Medium
2016-01-07 00:45:33 Mathew Hodson maas (Ubuntu Trusty): importance Undecided Medium
2016-01-11 15:11:46 Andres Rodriguez maas/trunk: status Fix Committed Fix Released
2016-03-09 17:53:32 Launchpad Janitor curtin (Ubuntu Trusty): status Fix Committed Fix Released
2016-03-09 17:59:12 Launchpad Janitor curtin (Ubuntu Vivid): status Fix Committed Fix Released
2016-04-29 16:57:30 Launchpad Janitor branch linked lp:~smoser/ubuntu/trusty/curtin/trusty-proposed
2016-08-22 20:54:48 Andres Rodriguez bug task deleted maas/1.7
2017-12-15 22:09:04 Scott Moser curtin: status Fix Committed Fix Released
2018-09-05 22:00:33 Andres Rodriguez maas: milestone next
2018-09-05 22:01:42 Andres Rodriguez maas/trunk: milestone next
2021-10-14 01:27:21 Steve Langasek maas (Ubuntu Precise): status Confirmed Won't Fix