/srv/irclogs.ubuntu.com/2016/01/12/#ubuntu-installer.txt

jibelcyphermox, hey, do you know a recent change that would trigger this question during a preseeded install with d-i ? http://paste.ubuntu.com/14478132/14:53
cyphermoxah, nice14:54
cyphermoxwell, could be that I messed up a merge, didn't notice that one14:54
jibelcyphermox, it's the same preseeed we've been using for years for servers. would it be harmless to preseed this key or it's a real issue?14:54
cyphermoxit's probably harmless to preseed14:54
jibelcyphermox, although on a manual installation there is no difference14:54
cyphermoxbut I think the intent in the past was to not have to, I'll need to check14:55
cyphermoxcould you file a bug?14:55
jibelcyphermox, will do14:56
jibelcyphermox, bug 153324315:04
jibelcyphermox, sorry I don't have more info, test artifacts are not attached to the result15:04
cyphermoxthanks15:05
cyphermoxno worries, that will be plenty I pretty much can just grep for that key and look again at my merge15:05
infinityxnox: Why did you add block-modules to generic/s390x.cfg?20:27
infinityxnox: "Add block-modules (depends on virtio-modules) for s390x" seems pointless with the kernel change to make those builtin.  I think I'll revert that commit.20:29
infinityxnox: Or was there some other reason?20:30
* infinity shrugs and leaves it there for now.20:31
xnoxinfinity, so apw wanted to make it a module on all platforms, rather than changing it to =y on s390x.21:53
xnoxinfinity, so i made the change, just in case. The net difference now is adding nbd module to the installer.21:53
xnoxinfinity, i don't know if apw will revert it to a =m everywhere. If he does, we will need block-modules in all/most d-i flavours.21:54
infinityxnox: It should still be in virtio-modules anyway (if it's =m), I'd say it's a bug that it was in block-modules.21:54
infinityxnox: But this works for now, I won't be too picky (already uploaded your change).21:54
xnoxok.21:56
xnoxinfinity, let me point that virtio-blk was in the wrong package to apw. cause we were slightly dilusional about it.21:56
xnoxinfinity, the whole bug was i went to boot cloud images, and virtio-blk was not in the initramfs, and was only in the -extra image....21:57
infinityxnox: Err, surely this was about virtio-net anyway?21:57
xnoxanyway all good now.21:57
infinityxnox: virtio-blk shouldn't be needed to boot the installer, it should be fetched over the network.21:57
xnoxvirtio-net and virtio-blk. Both were =m in -extra package and in block/virtio-udebs split21:57
xnoxand i do need virtio-net and virtio-blk in the cloud-images initramfs.21:57
xnoxvirtio-blk can be fetched over the network true, in the d-i case.21:58
infinityRight, moving them to -virtual (either in image or =y) is correct, and yes, you need them in the cloud-init initrd, but that has nothing to do with d-i.21:58
infinitySo, we probably should revert your change before we forget this conversation. :P21:58
xnoxright. and apw wants to drop them from =y -> =m everywhere, as otherwise they are always loaded, and never unloaded.21:59
infinityAnd then make sure the kernel debs/udebs are correct if they go =m21:59
xnoxright.21:59
xnoxlet me copy the irc log into a bug with actions to be done.21:59
infinityI'm not convinced virtio-modules needs to exist at all, except maybe as a dep of block-modules and nic-modules, if it has some common bits.22:00
infinityvirtio-net should be in nic- and virtio-blk should be in block-, the more I think about it.22:00
infinityThere's nothing about those drivers that make them any more "special" than any other block or nic device.22:01
xnoxthere is virtio-scsi and virtio-something else.22:01
xnoxin virtio-udebs.22:01
xnoxbut yeah vitio-nic should be in nic-modules22:01
infinityAhh, virtio-rng in virtio-modules.  I wonder if that could just live somewhere else like kernel-image.22:03
infinityAnyhow, if you file a master bug with an IRC dump, I can work it out with Andy, we have a big TODO of "stuff that sucks with the kernel" that we're working through.22:03
xnox&> #ubuntu-kernel22:04

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!