14:30:15 #startmeeting Pulp Triage 2017-08-04 14:30:15 #info ttereshc has joined triage 14:30:15 Meeting started Fri Aug 4 14:30:15 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:15 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:30:15 The meeting name has been set to 'pulp_triage_2017_08_04' 14:30:15 ttereshc: ttereshc has joined triage 14:30:22 pulpbot \o/ 14:30:22 ttereshc: Error: "\o/" is not a valid command. 14:30:28 !here 14:30:28 #info mansari has joined triage 14:30:28 mansari: mansari has joined triage 14:30:29 heh 14:30:31 !here 14:30:31 #info pcreech has joined triage 14:30:31 pcreech: pcreech has joined triage 14:30:33 !here 14:30:33 #info mhrivnak has joined triage 14:30:33 mhrivnak: mhrivnak has joined triage 14:30:43 !next 14:30:44 ttereshc: 3 issues left to triage: 2927, 2952, 2954 14:30:45 #topic Messages in resource manager queue are not persisted after restarting Qpid and cause tasks never start - http://pulp.plan.io/issues/2927 14:30:46 Issue #2927 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:30:47 Messages in resource manager queue are not persisted after restarting Qpid and cause tasks never start - http://pulp.plan.io/issues/2927 14:31:27 !here 14:31:27 #info bmbouter has joined triage 14:31:27 bmbouter: bmbouter has joined triage 14:31:34 !here 14:31:34 #info daviddavis has joined triage 14:31:35 daviddavis: daviddavis has joined triage 14:31:46 daviddavis should we skip again? 14:31:53 so it's important one to fix but it's not a pulp bug, is that correct? 14:32:04 seems like we need to dig into kombu to find the bug? 14:32:37 yeah that is right but all of the celery capacity has been going to another bug 14:32:37 I'm not 100% sure it's not pulp 14:32:50 so that is why this just keeps getting bumped 14:33:09 I think this is ready to accept and get assigned to someone to work on. 14:33:20 +1 from me 14:33:24 that sounds fine 14:33:33 +1 14:33:40 what about prio/sev? 14:33:41 !propose other high high add to sprint 14:33:41 #idea Proposed for #2927: high high add to sprint 14:33:41 mhrivnak: Proposed for #2927: high high add to sprint 14:33:43 hign? 14:33:46 ah :) 14:33:59 !accept 14:33:59 #agreed high high add to sprint 14:33:59 ttereshc: Current proposal accepted: high high add to sprint 14:34:00 ttereshc: 2 issues left to triage: 2952, 2954 14:34:01 #topic Use 'endpoint' in /etc/crane.conf in all response headers - http://pulp.plan.io/issues/2952 14:34:01 Issue #2952 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:34:02 Use 'endpoint' in /etc/crane.conf in all response headers - http://pulp.plan.io/issues/2952 14:34:53 I don't think this is the right change to make. 14:34:55 I think this bug/rfe is no longer needed 14:35:01 Agreed. 14:35:17 thomasmckay fixed his problem by seting the url on the distributor 14:35:19 I suspect Tom agrees it was based on a misunderstanding, but we can confirm. 14:35:27 +1 14:35:33 !propose skip 14:35:33 #idea Proposed for #2952: Skip this issue for this triage session. 14:35:33 mhrivnak: Proposed for #2952: Skip this issue for this triage session. 14:35:34 I'll update the bug 14:35:40 +1 14:35:42 thanks daviddavis 14:35:49 !accept 14:35:49 #agreed Skip this issue for this triage session. 14:35:49 ttereshc: Current proposal accepted: Skip this issue for this triage session. 14:35:50 ttereshc: 1 issues left to triage: 2954 14:35:50 #topic Ensure that queued tasks are not lost by enabling task_reject_on_worker_lost for Celery 4 - http://pulp.plan.io/issues/2954 14:35:51 Issue #2954 [ASSIGNED] (daviddavis@redhat.com) - Priority: Normal | Severity: Medium 14:35:52 Ensure that queued tasks are not lost by enabling task_reject_on_worker_lost for Celery 4 - http://pulp.plan.io/issues/2954 14:37:08 daviddavis: is this the issue we are currently working on? 14:37:08 assigned already, should it be on the sprint? 14:37:18 yeah it should be 14:37:23 yes please. it's a clone of 2861 14:37:25 this was split out from an existing bug 14:37:27 yeah 14:37:29 but with celery 4 14:37:46 is it not a bug with celery 3? 14:38:00 2861 is for celery 3 14:38:09 2954 is for celery 4 14:38:12 but wontfix... 14:38:13 * mhrivnak reads 14:38:23 yea, we'd have to monkeypatch celery 3 to fix it 14:38:29 celery 4 has an option to fix it 14:38:39 Gotcha. 14:38:49 happy to revisit 2861 if anyone thinks it's worth monkeypatching celery 3 14:39:22 or we could try to backport the feature from celery 4 ourselves 14:39:46 !propose other triage normal/med, add to sprint 14:39:46 #idea Proposed for #2954: triage normal/med, add to sprint 14:39:46 ttereshc: Proposed for #2954: triage normal/med, add to sprint 14:40:04 works for me. 14:40:24 !accept 14:40:24 #agreed triage normal/med, add to sprint 14:40:24 ttereshc: Current proposal accepted: triage normal/med, add to sprint 14:40:26 ttereshc: No issues to triage. 14:40:28 !end 14:40:28 #endmeeting