15:30:52 #startmeeting Pulp Triage 2018-03-06 15:30:52 Meeting started Tue Mar 6 15:30:52 2018 UTC and is due to finish in 60 minutes. The chair is dalley. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:30:52 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:30:52 The meeting name has been set to 'pulp_triage_2018_03_06' 15:30:52 #info dalley has joined triage 15:30:52 dalley: dalley has joined triage 15:30:57 !here 15:30:57 #info ttereshc has joined triage 15:30:58 ttereshc: ttereshc has joined triage 15:31:01 dalley: found some grammar issues but then again my night job is as an editor :) 15:31:03 !here 15:31:03 #info daviddavis has joined triage 15:31:03 daviddavis: daviddavis has joined triage 15:31:04 !here 15:31:04 #info bmbouter has joined triage 15:31:05 bmbouter: bmbouter has joined triage 15:31:24 !next 15:31:25 dalley: 6 issues left to triage: 3416, 3420, 3424, 3429, 3432, 3433 15:31:25 #topic Pulp Jenkins jobs use the development webserver - http://pulp.plan.io/issues/3416 15:31:26 Issue #3416 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:31:27 Pulp Jenkins jobs use the development webserver - http://pulp.plan.io/issues/3416 15:31:36 !here 15:31:36 #info asmacdo has joined triage 15:31:36 asmacdo: asmacdo has joined triage 15:32:28 task or story? 15:32:44 !here 15:32:44 #info ipanova has joined triage 15:32:44 ipanova: ipanova has joined triage 15:32:51 I think the dev tests are moving off jenkins 15:32:55 !here 15:32:55 #info milan_ has joined triage 15:32:55 milan_: milan_ has joined triage 15:33:12 !here 15:33:12 #info dkliban has joined triage 15:33:13 dkliban: dkliban has joined triage 15:33:18 ichimonji10: is this a QE story/task? 15:33:39 to be a story it should be rewritten as "jenkins jobs should use X webserver instead of the development webserver" 15:34:13 I can rewrite it as a story if that gets it done faster. 15:34:29 Yeah, I guess it's a QE story/task. 15:35:14 The lack of HTTPS usage is a major gap in coverage. 15:35:27 seems dependent on https://pulp.plan.io/issues/2921 15:35:42 or the nginx equivalent 15:35:48 Definitely. 15:36:55 I can see that, so is this where you want to track it and someone from QE will take it as assigned? 15:37:21 or is it moved to github issues? 15:38:03 !here 15:38:03 #info jortel has joined triage 15:38:03 jortel: jortel has joined triage 15:38:14 !propose other convert to task, mark as parent to #2921 15:38:14 #idea Proposed for #3416: convert to task, mark as parent to #2921 15:38:14 dalley: Proposed for #3416: convert to task, mark as parent to #2921 15:38:21 I think it's appropriate for devs to work on that task. 15:38:44 i propose we skipp this issue and discuss after triage 15:38:46 I (or someone else from QE) will if necessary. 15:39:20 yeah there is more to discuss since dev is moving off of jenkins 15:39:27 Jenkins isn't the critical bit here. 15:39:28 ok 15:39:34 #idea Proposed for #3416: Skip this issue for this triage session. 15:39:34 !propose skip 15:39:34 daviddavis: Proposed for #3416: Skip this issue for this triage session. 15:39:55 dalley: not parent, just blocked-by i think 15:41:18 +1 skip for now 15:41:25 +1 15:41:28 +1 15:41:30 +1 15:41:45 +1 to skip 15:41:53 !accept 15:41:53 #agreed Skip this issue for this triage session. 15:41:53 dalley: Current proposal accepted: Skip this issue for this triage session. 15:41:55 #topic Not able to access documentation endpoint - http://pulp.plan.io/issues/3420 15:41:55 dalley: 5 issues left to triage: 3420, 3424, 3429, 3432, 3433 15:41:56 Issue #3420 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:41:57 Not able to access documentation endpoint - http://pulp.plan.io/issues/3420 15:42:16 i think we have a PR to fix this already 15:42:21 dkliban: link? 15:42:25 or at elast temporarily fix it ... 15:42:28 one sec 15:42:56 I reverted the change that created this temporarily but we need a long term solution 15:43:02 oh ok 15:43:15 daviddavis: that's what i was talking about 15:43:23 ah ok 15:43:30 yes and afaik it is already merged 15:43:34 well, this needs to be fixed ASAP in my opinion 15:43:35 so accept? 15:43:41 and add to sprint? 15:43:44 yes 15:43:50 +1 15:44:05 +1 15:44:25 !propose other accept and add to sprint 15:44:25 #idea Proposed for #3420: accept and add to sprint 15:44:25 dalley: Proposed for #3420: accept and add to sprint 15:45:45 !accept 15:45:45 #agreed accept and add to sprint 15:45:45 dalley: Current proposal accepted: accept and add to sprint 15:45:46 dalley: 4 issues left to triage: 3424, 3429, 3432, 3433 15:45:47 #topic FileContent relative_path is not validated - http://pulp.plan.io/issues/3424 15:45:47 Issue #3424 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:45:48 FileContent relative_path is not validated - http://pulp.plan.io/issues/3424 15:47:23 accept 15:47:24 accept and add to sprint? 15:47:26 accept 15:47:38 and add to sprint 15:47:42 +1 15:48:07 though, the priority may be lower than core issues 15:48:07 +1 15:48:10 !propose accept and add to sprint 15:48:10 dalley: propose accept Propose accepting the current issue in its current state. 15:48:19 !accept 15:48:19 dalley: No action proposed, nothing to accept. 15:48:28 jortel: i agree ... but it should be a pretty simple fix 15:48:29 !propose other accept and add to sprint 15:48:29 #idea Proposed for #3424: accept and add to sprint 15:48:29 dalley: Proposed for #3424: accept and add to sprint 15:48:34 +1 15:48:38 !accept 15:48:38 #agreed accept and add to sprint 15:48:38 dalley: Current proposal accepted: accept and add to sprint 15:48:39 dalley: 3 issues left to triage: 3429, 3432, 3433 15:48:39 #topic The "projects" field on the importer should be stored as type "pulpcore.app.fields.JSONField" and should be serialized as JSON - http://pulp.plan.io/issues/3429 15:48:40 Issue #3429 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:48:41 The "projects" field on the importer should be stored as type "pulpcore.app.fields.JSONField" and should be serialized as JSON - http://pulp.plan.io/issues/3429 15:49:00 note: pulp_python issue 15:49:08 dkliban: agreed. just proposing a relative priority 15:49:53 i am not very familiar with the pulpcore.app.fields.JSONField, but since we've got it, this seems right 15:50:14 !propose triage low/medium 15:50:14 dalley: (propose triage [target_release]) -- Propose triage values including priority, severity, and an optional target release. 15:50:26 !propose triage low medium 15:50:26 #idea Proposed for #3429: Priority: Low, Severity: Medium 15:50:26 dalley: Proposed for #3429: Priority: Low, Severity: Medium 15:50:30 +1 15:50:40 !accept 15:50:40 #agreed Priority: Low, Severity: Medium 15:50:40 dalley: Current proposal accepted: Priority: Low, Severity: Medium 15:50:41 dalley: 2 issues left to triage: 3432, 3433 15:50:42 #topic Misleading documentation (or bug?): update a distributor - http://pulp.plan.io/issues/3432 15:50:42 Issue #3432 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:50:43 Misleading documentation (or bug?): update a distributor - http://pulp.plan.io/issues/3432 15:51:54 I suggest to accept as a docs bug 15:52:14 even though I'm not sure what the original idea was 15:52:34 i think this probably used to accept config={} as a default 15:52:34 from the code delta and config is not provided then it's none https://github.com/pulp/pulp/blob/master/server/pulp/server/controllers/distributor.py#L189 15:52:43 that's why in docs it was optional 15:53:13 yeah, so we should `if config is None: config={}` 15:53:33 lets accept as a regular bug, easyfix 15:54:19 +0 15:54:27 same 15:54:52 !propose other accept, easyfix 15:54:52 #idea Proposed for #3432: accept, easyfix 15:54:52 dalley: Proposed for #3432: accept, easyfix 15:54:59 +1 15:55:32 +1 15:55:37 +1 15:55:39 !accept 15:55:39 #agreed accept, easyfix 15:55:39 dalley: Current proposal accepted: accept, easyfix 15:55:41 #topic Travis doesn't log exceptions in pulp when running pulp_smash - http://pulp.plan.io/issues/3433 15:55:41 dalley: 1 issues left to triage: 3433 15:55:42 +1 15:55:43 Issue #3433 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:55:44 Travis doesn't log exceptions in pulp when running pulp_smash - http://pulp.plan.io/issues/3433 15:56:03 +1 to this and adding it to sprint 15:56:06 +1 15:56:09 +1 15:56:10 +1 15:56:13 the travis work is already paying off big 15:56:18 :O 15:56:30 !propose accept and add to sprint 15:56:30 dalley: propose accept Propose accepting the current issue in its current state. 15:56:32 !accept 15:56:32 dalley: No action proposed, nothing to accept. 15:56:35 !end 15:56:35 #endmeeting