=== jalcine is now known as webjadmin_ === webjadmin_ is now known as jalcine === jalcine is now known as webjadmin_ === webjadmin_ is now known as jalcine === hggdh_ is now known as hggdh [01:00] Hi [01:00] How can i be one of Membership ? [01:00] I've from thailand [01:01] and i'm not ensure to convert the timezome [01:01] of UTC into GMT [01:01] Oh sorry I've see the structions [01:02] Apologize for my less participate === jalcine is now known as JackyAlcine === JackyAlcine is now known as jalcine === freeflyi1g is now known as freeflying === smb` is now known as smb === greyback is now known as greyback|lunch === bladernr_afk is now known as bladernr_ === greyback|lunch is now known as greyback [15:01] * slangasek waves [15:01] o/ [15:01] * stgraber waves [15:02] * ogra_ shores [15:02] ~~~___~~~ [15:03] There seem to be some snakes on your plane. [15:03] there's a complex analysis joke in there somewhere [15:03] hi [15:04] #startmeeting [15:04] Meeting started Wed Mar 14 15:04:14 2012 UTC. The chair is slangasek. Information about MeetBot at http://wiki.ubuntu.com/meetingology. [15:04] Available commands: #accept #accepted #action #agree #agreed #chair #commands #endmeeting #endvote #halp #help #idea #info #link #lurk #meetingname #meetingtopic #nick #progress #rejected #replay #restrictlogs #save #startmeeting #subtopic #topic #unchair #undo #unlurk #vote #voters #votesrequired [15:04] [TOPIC] lightning round === meetingology changed the topic of #ubuntu-meeting to: lightning round [15:04] $ echo $(shuf -e barry doko stgraber jodh ev bdmurray slangasek ogra infinity cjwatson) [15:04] cjwatson ev stgraber barry ogra jodh doko slangasek infinity bdmurray [15:06] Release meeting: [15:06] - Do a better job of writing hybrid MBRs on Intel Macs (bug 856826). [15:06] - Reuse existing swap partitions and EFI System Partitions rather than creating new ones (bug 311299). [15:06] Launchpad bug 856826 in parted (Ubuntu Precise) "'guided resize' partioning leaves Mac unbootable" [High,Fix released] https://launchpad.net/bugs/856826 [15:06] Launchpad bug 311299 in partman-auto (Ubuntu) "automatic partitioning does not reuse existing swap / boot partitions" [Wishlist,Fix released] https://launchpad.net/bugs/311299 [15:06] - Arrange for Ubuntu Studio images to reconfigure jackd on boot (bug 923810). [15:06] Launchpad bug 923810 in casper (Ubuntu) "preseeding passwd/user-default-groups is ineffective" [Medium,Fix released] https://launchpad.net/bugs/923810 [15:06] - Work on several EFI partitioning bugs (bug 769669, bug 811485). [15:06] Launchpad bug 769669 in partman-efi (Ubuntu) "Installer should not format an existing EFI System Partition" [High,Fix released] https://launchpad.net/bugs/769669 [15:06] Other: [15:06] Launchpad bug 811485 in partman-efi (Ubuntu) "EFI SYSTEM PARTITION should be atleast 100 MiB size and formatted as FAT32, not FAT16" [High,Fix released] https://launchpad.net/bugs/811485 [15:06] - Planning for installer sprint. [15:06] - Some work on setting interview problems. [15:06] - Various installer translation updates. [15:06] - Multiarch fixes for pciutils (bug 948205) and libidl (bug 931388). [15:06] Launchpad bug 948205 in pciutils (Debian) " './usr/include/pci/config.h' is different from the same file on the system" [Unknown,New] https://launchpad.net/bugs/948205 [15:06] - Improve debootstrap's handling of corrupted downloads (bug 954197). [15:06] Launchpad bug 931388 in wine1.4 (Ubuntu Precise) "overenthusiastic adoption of "Depends: cpp:any" syntax" [High,Fix released] https://launchpad.net/bugs/931388 [15:06] Launchpad bug 954197 in base-installer (Ubuntu) "base system installation is not robust against transient network failures" [Medium,Triaged] https://launchpad.net/bugs/954197 [15:06] - Upgrade fixes for desktopcouch (bug 864328, bug 900570). [15:06] Launchpad bug 864328 in desktopcouch (Ubuntu) "package python-desktopcouch-records 1.0.7-0ubuntu2 failed to install/upgrade: trying to overwrite '/usr/share/pyshared/desktopcouch/__init__.py', which is also in package python-desktopcouch 0.6.4-0ubuntu3.2" [Undecided,Fix released] https://launchpad.net/bugs/864328 [15:06] Launchpad bug 900570 in desktopcouch (Ubuntu) "undeclared package conflict between python-desktopcouch-application (in precise) and python-desktopcouch (in lucid)" [Undecided,Fix released] https://launchpad.net/bugs/900570 [15:06] - Currently trying to get my head around some more Mac partitioning bugs: bug 855871 and bug 856763. Brain hurts, send cake. [15:06] Launchpad bug 855871 in ubiquity (Ubuntu Oneiric) "Grub install fails after manual xfs partitioning" [Medium,Confirmed] https://launchpad.net/bugs/855871 [15:06] .. [15:06] Launchpad bug 856763 in parted (Ubuntu Precise) "Mac cannot boot after manual partitioning, unless using ext4" [Medium,Confirmed] https://launchpad.net/bugs/856763 [15:06] - Ported the daisy code from Pika to amqplib, as that's what all other [15:06] Canonical services are using. [15:06] - Build out a configuration for retracing machines. [15:06] - Bug fixes in the WSGI applications (crash and core file acceptance). [15:06] - Packaging oops-repository, python-pycassa, and python-thrift for IS (RT 48667) [15:07] - Refactored whoopsie and broke out some separate utility code, adding tests [15:07] for all of it along the way. This was in support of fixing how we determine [15:07] whether or not to submit a crash. Previously, we were only sending them on [15:07] the first instance. We now send it every time that crash occurs. [15:07] - Did research and performance measurements around counting the contents of [15:07] crash buckets in Cassandra. [15:07] - Finished initial bucketing code. There may be room for improvement around [15:07] how we're counting and recording bucket sizes, but some real idea from the [15:07] deployment will give me a much better indication. [15:07] - Built a test harness around the WSGI applications (crash and core file [15:07] acceptance). Already quite a few serious fixes from this. Yay. [15:07] - Started moving Cassandra-facing functionality into oops-repository, per my [15:07] discussion with Robert last week where he made the suggestion to use it [15:07] alone as the layer over the database. [15:07] - Meeting with Amanda to discuss keeping her in the loop with respect to [15:07] changes to ubuntu-restricted-* [15:07] (done) [15:08] Bugs, bugs, bugs... [15:08] - Networking [15:08] - RELEASE: Uploaded new isc-dhcp, converting to upstart and creating separate IPv4 and IPv6 jobs [15:08] - Investigated bug 919068 (bridge_ports all not working at boot time) [15:08] Launchpad bug 919068 in bridge-utils (Ubuntu) "bridge_ports all doesn't work at boot time" [Medium,Triaged] https://launchpad.net/bugs/919068 [15:08] - Installer [15:08] - RELEASE (generic): Bugfix work on ubiquity, casper and wubi [15:08] - Went through wubi merge proposals, merged a lot of bugfixes from bcbc [15:08] - Started going through the wubi bugs too [15:08] - Spent some time implementing bluetooth support in the installer [15:08] - Fixed the keyboard indicator [15:08] - Fixed a few bugs in casper related to persistent usb stick [15:08] - Various other ubiquity fixes and a few improvements to tests [15:08] - Containers [15:08] - Worked/debugged apparmor and LXC with the security team, hoping to have an updated lxc today [15:08] - TPM [15:08] - Started digging into opencryptoki, fixed bug 926305 [15:08] Launchpad bug 926305 in opencryptoki (Ubuntu) "fails to load modules for pkcs11 backends" [Undecided,Fix released] https://launchpad.net/bugs/926305 [15:08] - Other [15:08] - RELEASE: Merged James' friendly-recovery branch adding support for LVM and APT status to the system-summary screen [15:08] - Had to deal with a security issue found in LTSP, fixing upstream and in Ubuntu + CVE and other paperwork [15:08] - Patch pilot on Monday [15:08] - TODO this week [15:08] - Work on some ubiquity debugging wiki pages [15:08] - Continue working on bugs [15:08] (DONE) [15:09] heh, barry doesn't seem to be awake [15:10] ogra: ^^^ [15:10] done: [15:11] * not gotten to oem-config issues, will do so next week [15:11] * trying to stopwatchg the initrd made me struggle on bug 936667 (--no-log helps) [15:11] * did some more weeding through ubuntu-arm buglist (starts to look more sane) [15:11] * missed my piloting on monday, will try to do before teh weekend [15:11] * worked on Bug 948163 (fix in local branch, pending a push) [15:11] Launchpad bug 936667 in upstart (Ubuntu) "Upstart early job logging causes boot failure for systems with no initramfs (error is "No available ptys")" [High,Confirmed] https://launchpad.net/bugs/936667 [15:11] * lots of community support issues (helped RTC issues with arm devices) [15:11] Launchpad bug 948163 in casper (Ubuntu) "System suspends upon closing the lid while installing/partitioning using a live CD" [Wishlist,New] https://launchpad.net/bugs/948163 [15:11] * looked into a few ac100 installer bugs [15:11] * booked my flight to UDS [15:11] tod: [15:11] * oem-config (this week for sure) [15:11] * finish initrd timing [15:11] * lookj into fsck delay on boot issue [15:11] done [15:11] Off sick Monday. Been hit by bug 751689 this week which has hampered [15:11] efforts somewhat until I realized what was happening. Investigated [15:11] bug 940396. Initially thought it might be multi-arch related, but [15:11] seemingly its a subtle dependency bug: mvo has now recreated and [15:11] Launchpad bug 751689 in linux (Ubuntu Maverick) "ThinkPads overheat due to slow fans when on 'auto'" [Critical,In progress] https://launchpad.net/bugs/751689 [15:11] Launchpad bug 940396 in apt (Ubuntu Precise) "lucid -> precise main all failed to upgrade: dpkg: dependency problems prevent configuration of kde-runtime" [Critical,Confirmed] https://launchpad.net/bugs/940396 [15:11] identified fixes. Worked on bug 508083 (currently testing). Writing some [15:11] Launchpad bug 508083 in eglibc (Ubuntu) "cron crashed with SIGSEGV in __pthread_initialize_minimal_internal()" [Medium,Triaged] https://launchpad.net/bugs/508083 [15:11] new tests for Upstart as a precursor to syncing branches and updating [15:11] Ubuntu package. [15:11] ⛀ [15:11] [15:11] - PyCon week, report hopefully later ... [15:11] - GCC-4.7 release candidates [15:11] - GCC-4.4.7 release [15:11] - eglibc ARM getcontext/setcontext patch [15:11] - libgo fix for ARM [15:12] - gdb Linaro update [15:12] - went through the list of MIR's (three pending) [15:12] - update python2.7 and 3.2 to the release candidates [15:12] - package python3.3.0 alpha1 (doko/toolchain PPA) [15:12] - PyCon sprint work: namespace peps, python core cross compilation [15:12] (done) [15:12] ogra_: hm, jodh was working on 936667 as well - missing coordination? [15:13] ogra_: oh, sorry, never mind that, I was confused by the interleaving of your comments and ubottu's - ignore me [15:13] cjwatson, i wasnt working on it, i was affected by it with my other stuff :) [15:13] but the --no-log option seems to help so i can move forward now [15:14] (only saw the original bug # today adn got some help from that) [15:14] slangasek: *nudge* [15:14] still reading, gimme a second ;) [15:15] ogra_: just ping me if you see any other init oddities ;) [15:15] jodh, i only see serial consoles that dont start by default *g* [15:16] jodh: I have this weird issue where, a couple of years ago, my sysvinit was replaced. Does that count? [15:16] infinity: :) [15:17] * RELEASE: transition soprano off of iodbc (bug #901638) [15:17] * acpi-support upload to support a couple more events not handled elsewhere in the system (bug #931614, #953296) [15:17] Launchpad bug 901638 in soprano (Debian) "Remove iodbc2 (causes upgrade failure from Oneiric to Precise)" [Unknown,New] https://launchpad.net/bugs/901638 [15:17] * add back a few dependencies to ia32-libs for compatibility (bug #946381, #953404) [15:17] Launchpad bug 931614 in acpi-support (Ubuntu) "ASUS backlit keys buttons don't work" [Medium,Fix released] https://launchpad.net/bugs/931614 [15:17] Launchpad bug 953296 in acpi-support (Ubuntu) "Elantech touchpad cannot be toggled" [Medium,Fix released] https://launchpad.net/bugs/953296 [15:17] * fix a multiarch issue with libvisual-plugins that was causing weird behavior on upgrade (bug #947639) [15:17] Launchpad bug 946381 in ia32-libs (Ubuntu) "Should install libxtst6:i386" [Medium,Fix released] https://launchpad.net/bugs/946381 [15:17] * fix a regression in python2.7, bdb support dropped in the latest merged (bug #440889) [15:17] * SRU nis to lucid, fixing longstanding issues resulting from it not being converted to upstart [15:17] * planning for installer sprint [15:17] Launchpad bug 953404 in ia32-libs (Ubuntu) "ia32-libs in precise is missing libgconf-2-4:i386" [Undecided,Fix released] https://launchpad.net/bugs/953404 [15:17] Launchpad bug 947639 in apt (Ubuntu) "package libgudev-1.0-0 1:175-0ubuntu5 failed to install/upgrade: libgudev-1.0-0:amd64 1" [High,New] https://launchpad.net/bugs/947639 [15:17] * partner archive work [15:17] Launchpad bug 440889 in python2.7 (Ubuntu Precise) "software-center crashed with ImportError in /usr/lib/python2.7/bsddb/__init__.py: No module named _bsddb" [Critical,Fix released] https://launchpad.net/bugs/440889 [15:18] (done) [15:18] * doko thanks for the bdb fix ... [15:18] ogra_: well, technically I provided review feedback on that feature and it hasn't yet been acted on. I'll try to look at it but it's not on my priority list right now. Maybe SpamapS has some spare cycles? [15:19] doko: sure [15:19] Done this week: [15:19] - Fixed armhf FTBFS of qtwebkit-source [15:19] - Worked on packaging new vmware-view-client [15:19] - Worked on cdimage bugs [15:19] - Patch piloted on Friday, including sponsoring a bunch of multi-arch build-dep chain fixes for cross-bootstrapping [15:19] - Started looking into grub upgrade bug (bug 759545) [15:19] Launchpad bug 759545 in grub2 (Ubuntu Precise) "user prompted to update unmodified grub configuration during Ubuntu server upgrade" [Medium,Triaged] https://launchpad.net/bugs/759545 [15:19] - Started working on bug 876626 [15:19] Launchpad bug 876626 in upstart (Ubuntu Precise) "Unlocking the second crypto disk (/home) echos password on console" [High,Confirmed] https://launchpad.net/bugs/876626 [15:19] - Reviewed armadaxp kernel mess for license sanity (3.2 should go in today) [15:19] - Review flash-kernel changes for calxeda support (sponsoring today) [15:19] jodh, well, i thought the .conf file we had was fine, isnt it just a matter of including it in the package ? [15:19] - Generic archive admin and other random bits [15:19] - Discussions with IS and webops about shuffling ARM buildds, upgrading some (or all) distro buildds to precise, and other fun projects [15:19] - Discovered I need to keep better track of what I've done, so these cut and paste reports are a bit more useful [15:19] - PS: &*^!$@ daylight savings [15:19] (done) [15:19] infinity, move to russia ! [15:20] infinity, does the discussion include non ARM buildds as well? [15:20] doko: The upgrading discussion? Yes. [15:20] ogra_: https://code.launchpad.net/~clint-fewbar/ubuntu/natty/upstart/add-serial-console/+merge/46191/comments/104693 [15:20] infinity: do you know if anyone has a master list of outstanding issues preventing cross-bootstrapping? [15:20] slangasek: wookey does, but I don't have the URL off the top of my drowsy head. [15:20] slangasek: Poke him, or poke me after caffeine. ;) [15:20] ok [15:21] bug triage [15:21] research into bug 945826 regarding sources.list on a live cd [15:21] updated dash bug reporting guidelines to mention unity [15:21] updated screenshots on h.u.c/community/BugReportingGuidelines [15:21] Launchpad bug 945826 in apt (Ubuntu) "12.04 AMD64 Beta Live CD sources.list has a bad entry" [Medium,Triaged] https://launchpad.net/bugs/945826 [15:21] added in 'ubuntu-bug -w' at h.u.c/community/BugReportingGuidelines [15:21] recorded a how to confirm a bug screencast and uploaded to ubuntucontributers youtube channel [15:21] investigation into to no package bug reports (getting autoconfirmed) [15:21] uploaded new version of firefox-lp-improvements after fixing karma_suffix [15:21] jodh, oh, i didnt see that [15:21] created hottest bugs charts for package sets [15:21] modified hottest and recent package bug charts to have a date updated in them [15:21] made links bold if there is a spike in this week's bug reporting volume compared to last week in recent package bug tasks [15:21] created a team assigned bug report using cbd / arsenal [15:21] bug control application review for Vadim Rutkovsky [15:21] * done * [15:21] cjwatson: does 945826 ring any bells with you? [15:22] I'm not sure apt-cdrom knows about gz files [15:22] it's not the first time this has broken, I think, but it *definitely* used to work [15:23] e.g. it's faintly reminiscent of bug 924182 [15:23] Launchpad bug 924182 in apt (Ubuntu Precise) "d-i images (server, alternate) failed to install: no kernels found" [Critical,Fix released] https://launchpad.net/bugs/924182 [15:24] let me just see if I can reproduce 945826 ... [15:24] * slangasek nods [15:25] doko: https://rt.admin.canonical.com/Ticket/Display.html?id=50414 [15:25] depending on what you mean by upgrading I guess :) [15:25] https://rt.admin.canonical.com/Ticket/Display.html?id=51115 is for the software upgrade [15:25] [TOPIC] installer vsprint === meetingology changed the topic of #ubuntu-meeting to: installer vsprint [15:25] installer virtual sprint next week [15:26] (hmm, should that be 'virtual sprint, foundations' --> vsprintf?) [15:26] * cjwatson awards slangasek one-third of a point [15:26] * infinity groans. [15:26] have people looked at the first-line bug lists I suggested? do we think it should be broadened a bit? [15:26] please be sure to read through the wiki page that's been sent around with information on prep work to do ahead of time [15:26] cjwatson: :) [15:27] I just wanted to get the most urgent ones down [15:27] in particular, you should have both an image and a machine (VM or hardware) to install it on, ready to go Monday morning, not spend the morning downloading ISOs [15:28] cjwatson: the bug lists looked to me like the right place to start [15:28] I've spoken to Nicholas Skaggs about some testing arrangements [15:29] actually I wanted to talk through that with Steve yesterday but we missed our call due to the bandwidth pixies [15:29] does kvm work as VM? [15:29] yes [15:29] we've agreed that at least we're going to try to organise something along the lines of milestone ISO testing, only based on the daily (or more) builds we emit during the sprint [15:29] and that we'll prepare a daily changelist for him so that his minions can do more focused testing work [15:30] ah, cool [15:30] further ideas welcome [15:31] yes, kvm is fine for most purposes, it's what I use. (I haven't managed to get it to pretend to be EFI well enough to boot Linux yet though, despite some attempts today) [15:32] I'm running most tests in kvm too and some others (webcam, wireless, ...) in a LXC container with access to the required devices (and some hacks in ubiquity to run just the steps I need) [15:33] so I would like to suggest that we require any changes made as part of the installer mini sprint go through code review in LP [15:33] ideally, with two pairs of eyes on it [15:33] normally I'd suggest avoiding a monoculture, but I don't think it's especially harmful in this case - more important for people to be able to do development efficiently [15:33] I'm also using kvm, but doesn't that preclude Unity 3D? [15:34] I think one review should be sufficient, but I agree that we should use the review system [15:34] jodh: I don't expect any of our installer bugs are going to be unity related [15:34] jodh: yes; for some bugs it matters, for most it doesn't [15:34] or at least s/any/many/ [15:35] slangasek: I wasn't necessarily thinking of the installer per se, more the resultant experience. But that was another question - when does an installer test terminate? On first reboot? [15:35] we aren't doing testing, we're doing developmenet [15:35] -e [15:35] or rather on first login after first reboot? [15:35] on first successfull boot after installation ;) [15:35] jodh: this isn't for installer testing, but for installer development :) [15:36] ideally, I [15:36] and we're not working on the whole experience, but on the installer :) [15:36] ideally, I'd like to see us move towards doing code review across all our projects, and I think this sprint would be a good trial run of that [15:36] but we'll test the results of our work [15:36] ev: agreed, I think this is a good idea [15:36] yes; but really, it's not likely that you'll need to get far enough for unity 3d to work [15:37] indeed [15:38] flip side of course is that ev, stgraber, and I will probably be spending a lot of time doing reviews; but that's probably all to the good [15:38] well, I would hope that we all do reviews [15:38] ev: As long as you mean "move toward a culture of code review" rather than "move to awkward processes that force busy work in the name of mandatory code review", I'm with you. [15:38] different people will approach from different angles [15:38] but yes, you're probably right :) [15:38] sure, but realistically ... [15:38] absolutely [15:38] jodh: many of these bugs are going to be quite self-contained; we'll want end-to-end testing after the fixes have landed, but that's why we'll have community testers coordinated to stand by [15:39] but that's what I'm getting at with suggesting that two people review. This way we're all looking at each other's code, not just those of us most familiar with it. [15:39] but you'd be hard pressed to regress the post-install desktop by fixing a bug with manual partitioning [15:39] ev: well, let's apply common sense, there's a wide range of complexity here [15:39] sure [15:40] * infinity is glad he knows nothing about d-i and ubiquity. [15:40] I'm happy for reviews of complex changes to say "oh and somebody else should look at this", although I also think it's important to keep the sprint pace moving along [15:40] * infinity wistles nonchalantly and hides in the corner. [15:40] we'll teach you ! [15:40] ev: so you're suggesting two reviewers, in addition to the author? [15:41] +1 on common sense ;) FWIW I've always been reviewing any commit in the ubiquity branch before uploading and I'm guessing others might have done so too, we really just need to make sure it always happens and that when uploader == author, someone else looks at it before upload at least [15:41] I was thinking 1 reviewer, + author, should be enough [15:41] and shouldn't slow us down [15:42] I'm not against using separate branches and doing merges and MP review on LP but we'll have to adapt our workflow a bit and make sure the commit and see how we can get CIA work with that :) [15:42] I do think that we should view this as an opportunity for people who haven't done a lot of installer work to learn by reviewing, too [15:42] should still land trivial changes directly, as long as we have a reasonably shared understanding of that [15:42] stgraber: To be honest, I think MPs kinda slow the whole thing down, but maybe I'm alone in that. [15:42] (cf. e.g. Launchpad dev process) [15:42] infinity, ++ [15:42] * ogra_ prefers IRC and pastebin [15:42] infinity: you're not alone :) I think MPs are fine for new features, not for bugfixes [15:43] slangasek: yes. I think it solves the problem of only people who are intimately familiar with a codebase being the reviewers. I'm trying to increase our understanding of each other's silos with this. [15:43] I don't think it would slow us down, as these are bug fixes. They should be fairly isolated. Equally, I think once it becomes a reflex for us to engage in code review, the pace will pick up drastically. [15:43] infinity: I much prefer IRC + pastebin for trivial fixes review, just before commiting the change [15:43] stgraber: Unless it's a complex merge/branch that should obviously not be landed without review, for fear of exploding everything everyone else is working on. [15:43] stgraber: Yeah, agreed. Lightweight, FTW. [15:43] But I do not pretend to know for certain how it will pan out. That's why it's an experiment :) [15:43] ev: well, time spent reviewing is time not spent writing your own code; so yes, I do expect there to be some slowdown no matter how we structure it [15:44] but this is a good opportunity to subjectively measure that slowdown [15:44] ev: To be fair, most people who understand a programming language or two can review 75% (or more, made up statistics are fun) of proposed fixed just based on comparing the changelog's description of the change to the logical path the code takes. [15:44] sorry, I meant to imply that it wont waste time. I would argue that time spent reviewing is time spent learning. [15:44] if we have everyone in #ubuntu-installer, doing "bzr diff | pastebinit" before a commit and posting the URL in the channel should work fine and is really quick to do [15:44] since we'll have people with varying degrees of experience with the code, and we want people to learn from their peers - exactly [15:44] the main issue with that is tracability as we don't have a clear MP with the votes but well ... we have the IRC log :) [15:45] infinity: exactly, which is why I think it's entirely reasonable that we put two bodies on every merge. [15:45] stgraber: what's wrong with the LP code review stuff? (other than it doesn't do inline comments) [15:45] ev: That was actually my argument for why you only need one reviewer, even if that reviewer isn't an old skool ubuntu-installer committer. :) [15:46] ev: LP's code review stuff is just a bit heavyweight for a 3-line patch. [15:46] infinity: my thought there is that when a second person comes in to review the code, they're going to be inclined to approach the code from a different angle than the one already covered [15:46] ev: I don't think we should *mandate* two peers for each change, and I'm inclined to agree with cjwatson that for trivial changes we might land them with no review at all... but if any of you guys have a change that's particularly expository of an aspect of the installer code, no reason it can't be shared around [15:46] ev: takes longer. You need to commit, push a separate branch, send a merge proposal, wait for LP to diff the code, have people review, then merge, commit and push [15:46] and that by just reading what's already been said, they may learn something [15:46] ev: I spend more time pushing branches around than I do creating, reviewing, and committing the fix. [15:46] ev: instead of doing the changes, bzr diff | pastebinit, posting to IRC, wait for a +1, commit, push [15:46] having review is more important than how the review is done [15:47] * slangasek nods [15:47] cjwatson: Crazy talk. Next you'll imply that a submitted patch is more important than the format, and the entire open source elite will crumble. [15:48] so for complex things, sure, let's use merge proposals, otherwise, I'm easy [15:48] stgraber: merge proposals have an advantage too, however, in that if the reviewer is happy, there's no additional round-trip... just land the branch directly. So there's a bit more overhead for the author and a bit less for the reviewer [15:48] sure, I agree with Colin's point on having review being more important. But I would like to see some clear structure to this. My hope is that we made code review a core part of what we do. [15:48] cjwatson: I think that's the same opinion stgraber and I had. :P [15:48] Doesn't bzr have commands to automatically propose a branch? [15:48] yes [15:49] structure is there to serve us, not the other way round :) [15:49] sometimes they work :) [15:49] lol [15:49] cjwatson: sure, I just mean, it'd be great if there was a page I could go to in order to see the list of things awaiting comment [15:49] rather than having to parse scrollback [15:49] in the general sense [15:49] Like I said before, I want a culture of review, but I'm annoyed by attempts to force process on the whole thing. [15:49] obviously pastebin will be easy when we're in sprint mode [15:49] MPs are awesome for complex merges/features. [15:50] Especially things that require a lot of iteration. [15:50] mind you, I'm flexible on all of this. I am by no means saying this is the one true way to do everything. [15:50] yeah, I'll be kind of annoyed if I have to run say debconf-updatepo runs through MPs :-) [15:51] just want to make sure we're being sensible about it is all [15:51] We should gate all installer commits through PQM while we're at it. *nod* [15:51] understandable. But I'd also like to make the few lines patch fairly easy to do a MP on while still being visible in the big queue of things [15:51] hahaha, oh dear [15:52] so I don't think we have agreement on a single right way to handle merges [15:52] ev: might be just me, but I'm usually flooded by LP while trafic in #ubuntu-installer is low, so I end up checking #ubuntu-installer a lot more often and read the scrollback very carefuly, I can't always say the same for all the LP merge proposals, bugs, ... (unless they are assigned to me) [15:53] so let's plan to use a mix of LP MPs and pastebins (and direct commits for the trivial stuff) [15:53] stgraber: ah, but you'll also be on G+ the whole time, so people can nudge you to ask for reviews directly :) [15:53] if people want to say "that's too hard for a paste, MP it" I think that's fine [15:53] agreed then? [15:54] Works for me. [15:54] FWIW I intend to make sure everything is uploaded by the end of each day, for the purpose of daily builds [15:54] so please don't break the trunk or I'll be sad :P [15:54] Keep flexible, keep lower-case agile (because upper-case Agile is anything but). [15:54] sure [15:54] [TOPIC] bugs === meetingology changed the topic of #ubuntu-meeting to: bugs [15:54] bdmurray: sorry, I haven't left you much time [15:55] fwiw I can't reproduce bug 945826 [15:55] cjwatson: are you going to be the sheriff? :) [15:55] Launchpad bug 945826 in apt (Ubuntu) "12.04 AMD64 Beta Live CD sources.list has a bad entry" [Medium,Triaged] https://launchpad.net/bugs/945826 [15:55] (context: http://www.chromium.org/developers/tree-sheriffs) [15:55] ev: pass me the silver star and the six-shooter [15:55] lol [15:55] you heard it here, ladies and gentlemen [15:56] if you break trunk [15:56] slangasek: is bug 953289 related to the one you mentioned? [15:56] Launchpad bug 953289 in unixodbc (Ubuntu) "package odbcinst1debian2 2.2.14p2-5ubuntu2 failed to install/upgrade: ErrorMessage: dependency problems - leaving unconfigured" [Medium,Confirmed] https://launchpad.net/bugs/953289 [15:56] cjwatson will gun you down [15:56] I'd rather be the tree deputy. No one ever shoots that guy. [15:56] lol [15:56] bdmurray: looking [15:56] cjwatson: what architecture did you test? [15:56] i386 [15:57] don't have kubuntu-precise-desktop-amd64 locally [15:57] (would be surprised if that mattered here?) [15:57] I used ubuntu-desktop-amd64 [15:57] bdmurray: bug #953289 is an apt bug, fixed in precise but not in oneiric [15:57] Launchpad bug 953289 in unixodbc (Ubuntu) "package odbcinst1debian2 2.2.14p2-5ubuntu2 failed to install/upgrade: ErrorMessage: dependency problems - leaving unconfigured" [Medium,Confirmed] https://launchpad.net/bugs/953289 [15:57] ok, I do have that [15:58] bdmurray: I see odbcinst:i386 being pulled in as a dep, and it should always have odbcinst [15:58] bdmurray: and, to be clear, you could reproduce the bug there? [15:59] cjwatson: yes apt-get update printed errors and using apt-cdrom failed too [15:59] bdmurray: hm, yes [15:59] bdmurray: oh, backing up a bit, maybe there are two different issues in this log... either way, no, it's not the same bug I was talking about [16:00] * infinity looks at the clock. [16:00] cjwatson: I was using the apt-cdrom comand from 41apt-cdrom [16:01] bdmurray: oho, it's the *i386* entries on amd64, now I understand [16:01] Oh, some confusion about how multiarch should or shouldn't work WRT apt-cdrom? [16:01] running apt-cdrom is more likely to confuse than help, I think [16:01] cjwatson: can I assign the bug to you for follow-through? [16:02] infinity: there's some care to try to make i386 index files available on the live CD, yes [16:02] slangasek: yep [16:02] done [16:02] bdmurray: any others? [16:02] looks to me like I just forgot to do it for restricted [16:02] cjwatson: Yeah, but having two entries seems wrong, shouldn't it just transparently DTRT if binary-i386 exists, like a file:/ or http:// URI does? [16:03] infinity: true, there's something odd there [16:03] bug 946663 - but I haven't looked into it much [16:03] Launchpad bug 946663 in ubiquity (Ubuntu) "Installer stuck at "Removing conflicting operating system files..."" [High,Confirmed] https://launchpad.net/bugs/946663 [16:03] can we put that one in the hopper for the sprint? [16:04] * slangasek gives it a rls-mgr-p-tracking [16:04] The apt-cdrom/casper one seems vaguely installer sprinty too. [16:05] I'll fix that today, I think [16:05] Or that. :P [16:05] * cjwatson isn't planning to slow down to leave more for the installer sprint, sorry :) [16:05] running out of days ... [16:05] cjwatson: I'm kinda curious if the one non-arch-specified line (ie: the second line listed) is the only one present, if it DTRT WRT multi-arch. [16:06] well I was going to fix the obvious bug I see in the CD layout first [16:06] cjwatson: Cause if so, it's probably just a casper bug that it's adding the i386 one. [16:06] cjwatson: Oh, is the archive layout also not sane? :) [16:06] yep, it's entirely missing restricted/binary-i386 [16:06] [TOPIC] AOB === meetingology changed the topic of #ubuntu-meeting to: AOB [16:06] anything else, or shall we adjourn? :) [16:07] *crickets* [16:07] #endmeeting === meetingology changed the topic of #ubuntu-meeting to: Ubuntu Meeting Grounds | Calendar/Scheduled meetings: http://fridge.ubuntu.com/calendar | Logs: https://wiki.ubuntu.com/MeetingLogs | Meetingology documentation: https://wiki.ubuntu.com/meetingology [16:07] Meeting ended Wed Mar 14 16:07:48 2012 UTC. [16:07] Minutes (wiki): http://ubottu.com/meetingology/logs/ubuntu-meeting/2012/ubuntu-meeting.2012-03-14-15.04.moin.txt [16:07] Minutes (html): http://ubottu.com/meetingology/logs/ubuntu-meeting/2012/ubuntu-meeting.2012-03-14-15.04.html [16:07] done [16:07] thanks :) [16:07] thanks [16:08] ta [16:09] thanks [17:03]