CIA-45 | ubiquity-slideshow-ubuntu: cjwatson * r390 ubiquity-slideshow-ubuntu/debian/changelog: changelog for last commit | 09:35 |
---|---|---|
CIA-45 | ubiquity-slideshow-ubuntu: evand * r391 ubiquity-slideshow-ubuntu/ (381 files in 7 dirs): | 09:47 |
CIA-45 | ubiquity-slideshow-ubuntu: Update translations from Launchpad at the request of the Lithuanian | 09:47 |
CIA-45 | ubiquity-slideshow-ubuntu: translation team. | 09:47 |
CIA-45 | ubiquity-slideshow-ubuntu: evand * r392 ubiquity-slideshow-ubuntu/debian/changelog: releasing version 49 | 09:54 |
CIA-45 | ubiquity: cjwatson * r5015 trunk/ (3 files in 2 dirs): | 09:55 |
CIA-45 | ubiquity: Preserve ordering of disks returned by partman-auto rather than | 09:55 |
CIA-45 | ubiquity: shuffling them based on dictionary ordering (LP: #770711). This uses | 09:55 |
CIA-45 | ubiquity: collections.OrderedDict, which requires Python >= 2.7. | 09:55 |
infinity | cjwatson: Have I been smoking the good stuff all weekend, or didn't you commit a partman/OrderedDict change already? | 10:14 |
infinity | Hrm. Maybe you were just discussing it. | 10:14 |
cjwatson | just discussing - I hadn't tested it at that point | 10:17 |
cjwatson | jibel: bug 743359 - have you seen this yourself with 2.8.1? do you have up-to-date logs? I couldn't see any logs from 2.8.1 in that bug or its dups | 10:28 |
ubot2 | Launchpad bug 743359 in ubiquity "Installer: LockFailedException: Failed to lock /target/var/cache/apt/archives/lock" [High,Triaged] https://launchpad.net/bugs/743359 | 10:28 |
cjwatson | jibel: there are reports from beta 2 but that's an earlier version | 10:30 |
jibel | cjwatson, yes I reproduced it this morning. I'll upload the logs | 10:45 |
cjwatson | thanks | 10:48 |
* cjwatson reproduces bug 856418. How curious | 11:19 | |
ubot2 | Launchpad bug 856418 in ubiquity "KDE OEM Mode Hang On Shutdown" [High,Incomplete] https://launchpad.net/bugs/856418 | 11:19 |
* ev is on bug 769350 - have a solution but looks like my code is interacting poorly with partman, investigating | 11:24 | |
ubot2 | Launchpad bug 769350 in partman-auto "resized partition size doesn't count swap space size." [High,Triaged] https://launchpad.net/bugs/769350 | 11:24 |
ev | ah no, it's just monday morning not using logger correctlyt | 11:31 |
gema_ | I had a wiki that had turned into an immutable page... any idea what I may have done wrong? | 11:39 |
cjwatson | you probably got logged out | 11:41 |
gema_ | and there goes my Monday morning moment! thanks, cjwatson | 11:44 |
stgraber | ev: going to be poking at ubiquity this week, trying to fix as many milestoned bugs as possible before the sprint. Do you have some in particular you want me to look at or should I just go through the current list? | 13:17 |
ev | hmm | 13:18 |
ev | seeing where we stand with bug 800760 would be good | 13:18 |
ubot2 | Launchpad bug 800760 in ubiquity "Needs gconf to gsettings updates" [Medium,Confirmed] https://launchpad.net/bugs/800760 | 13:18 |
stgraber | ok, I'll start with that then. | 13:19 |
ev | thanks | 13:20 |
cjwatson | bug 830946 just appeared on our milestone list | 14:02 |
ubot2 | Launchpad bug 830946 in ubiquity "Nothing displayed on embedded terminal." [Medium,Confirmed] https://launchpad.net/bugs/830946 | 14:02 |
ev | really? That's a bit silly. It's a UI nit at best. | 14:24 |
ev | but okay | 14:24 |
cjwatson | jibel: would you like to explain your milestoning? | 14:27 |
CIA-45 | ubiquity: cjwatson * r5016 trunk/ (bin/oem-config-prepare debian/changelog): | 14:40 |
CIA-45 | ubiquity: Run only those parts of oem-config-prepare that require root access as | 14:40 |
CIA-45 | ubiquity: root; in particular displaying confirmation dialogs is now done as the | 14:40 |
CIA-45 | ubiquity: calling user, which avoids KDE confusion (LP: #856418). | 14:40 |
cjwatson | ev: I'm happy to dig out APUE and have another crack at bug 743359 if you like | 14:56 |
ubot2 | Launchpad bug 743359 in ubiquity "Installer: LockFailedException: Failed to lock /target/var/cache/apt/archives/lock" [High,Triaged] https://launchpad.net/bugs/743359 | 14:56 |
ev | please do | 14:58 |
ev | I'm at my wits end with that one | 14:58 |
ev | thanks | 14:58 |
cjwatson | I think you were actually right with the process group idea now that I look at it more closely, but we need to take more care to ensure that update-apt-cache is actually a separate pgrp | 14:58 |
cjwatson | and maybe repeated-retry-for-a-bit-until-dead would be a good idea | 14:59 |
jibel | cjwatson, I milestoned bug 830946 because it is a regression over Natty, has duplicates, clear instructions how to reproduce it and it can not be fixed with a SRU. | 15:02 |
ubot2 | Launchpad bug 830946 in ubiquity "Nothing displayed on embedded terminal." [Medium,Confirmed] https://launchpad.net/bugs/830946 | 15:02 |
jibel | It is medium importance because it doesn't limit the functionality of a core application. | 15:02 |
cjwatson | As long as you don't mind it being ignored | 15:04 |
cjwatson | (we still have a large High list) | 15:04 |
cjwatson | ev: do you have a copy of APUE, by the way? bit of a brick of a book, but very useful | 15:16 |
ev | I do! | 15:16 |
ev | I haven't leafed through it in ages though | 15:16 |
ev | should really get back on that | 15:16 |
ev | I really wish there was an ebook version though | 15:16 |
ev | always a pain to have that extra weight in the suitcase for UDS and the sprints | 15:17 |
cjwatson | yeah, I don't carry it | 15:22 |
cjwatson | I think the way I use it (lots of flicking about) would not be very e-book friendly, although searching would be useful | 15:23 |
cjwatson | though OTOH it has excellent contents/indexing | 15:23 |
cjwatson | ev: I guess the probability of encountering this bug goes up the older the image you're using has got | 15:41 |
cjwatson | since 'apt-get upgrade' will take longer | 15:41 |
ev | indeed | 15:41 |
ev | or if, say, you're on fibre to the canonical datacenter | 15:41 |
cjwatson | sadly I only have current images :-) I'll see if I can fake something up | 15:41 |
cjwatson | for example "drop all outgoing packets" might slow apt-get down enough that copy_all beats it | 15:42 |
ev | heh, nice | 15:42 |
CIA-45 | ubiquity: stgraber * r5017 ubiquity/bin/ubiquity-dm: temp | 15:43 |
stgraber | doh, I need to make CIA clever when I do temporary commits ;) | 15:43 |
ev | hah! I'm an idiot | 15:55 |
ev | was wondering why int(value * 1024 * 1024) wasn't working | 15:55 |
ev | beacause that's a rather large string it's turning into an integer | 15:56 |
ev | because* | 15:56 |
cjwatson | d'oh | 15:56 |
ev | I'm blaming that one on it being Monday | 15:56 |
cjwatson | I blame many things on that | 15:56 |
cjwatson | Although it has the disadvantage of being an excuse that only works for 20% of the working week. Blaming things on there being a 'y' in the day works better. | 15:57 |
ev | lol | 15:57 |
cjwatson | boo, that doesn't work because apt-get update isn't pointed at /target/var/cache/apt | 15:58 |
cjwatson | maybe if I stick the iptables between update and upgrade ... | 16:01 |
bdmurray | Have you all seen bug 825238? | 16:24 |
ubot2 | Launchpad bug 825238 in casper "screen reader does not start in a11y installation" [High,Confirmed] https://launchpad.net/bugs/825238 | 16:24 |
bdmurray | translation bugs, bug 838141, don't require a ubiquity task correct? | 16:31 |
ubot2 | Launchpad bug 838141 in ubiquity "English strings in Swedish translation" [Undecided,New] https://launchpad.net/bugs/838141 | 16:31 |
ev | I could definitely use some extra pairs of eyes on the following merges: | 16:31 |
ev | https://code.launchpad.net/~ev/partman-partitioning/save-swap-size/+merge/77970 | 16:31 |
ev | https://code.launchpad.net/~ev/ubiquity/factor-swap-in-resize/+merge/77973 | 16:31 |
cjwatson | bdmurray: translation> there's only any point in making it ubiquity's problem if the string is untranslatable (i.e. doesn't appear in the pot files) | 16:32 |
bdmurray | cjwatson: okay, thanks | 16:32 |
cjwatson | ok, that seems to reliably reproduce this; nice wide race window | 16:37 |
bdmurray | cjwatson: where does this dialog come from? https://launchpadlibrarian.net/78625495/translation1.png | 16:37 |
cjwatson | bdmurray: console-setup | 16:41 |
cjwatson | bdmurray: that part should be fixed as of 26 Sep though | 16:42 |
cjwatson | yep, installation media from 20110831 | 16:43 |
cjwatson | so you should just be able to mark that part as done | 16:43 |
ev | hm, I might have misunderstood which partition min_size and max_size refer to. Rubbish. | 16:43 |
bdmurray | cjwatson: okay, thanks | 16:43 |
cjwatson | min_size and max_size refer to the partition being resized | 16:44 |
cjwatson | https://code.launchpad.net/~ev/partman-partitioning/save-swap-size/+merge/77970 is entirely full of conflicts, and in any case wanted to be proposed against lp:~ubuntu-core-dev/partman-partitioning/ubuntu not lp:partman-partitioning | 16:45 |
ev | yikes | 16:46 |
ev | will fix | 16:46 |
ev | and indeed | 16:46 |
cjwatson | um, wow, um, complicated | 16:46 |
cjwatson | it's not feasible to have something we call out to from ubiquity directly rather than doing it in the resizing library? | 16:47 |
cjwatson | I can't actually find a bug in it as such, just worried about testing | 16:49 |
cjwatson | 'decode_recipe $(get_recipedir)/[0-9][0-9]atomic linux-swap' is a scary lilne | 16:49 |
cjwatson | *line | 16:49 |
cjwatson | but I guess as long as we make sure it's tested both in d-i and ubiquity ... | 16:49 |
cjwatson | perhaps it would be worth having an NIHed python recipe decoder in P | 16:50 |
cjwatson | although there's some non-trivial logic there; perhaps a wrapper that hides the evil of shelling out | 16:50 |
ev | I just wanted to be sure we weren't hardcoding an assumed swap size | 16:51 |
cjwatson | oh, agreed | 16:51 |
ev | but yes, I suppose something in ubiquity itself would work | 16:51 |
ev | ah | 16:51 |
cjwatson | I guess I was thinking of shipping a tiny script that did . /lib/partman/lib/recipes.sh; decode_recipe blah; echo "$scheme" and then parse its output in a python library | 16:52 |
cjwatson | but this is probably a smaller change for 11.10 | 16:52 |
CIA-45 | ubiquity: stgraber * r5017 ubiquity/ (bin/ubiquity-dm debian/changelog): Update ubiquity-dm to set gsettings keys instead of gconf | 17:12 |
stgraber | ev: ^ that bit is just for ubiquity-dm. The code seems to "work" here (when copy/pasted and ran separately) but I didn't try ubiquity-dm itself | 17:13 |
cjwatson | does that need a bug closure in the changelog? | 17:14 |
stgraber | I didn't include the bug closure because the bug includes quite a bit more than just ubiquity-dm. I'll now be working on the others. | 17:14 |
cjwatson | ok | 17:14 |
stgraber | bug 800760 | 17:15 |
ubot2 | Launchpad bug 800760 in ubiquity "Needs gconf to gsettings updates" [Medium,Confirmed] https://launchpad.net/bugs/800760 | 17:15 |
CIA-45 | ubiquity: cjwatson * r5018 trunk/ (debian/changelog scripts/install.py scripts/update-apt-cache): | 17:16 |
CIA-45 | ubiquity: Go back to killing update-apt-cache's process group, but this time make | 17:16 |
CIA-45 | ubiquity: sure that it's in a separate process group and make more of an effort to | 17:16 |
CIA-45 | ubiquity: ensure that it terminates (LP: #743359). | 17:16 |
cjwatson | ev: ^- a double-check of that wouldn't hurt, though it passes my torture test now and I watched it killing the process group | 17:17 |
ev | cjwatson: will do. About to head out to the cinema with some millbank folk, but I'll have a look first thing in the morning | 17:18 |
ev | as I also fix up these merge proposals | 17:18 |
cjwatson | I think I may be mostly out of steam for today, but should I do an upload? | 17:18 |
stgraber | ev: in plugininstall.py the code won't work with current NM. Should I port it to the new NM or just drop it (no longer using gconf + gnome-keyring but instead ini files in /etc/NetworkManager/system-connections)? | 17:18 |
cjwatson | max testing time and all that | 17:18 |
stgraber | ev: should be fairly trivial to change (copytree of /etc/NetworkManager/system-connections should do the trick) | 17:19 |
stgraber | cjwatson: I'll probably have a few more commits today. I can do an upload when I'm done. When is the deadline to get something in tomorrow's dailies? | 17:20 |
ev | cjwatson: by all means | 17:20 |
ev | stgraber: we definitely still need the functionality | 17:20 |
cjwatson | stgraber: I wouldn't worry about exactness of that | 17:20 |
ev | as the installer now has that wireless page | 17:20 |
ev | but whatever is cheapest, really | 17:20 |
cjwatson | sometime tonight your time should be OK | 17:21 |
cjwatson | I'll do a translation refresh now | 17:21 |
stgraber | ev: ok, I'll replace that code by a simple copytree of all of NM's config then | 17:21 |
ev | brill | 17:21 |
ev | thanks | 17:21 |
stgraber | cjwatson: ok, will upload before leaving the office then | 17:21 |
cjwatson | cool, thanks | 17:21 |
CIA-45 | ubiquity: stgraber * r5019 ubiquity/ (debian/changelog scripts/plugininstall.py): Update copy_network_config to work with Network Manager 0.9 storing the configuration system wide | 17:31 |
CIA-45 | ubiquity: cjwatson * r5019 trunk/debian/ (changelog real-po/hr.po real-po/sk.po): Update translations from Launchpad. | 17:32 |
cjwatson | um, sorry, I beat you to that, you'll need to merge | 17:32 |
cjwatson | (bound branches FTW) | 17:32 |
CIA-45 | ubiquity: stgraber * r5020 ubiquity/ (debian/changelog scripts/plugininstall.py): Update copy_network_config to work with Network Manager 0.9 storing the configuration system wide | 17:33 |
cjwatson | or uncommit/recommit, fair enough :) | 17:33 |
stgraber | merging easily gets messy, uncommit/recommit is easy when it's only one commit, gets tricky when it's multiple including multiple changes to the same file | 17:34 |
* cjwatson is out of steam. See you later | 17:46 | |
CIA-45 | ubiquity: stgraber * r5021 ubiquity/ubiquity/gsettings.py: Add initial implementation of gsettings module | 18:48 |
CIA-45 | ubiquity: stgraber * r5022 ubiquity/bin/ubiquity-dm: Port ubiquity-dm to the new gsettings module | 18:52 |
stgraber | ok, now that I have that gsettings module, porting the rest of the code should be pretty easy | 18:53 |
CIA-45 | ubiquity: stgraber * r5023 ubiquity/ubiquity/components/apt_setup.py: Port apt_setup proxy configuration to gsettings | 19:01 |
stgraber | ouch, gtk_ui.py is going to be fun to switch to gsettings... | 19:03 |
CIA-45 | ubiquity: stgraber * r5024 ubiquity/ubiquity/gsettings.py: If SUDO_USER is set, use that for gsettings by default | 19:08 |
CIA-45 | ubiquity: stgraber * r5025 ubiquity/ubiquity/frontend/gtk_ui.py: Switch gtk_ui from gconf to gsettings | 19:32 |
infinity | Ugh. This anacron thing is killing me. :/ | 19:42 |
CIA-45 | ubiquity: stgraber * r5026 ubiquity/ubiquity/misc.py: Add a note that we need to convert misc.py to gconf (currently not needed as the function is disabled) | 19:43 |
infinity | And I realised that just touching stamp files probably won't help, since it's already running by the time the installer launches, and the delay is ticking. | 19:43 |
infinity | I'm at the point where I'm considering just stopping anacron and cron before the install, and starting it/them at the end. | 19:43 |
infinity | Or maybe changing the default cron.daily delay from 5 minutes to an hour or something... | 19:44 |
infinity | Or make the apt update cronjob exit if [ -x /usr/bin/ubiquity ] ... How evil would that be? :P | 19:45 |
CIA-45 | ubiquity: stgraber * r5027 ubiquity/debian/changelog: Update changelog | 19:46 |
stgraber | ok, there we go for gsettings. Now the fun part, testing all of that... | 19:47 |
stgraber | going to run 2-3 test installs trying to stress test the changes, if that works I'll upload to the queue | 19:48 |
infinity | Any brilliant thoughts on my cron.daily-during-install headaches? :) | 19:48 |
stgraber | checking for oem-config/ubiquity would definitely be the easiest way to fix your use case with minimal impact to other ways of installing Ubuntu | 19:51 |
stgraber | though I think it'd be good for everyone not to have these start on first boot | 19:51 |
infinity | Hrm. | 19:52 |
infinity | -u, --update | 19:52 |
infinity | incremental update, reindexing only those packages whose version has changed since | 19:52 |
infinity | the last run | 19:52 |
infinity | ^-- Note that the call in cron.daily doesn't use -u | 19:52 |
infinity | I wonder if this would make things significantly happier.. | 19:52 |
stgraber | oh, interesting, would probably be a good idea to poke mvo to confirm that -u does what it's supposed to and check that we can indeed use it in the cron | 19:53 |
stgraber | but that sounds like a good solution both for the first boot usecase and for the "this cron job takes hours to run" problem :) | 19:53 |
infinity | Testing to see if it actually matters in practice. | 19:54 |
infinity | It might just be I/O bound on writing the massive DB regardless. | 19:54 |
infinity | Hrm, no, -u does seem to be significantly faster. | 19:55 |
infinity | And you'd think it was introduced for just this use case. :P | 19:55 |
infinity | If only mvo was around. | 19:55 |
superm1 | infinity, you have casper in your ubiquity setup no? how about nuking the job from casper? | 19:57 |
infinity | No casper, no. | 19:58 |
infinity | We have jasper, which doesn't do anything on second boot. (It resizes the root fs, massages some stuff on the filesystem, then reboots into oem-config) | 19:59 |
infinity | Could add some code to run on second boot, though, we don't remove it until later. | 20:00 |
infinity | Seems unpleasant, however. | 20:00 |
infinity | And I don't want this to be ARM-specific. | 20:00 |
infinity | That said, '-u' seems to make a HUGE difference here. | 20:00 |
infinity | Next test is to alter an install image with -u and see if my timeouts go away. | 20:02 |
infinity | (PS: dbus timeouts killing installer processes is a pretty awful failure mode) | 20:03 |
superm1 | i feel like there really shouldn't be any cron jobs allowed to really run during install or during oem-config | 20:03 |
infinity | Yeah, I'm inclined to agree. | 20:03 |
infinity | And stopping anacron/cron at the beginning of the install and starting them at the end does sound sane. | 20:04 |
superm1 | so maybe it's better to tell ubiquity to stop it / restart it | 20:04 |
superm1 | yeah | 20:04 |
infinity | So, 42s for -u, 346s without. | 20:04 |
infinity | I have a feeling this should be done anyway. :P | 20:04 |
CIA-45 | ubiquity: stgraber * r5028 ubiquity/ubiquity/ (frontend/gtk_ui.py gsettings.py): Fix obvious mistakes found by pyflakes | 20:06 |
infinity | So, some sort of disable_cron() called from main, right after acquire_lock(), and an atext.register(enable_cron) | 20:10 |
infinity | This might not be that ugly. | 20:10 |
infinity | I'm assuming there are no python upstart bits, I probably just need to fork shells? | 20:11 |
infinity | apt-cache search upstart python seems to agree. | 20:11 |
superm1 | well actually it looks like anacron is disabled for a standard ubiquity install already to me | 20:12 |
superm1 | there is some code in casper's 25configure_init to do it | 20:12 |
stgraber | you can control upstart over dbus, but subprocess.Popen is going to be a lot easier :) | 20:12 |
infinity | superm1: Ahh, but that's in casper. It should be in the installer itself, methinks. | 20:12 |
stgraber | well, if you just use the live environment and don't start ubiquity, you still don't want xapian update running as it's going to eat your memory (aufs) for no good reason | 20:13 |
superm1 | but you also don't want cron being mean in oem-config, so maybe needs to be both then | 20:13 |
infinity | It completey diverts anacron? | 20:13 |
superm1 | yeah | 20:13 |
superm1 | another way to solve it then is to divert during oem-config-prepare | 20:14 |
infinity | What un-diverts them? | 20:14 |
stgraber | probably nothing | 20:14 |
superm1 | nothing needs to, it's only effective in the live environment | 20:14 |
infinity | Oh, bleh. | 20:15 |
infinity | Right. | 20:15 |
infinity | Not the case for me. | 20:15 |
infinity | My live == My installed system. | 20:15 |
stgraber | yeah, PPA build of ubiquity succeeded! | 20:44 |
stgraber | now to wait 2 hours to get the i386 one and be able to do some tests :) | 20:44 |
stgraber | oh actually, just having ubiquity and ubiquity-frontend-gtk may be enough for my tests | 20:45 |
CIA-45 | ubiquity: stgraber * r5029 ubiquity/ubiquity/gsettings.py: Convert booleans before giving them to subprocess | 21:11 |
stgraber | interesting, apparently we can't hide "suspend" from the session indicator... | 21:14 |
stgraber | so during install the user can still suspend the machine. I guess it's still better than shutdown or reboot though :) | 21:15 |
=== NCommander is now known as Guest45434 | ||
stgraber | infinity: planning on pushing some changes to ubiquity in the next hour or can I upload it? | 21:31 |
infinity | stgraber: No, go to town. | 21:32 |
CIA-45 | ubiquity: stgraber * r5030 ubiquity/debian/changelog: releasing version 2.8.2 | 21:32 |
infinity | stgraber: I need to talk to mvo tonight when he gets in to work. | 21:32 |
CIA-45 | ubiquity: stgraber * r5031 ubiquity/scripts/plugininstall.py: Skip LTSP live when copying network to the target | 21:38 |
Guest45434 | cjwatson: ping, do you think its possible to get one last d-i upload to boost the size of the OMAP4 images? | 23:02 |
=== Guest45434 is now known as NCommander | ||
=== NCommander is now known as Guest61019 | ||
infinity | Guest61019: Commit fix, upload, ask forgiveness? | 23:14 |
infinity | Guest61019: It'll end up being me or Colin that reviews it anyway. :P | 23:14 |
Guest61019 | infinity: k | 23:20 |
cjwatson | NCommand1r: go ahead | 23:23 |
cjwatson | NCommand1r: actually | 23:23 |
cjwatson | NCommand1r: could you just commit but not upload? if you're going to do that, I'd like to refresh translations in the same upload | 23:23 |
cjwatson | NCommand1r: and I have the scripts for that | 23:24 |
NCommand1r | cjwatson: sure, I'll commit in about an hour or so (network here is going about the same speed as a tortise) | 23:30 |
cjwatson | NCommand1r: I'll commit the translations before that, then, and you can upload | 23:31 |
cjwatson | I'll be in bed in an hour | 23:31 |
NCommand1r | k | 23:31 |
NCommand1r | k | 23:31 |
cjwatson | just awaiting the tarball now | 23:31 |
=== NCommand1r is now known as NCommander | ||
=== Guest16725 is now known as StevenK | ||
CIA-45 | debian-installer: cjwatson * r1546 ubuntu/ (3 files in 2 dirs): Update help text translations from Launchpad. | 23:53 |
cjwatson | NCommander: ^- there, just make sure to include that | 23:54 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!