14:30:08 #startmeeting Pulp Triage 2019-09-27 14:30:08 !start 14:30:08 #info fabricioo0 has joined triage 14:30:08 Meeting started Fri Sep 27 14:30:08 2019 UTC. The chair is fabricioo0. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:30:08 Useful Commands: #action #agreed #help #info #idea #link #topic. 14:30:08 The meeting name has been set to 'pulp_triage_2019-09-27' 14:30:08 fabricioo0: fabricioo0 has joined triage 14:30:31 #info daviddavis has joined triage 14:30:31 !here 14:30:31 daviddavis: daviddavis has joined triage 14:31:07 !next 14:31:08 #topic https://pulp.plan.io/issues/5510 14:31:08 fabricioo0: 3 issues left to triage: 5510, 5507, 5502 14:31:09 RM 5510 - Skullman - NEW - I would like to have an option to associate key-value pair to repository - former "Notes" field in Pulp 2 14:31:10 https://pulp.plan.io/issues/5510 14:31:10 #info bmbouter has joined triage 14:31:10 !here 14:31:11 bmbouter: bmbouter has joined triage 14:31:17 #info dkliban has joined triage 14:31:17 !here 14:31:17 dkliban: dkliban has joined triage 14:31:33 this is a story 14:31:51 #idea Proposed for #5510: change to story 14:31:51 !propose other change to story 14:31:51 fabricioo0: Proposed for #5510: change to story 14:31:56 we also need to remove 3.0 14:32:02 +1 14:32:04 +1 14:32:16 #idea Proposed for #5510: change to story and remove 3.0 14:32:16 !propose other change to story and remove 3.0 14:32:16 fabricioo0: Proposed for #5510: change to story and remove 3.0 14:32:24 #agreed change to story and remove 3.0 14:32:24 !accept 14:32:24 fabricioo0: Current proposal accepted: change to story and remove 3.0 14:32:25 fabricioo0: 2 issues left to triage: 5507, 5502 14:32:25 #topic https://pulp.plan.io/issues/5507 14:32:26 RM 5507 - dkliban@redhat.com - NEW - content-app always sets Content-Type header to 'application/octet-stream' 14:32:27 does rpm want to do this in its plugin "first"? 14:32:28 https://pulp.plan.io/issues/5507 14:32:40 accept 14:32:49 it's already in POST ... should be at least 14:32:51 bmbouter: sure, or pulp_file 14:32:52 i'll update it 14:33:48 fabricioo0: i updated this issue ... we can go to the next 14:33:56 #idea Proposed for #5507: Leave the issue as-is, accepting its current state. 14:33:56 !propose accept 14:33:56 dkliban: Proposed for #5507: Leave the issue as-is, accepting its current state. 14:34:13 #info ggainey has joined triage 14:34:13 !here 14:34:13 ggainey: ggainey has joined triage 14:35:04 fabricioo0_: i updated the issue and it's ready to be accepted 14:35:15 I had some connectivity issue 14:35:37 #idea Proposed for #5507: Leave the issue as-is, accepting its current state. 14:35:37 #info fabricioo0_ has joined triage 14:35:37 !propose accept 14:35:37 fabricioo0_: fabricioo0_ has joined triage 14:35:38 fabricioo0_: Proposed for #5507: Leave the issue as-is, accepting its current state. 14:36:36 #idea Proposed for #5507: Leave the issue as-is, accepting its current state. 14:36:36 !propose accept 14:36:36 fabricioo0: Proposed for #5507: Leave the issue as-is, accepting its current state. 14:36:46 #agreed Leave the issue as-is, accepting its current state. 14:36:46 !accept 14:36:46 fabricioo0: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:36:46 #topic https://pulp.plan.io/issues/5502 14:36:47 fabricioo0: 1 issues left to triage: 5502 14:36:48 RM 5502 - mihai.ibanescu@gmail.com - NEW - worker fails to mark a task as failed when critical conditions are encountered 14:36:49 https://pulp.plan.io/issues/5502 14:37:23 this needs Pulp 2 label 14:37:29 yeah 14:37:37 yeah ... and we should accept 14:37:41 +1 14:37:43 concur 14:37:52 can we ask him for a patch... maybe? 14:38:02 yea, if not we should maybe close out 14:38:07 #idea Proposed for #5502: add pulp2 label and accept 14:38:07 !propose other add pulp2 label and accept 14:38:07 fabricioo0: Proposed for #5502: add pulp2 label and accept 14:38:25 bmbouter: we can always aask 14:39:42 #agreed add pulp2 label and accept 14:39:42 !accept 14:39:42 fabricioo0: Current proposal accepted: add pulp2 label and accept 14:39:43 fabricioo0: No issues to triage. 14:39:51 Open floor! 14:40:06 I got one 14:40:11 https://pulp.plan.io/issues/5509 14:40:42 !issue 5509 14:40:43 #topic https://pulp.plan.io/issues/5509 14:40:43 RM 5509 - daviddavis - NEW - Can't sync with 3.0.0b5 release of pulp_rpm 14:40:44 https://pulp.plan.io/issues/5509 14:41:45 I think we should release a new core 14:41:52 yes 14:41:56 that will mean releasing new plugins too 14:41:57 let's release a new core 14:42:17 I can work on it monday unless someone has time today 14:42:29 bmbouter: any objection? 14:43:18 #info dawalker has joined triage 14:43:18 !here 14:43:18 dawalker: dawalker has joined triage 14:43:21 #idea Proposed for #5509: Leave the issue as-is, accepting its current state. 14:43:21 !propose accept 14:43:21 fabricioo0: Proposed for #5509: Leave the issue as-is, accepting its current state. 14:44:07 #idea Proposed for #5509: accept and add to sprint 14:44:07 !propose other accept and add to sprint 14:44:08 daviddavis: Proposed for #5509: accept and add to sprint 14:44:27 well bmbouter can object when he gets back 14:44:28 #agreed accept and add to sprint 14:44:28 !accept 14:44:28 fabricioo0: Current proposal accepted: accept and add to sprint 14:44:29 fabricioo0: No issues to triage. 14:44:47 sorry back 14:44:55 +1 14:45:08 Open floor! 14:46:34 daviddavis: we doing core issue stuff here? 14:47:10 bmbouter: sure 14:48:57 here's one https://pulp.plan.io/issues/5442 14:49:07 Replace pulpcore.plugin.models Task with the actual Task from pulpcore 14:49:12 !issue 5442 14:49:12 #topic https://pulp.plan.io/issues/5442 14:49:13 RM 5442 - bmbouter - NEW - Replace pulpcore.plugin.models Task with the actual Task from pulpcore 14:49:14 https://pulp.plan.io/issues/5442 14:49:17 !friday 14:49:17 ♪ It's Friday, Friday, gotta get down on Friday ♪ 14:49:21 lgtm 14:49:26 let's accept + add to sprint 14:49:28 +1 14:49:41 #idea Proposed for #5442: accept and to sprint 14:49:41 !propose other accept and to sprint 14:49:41 fabricioo0: Proposed for #5442: accept and to sprint 14:49:48 #agreed accept and to sprint 14:49:48 !accept 14:49:48 fabricioo0: Current proposal accepted: accept and to sprint 14:49:49 fabricioo0: No issues to triage. 14:50:07 https://pulp.plan.io/issues/5086 14:50:14 that's another one I think is ready 14:50:20 +1 14:50:23 I'm not sure if these are in your six 14:51:23 I may have found a bug, but not sure if it's filed already? Whenever a task completes successfully, you can see the results, but if it fails, I get a 500 error: https://paste.fedoraproject.org/paste/0zkweANxTGxL4j77NC2Zlg 14:51:28 Anyone run into this before? 14:51:42 bmbouter: I didn't have a list of actual issues. just said we would probably put on ~6 issues. 14:51:49 cool 14:51:56 daviddavis, does this look good? https://pulp.plan.io/issues/5502#note-2 14:52:45 dalley: I won't be able to work on it any time soon 14:53:17 dalley: and I don't have enough knowledge of celery to even know how to fix it 14:53:26 any more bugs or should i end this triage? 14:53:26 ok 14:53:36 re https://pulp.plan.io/issues/5509: shameless plug https://pulp.plan.io/issues/4386 :-) 14:53:39 dalley, thank you for your comments, I'll answer after all my meetings 14:53:42 dalley: +1 14:54:10 daviddavis: this is a blocker currently but it's a P3? https://pulp.plan.io/issues/3809#note-11 14:54:27 dalley: I *think* one needs to create a function-scope task class and decorate it with current_app, or something like that 14:54:41 dalley: but I don't know if there's a better way to do it 14:54:53 bmbouter: it probably never got bumped to P1 or P2. cc jsherrill 14:55:06 once we got rid of P4 14:55:24 bmbouter: should we add it to the sprint? 14:55:38 I think so 14:55:51 and I can take as assigned, really the selinux team is going to do the work I was so pleased with their reply 14:57:33 thanks guys, not sure how i missed that a while back when i looked through them all 14:57:49 #endmeeting 14:57:49 !end