=== matsubara is now known as matsubara-afk === noodles775 changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [sinzui, adiroiban(code,bug-146178),adiroiban(code,bug-525992), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [11:35] henninge: another one for you if/when you've time :) [11:36] noodles775: um, there is quite a queue and I am not ready yet ... [11:36] noodles775: you can append it and I will see if I can get to it. [11:36] I need to finish some urgent QA first, sorry. [11:37] henninge: yeah, no worries, I've already got in on the queue for whoever gets there first. Good luck with the QA! [12:01] another staging restore .... might as well do reviews ;) === henninge changed the topic of #launchpad-reviews to: On call: henninge || reviewing: sinzui || queue: [adiroiban(code,bug-146178),adiroiban(code,bug-525992), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === henninge changed the topic of #launchpad-reviews to: On call: henninge || reviewing: adiroiban(code,bug-525992) || queue: [adiroiban(code,bug-146178), noodles] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [12:09] sinzui's queue entry seems to have been stale. === mrevell is now known as mrevell-lunch === bigjools changed the topic of #launchpad-reviews to: On call: henninge || reviewing: adiroiban(code,bug-525992) || queue: [adiroiban(code,bug-146178), noodles, bigjools] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === salgado-afk is now known as salgado === StevenK changed the topic of #launchpad-reviews to: On call: henninge || reviewing: adiroiban(code,bug-525992) || queue: [adiroiban(code,bug-146178), noodles, bigjools, StevenK] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === mrevell-lunch is now known as mrevell === bac changed the topic of #launchpad-reviews to: On call: henninge,bac || reviewing: adiroiban(code,bug-525992) || queue: [adiroiban(code,bug-146178), noodles, bigjools, StevenK] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === bac changed the topic of #launchpad-reviews to: On call: henninge,bac || reviewing: adiroiban(code,bug-525992),adiroiban(code,bug-146178) || queue: [noodles, bigjools, StevenK] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [14:18] morning henninge. how's it going? [14:19] bac: Hi! Good, thanks. [14:20] bac: but I won't have a great OCR throughput today .... [14:21] henninge: that's ok === bac changed the topic of #launchpad-reviews to: On call: henninge,bac || reviewing: adiroiban(code,bug-525992),noodles || queue: [bigjools, StevenK] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [15:41] abentley_: ping [15:41] BjornT, pong. [15:42] abentley: about dailybuilds-archive. do i understand correctly, if i think that the archive column will be used only for daily builds? [15:44] BjornT, correct. When we do a daily build, we will use SourcePackageRecipe.archive as value for SourcePackageRecipeBuild.archive [15:45] abentley_: how about naming it daily_build_archive to make that clear? [15:46] BjornT, we might also like to use it as the default when doing a manual build. [15:49] BjornT, but if it gets it though review faster, I can change the name. [15:52] abentley: well, it's mostly that i'd like us to have an idea of what things will be used for. the name should make this clear, so that it's easier to understand from reading the code. === matsubara is now known as matsubara-lunch [15:58] Hi :) [15:58] Can I get some help with my branch? lp:~cody-somerville/launchpad/bug-444266/ [15:59] I created a IHasBugSupervisorEditSchema class to provide an editable field for bug_supervisor in IHasBugSupervisorEditView since bug_supervisor is defined as readoly in the IHasBugSupervisor interface. [15:59] abentley: if you're ok with naming it daily_build_archive, that's ok, though. that shows a clear use, and if you find different use cases, it will be more obvious that you should rename it. [15:59] Similar to what BranchEditSchema does to make the private field editable. [15:59] However, I get TypeError: ('Could not adapt', , ) [16:00] and TypeError: ('Could not adapt', , ) [16:03] :) [16:07] noodles775, danilo_: Can you please have a look at the first paragraph of my review for Adi and comment *if you object*? Thanks. ;) https://code.edge.launchpad.net/~adiroiban/launchpad/bug-525992/+merge/23285 === henninge changed the topic of #launchpad-reviews to: On call: bac || reviewing: noodles || queue: [bigjools, StevenK] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [16:09] bac: I am done. ;) [16:09] have a good evening henninge [16:09] thanks [16:11] henninge: I think your suggestion is a good one (stating the mode without "Enter").. [16:11] noodles775: ok, thanks ;) [16:12] noodles775: I was just thinking it could even be "You are in ... mode" [16:17] henninge, I concur :) [16:18] cool === deryck is now known as deryck[lunch] [16:49] I'm confused. [16:49] Who is reviewing? bac or noodles775? [16:50] on call: bac [16:50] what is bac doing? [16:50] reviewing: noodles [16:50] clearer cody-somerville? :) [16:50] bac is reviewing noodles? [16:50] indeed [16:51] noodles775, thats not clear at all :P [16:51] do you need a review cody-somerville? [16:51] err.. [16:51] (sorry, auto complete) [16:51] bac, Yes please :) [16:51] I created a IHasBugSupervisorEditSchema class to provide an editable field for bug_supervisor in IHasBugSupervisorEditView since bug_supervisor is defined as readoly in the IHasBugSupervisor interface. [16:51] Similar to what BranchEditSchema does to make the private field editable. [16:51] However, I get TypeError: ('Could not adapt', , ) [16:51] cody-somerville: tbh you're the first person to say it has confused them in the 2 years we've used this format. [16:51] and TypeError: ('Could not adapt', , ) [16:52] lp:~cody-somerville/launchpad/bug-444266/ [16:52] bac, Maybe it would be more clear if it listed a branch instead of a person [16:52] bac, But to me the current topic suggests that noodles775 is reviewing and you're 'on call' to help if needed. [16:52] cody-somerville: things to be reviewed show up at https://code.edge.launchpad.net/launchpad/+activereviews [16:53] so if noodles only has one branch ready to be reviewed there is no ambiguuity [16:53] cody-somerville: have you created a merge proposal? [16:54] bac, No. My branch doesn't work so I didn't create one. [16:54] ah, ok [16:55] * bac looks at branch [16:57] hi noodles775 === flacoste is now known as flacoste_lunch === flacoste_lunch is now known as flacoste [17:09] cody-somerville: how do i reproduce those errors? [17:09] ./bin/test -vvvc -tbug-release-management === mrevell changed the topic of #launchpad-reviews to: On call: bac || reviewing: noodles || queue: [bigjools, StevenK, mrevell] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === salgado changed the topic of #launchpad-reviews to: On call: bac || reviewing: noodles || queue: [bigjools, StevenK, mrevell,salgado] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === salgado is now known as salgado-lunch [17:28] Hi bac [17:37] cody-somerville: you need an adapters property on the view. look at how it was done for BranchEditSchema === gary_poster is now known as gary-lunch === deryck[lunch] is now known as deryck [18:14] bac, okay, I must have missed that [18:14] bac, I'll take another look === salgado-lunch is now known as salgado === gary-lunch is now known as gary-poster === gary-poster is now known as gary_poster [19:36] noodles775: ping === bac changed the topic of #launchpad-reviews to: On call: bac || reviewing: stevenk || queue: [bigjools, salgado] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [20:21] hi bac [20:21] hi noodles775. bad timing -- i thought you'd gone so i just marked your MP 'abstain' === salgado changed the topic of #launchpad-reviews to: On call: bac || reviewing: stevenk || queue: [bigjools] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [20:33] bac, can i add https://code.edge.launchpad.net/~leonardr/lazr.restfulclient/retry_on_502/+merge/24252 to the queue? [20:47] leonardr: yes [20:47] cool === leonardr changed the topic of #launchpad-reviews to: On call: bac || reviewing: stevenk || queue: [bigjools,leonardr] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews === bac changed the topic of #launchpad-reviews to: On call: bac || reviewing: leonardr || queue: [bigjools] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [20:55] leonardr: inexplicably your MP is private again. [20:55] bac: fixing [20:58] bac: i don't see any 'public' flag for mps. is it possible you don't belong to lazr-developers?? [20:58] i've asked you specifically for a review [20:58] so you should be able to see it now [20:59] actually i could see it before but noticed it was private [20:59] now i cannot see it at all [21:00] aaaargh [21:00] if only i knew how to use this launchpad thing [21:00] and i am in ~lazr-developers as are all of ~launchpad [21:01] brad: ah, the _branch_ is private [21:01] odd, the MP moved URLs when you changed it. it now lives at https://code.edge.launchpad.net/~leonardr/lazr.restfulclient/retry_on_502/+merge/24257 [21:01] bac: try again, i made itp ublic [21:01] how'd you do that? [21:18] leonardr: what is the purpose of the BrokenApplication class in your test? [21:19] bac: i think that might not be necessary anymore [21:20] leonardr: me too [21:20] yeah, i tried to do it as a class and decided it was easier as a function === bac changed the topic of #launchpad-reviews to: On call: bac || reviewing: - || queue: [bigjools] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.edge.launchpad.net/launchpad/+activereviews [21:33] bac, woot. ty. [21:33] np [21:35] bac, Now, where would you recommend I add tests to set security contact and bug supervisor via API? [21:36] cody-somerville: we cannot yet do automated tests using launchpadlib but will be able to very soon now [21:36] until then, we use the web service directly, without the lplib wrapper [21:36] cody-somerville: look in lp/bugs/stories/webservices [21:37] bac, right right [21:37] bac, but there is no doctest in there yet for this sort of thing [21:37] bac, I'm looking for a good name to use for new doctest [21:40] cody-somerville: i'm not familar with how the bug tests are organized right now. if they don't fit in one of the existing tests then just use your best judgment === salgado is now known as salgado-afk === matsubara is now known as matsubara-afk [23:23] bac: hi. I have create a new MP for bug 146178 and this one is targeted to devel (not db-deve). https://code.edge.launchpad.net/~adiroiban/launchpad/bug-146178/+merge/24265 [23:23] Bug #146178: Add links to latest full and delta language pack exported [23:24] Should I get a new UI review?