14:30:25 #startmeeting Pulp Triage 2017-08-08 14:30:25 #info ttereshc has joined triage 14:30:25 !start 14:30:25 Meeting started Tue Aug 8 14:30:25 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:25 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:30:25 The meeting name has been set to 'pulp_triage_2017_08_08' 14:30:25 ttereshc: ttereshc has joined triage 14:30:34 #info mhrivnak has joined triage 14:30:34 !here 14:30:34 mhrivnak: mhrivnak has joined triage 14:30:58 pcreech, Let's talk about this after triage. 14:31:04 !here 14:31:04 #info mansari has joined triage 14:31:05 mansari: mansari has joined triage 14:31:09 ichimonji10: ack, didn't realize triage was coming up :) 14:31:11 !here 14:31:11 #info pcreech has joined triage 14:31:12 pcreech: pcreech has joined triage 14:31:14 !next 14:31:15 ttereshc: 6 issues left to triage: 2956, 2957, 2959, 2960, 2961, 2963 14:31:15 (: 14:31:15 #topic Pulp docker ignores basic auth credentials - http://pulp.plan.io/issues/2956 14:31:16 Issue #2956 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:31:17 Pulp docker ignores basic auth credentials - http://pulp.plan.io/issues/2956 14:31:33 #info ipanova has joined triage 14:31:33 !here 14:31:33 ipanova: ipanova has joined triage 14:32:05 !propose accept 14:32:05 #idea Proposed for #2956: Leave the issue as-is, accepting its current state. 14:32:06 mhrivnak: Proposed for #2956: Leave the issue as-is, accepting its current state. 14:32:08 #info bmbouter has joined triage 14:32:08 !here 14:32:08 bmbouter: bmbouter has joined triage 14:32:13 !here 14:32:13 #info jortel has joined triage 14:32:14 jortel: jortel has joined triage 14:32:14 good candidate to talk about during sprint planning. 14:32:22 agree 14:32:24 !here 14:32:24 #info preethi has joined triage 14:32:25 preethi: preethi has joined triage 14:32:29 it should converted to to story 14:32:46 we will describe and groom with asmacdo 14:32:54 !here 14:32:54 #info dkliban has joined triage 14:32:55 dkliban: dkliban has joined triage 14:33:14 ipanova sounds fine. 14:33:21 #idea Proposed for #2956: convert to story 14:33:21 !propose other convert to story 14:33:21 ttereshc: Proposed for #2956: convert to story 14:33:27 +1 14:33:30 !accept 14:33:30 #agreed convert to story 14:33:30 ttereshc: Current proposal accepted: convert to story 14:33:32 #topic pulp-admin auth user create allows creation of duplicate userid - http://pulp.plan.io/issues/2957 14:33:32 ttereshc: 5 issues left to triage: 2957, 2959, 2960, 2961, 2963 14:33:34 Issue #2957 [NEW] (unassigned) - Priority: Normal | Severity: High 14:33:35 pulp-admin auth user create allows creation of duplicate userid - http://pulp.plan.io/issues/2957 14:33:42 * asmacdo is too slow, just https://pulp.plan.io/issues/2956#note-5 14:34:23 well that's weird. 14:34:49 btw i was not able to reproduce this 14:35:00 at leas it was always asking for the password 14:36:00 ipanova aside from that, did it allow you to create two users with the same id? 14:36:06 #info daviddavis has joined triage 14:36:06 !here 14:36:06 daviddavis: daviddavis has joined triage 14:36:06 no 14:36:32 looks like it is unique for [user, name] 14:37:03 can it be that he had the DB screwed up? 14:37:08 ipanova want to close it WORKSFORME and paste in what you saw? 14:37:17 https://github.com/pulp/pulp/blob/master/server/pulp/server/db/model/__init__.py#L1163 14:37:18 I would guess that person was missing an index. 14:37:22 mhrivnak: will do 14:37:26 +1 14:37:50 #idea Proposed for #2957: close as WORKSFORME, ipanova will comment 14:37:50 !propose other close as WORKSFORME, ipanova will comment 14:37:51 ttereshc: Proposed for #2957: close as WORKSFORME, ipanova will comment 14:38:03 #agreed close as WORKSFORME, ipanova will comment 14:38:03 !accept 14:38:03 ttereshc: Current proposal accepted: close as WORKSFORME, ipanova will comment 14:38:04 ttereshc: 4 issues left to triage: 2959, 2960, 2961, 2963 14:38:05 #topic Task results can be lost due to race condition - http://pulp.plan.io/issues/2959 14:38:05 Issue #2959 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:38:07 Task results can be lost due to race condition - http://pulp.plan.io/issues/2959 14:38:09 ipanova actually let's follow up on that later. 14:38:22 bmbouter and I were just discussing this 14:38:23 daviddavis and I looked at this filing, and we agree it's a bug 14:38:36 mhrivnak: ok 14:38:38 our only question is if it should go on a sprint or not 14:38:41 fwiw, it's pulp 2 only 14:38:56 oh yeah, it's pulp2 only this was fixed during the tasking system transition to pulp3 14:39:24 we could ask RCM if it's a blocker for them? 14:39:31 bmbouter: we can discuss this at spring planning 14:39:38 cool but it's not on my list 14:39:45 agreed, we should discuss that in sprint planning. 14:40:03 who will bring it back up then? 14:40:05 daviddavis: it should not be a blocker for them, but let's ask on the issue 14:40:18 !propose accept, ask RCM if it's a blocker 14:40:18 ttereshc: Error: "propose" is not a valid command. 14:40:22 bmbouter: in case it is a blocker i will bring this up 14:40:23 #idea Proposed for #2959: accept, ask RCM if it's a blocker 14:40:23 !propose other accept, ask RCM if it's a blocker 14:40:23 ttereshc: Proposed for #2959: accept, ask RCM if it's a blocker 14:40:28 thank you ipanova 14:40:45 pulpbot you were out for too long, I forgot everything :) 14:40:45 ttereshc: Error: "you" is not a valid command. 14:40:52 !here 14:40:52 #info dalley has joined triage 14:40:53 dalley: dalley has joined triage 14:40:53 !accept 14:40:53 #agreed accept, ask RCM if it's a blocker 14:40:54 ttereshc: Current proposal accepted: accept, ask RCM if it's a blocker 14:40:54 #topic UnicodeEncodeError in case of a non-ASCII character in comments provided with SSL cert/key/CA - http://pulp.plan.io/issues/2960 14:40:55 ttereshc: 3 issues left to triage: 2960, 2961, 2963 14:40:56 Issue #2960 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:40:57 UnicodeEncodeError in case of a non-ASCII character in comments provided with SSL cert/key/CA - http://pulp.plan.io/issues/2960 14:41:40 !propose accept 14:41:40 #idea Proposed for #2960: Leave the issue as-is, accepting its current state. 14:41:41 mhrivnak: Proposed for #2960: Leave the issue as-is, accepting its current state. 14:41:59 +1 14:41:59 +1 14:42:01 There's a reasonable work-around at least. 14:42:01 maybe low priority? workaround is easy 14:42:03 no? 14:42:07 heh 14:42:09 :) 14:42:35 but I'm ok with normal/medium as well 14:42:42 #agreed Leave the issue as-is, accepting its current state. 14:42:42 !accept 14:42:42 ttereshc: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:42:43 ttereshc: 2 issues left to triage: 2961, 2963 14:42:44 #topic Pulp 2.14 broken on Fedora 26 - http://pulp.plan.io/issues/2961 14:42:44 Issue #2961 [NEW] (unassigned) - Priority: Normal | Severity: High 14:42:45 Pulp 2.14 broken on Fedora 26 - http://pulp.plan.io/issues/2961 14:43:19 !propose other high high add to sprint 14:43:19 #idea Proposed for #2961: high high add to sprint 14:43:20 mhrivnak: Proposed for #2961: high high add to sprint 14:43:32 I'm happy to be talked out of that if it's overkill. :) 14:43:51 based on what i've seen so far, I can +1 14:43:59 +1 14:44:08 +1 14:44:14 is it an issue with pulp_packaging way of installation? 14:44:21 (to be clear, we are saying this isn't a blocker) 14:44:24 or in general 14:44:25 no? 14:44:36 some of it is pulp_packaging, but the bulk is selinux issues on f26 14:44:45 ah ok 14:45:12 I suppose the immediate priority is to figure out exactly what's going on and ensure next steps are clear to get it resolved. 14:45:20 whether that's on our side, Fedora's side, or both. 14:46:02 so we can make it a blocker later? 14:46:06 if needed 14:47:07 it looks like a blocker if it can poentially be the one 14:47:30 I'm not up-to-speed on the blocker conversation, but it does seem like we should get all the facts out and assessed before deciding to release. 14:47:47 I know a lot of that's happened, so maybe we just need to summarize on the issue. 14:48:03 Yeah, i'm finishing up some reasearch now via manual installs 14:48:21 ok, everyone agree on high/high + add to sprint? 14:48:52 I believe if we can install on all distros with selinux enabled then it should block 14:49:17 but we can wait until we have all the facts to make the call 14:49:44 ok, seems like no objections 14:49:46 #agreed high high add to sprint 14:49:46 !accept 14:49:46 ttereshc: Current proposal accepted: high high add to sprint 14:49:47 ttereshc: 1 issues left to triage: 2963 14:49:47 #topic Use the new lower case celery config settings - http://pulp.plan.io/issues/2963 14:49:48 Issue #2963 [NEW] (unassigned) - Priority: Low | Severity: Medium 14:49:49 Use the new lower case celery config settings - http://pulp.plan.io/issues/2963 14:50:01 story? 14:50:02 Oh good, no more yelling. :) 14:50:08 or a refactor? 14:50:35 +1 refactor 14:50:39 +1 14:50:43 also, low/low 14:50:47 +1 14:50:56 Sounds good. 14:50:57 #idea Proposed for #2963: Priority: Low, Severity: Low 14:50:57 !propose triage l l 14:50:57 ttereshc: Proposed for #2963: Priority: Low, Severity: Low 14:51:03 !accept 14:51:03 #agreed Priority: Low, Severity: Low 14:51:03 ttereshc: Current proposal accepted: Priority: Low, Severity: Low 14:51:05 ttereshc: No issues to triage. 14:51:08 !end 14:51:08 #endmeeting