17:00:30 #startmeeting FESCO (2011-10-03) 17:00:30 Meeting started Mon Oct 3 17:00:30 2011 UTC. The chair is ajax. Information about MeetBot at http://wiki.debian.org/MeetBot. 17:00:30 Useful Commands: #action #agreed #halp #info #idea #link #topic. 17:00:30 #meetingname fesco 17:00:30 #chair ajax notting nirik cwickert mjg59 mmaslano t8m pjones sgallagh 17:00:30 #topic init process 17:00:30 The meeting name has been set to 'fesco' 17:00:30 Current chairs: ajax cwickert mjg59 mmaslano nirik notting pjones sgallagh t8m 17:00:40 * nirik is around. 17:00:44 hello. 17:01:13 * notting is here 17:01:14 * sgallagh is here more or less (busy day) 17:01:17 Here (obv) 17:01:22 hi 17:01:46 right, that's enough for quota, let's dive in. 17:01:51 #topic #663 Late F16 Feature Java7 17:01:53 .fesco 663 17:01:54 ajax: #663 (Late F16 Feature Java7) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/663 17:02:22 this is pending the rebuild of some packages. 17:02:27 Yeah 17:02:39 Given there's a rel-eng ticket I guess it'll be done 17:02:41 Hopefully dgilmore can get to it later this week... he's been traveling/at fucon/getting beta done. 17:03:28 do we need someone to keep an eye on this or shall we assume rel-eng is on the job? 17:03:56 did rel-eng promised to do it? 17:03:58 we can leave it on the agenda but defer? 17:04:34 mmaslano: well, no, but nobody promises to do anything. 17:04:47 i'll poke rel-eng on the ticket, we'll come back next week. 17:05:04 #action ajax to follow up with rel-eng 17:05:18 anything else on this? 17:05:33 Hi, I'm sorry for being late 17:05:49 * nirik has nothing more on this topic 17:05:52 t8m: np, didn't miss much yet 17:05:59 next up 17:06:03 #topic #667 Request to fix CRITPATH update process 17:06:03 .fesco 667 17:06:05 ajax: #667 (Request to fix CRITPATH update process) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/667 17:06:22 heh, now the fun begins :) 17:06:23 so, should we revisit the timeout proposal? or is everyone set on their votes? 17:06:30 * sgallagh continues to be in favor of the "Allow it after two weeks if there is no negative karma" 17:06:44 * t8m too 17:06:49 * mmaslano too 17:06:55 * nirik is also +1 for that. I think toshio's reasoning made sense. 17:07:11 I don't think there's any new reasoning 17:07:22 mjg59: i don't think there is either 17:07:43 no reason to think anything has changed. 17:07:47 i'm going to +1 that as well just so i can stop hearing about this 17:07:50 If this needs to be fixed then we need to fix it properly 17:08:15 If I count right this means +5 now? 17:08:15 mjg59: yes, but in the mean time we are frustrating maintainers and our users... 17:08:15 i don't disagree 17:08:17 Rather than just satisfying the noisiest (and I don't mean that in a bad way) people without actually fixing the other cases that are being caught 17:08:44 but i think adding the timeout here does get us to a less bad place 17:08:50 even though it's still manifestly broken 17:08:57 I do not think adding the timeout prevents us to fix the problems 17:08:57 it's a band aid. 17:09:07 t8m: It doesn't prevent us. It just means that we won't bother. 17:09:10 But, eh. 17:09:11 What's the "proper" fix? 17:09:12 i would like to track how many updates time out 17:09:14 but sometimes you have to stem the blood flow so you can fix the injury. ;) 17:09:23 notting: +1 17:09:37 dledford: Making sure that these packages actually get appropriately tested, whether by people or by automated infrastructure 17:09:46 Actually, I'd like to amend the proposal for clarity: #proposal CRITPATH packages can go to stable if they either receive no negative karma after two weeks, or they receive the appropriate positive karma. If there is negative karma, it must be negated by proventester positive karma. 17:10:08 +1 17:10:09 That last sentence is hard to parse. 17:10:16 sgallagh: isn't the second sentence a no-op? 17:10:23 s/negated/canceled/ i think? 17:10:24 negative, negative. 17:10:27 "counteracted", perhaps? 17:10:33 mjg59: Well, automated infrastructure is on hold because of other work to do, and treating a volunteer project like it's a paid job and we can simply wave a magic wand and tell people to test and they will is unrealistic. 17:10:35 Sure, poor choice of words 17:10:39 sgallagh: i.e., if they get negative karma, the first condition fails, so they're stuck on relying on getting the appropriate karma anyway 17:10:59 notting, +1 17:11:10 notting: A fair point\ 17:11:16 dledford: I don't disagree 17:11:23 I suppose I was trying to overclarify 17:11:51 anyway, rough consensus? 17:11:53 so, implementing this would need bodhi changes, correct? or... 17:11:56 #proposal CRITPATH packages can go to stable if they either receive no negative karma after two weeks, or they receive the appropriate positive karma, which includes at least one proventester. 17:12:15 sgallagh, +1 17:12:19 do we want to ask people who want to do this to ping/file a ticket so we can more easily trac it. 17:12:21 nirik: yes. although it already has the concepts of karma & timeouts, so ideally it's just a rules tweak 17:12:51 notting: yeah. 17:12:55 nirik: That was pun-tastic. (Sorry) 17:12:55 +1 (again) to sgallagh's proposal 17:12:57 sgallagh: +1 17:13:22 +1 to sgallagh 17:13:33 dledford: this change would at least help you, correct? 17:13:40 nirik: absolutely 17:13:46 not a huge fan, but we're not making progress on the proper fix yet. +1 to bandage the patient. 17:14:09 #agreed critpath package rules to be modified per sgallagh's proposal above 17:14:37 what about the self votes proposal? 17:14:41 so, we need a bodhi ticket and then updating the updates_policy on the wiki once the bodhi change is live and announcing it then. 17:14:49 t8m: I think that's a separate issue 17:15:13 sgallagh, yes, but we can discuss it here - or is there separate open ticket for that? 17:15:13 * nirik guesses he could file the bodhi ticket and update things once it's done. 17:15:25 nirik: if'n you don't mind, that'd be great. 17:15:32 can do 17:15:45 #action nirik to file bodhi ticket for critpath change 17:15:45 t8m: I don't think there's a ticket currently, but let's save that for the Open Discussion 17:15:53 indeed. 17:16:04 sgallagh, OK 17:16:18 last item 17:16:18 #topic #671 Packages packaging yum repo files? 17:16:18 .fesco 671 17:16:20 ajax: #671 (Packages packaging yum repo files?) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/671 17:16:52 I thought we decided to let FPC poke at this? or? 17:16:57 Sorry 17:17:01 I think I forgot to close it 17:17:08 yeah 17:17:10 My fault 17:17:17 tsk! 17:17:31 #action mjg59 to close out #671 per last week's discussion 17:18:12 well that was pleasantly quick 17:18:18 mjg59: and you forgot send meeting minutes from last week 17:18:18 #topic Next week's chair 17:18:23 * nirik notes https://fedorahosted.org/bodhi/ticket/642 is the bodhi ticket for the critpath change. 17:18:31 nirik: thanks! 17:18:45 ajax, I can do it 17:18:52 t8m: excellent 17:19:00 #action t8m to chair next week's meeting 17:19:05 #topic Open Floor 17:19:23 OK then for the self-vote in bodhi proposal 17:19:29 I may not be around next week. 17:19:54 I'd like to thank everyone who worked hard to get F16 beta out this week. Much hard work and testing. ;) 17:19:57 t8m: I'm inclined to allow the "vote on someone else's behalf" 17:20:08 Yes, excellent work! 17:20:11 #proposal A self vote in bodhi is allowed if you vote on someone else's behalf. 17:20:25 t8m: why couldn't they vote themselves? 17:20:26 Amended: With a comment to that effect. 17:20:35 seems... gameable, but sure. 17:20:48 nirik, they do not have the bodhi account and they don't bother to create one? 17:20:51 nirik: Not everyone wants to join FAS just to add karma for a single package 17:20:59 I know that's a little bit lazy. 17:21:10 bodhi has anonymous karma though 17:21:14 sgallagh, +1 with the amendment 17:21:19 so, does this happen much? is this going to help us any? 17:21:22 ajax, which is not counted 17:21:26 ajax: anonymous karma is useless from a karma perspective 17:21:39 (then why have it, i wonder) 17:21:41 also, note there is a ticket to make bodhi not allow the maintainer to +1 their own update. We would have to drop that. 17:21:55 nirik, sometimes people just comment in the bz 17:21:55 I've said before, in general self-karma doesn't have any negative effect, because the maintainer could always opt to just reduce the karma requirement 17:22:06 What about doing an auto +1 to karma when a bug listed in the ticket goes from ON_QA to VERIFIED? 17:22:07 the negative karma thing is broken though "it does not fix $unrelated bug" 17:22:09 ideally, all you'd need is a email + passwd 17:22:14 but that would require FAS changes 17:22:15 -> update sits there forever 17:22:23 the timeout is no real fix imo 17:23:03 dledford: not a bad idea, we don't have any process in place to do that currently tho... 17:23:08 we could allow the maintainer to mark karma as invalid .... but that can be abused 17:23:53 nirik: We have at least minimal integration, a bodhi tickets puts the bug into ON_QA when the package hits testing, so this would just need to be either a cron job that checks bug state or a reverse trigger in bugzilla that updates bodhi. 17:23:57 sgallagh, not in the case of the hard limit requirement as in case of critpath 17:24:18 t8m: Valid point 17:24:31 dledford: yeah, may not be too bad... not sure. 17:24:47 so any votes for the proposal? 17:24:50 of course when it goes to verified, the same person might go and +1 it... leading to 2. 17:24:56 t8m: Perhaps "allow maintainers to +1 their own tickets, but proventester +1 must NOT be the submitter"? 17:25:22 Why would the maintainer have uploaded the package if they hadn't tested it? 17:25:30 mjg59: Happens all the time 17:25:37 mjg59: have you _seen_ the glibc updates? 17:25:47 mjg59: Test on F15, submit the same update for F14 assuming it works too 17:25:59 I'm fine with allowing maintainers to +1 their own update providing the karma requirements also increase by 1 17:26:08 mjg59, he could regression test it but perhaps the bug that was the cause of the update is not reproduceable for him? 17:26:18 But the values we set were based on the assumption that the update was, you know, actually tested. 17:26:25 mjg59, yeah and the old release updates 17:27:14 mjg59: but in that case, what's the point? 17:27:38 notting: Our requirements then match reality rather than what we originally implemented? 17:27:47 The point of not allowing the maintainer to +1 is that the maintainer has presumably already tested 17:28:15 which they certainly ought to have. 17:28:18 So letting them +1 would be them saying "I've tested this thing that I tested" 17:28:20 nirik: for the +2 update scenario, you could have karma from the same person as the bug reporter (assuming an automatic karma from the bug going to VERIFIED) ignored for counting purposes. 17:28:29 mjg59: i just mean that your proposal is entirely equal to 'not allow maintainers to +1', which is much simpler 17:28:43 also, that we now appear to be discussing entirely disparate proposals at once 17:28:50 notting: Well, it distinguishes the case where the maintainer (a) doesn't test and (b) doesn't even pretend to have tested 17:29:40 In general, I'm opposed to any proposal that adds more effort to the maintainer for little-to-no visible gain 17:29:48 Well I would like the proposal if we hadn't the problem with already too strict requirements. 17:29:59 t8m: that's not the problem. 17:30:03 sgallagh, +1 17:30:10 #proposal Leave things as they are and handle individual abuse cases if-and-when they come up 17:30:19 pjones, yeah and what is? 17:30:25 t8m: not enough testing! 17:30:43 I think we are trying to increase our testing pool from our maintainers testing their own packages, which seems bad. 17:30:52 pjones, no, not enough testers giving karma points in updates is the problem 17:30:55 trying to solve it by changing the way we count the testing doesn't make more testing happen. 17:31:05 pjones, we simply do not know if we have enough real testing or not 17:31:09 it doesn't solve that problem either! 17:31:45 pjones, it does if you allow giving the vote only on behalf of someone who doesn't have FAS account 17:31:48 pjones: waiting for testers didn't help either 17:31:59 Well it comes down to this 17:32:00 pjones: so we need to somehow encourage people to do more testing but ... how 17:32:05 Is it better to have no updates or untested updates 17:32:08 mmaslano: no, but that's not exactly something we're proposing either. 17:32:14 drago01, actually we do not know that 17:32:26 mjg59: depends on the actual update 17:32:30 drago01, we need to encourage them to give karma points after they tested 17:32:31 We've tried the untested updates approach 17:32:35 It didn't work out so well 17:32:36 mjg59: and the answer is: in almost all cases, the former. in critical security updates - eh, hard to say. 17:32:49 or better what pjones said 17:32:59 Yeah. There's definitely a problem here. 17:33:22 yeah, but adding more work on maintainers seems like the wrong place to be trying to fix this. 17:33:23 mjg59: That's not quite accurate, a more correct summation would be: Is it better to have untested by proventester updates or no updates. At least in my case, if all the people testing gave karma I would hit the number, but would still be blocked by the proventester requirement. 17:33:38 * sgallagh notes that there's an unvoted proposal out there. 17:33:46 dledford: Well right that's certainly one possible issue 17:33:55 Maybe the proventester distinction isn't actually useful 17:34:02 We don't have any mechanism for determining that at the moment 17:34:03 sgallagh: several. 17:34:22 Can we mine bodhi and find out how many updates had postive karma but negative proventester karma? 17:34:27 mjg59: And there comes a point where, statistically speaking, more testing by non-proventester testers is in fact better than a single proventester's testing. 17:34:35 Because if that number is small then maybe we are taking the wrong approach here 17:34:55 We were supposed to try to work out metrics on this. That seems like an obvious one. 17:35:03 mjg59: is it really about negative proventester karma or *no* proventester karma? 17:35:09 drago01: Negative. 17:35:17 mjg59: we can, but not in this meeting 17:35:26 drago01: ie, how many times has a proventester found a package to be broken when other testers have said it's fine 17:35:35 drago01: the question is "how good is !proventester karma" 17:35:35 mjg59: ah ok 17:35:45 so, where do we go here? vote on the several proposals? I fear we are going in circles. 17:36:19 I'm somewhat moved by the argument that proventester karma is an unnecessary blocker 17:36:25 But we should get numbers on that 17:36:25 nirik: well, we have found a case where there's a clear policy change we can make after measuring something. let's measure it, and vote on the proposed change depending on the result? 17:36:26 mjg59: that depends on the package the situation though ... it might be broken in the proventester's configuration but not on the "non proven" tester's 17:36:37 drago01: Right but that's still valuable 17:36:39 ./package/package and/ 17:36:44 drago01: that converges on the same thing 17:36:55 sure, if folks want to gather more info thats great. 17:37:12 Who would be able to get that information? 17:37:18 yeah without data this is going nowhere 17:37:33 well, anyone with a copy of wget, but there's probably more practical methods. 17:37:43 mjg59: assuming it is stored in a database ... no technical reason why we couldn't 17:37:45 ajax: Yeah exactly 17:37:52 Ok. How about I talk to Luke? 17:38:03 sounds like a plan 17:38:09 nice 17:38:24 #action mjg59 to get data on proven/normal karma on updates 17:38:28 * nirik is fine with that. 17:38:37 lmacken: Oh hey perfect timing. Don't go anywhere. 17:38:53 and that's >15 minutes on that topic so i'm fine with moving on... 17:38:59 * t8m too 17:39:06 anything else from anybody else? 17:39:09 +1 to move on 17:39:54 * nirik has nothing. 17:39:56 * ajax will close in a minute if there's nothing further 17:39:58 ditto 17:40:13 ticket #674 17:40:43 .fesco 674 17:40:47 sgallagh: #674 (Need Jes Sorenson added to one of the packager groups) - FESCo - Trac - https://fedorahosted.org/fesco/ticket/674 17:41:09 dledford: yeah, I can take care of that after the meeting (unless notting beats me to it) 17:41:14 nirik: go for it 17:41:17 yeah, this doesn't even need a vote. 17:41:18 No objections 17:41:21 Thanks ;-) 17:41:29 oh, so that's what he's working on now. 17:42:00 Yeah, he's been brought in to help me with RAID stuff, and I'm having him help in both Fedora and RHEL. 17:42:27 * ajax repeats the "close in one minute" warning bell 17:42:32 +1 to adding him to packagers 17:43:37 #endmeeting