15:30:42 !start 15:30:42 #startmeeting Pulp Triage 2018-12-14 15:30:42 #info dawalker has joined triage 15:30:42 I'm making those now 15:30:42 Meeting started Fri Dec 14 15:30:42 2018 UTC and is due to finish in 60 minutes. The chair is dawalker. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:30:42 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:30:42 The meeting name has been set to 'pulp_triage_2018_12_14' 15:30:42 dawalker: dawalker has joined triage 15:30:43 !here 15:30:43 #info bmbouter has joined triage 15:30:44 bmbouter: bmbouter has joined triage 15:30:51 bmbouter: ah ok 15:30:58 #info daviddavis has joined triage 15:30:58 !here 15:30:58 daviddavis: daviddavis has joined triage 15:31:01 #info ttereshc has joined triage 15:31:01 !here 15:31:01 ttereshc: ttereshc has joined triage 15:31:10 #info dkliban has joined triage 15:31:10 !here 15:31:10 dkliban: dkliban has joined triage 15:31:22 #info dalley has joined triage 15:31:22 !here 15:31:22 dalley: dalley has joined triage 15:31:31 !next 15:31:32 dawalker: 3 issues left to triage: 4264, 4266, 4267 15:31:32 #topic 500 IntegrityErrors due to race conditions - http://pulp.plan.io/issues/4264 15:31:33 Issue #4264 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:31:34 500 IntegrityErrors due to race conditions - http://pulp.plan.io/issues/4264 15:32:28 #idea Proposed for #4264: Leave the issue as-is, accepting its current state. 15:32:28 !propose accept 15:32:28 dawalker: Proposed for #4264: Leave the issue as-is, accepting its current state. 15:32:32 #info asmacdo has joined triage 15:32:32 !here 15:32:32 asmacdo: asmacdo has joined triage 15:32:35 +1 15:32:37 +1 15:33:02 can we accept this? i'm not sure what we are supposed to do? 15:33:21 +1 15:33:34 it needs discussion 15:33:36 I think jortel proposed creating a base viewset that can rescue IntegrityError and return 400s 15:33:39 +1 to accept and handle the IntegrityError 15:34:12 sounds good to me, +1 15:34:14 +1 accept for now, have more discussion in the comments to the issue? 15:34:19 #agreed Leave the issue as-is, accepting its current state. 15:34:19 !accept 15:34:19 dawalker: Current proposal accepted: Leave the issue as-is, accepting its current state. 15:34:20 dawalker: 2 issues left to triage: 4266, 4267 15:34:20 #topic The pulp 3 release docs are out of date - http://pulp.plan.io/issues/4266 15:34:21 Issue #4266 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:34:22 The pulp 3 release docs are out of date - http://pulp.plan.io/issues/4266 15:34:49 maybe more of a task but would like to maybe add this to the sprint 15:34:51 or next sprint 15:34:52 accept and add to sprint 15:34:59 should we move these docs from the wiki to the contributor docs? 15:34:59 #idea Proposed for #4266: accept and add to sprint 15:34:59 !propose other accept and add to sprint 15:34:59 dawalker: Proposed for #4266: accept and add to sprint 15:35:09 this seems important to do sooner than later 15:35:16 yep 15:35:23 I don't think this is contributor related. it's specific to pulpcore I think? 15:35:34 oh the first part is plugin specific 15:35:35 we have a stub for build guide https://docs.pulpproject.org/en/3.0/nightly/contributing/build-guide.html 15:35:35 I agree 15:35:37 maybe move that out 15:35:48 +1 accept and add to sprint 15:35:50 +1 15:35:56 #agreed accept and add to sprint 15:35:56 !accept 15:35:56 dawalker: Current proposal accepted: accept and add to sprint 15:35:58 #topic force_full is not honored for distribution units - http://pulp.plan.io/issues/4267 15:35:58 dawalker: 1 issues left to triage: 4267 15:35:59 Issue #4267 [POST] (cduryee) - Priority: Normal | Severity: Medium 15:36:00 force_full is not honored for distribution units - http://pulp.plan.io/issues/4267 15:36:17 accept and add to sprint 15:36:25 #idea Proposed for #4267: accept and add to sprint 15:36:25 !propose other accept and add to sprint 15:36:25 dawalker: Proposed for #4267: accept and add to sprint 15:36:51 +1 15:36:52 there is a PR for that 15:37:06 +1 15:37:09 beav fixed it already 15:37:10 yeah, it's in post 15:37:13 we need a review 15:37:22 i was planning to do that today 15:37:28 I'm not sure if we need to add it to the sprint in that case 15:37:33 #info jortel has joined triage 15:37:33 !here 15:37:33 jortel: jortel has joined triage 15:37:33 yeah 15:37:38 #idea Proposed for #4267: Leave the issue as-is, accepting its current state. 15:37:38 !propose accept 15:37:39 dawalker: Proposed for #4267: Leave the issue as-is, accepting its current state. 15:37:40 none of us is going to work on it 15:37:44 agreed 15:37:45 sounds good 15:37:53 i think we should add to sprint 15:37:54 +1 15:38:13 asmacdo, we used to add work which is for us only 15:38:19 not for external contributors 15:38:25 ^what she said 15:38:26 its helpful to add to sprint for daviddavis and I when we meet with QE 15:38:39 I'm ok both ways 15:38:47 but thats ok too, we might just need to adjust our query 15:39:01 yea hmm 15:39:03 well, it's not like our sprints are really sprints anyway so... 15:39:09 heh 15:39:39 mind just adding it to the sprint, and we can chat about the process later? 15:39:44 sure 15:39:50 #idea Proposed for #4267: accept and add to sprint 15:39:50 !propose other accept and add to sprint 15:39:51 dawalker: Proposed for #4267: accept and add to sprint 15:40:02 look into your query options and get back to us next time 15:40:06 #agreed accept and add to sprint 15:40:06 !accept 15:40:06 dawalker: Current proposal accepted: accept and add to sprint 15:40:07 dawalker: No issues to triage. 15:40:14 #endmeeting 15:40:14 !end