=== rangerpbzzzz is now known as rangerpb [13:28] smoser: utlemming: I've attached a debdiff to https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/1398997; is it worth trying to get that into vivid? [13:28] (It also affects trusty and utopic, but not precise) [13:29] Odd_Bloke, its by design [13:29] well... maybe [13:29] let me find bug [13:30] smoser: Ah, is this because SmartOS uses the serial console which can make other things sad? [13:31] Looking at precise, it's included in the choices, but _not_ in defaults. [13:31] bug 1316475 is the bug that disabled it. [13:32] smoser: That's (mostly about) CloudSigma, not SmartOS. [13:33] Or is it also SmartOS because SmartOS falls in to the 'negative side effects' category? [13:40] Oh, but CloudSigma is in the default list still. [13:42] Odd_Bloke, yeah, cloudsigma had negative side effects so it got taken out of the image. [13:42] and just never put back in [13:43] but the fix that wawas added should be sufifcient [13:43] smoser: Looking at the latest packaging, CloudSigma is in the default list... [13:43] i'm confused. [13:44] yeah, you're right. its in vivid/debian/cloud-init.templates as defaults. [13:44] cloud biuld process seedxs that list maybe ? [13:45] For cloud-specific images, we generally drop a file in to cloud.cfg.d that overrides the list completely. [13:47] So I don't think this is a problem for us, necessarily, just people who want to install cloud-init themselves. [13:47] ok. so, we *should* re-enable it. as the fix that was put in should be sufficient to avoid the negative side affect that caused its removal [13:48] but i wouldn't do that for vivid "right now". just to avoid dealing with regression 3 days before release. escpecially as you said, because it has a work around [13:48] which makes it "medium" by definition i think [13:48] and not "kitten killer" [13:48] Ack. [13:48] but i woudl support it for SRU [13:55] Odd_Bloke, why did you say lcoudsigma above ? [13:55] oh. i see. i did that. [13:55] i'm osrry . confusing. i was just mis-remembering [13:55] i dont know why SmartOS is not i nthat list. probalby just missed it. [13:56] it would appear it SmartOS just never got in to that list. [13:57] but it seems that smartos does the same look (dmi info) so it should be fine. [13:57] Yeah. [13:57] I'll revisit it early next cycle. [14:02] can you confirm i nominated approrrately ? [14:02] and mark 'confirmed' and importance on Trusty anhd W ? [14:02] (i just ignored utopic. i'd not bother with SRU for this to utopic) [14:04] smoser: Confirmed; I can't do importance. [14:05] k. there ya go [14:06] smoser: Thanks! [14:06] sorry for confusing cloudsigma and smartos [14:06] :) [14:07] No worries; the reasons for them being off this list are very similar. :p [14:07] interestingly, they are the same in this regard (they both expect something ont he othe rside of /dev/ttySX, and now both look for dmi data before polling around there) [14:16] Odd_Bloke, i put a comment in https://code.launchpad.net/~daniel-thewatkins/cloud-init/lp1403617/+merge/256812 [14:16] but i'm willing to be wrong on it [14:16] since we're already able to test read_url . [14:20] smoser: No, it makes sense; I was initially relying on more state in that function, but refactored it to avoid that need. [14:24] smoser: I've made the change locally, will push it up some time this afternoon after doing a quick smoke test. [14:25] (About to head in to a meeting) [15:25] smoser: Updated that MP. === Guest36304 is now known as mgagne === harlowja_away is now known as harlowja === shardy is now known as shardy_z