| === rangerpbzzzz is now known as rangerpb | ||
| 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:28 |
| smoser | Odd_Bloke, its by design | 13:29 |
| smoser | well... maybe | 13:29 |
| smoser | let me find bug | 13:29 |
| Odd_Bloke | smoser: Ah, is this because SmartOS uses the serial console which can make other things sad? | 13:30 |
| 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:31 |
| Odd_Bloke | smoser: That's (mostly about) CloudSigma, not SmartOS. | 13:32 |
| Odd_Bloke | Or is it also SmartOS because SmartOS falls in to the 'negative side effects' category? | 13:33 |
| Odd_Bloke | Oh, but CloudSigma is in the default list still. | 13:40 |
| 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:42 |
| 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:43 |
| 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:44 |
| Odd_Bloke | For cloud-specific images, we generally drop a file in to cloud.cfg.d that overrides the list completely. | 13:45 |
| 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:47 |
| 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:48 |
| 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:55 |
| smoser | it would appear it SmartOS just never got in to that list. | 13:56 |
| 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. | 13:57 |
| 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:02 |
| Odd_Bloke | smoser: Confirmed; I can't do importance. | 14:04 |
| smoser | k. there ya go | 14:05 |
| Odd_Bloke | smoser: Thanks! | 14:06 |
| smoser | sorry for confusing cloudsigma and smartos | 14:06 |
| smoser | :) | 14:06 |
| 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:07 |
| 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:16 |
| 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:20 |
| Odd_Bloke | smoser: I've made the change locally, will push it up some time this afternoon after doing a quick smoke test. | 14:24 |
| Odd_Bloke | (About to head in to a meeting) | 14:25 |
| Odd_Bloke | smoser: Updated that MP. | 15:25 |
| === Guest36304 is now known as mgagne | ||
| === harlowja_away is now known as harlowja | ||
| === shardy is now known as shardy_z | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!