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