14:30:40 #startmeeting Pulp Triage 2017-09-12 14:30:40 #info ttereshc has joined triage 14:30:41 Meeting started Tue Sep 12 14:30:40 2017 UTC and is due to finish in 60 minutes. The chair is ttereshc. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:30:41 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:30:41 The meeting name has been set to 'pulp_triage_2017_09_12' 14:30:41 ttereshc: ttereshc has joined triage 14:30:42 !here 14:30:42 #info daviddavis has joined triage 14:30:43 daviddavis: daviddavis has joined triage 14:30:48 !here 14:30:48 #info dalley_ has joined triage 14:30:48 dalley_: dalley_ has joined triage 14:31:10 !here 14:31:10 #info mansari has joined triage 14:31:10 mansari: mansari has joined triage 14:31:14 !here 14:31:14 #info jortel has joined triage 14:31:14 jortel: jortel has joined triage 14:31:16 !next 14:31:17 ttereshc: 3 issues left to triage: 2988, 3004, 3005 14:31:17 #topic Exception when raising a user-Defined Exception that has a custom __init__ signature - http://pulp.plan.io/issues/2988 14:31:18 Issue #2988 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:31:19 Exception when raising a user-Defined Exception that has a custom __init__ signature - http://pulp.plan.io/issues/2988 14:32:22 !here 14:32:22 #info asmacdo has joined triage 14:32:22 asmacdo: asmacdo has joined triage 14:32:24 !here 14:32:24 #info bmbouter has joined triage 14:32:24 bmbouter: bmbouter has joined triage 14:32:32 !here 14:32:32 #info dkliban has joined triage 14:32:32 dkliban: dkliban has joined triage 14:32:43 I wrote my recommendation at the bottom in the last comment 14:32:43 !propose notskip 14:32:43 asmacdo: Error: "propose" is not a valid command. 14:33:06 ha 14:33:15 which is to accept and use non-concrete exceptions for now and switch them to concrete exceptions later (iirc) 14:34:30 !propose accept 14:34:30 #idea Proposed for #2988: Leave the issue as-is, accepting its current state. 14:34:30 ttereshc: Proposed for #2988: Leave the issue as-is, accepting its current state. 14:35:02 +1 14:35:30 I think it's the issue which has to be solved post alpha and how we are going to solve it, we can decide later 14:36:05 +1 to accept as is 14:36:08 !accept 14:36:08 #agreed Leave the issue as-is, accepting its current state. 14:36:08 ttereshc: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:36:09 ttereshc: 2 issues left to triage: 3004, 3005 14:36:10 #topic recursive errata copy does not copy rpms - http://pulp.plan.io/issues/3004 14:36:10 Issue #3004 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:36:11 recursive errata copy does not copy rpms - http://pulp.plan.io/issues/3004 14:37:28 This is a regression in 6.2.11 and I guess it's becasue of my changes to errata handling, so likely a regression in 2.13.3 14:37:30 bz has a pm score of 2100 seems important 14:37:43 I think it is important 14:37:45 high? 14:37:48 add to sprint? 14:37:55 I suggest high/high and ad to sprint 14:37:58 yes :) 14:38:01 +1 14:38:03 +1 14:38:03 sounds good 14:38:16 !propose other triage h h, add to sprint 14:38:16 #idea Proposed for #3004: triage h h, add to sprint 14:38:17 ttereshc: Proposed for #3004: triage h h, add to sprint 14:38:21 !accept 14:38:21 #agreed triage h h, add to sprint 14:38:21 ttereshc: Current proposal accepted: triage h h, add to sprint 14:38:22 ttereshc: 1 issues left to triage: 3005 14:38:22 #topic redmine_bugzilla script restores NEEDSINFO flag on developer after it has been removed - http://pulp.plan.io/issues/3005 14:38:24 Issue #3005 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:38:24 lol @ pulp-admin rpm repo sync --repo-id=foo2 and then pulp-admin rpm repo sync run --repo-id=foo2 14:38:25 redmine_bugzilla script restores NEEDSINFO flag on developer after it has been removed - http://pulp.plan.io/issues/3005 14:38:28 low / low 14:38:45 just an annoyance 14:38:46 asmacdo: ha 14:39:03 daviddavis: ive done that so many (like every?) time 14:39:17 dalley_: how do we fix it 14:39:25 asmacdo: lol me too 14:39:37 bmbouter wrote thae script iirc 14:39:51 is it an issue at all? I think it probably makes sense to keep developer on the ticket. And the workflow currently is that if no more input from upstream is required, FailedQA should be unset as well 14:39:57 along with needinfo 14:40:00 not sure, I haven't spent much time on it yet, I just recorded the bug when I came across it 14:40:29 ttereshc, that's a good point 14:40:38 yea, I kinda agree too 14:40:42 but, once someone comments on the issue, don't they get emails anyway? 14:40:51 does needsinfo need to be set also 14:41:08 unless you mean for reference purposes 14:41:18 in which case... I agree with you, that sounds fine 14:41:36 I don't see the harm in NEEDSINFO I guess. unless people start complaining. 14:41:40 maybe they get into CC but you won't see as an upstream developer that you have needinfo flag, that something is still waiting on you 14:42:04 +1 14:42:11 sounds fine to me 14:42:23 maybe some feedback from bmbouter will be helpful and we can close it if needed after triage 14:42:23 NOTABUG ? 14:42:23 _ 14:42:25 ? 14:42:39 !propose skip 14:42:39 #idea Proposed for #3005: Skip this issue for this triage session. 14:42:40 ttereshc: Proposed for #3005: Skip this issue for this triage session. 14:42:43 +1 14:42:43 +1 14:42:50 !accept 14:42:50 #agreed Skip this issue for this triage session. 14:42:50 ttereshc: Current proposal accepted: Skip this issue for this triage session. 14:42:52 ttereshc: No issues to triage. 14:42:56 !end 14:42:56 #endmeeting