[Merge] ~jchittum/livecd-rootfs:vagrant_slow_boot_workaround_with_null_file into livecd-rootfs:ubuntu/master

Robert C Jennings robert.jennings at canonical.com
Thu Aug 27 20:18:13 UTC 2020


Review: Needs Fixing

Just need a fixup on the kernel cmdline arg in the comment

Diff comments:

> diff --git a/live-build/ubuntu-cpc/hooks.d/base/vagrant.binary b/live-build/ubuntu-cpc/hooks.d/base/vagrant.binary
> index f9b21d3..b236ea9 100755
> --- a/live-build/ubuntu-cpc/hooks.d/base/vagrant.binary
> +++ b/live-build/ubuntu-cpc/hooks.d/base/vagrant.binary
> @@ -154,8 +154,16 @@ Vagrant.configure("2") do |config|
>  
>    config.vm.provider "virtualbox" do |vb|
>       vb.customize [ "modifyvm", :id, "--uart1", "0x3F8", "4" ]
> -# Creating a console log file is not an expected behavior for vagrant boxes. LP #1777827
> -#     vb.customize [ "modifyvm", :id, "--uartmode1", "file", File.join(Dir.pwd, "${prefix}-console.log") ]
> +    # Create a NULL serial port to skip console logging by default
> +      vb.customize [ "modifyvm", :id, "--uartmode1", "file", File::NULL ]
> +    # If console logging is desired, uncomment this line and remove prior
> +    # vb.customize [ "modifyvm", :id, "--uartmode1", "file", File.join(Dir.pwd, "${prefix}-console.log") ]
> +    # Ubuntu cloud images, by default, enable tty=ss0. This enables serial consoles to

Shouldn't "tty=ss0" be "console=ttyS0"?

> +    # connect to the images. With the change related to LP #1777827, removing a serial
> +    # file logger, Vagrant image boot times increased and now run greater than 5 minutes
> +    # Creating a console log file is not an expected defaul behavior for vagrant boxes. LP #1777827

Probably don't need lp 1777827 in here twice.

> +    # As a workaround, we create a console connection to File:NULL. LP #1874453
> +    # This is overwriteable in user files to write to a local file
>    end
>  end
>  EOF


-- 
https://code.launchpad.net/~jchittum/livecd-rootfs/+git/livecd-rootfs/+merge/389918
Your team Ubuntu Core Development Team is requested to review the proposed merge of ~jchittum/livecd-rootfs:vagrant_slow_boot_workaround_with_null_file into livecd-rootfs:ubuntu/master.



More information about the Ubuntu-reviews mailing list