=== cpaelzer__ is now known as cpaelzer === vrubiolo1 is now known as vrubiolo === vrubiolo1 is now known as vrubiolo === hggdh is now known as hggdh-msft === vrubiolo1 is now known as vrubiolo [18:29] rharper: Can you take a look at PR:229 when you have a minute. It's a follow up from our chat last week about the cached datasource object? [18:29] https://github.com/canonical/cloud-init/pull/229 [18:30] fredlef1: I also owe you a followup on https://github.com/canonical/cloud-init/pull/216 apologies I got pulled into a minor escalation [18:30] plan is still to get you an update today [18:34] rharper: I think the bug we were hitting is in part related to https://bugs.launchpad.net/ubuntu/+source/mutter/+bug/1727356 [18:34] Ubuntu bug 1727356 in mutter (Ubuntu Cosmic) "Login screen never appears on early generation Intel GPUs (Core2 and Atom etc)" [High,Fix released] [18:35] fredlef1: definitely [18:35] woah wrong bug [18:35] blackboxsw: maybe; but it's out-of-scope for our bug [18:35] oh [18:35] hehe [18:35] https://bugs.launchpad.net/cloud-init/+bug/1848617 [18:35] Ubuntu bug 1848617 in ifupdown (Ubuntu) "Installing ifupdown on bionic does not install a /etc/network/interfaces that sources /etc/network/interfaces.d/*" [Undecided,New] [18:35] but still out of scope for our bug specifically [18:35] blackboxsw: yes, that is true [18:35] I agree [18:35] though bionic ifupdown on bionic should have a proper include [18:35] the source file is from the image build [18:35] and on bionic it's not installed [18:36] I don't think ifupdown packaging ever contained that [18:36] it was always an image build add [18:36] ahh i see [18:36] but, that sounds exactly correct [18:37] confirmed on xenial dpkg -S /etc/network/interfaces finds no owning pkgs [18:37] blackboxsw: worth asking cpc how best to handle this w.r.t legacy cloud-init [18:37] will do [18:37] that is, cloud-init *could* emit that source stanza in /etc/network/interfaces [18:38] blackboxsw: actually; the xenial image should have that [18:39] rharper: xenial image does contain the file, just not delivered by a package [18:39] https://pastebin.ubuntu.com/p/dNGHqNzKvt/ [18:39] so image build process did it [18:39] correct [18:39] however , what clobbers the file ? [18:39] if you launch a xenial image , it should be present, no? [18:39] so what happens during package install, snapshot, upgrade ? [18:39] rharper: ubuntu@35.247.83.53 [18:40] in [18:40] I see the file looks like it was clobbered by the install of the package [18:41] look at ifupdown postinst [18:41] it will create the file [18:41] +1 see it [18:42] rharper: though see that postinst creates with different incorrect content too [18:42] not incorrect [18:42] just does not filter *.cfg from the .d dir [18:42] source /etc/network/interfaces instead of 'source /etc/network/interfaces.d/*.cfg' [18:43] ahh I overlooked source-directory [18:43] right [18:46] blackboxsw: so; if I launch a new xenial image, the /etc/network/interfaces is what we expect [18:46] rharper: and I don't think the original issue was on xenial at all [18:46] oh ? [18:47] just bionic as I re-readit [18:47] how did I see xenial mentioned as well / [18:47] that makes a lot more sense [18:47] as the in-image eni file is sane [18:47] right, xenial was mentioned as not replacing it [18:47] ok [18:47] *reproducing* [18:47] there we go [18:47] ok so filing a bug and adding workaround notes. [18:48] so, then yes, this comes down to bionic images not having the cpc eni file because it's not ifupdown enabled; and further, ifupdown packging does not provide a file with the cloud-init expected source directive [18:48] it can be resolved in a few places; for now; I suspect we want to purge ifupdown in that image unless pppoe is needed (it is not in the cloud AFAICT) [18:49] rharper: you thinks it's worth demoting pppeoconf installed by ubuntu-desktop from required to recommends too? maybe I'm misreading the bionic pkg debian control [18:49] this should result in rendereing networkd just fine and all is well [18:49] blackboxsw: unclear [18:49] I think workaround first (uninstall ifupdown) [18:49] yeah +1. [18:50] blackboxsw: follow is, if you install bionic from desktop iso, do you get ifupdown ? [18:50] if not, why [18:50] and cloud-init can be defensive in the bionic case w.r.t adding a source directive if the /etc/network/interfaces files does not contain it [18:51] floating the idea of whether the ifupdown package itself should have that in it's auto-generated eni file [19:07] rharper: I tested the PR looks to be working, can you double check? [19:07] ahosmanMSFT: excellent, I'll give it a spin [19:52] rharper: http://releases.ubuntu.com/18.04/ubuntu-18.04.4-desktop-amd64.manifest doesn't that mean the Ubuntu Desktop ISO installs both ifupdown and ppoeconfig pkgs? [19:52] I'm mid download and install run now to be certain [19:53] blackboxsw: well, it means the deb packages are installed [19:53] it does not say, what, if any, iso specific changes are made in livecd-rootfs [19:53] blackboxsw: so, unfortunately you'll need to complete an install [19:54] paride is gone, but I wonder if utah has final image qcow2 images we could post-mortem the filesystem [19:54] * rharper inspects a local bionic desktop install [19:56] blackboxsw: my local bionic desktop does not have any source * stuff in it's eni file [19:58] rharper: my orig cpc image had the netplan disabled comments in /etc/network/interfaces [19:58] yes, cloud images [19:58] but standard desktop wont (just ifupdown installed due to the pppoe dep IIUC)