15:30:00 #startmeeting Pulp Triage 2017-02-17 15:30:00 #info bizhang has joined triage 15:30:00 Meeting started Fri Feb 17 15:30:00 2017 UTC and is due to finish in 60 minutes. The chair is bizhang. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:30:00 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:30:00 The meeting name has been set to 'pulp_triage_2017_02_17' 15:30:00 bizhang has joined triage 15:30:03 #info mhrivnak has joined triage 15:30:03 !here 15:30:04 mhrivnak has joined triage 15:30:06 !here 15:30:06 #info daviddavis has joined triage 15:30:07 daviddavis has joined triage 15:30:09 !here 15:30:09 #info jortel has joined triage 15:30:09 jortel has joined triage 15:30:12 !next 15:30:13 #topic Disallow re-uploading the same package twice - http://pulp.plan.io/issues/1389 15:30:14 6 issues left to triage: 1389, 2588, 2589, 2591, 2592, 2593 15:30:14 Pulp Issue #1389 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:30:15 Disallow re-uploading the same package twice - http://pulp.plan.io/issues/1389 15:30:53 !here 15:30:53 #info dalley has joined triage 15:30:54 dalley has joined triage 15:30:56 !here 15:30:56 #info ipanova has joined triage 15:30:56 ipanova has joined triage 15:31:05 !here 15:31:05 #info ttereshc has joined triage 15:31:05 ttereshc has joined triage 15:31:11 !here 15:31:11 #info dkliban has joined triage 15:31:11 dkliban has joined triage 15:31:20 accept and add to sprint? 15:31:36 bizhang: +1 15:31:41 +1 15:32:05 +1 15:32:07 !propose other accept and add to sprint 15 15:32:07 #idea Proposed for #1389: accept and add to sprint 15 15:32:08 Proposed for #1389: accept and add to sprint 15 15:32:11 !accept 15:32:11 #agreed accept and add to sprint 15 15:32:11 Current proposal accepted: accept and add to sprint 15 15:32:13 5 issues left to triage: 2588, 2589, 2591, 2592, 2593 15:32:13 #topic Pulp Gets Stuck On A Particular Mirror - http://pulp.plan.io/issues/2588 15:32:14 Pulp Issue #2588 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:32:14 Pulp Gets Stuck On A Particular Mirror - http://pulp.plan.io/issues/2588 15:33:15 sounds quite bad :( prio high? 15:34:12 is the problem that pulp is storing the destination url and retrying that instead of the original url? 15:35:17 the catalog contains the url as would be used by the importer for 'immediate' 15:35:17 Oh, I might know what's going on here. Interesting problem. 15:35:43 jortel, the yum importer can have a mirror list URL. 15:35:54 Which gets resolved to a specific mirror during sync. 15:36:06 I wonder if the catalog entries are made using that specific mirror URL. 15:36:46 ah, yea 15:36:59 !here 15:36:59 #info bmbouter has joined triage 15:37:00 could be. I'd have to look. 15:37:00 bmbouter has joined triage 15:37:04 high and add to sprint? 15:37:09 Not real sure what the right approach is. 15:37:29 yeah, i'd need to think/dig through it 15:37:34 needsinfo and retriage after jortel takes a look? 15:37:46 I think that's a fine idea. 15:37:49 works for me 15:37:51 +1 15:37:52 !propose other needsinfo, jortel to take a look 15:37:52 #idea Proposed for #2588: needsinfo, jortel to take a look 15:37:53 We can brainstorm on it and come back in a few days. 15:37:53 Proposed for #2588: needsinfo, jortel to take a look 15:37:58 !accept 15:37:58 #agreed needsinfo, jortel to take a look 15:37:58 Current proposal accepted: needsinfo, jortel to take a look 15:37:59 #topic Pulp 2.13 nightly fails to start on RHEL 7.3 - http://pulp.plan.io/issues/2589 15:38:00 4 issues left to triage: 2589, 2591, 2592, 2593 15:38:00 Pulp Issue #2589 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:38:01 Pulp 2.13 nightly fails to start on RHEL 7.3 - http://pulp.plan.io/issues/2589 15:38:09 looks like celery just needs to be updated 15:38:15 high and add to sprint 15:38:19 +1 15:38:50 !propose other triage high/high add to sprint 15:38:50 #idea Proposed for #2589: triage high/high add to sprint 15:38:51 Proposed for #2589: triage high/high add to sprint 15:39:20 What about medium priority? 15:39:20 +1 15:39:24 Since it's not released yet. 15:39:36 +1 15:39:38 No users affected. 15:39:39 +1 15:40:22 +1 15:40:30 I'd be ok with med too, but we probably want to get the nightly working sooner rather than later 15:40:35 are these +1 to med prio? 15:41:02 actually, I vote for high/high 15:41:21 agreed, and it's fairly low difficulty 15:41:29 What is this blocking right now? 15:41:39 it should block 2.13 15:41:50 the tag got removed by accident because of that redmine bug 15:42:08 Is testing not working because of it? 15:42:49 mhrivnak, pulp basically does not work on rhel7 without a newer version of celery 15:42:54 I can easily be sold on high priority. :) I'm just looking for what the current impact is of this bug. 15:43:18 workers and resource manager immediately crash 15:43:46 vote not to add it to the sprint. 15:44:27 jortel, can you elaborate on why you're advocating high priority and not adding to the sprint? 15:45:21 i think they are separate things. not all high prio needs to be rush onto a sprint. 15:45:45 but should be considered first for future sprints 15:45:47 Agreed. I'm just looking for any kind of explanation of what you're thinking along both lines. :) 15:46:48 i'm concerned med/high it will not be visible enough. higher vis will help ensure it will be fixed before blocking anything. 15:46:55 (I'm also in another meeting), but I think this will block 2.13 15:47:40 I think we're all in agreement that it blocks 2.13 15:47:46 yep 15:48:14 cool, +1 to sprint for that 15:48:18 I'm ok with high prio and not on the sprint, or med prio and add it to the sprint. 15:48:29 I'm -0 for adding to the sprint. I think next sprint would be fine. 15:48:44 the thing is that when we upgrade to 3.1.20 pulp-smash is going to break 15:48:59 Fine with me. That gets back to what the current impact is of the bug, which so far I'm hearing "None". 15:49:00 so the dev fix is easy, but we will then have a new problem 15:50:09 mhrivnak: do you mean impact is none due to it not being released? 15:50:22 We've been on this 10 minutes. I'm ok with virtually anything at this point. :) 15:50:30 lol 15:50:45 FYI, if we were to upgrade to 3.1.17 instead of 3.1.20 it would not break pulp-smash 15:50:52 !propose other triage med/high add to sprint 15:50:52 #idea Proposed for #2589: triage med/high add to sprint 15:50:53 Proposed for #2589: triage med/high add to sprint 15:50:54 bmbouter, yes. If this is causing a problem for someone right now, then that suggests we should prioritize it more highly. 15:51:08 !accept 15:51:08 #agreed triage med/high add to sprint 15:51:08 Current proposal accepted: triage med/high add to sprint 15:51:08 but it would still resolve the issue 15:51:10 #topic pulp not starting properly, apache pulp-manage-db throwing errors - http://pulp.plan.io/issues/2591 15:51:10 3 issues left to triage: 2591, 2592, 2593 15:51:11 Pulp Issue #2591 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:51:12 pulp not starting properly, apache pulp-manage-db throwing errors - http://pulp.plan.io/issues/2591 15:51:40 ^ we can argue more about it after triage 15:52:36 close worksforme? or should we take a closer look at this? 15:53:12 If someone could comment on it to help that person work around the problem, I think that would be fine. 15:53:18 It doesn't seem like we're going to be able to reproduce. 15:53:29 agreed 15:53:31 it looks like user reported upgrade from 2.10 15:53:46 preethi tried from 2.11 15:53:48 bmbouter: I am back 15:54:16 maybe it's a typo 15:54:23 preethi, ^ 15:54:52 ttereshc: i saw that, I can try it today 15:55:10 !propose other needsinfo, preethi to try 2.10 upgrade 15:55:10 #idea Proposed for #2591: needsinfo, preethi to try 2.10 upgrade 15:55:11 Proposed for #2591: needsinfo, preethi to try 2.10 upgrade 15:55:20 +1 15:55:41 works for me. 15:55:43 !accept 15:55:43 #agreed needsinfo, preethi to try 2.10 upgrade 15:55:43 Current proposal accepted: needsinfo, preethi to try 2.10 upgrade 15:55:44 #topic repos.fedorapeople.org repository certificate changed since 2017-02-14 - http://pulp.plan.io/issues/2592 15:55:45 2 issues left to triage: 2592, 2593 15:55:45 Pulp Issue #2592 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:55:46 repos.fedorapeople.org repository certificate changed since 2017-02-14 - http://pulp.plan.io/issues/2592 15:56:08 Seems this is el6 only. 15:56:48 And will be difficult/impossible to solve if the linked stackoverflow approach doesn't work. 15:57:23 I lean toward "wontfix". 15:57:47 Especially considering that 2.12, our current bugfix release, isn't affected. 15:58:20 Only thing I'd recommend that isn't "wontfix" is change it to add a note to the install docs 15:58:22 I'm fine with wontfix 15:58:37 But even that is "eh", because it's also reasonable to remove el6 from the install docs starting with 2.12.0 15:59:01 !propose other close wontfix 15:59:01 #idea Proposed for #2592: close wontfix 15:59:02 Proposed for #2592: close wontfix 15:59:16 +1 wontfix 15:59:42 !accept 15:59:42 #agreed close wontfix 15:59:42 Current proposal accepted: close wontfix 15:59:43 1 issues left to triage: 2593 15:59:44 #topic Single-package error during sync is not being propogated up through the CLI - http://pulp.plan.io/issues/2593 15:59:44 Pulp Issue #2593 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:59:44 Single-package error during sync is not being propogated up through the CLI - http://pulp.plan.io/issues/2593 16:00:02 !propose accept 16:00:02 #idea Proposed for #2593: Leave the issue as-is, accepting its current state. 16:00:03 Proposed for #2593: Leave the issue as-is, accepting its current state. 16:00:29 low/low 16:00:49 I assume this is rpm-specific? 16:00:55 I believe so 16:01:28 Gotcha. We can move it then to the "rpm" project in redmine. 16:01:31 !propose other move to pulp_rpm project, triage low/low 16:01:31 #idea Proposed for #2593: move to pulp_rpm project, triage low/low 16:01:31 Proposed for #2593: move to pulp_rpm project, triage low/low 16:01:39 works for me. 16:01:42 +1 16:01:43 +1 16:01:45 one sec 16:01:48 !accept 16:01:48 #agreed move to pulp_rpm project, triage low/low 16:01:48 Current proposal accepted: move to pulp_rpm project, triage low/low 16:01:50 No issues to triage. 16:01:56 isn't this issue in platform 16:02:01 in the CLI of platform 16:02:21 !issue 2593 16:02:22 #topic Single-package error during sync is not being propogated up through the CLI - http://pulp.plan.io/issues/2593 16:02:22 Pulp Issue #2593 [NEW] (unassigned) - Priority: Normal | Severity: Medium 16:02:22 Single-package error during sync is not being propogated up through the CLI - http://pulp.plan.io/issues/2593 16:02:35 +1 to low low, but isn't the progress reporting generic or rpm specific? 16:02:38 pulp_rpm sync has mostly custom CLI rendering. 16:02:48 ok 16:02:57 then this sounds right I thought it was generic 16:03:08 Because it pre-dates the step framework, which gave us a unified progress structure. 16:03:14 oic 16:03:16 ty 16:03:20 do we also want to talk more about #2589 16:03:20 Sure thing! 16:03:20 +1 to the previous resolution 16:03:28 yes please 16:03:35 !issue 2589 16:03:36 #topic Pulp 2.13 nightly fails to start on RHEL 7.3 - http://pulp.plan.io/issues/2589 16:03:36 Pulp Issue #2589 [NEW] (unassigned) - Priority: Normal | Severity: Medium 16:03:37 Pulp 2.13 nightly fails to start on RHEL 7.3 - http://pulp.plan.io/issues/2589 16:04:15 so we know we need to upgrade python-celery 16:04:27 any version of celery > 3.1.11 should work, but versions > 3.1.17 will break pulp-smash 16:04:40 the thing is that fedoras already have 3.1.20 16:04:43 so pulp-smash is already broken 16:04:49 true 16:05:52 that is why I +1 to upgrading to 3.1.20 for consistency on all platforms and have pulp-smash fix whatever the problem is 16:06:08 That sounds reasonable. 16:06:56 give that we want to fix pulp instead of installing a different celery version do we still want to add it to the sprint? 16:08:18 we need to upgrade the celery version and fix pulp-smash 16:08:32 upgrading celery will resolve the bug, but pulp-smash will need to be updated to match 16:09:19 this is a relevant celery issue to the pulp-smash problems 16:09:20 https://github.com/celery/celery/issues/2663 16:09:21 Title: celery 3.1.18 doesn't work w/ Django dummy db · Issue #2663 · celery/celery · GitHub (at github.com) 16:09:50 they don't plan to fix it themselves 16:09:58 I've found the reason why 2.13 jobs are timing out, who was looking into it? 16:10:20 gotcha, I thought it was a pulp celery 3.2.20 issue not a pulp-smash celery issue 16:10:46 ok so we're all good with the preciously propose resolution? 16:11:20 !end 16:11:20 #endmeeting