CIA-45ubiquity-slideshow-ubuntu: cjwatson * r390 ubiquity-slideshow-ubuntu/debian/changelog: changelog for last commit09:35
CIA-45ubiquity-slideshow-ubuntu: evand * r391 ubiquity-slideshow-ubuntu/ (381 files in 7 dirs):09:47
CIA-45ubiquity-slideshow-ubuntu: Update translations from Launchpad at the request of the Lithuanian09:47
CIA-45ubiquity-slideshow-ubuntu: translation team.09:47
CIA-45ubiquity-slideshow-ubuntu: evand * r392 ubiquity-slideshow-ubuntu/debian/changelog: releasing version 4909:54
CIA-45ubiquity: cjwatson * r5015 trunk/ (3 files in 2 dirs):09:55
CIA-45ubiquity: Preserve ordering of disks returned by partman-auto rather than09:55
CIA-45ubiquity: shuffling them based on dictionary ordering (LP: #770711). This uses09:55
CIA-45ubiquity: collections.OrderedDict, which requires Python >= 2.7.09:55
infinitycjwatson: Have I been smoking the good stuff all weekend, or didn't you commit a partman/OrderedDict change already?10:14
infinityHrm.  Maybe you were just discussing it.10:14
cjwatsonjust discussing - I hadn't tested it at that point10:17
cjwatsonjibel: 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 dups10:28
ubot2Launchpad bug 743359 in ubiquity "Installer: LockFailedException: Failed to lock /target/var/cache/apt/archives/lock" [High,Triaged] https://launchpad.net/bugs/74335910:28
cjwatsonjibel: there are reports from beta 2 but that's an earlier version10:30
jibelcjwatson, yes I reproduced it this morning. I'll upload the logs10:45
* cjwatson reproduces bug 856418. How curious11:19
ubot2Launchpad bug 856418 in ubiquity "KDE OEM Mode Hang On Shutdown" [High,Incomplete] https://launchpad.net/bugs/85641811:19
* ev is on bug 769350 - have a solution but looks like my code is interacting poorly with partman, investigating11:24
ubot2Launchpad bug 769350 in partman-auto "resized partition size doesn't count swap space size." [High,Triaged] https://launchpad.net/bugs/76935011:24
evah no, it's just monday morning not using logger correctlyt11:31
gema_I had a wiki that had turned into an immutable page... any idea what I may have done wrong?11:39
cjwatsonyou probably got logged out11:41
gema_and there goes my Monday morning moment! thanks, cjwatson11:44
stgraberev: 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
evseeing where we stand with bug 800760 would be good13:18
ubot2Launchpad bug 800760 in ubiquity "Needs gconf to gsettings updates" [Medium,Confirmed] https://launchpad.net/bugs/80076013:18
stgraberok, I'll start with that then.13:19
cjwatsonbug 830946 just appeared on our milestone list14:02
ubot2Launchpad bug 830946 in ubiquity "Nothing displayed on embedded terminal." [Medium,Confirmed] https://launchpad.net/bugs/83094614:02
evreally? That's a bit silly. It's a UI nit at best.14:24
evbut okay14:24
cjwatsonjibel: would you like to explain your milestoning?14:27
CIA-45ubiquity: cjwatson * r5016 trunk/ (bin/oem-config-prepare debian/changelog):14:40
CIA-45ubiquity: Run only those parts of oem-config-prepare that require root access as14:40
CIA-45ubiquity: root; in particular displaying confirmation dialogs is now done as the14:40
CIA-45ubiquity: calling user, which avoids KDE confusion (LP: #856418).14:40
cjwatsonev: I'm happy to dig out APUE and have another crack at bug 743359 if you like14:56
ubot2Launchpad bug 743359 in ubiquity "Installer: LockFailedException: Failed to lock /target/var/cache/apt/archives/lock" [High,Triaged] https://launchpad.net/bugs/74335914:56
evplease do14:58
evI'm at my wits end with that one14:58
cjwatsonI 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 pgrp14:58
cjwatsonand maybe repeated-retry-for-a-bit-until-dead would be a good idea14:59
jibelcjwatson, 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
ubot2Launchpad bug 830946 in ubiquity "Nothing displayed on embedded terminal." [Medium,Confirmed] https://launchpad.net/bugs/83094615:02
jibelIt is medium importance because it doesn't limit the functionality of a core application.15:02
cjwatsonAs long as you don't mind it being ignored15:04
cjwatson(we still have a large High list)15:04
cjwatsonev: do you have a copy of APUE, by the way?  bit of a brick of a book, but very useful15:16
evI do!15:16
evI haven't leafed through it in ages though15:16
evshould really get back on that15:16
evI really wish there was an ebook version though15:16
evalways a pain to have that extra weight in the suitcase for UDS and the sprints15:17
cjwatsonyeah, I don't carry it15:22
cjwatsonI think the way I use it (lots of flicking about) would not be very e-book friendly, although searching would be useful15:23
cjwatsonthough OTOH it has excellent contents/indexing15:23
cjwatsonev: I guess the probability of encountering this bug goes up the older the image you're using has got15:41
cjwatsonsince 'apt-get upgrade' will take longer15:41
evor if, say, you're on fibre to the canonical datacenter15:41
cjwatsonsadly I only have current images :-)  I'll see if I can fake something up15:41
cjwatsonfor example "drop all outgoing packets" might slow apt-get down enough that copy_all beats it15:42
evheh, nice15:42
CIA-45ubiquity: stgraber * r5017 ubiquity/bin/ubiquity-dm: temp15:43
stgraberdoh, I need to make CIA clever when I do temporary commits ;)15:43
evhah! I'm an idiot15:55
evwas wondering why int(value * 1024 * 1024) wasn't working15:55
evbeacause that's a rather large string it's turning into an integer15:56
evI'm blaming that one on it being Monday15:56
cjwatsonI blame many things on that15:56
cjwatsonAlthough 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
cjwatsonboo, that doesn't work because apt-get update isn't pointed at /target/var/cache/apt15:58
cjwatsonmaybe if I stick the iptables between update and upgrade ...16:01
bdmurrayHave you all seen bug 825238?16:24
ubot2Launchpad bug 825238 in casper "screen reader does not start in a11y installation" [High,Confirmed] https://launchpad.net/bugs/82523816:24
bdmurraytranslation bugs, bug 838141, don't require a ubiquity task correct?16:31
ubot2Launchpad bug 838141 in ubiquity "English strings in Swedish translation" [Undecided,New] https://launchpad.net/bugs/83814116:31
evI could definitely use some extra pairs of eyes on the following merges:16:31
cjwatsonbdmurray: 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
bdmurraycjwatson: okay, thanks16:32
cjwatsonok, that seems to reliably reproduce this; nice wide race window16:37
bdmurraycjwatson: where does this dialog come from? https://launchpadlibrarian.net/78625495/translation1.png16:37
cjwatsonbdmurray: console-setup16:41
cjwatsonbdmurray: that part should be fixed as of 26 Sep though16:42
cjwatsonyep, installation media from 2011083116:43
cjwatsonso you should just be able to mark that part as done16:43
evhm, I might have misunderstood which partition min_size and max_size refer to.  Rubbish.16:43
bdmurraycjwatson: okay, thanks16:43
cjwatsonmin_size and max_size refer to the partition being resized16:44
cjwatsonhttps://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-partitioning16:45
evwill fix16:46
evand indeed16:46
cjwatsonum, wow, um, complicated16:46
cjwatsonit's not feasible to have something we call out to from ubiquity directly rather than doing it in the resizing library?16:47
cjwatsonI can't actually find a bug in it as such, just worried about testing16:49
cjwatson'decode_recipe $(get_recipedir)/[0-9][0-9]atomic linux-swap' is a scary lilne16:49
cjwatsonbut I guess as long as we make sure it's tested both in d-i and ubiquity ...16:49
cjwatsonperhaps it would be worth having an NIHed python recipe decoder in P16:50
cjwatsonalthough there's some non-trivial logic there; perhaps a wrapper that hides the evil of shelling out16:50
evI just wanted to be sure we weren't hardcoding an assumed swap size16:51
cjwatsonoh, agreed16:51
evbut yes, I suppose something in ubiquity itself would work16:51
cjwatsonI 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 library16:52
cjwatsonbut this is probably a smaller change for 11.1016:52
CIA-45ubiquity: stgraber * r5017 ubiquity/ (bin/ubiquity-dm debian/changelog): Update ubiquity-dm to set gsettings keys instead of gconf17:12
stgraberev: ^ 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 itself17:13
cjwatsondoes that need a bug closure in the changelog?17:14
stgraberI 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
stgraberbug 80076017:15
ubot2Launchpad bug 800760 in ubiquity "Needs gconf to gsettings updates" [Medium,Confirmed] https://launchpad.net/bugs/80076017:15
CIA-45ubiquity: cjwatson * r5018 trunk/ (debian/changelog scripts/install.py scripts/update-apt-cache):17:16
CIA-45ubiquity: Go back to killing update-apt-cache's process group, but this time make17:16
CIA-45ubiquity: sure that it's in a separate process group and make more of an effort to17:16
CIA-45ubiquity: ensure that it terminates (LP: #743359).17:16
cjwatsonev: ^- a double-check of that wouldn't hurt, though it passes my torture test now and I watched it killing the process group17:17
evcjwatson: will do.  About to head out to the cinema with some millbank folk, but I'll have a look first thing in the morning17:18
evas I also fix up these merge proposals17:18
cjwatsonI think I may be mostly out of steam for today, but should I do an upload?17:18
stgraberev: 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
cjwatsonmax testing time and all that17:18
stgraberev: should be fairly trivial to change (copytree of /etc/NetworkManager/system-connections should do the trick)17:19
stgrabercjwatson: 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
evcjwatson: by all means17:20
evstgraber: we definitely still need the functionality17:20
cjwatsonstgraber: I wouldn't worry about exactness of that17:20
evas the installer now has that wireless page17:20
evbut whatever is cheapest, really17:20
cjwatsonsometime tonight your time should be OK17:21
cjwatsonI'll do a translation refresh now17:21
stgraberev: ok, I'll replace that code by a simple copytree of all of NM's config then17:21
stgrabercjwatson: ok, will upload before leaving the office then17:21
cjwatsoncool, thanks17:21
CIA-45ubiquity: stgraber * r5019 ubiquity/ (debian/changelog scripts/plugininstall.py): Update copy_network_config to work with Network Manager 0.9 storing the configuration system wide17:31
CIA-45ubiquity: cjwatson * r5019 trunk/debian/ (changelog real-po/hr.po real-po/sk.po): Update translations from Launchpad.17:32
cjwatsonum, sorry, I beat you to that, you'll need to merge17:32
cjwatson(bound branches FTW)17:32
CIA-45ubiquity: stgraber * r5020 ubiquity/ (debian/changelog scripts/plugininstall.py): Update copy_network_config to work with Network Manager 0.9 storing the configuration system wide17:33
cjwatsonor uncommit/recommit, fair enough :)17:33
stgrabermerging 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 file17:34
* cjwatson is out of steam. See you later17:46
CIA-45ubiquity: stgraber * r5021 ubiquity/ubiquity/gsettings.py: Add initial implementation of gsettings module18:48
CIA-45ubiquity: stgraber * r5022 ubiquity/bin/ubiquity-dm: Port ubiquity-dm to the new gsettings module18:52
stgraberok, now that I have that gsettings module, porting the rest of the code should be pretty easy18:53
CIA-45ubiquity: stgraber * r5023 ubiquity/ubiquity/components/apt_setup.py: Port apt_setup proxy configuration to gsettings19:01
stgraberouch, gtk_ui.py is going to be fun to switch to gsettings...19:03
CIA-45ubiquity: stgraber * r5024 ubiquity/ubiquity/gsettings.py: If SUDO_USER is set, use that for gsettings by default19:08
CIA-45ubiquity: stgraber * r5025 ubiquity/ubiquity/frontend/gtk_ui.py: Switch gtk_ui from gconf to gsettings19:32
infinityUgh.  This anacron thing is killing me. :/19:42
CIA-45ubiquity: 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
infinityAnd 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
infinityI'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
infinityOr maybe changing the default cron.daily delay from 5 minutes to an hour or something...19:44
infinityOr make the apt update cronjob exit if [ -x /usr/bin/ubiquity ] ... How evil would that be? :P19:45
CIA-45ubiquity: stgraber * r5027 ubiquity/debian/changelog: Update changelog19:46
stgraberok, there we go for gsettings. Now the fun part, testing all of that...19:47
stgrabergoing to run 2-3 test installs trying to stress test the changes, if that works I'll upload to the queue19:48
infinityAny brilliant thoughts on my cron.daily-during-install headaches? :)19:48
stgraberchecking for oem-config/ubiquity would definitely be the easiest way to fix your use case with minimal impact to other ways of installing Ubuntu19:51
stgraberthough I think it'd be good for everyone not to have these start on first boot19:51
infinity       -u, --update19:52
infinity              incremental  update, reindexing only those packages whose version has changed since19:52
infinity              the last run19:52
infinity^-- Note that the call in cron.daily doesn't use -u19:52
infinityI wonder if this would make things significantly happier..19:52
stgraberoh, 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 cron19:53
stgraberbut 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
infinityTesting to see if it actually matters in practice.19:54
infinityIt might just be I/O bound on writing the massive DB regardless.19:54
infinityHrm, no, -u does seem to be significantly faster.19:55
infinityAnd you'd think it was introduced for just this use case. :P19:55
infinityIf only mvo was around.19:55
superm1infinity, you have casper in your ubiquity setup no?  how about nuking the job from casper?19:57
infinityNo casper, no.19:58
infinityWe 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
infinityCould add some code to run on second boot, though, we don't remove it until later.20:00
infinitySeems unpleasant, however.20:00
infinityAnd I don't want this to be ARM-specific.20:00
infinityThat said, '-u' seems to make a HUGE difference here.20:00
infinityNext 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
superm1i feel like there really shouldn't be any cron jobs allowed to really run during install or during oem-config20:03
infinityYeah, I'm inclined to agree.20:03
infinityAnd stopping anacron/cron at the beginning of the install and starting them at the end does sound sane.20:04
superm1so maybe it's better to tell ubiquity to stop it / restart it20:04
infinitySo, 42s for -u, 346s without.20:04
infinityI have a feeling this should be done anyway. :P20:04
CIA-45ubiquity: stgraber * r5028 ubiquity/ubiquity/ (frontend/gtk_ui.py gsettings.py): Fix obvious mistakes found by pyflakes20:06
infinitySo, some sort of disable_cron() called from main, right after acquire_lock(), and an atext.register(enable_cron)20:10
infinityThis might not be that ugly.20:10
infinityI'm assuming there are no python upstart bits, I probably just need to fork shells?20:11
infinityapt-cache search upstart python seems to agree.20:11
superm1well actually it looks like anacron is disabled for a standard ubiquity install already to me20:12
superm1there is some code in casper's 25configure_init to do it20:12
stgraberyou can control upstart over dbus, but subprocess.Popen is going to be a lot easier :)20:12
infinitysuperm1: Ahh, but that's in casper.  It should be in the installer itself, methinks.20:12
stgraberwell, 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 reason20:13
superm1but you also don't want cron being mean in oem-config, so maybe needs to be both then20:13
infinityIt completey diverts anacron?20:13
superm1another way to solve it then is to divert during oem-config-prepare20:14
infinityWhat un-diverts them?20:14
stgraberprobably nothing20:14
superm1nothing needs to, it's only effective in the live environment20:14
infinityOh, bleh.20:15
infinityNot the case for me.20:15
infinityMy live == My installed system.20:15
stgraberyeah, PPA build of ubiquity succeeded!20:44
stgrabernow to wait 2 hours to get the i386 one and be able to do some tests :)20:44
stgraberoh actually, just having ubiquity and ubiquity-frontend-gtk may be enough for my tests20:45
CIA-45ubiquity: stgraber * r5029 ubiquity/ubiquity/gsettings.py: Convert booleans before giving them to subprocess21:11
stgraberinteresting, apparently we can't hide "suspend" from the session indicator...21:14
stgraberso 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
stgraberinfinity: planning on pushing some changes to ubiquity in the next hour or can I upload it?21:31
infinitystgraber: No, go to town.21:32
CIA-45ubiquity: stgraber * r5030 ubiquity/debian/changelog: releasing version 2.8.221:32
infinitystgraber: I need to talk to mvo tonight when he gets in to work.21:32
CIA-45ubiquity: stgraber * r5031 ubiquity/scripts/plugininstall.py: Skip LTSP live when copying network to the target21:38
Guest45434cjwatson: 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
infinityGuest61019: Commit fix, upload, ask forgiveness?23:14
infinityGuest61019: It'll end up being me or Colin that reviews it anyway. :P23:14
Guest61019infinity: k23:20
cjwatsonNCommand1r: go ahead23:23
cjwatsonNCommand1r: actually23:23
cjwatsonNCommand1r: could you just commit but not upload?  if you're going to do that, I'd like to refresh translations in the same upload23:23
cjwatsonNCommand1r: and I have the scripts for that23:24
NCommand1rcjwatson: sure, I'll commit in about an hour or so (network here is going about the same speed as a tortise)23:30
cjwatsonNCommand1r: I'll commit the translations before that, then, and you can upload23:31
cjwatsonI'll be in bed in an hour23:31
cjwatsonjust awaiting the tarball now23:31
=== NCommand1r is now known as NCommander
=== Guest16725 is now known as StevenK
CIA-45debian-installer: cjwatson * r1546 ubuntu/ (3 files in 2 dirs): Update help text translations from Launchpad.23:53
cjwatsonNCommander: ^- there, just make sure to include that23:54

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!