14:30:36 <fabricioo0> #startmeeting Pulp Triage 2019-10-01 14:30:36 <fabricioo0> #info fabricioo0 has joined triage 14:30:36 <fabricioo0> !start 14:30:36 <pulpbot> Meeting started Tue Oct 1 14:30:36 2019 UTC. The chair is fabricioo0. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:30:36 <pulpbot> Useful Commands: #action #agreed #help #info #idea #link #topic. 14:30:36 <pulpbot> The meeting name has been set to 'pulp_triage_2019-10-01' 14:30:36 <pulpbot> fabricioo0: fabricioo0 has joined triage 14:30:47 <daviddavis> #info daviddavis has joined triage 14:30:47 <daviddavis> !here 14:30:47 <pulpbot> daviddavis: daviddavis has joined triage 14:30:50 <bmbouter> #info bmbouter has joined triage 14:30:50 <bmbouter> !here 14:30:50 <pulpbot> bmbouter: bmbouter has joined triage 14:30:59 <ggainey> #info ggainey has joined triage 14:30:59 <ggainey> !here 14:30:59 <pulpbot> ggainey: ggainey has joined triage 14:30:59 <fabricioo0> !next 14:31:00 <pulpbot> fabricioo0: 3 issues left to triage: 5516, 5513, 5511 14:31:01 <fabricioo0> #topic https://pulp.plan.io/issues/5516 14:31:01 <pulpbot> RM 5516 - fabricio.aguiar - POST - plugin managed repositories not found 14:31:02 <ppicka> #info ppicka has joined triage 14:31:02 <ppicka> !here 14:31:03 <pulpbot> https://pulp.plan.io/issues/5516 14:31:04 <pulpbot> ppicka: ppicka has joined triage 14:31:21 <fabricioo0> #idea Proposed for #5516: accept and add to sprint 14:31:21 <fabricioo0> !propose other accept and add to sprint 14:31:21 <pulpbot> fabricioo0: Proposed for #5516: accept and add to sprint 14:31:38 <ttereshc> #info ttereshc has joined triage 14:31:38 <ttereshc> !here 14:31:38 <pulpbot> ttereshc: ttereshc has joined triage 14:31:44 <dkliban> #info dkliban has joined triage 14:31:44 <dkliban> !here 14:31:44 <pulpbot> dkliban: dkliban has joined triage 14:32:00 <daviddavis> +1 14:32:12 <dkliban> +1 14:32:12 <ggainey> fabricioo0: concur 14:32:12 <ppicka> already in post so +1 14:32:14 <mikedep333> #info mikedep333 has joined triage 14:32:14 <mikedep333> !here 14:32:14 <pulpbot> mikedep333: mikedep333 has joined triage 14:32:24 <fabricioo0> #agreed accept and add to sprint 14:32:24 <fabricioo0> !accept 14:32:24 <pulpbot> fabricioo0: Current proposal accepted: accept and add to sprint 14:32:25 <pulpbot> fabricioo0: 2 issues left to triage: 5513, 5511 14:32:25 <fabricioo0> #topic https://pulp.plan.io/issues/5513 14:32:26 <pulpbot> RM 5513 - dkliban@redhat.com - NEW - Task.error includes traceback for fatal download exceptions 14:32:27 <pulpbot> https://pulp.plan.io/issues/5513 14:32:48 <dkliban> i am not sure if this is more of a story 14:32:58 <dkliban> but we should work on this before the GA 14:33:20 <bmbouter> I agree it's a story, we should consider it as a blocker 14:33:31 <ttereshc> yeah, it falls under UX I think 14:33:41 <daviddavis> works for me 14:33:41 <dkliban> #idea Proposed for #5513: convert to story 14:33:41 <dkliban> !propose other convert to story 14:33:41 <pulpbot> dkliban: Proposed for #5513: convert to story 14:33:50 <fabricioo0> #idea Proposed for #5513: change to story and mark as GA blocker 14:33:50 <fabricioo0> !propose other change to story and mark as GA blocker 14:33:50 <pulpbot> fabricioo0: Proposed for #5513: change to story and mark as GA blocker 14:33:57 <ttereshc> +1 14:33:57 <bmbouter> +1 fabricio 14:33:57 <dkliban> +1 14:33:59 <ggainey> concur 14:34:06 <fabricioo0> #agreed change to story and mark as GA blocker 14:34:06 <fabricioo0> !accept 14:34:07 <pulpbot> fabricioo0: Current proposal accepted: change to story and mark as GA blocker 14:34:08 <pulpbot> fabricioo0: 1 issues left to triage: 5511 14:34:08 <fabricioo0> #topic https://pulp.plan.io/issues/5511 14:34:09 <pulpbot> RM 5511 - CodeHeeler - NEW - Failed Task Returns 500 Error 14:34:10 <pulpbot> https://pulp.plan.io/issues/5511 14:35:03 <dalley> #info dalley has joined triage 14:35:03 <dalley> !here 14:35:03 <pulpbot> dalley: dalley has joined triage 14:35:16 <daviddavis> I suspect there is some other condition here as I've been able to query failed tasks but maybe I'm misremembering 14:35:26 <dkliban> i agree 14:35:34 <dkliban> daviddavis: could you comment on the issue? 14:35:46 <dkliban> maybe ask for more specific reproduction steps? 14:35:49 <daviddavis> +1 14:35:51 <bmbouter> +1 14:35:57 <ppicka> +1 for more info 14:35:59 <ggainey> aye sounds reasonable 14:36:01 <dalley> it's failing to deserialize 14:36:04 <ipanova> #info ipanova has joined triage 14:36:04 <ipanova> !here 14:36:04 <pulpbot> ipanova: ipanova has joined triage 14:36:05 <ttereshc> +1 14:36:11 <dalley> fwiw I converted it to native JSONfield recently 14:36:24 <daviddavis> ok, I'll try to reproduce 14:36:25 <dalley> so, that's probably why, I might have missed something 14:36:43 <ggainey> maybe add this discussion to the issue, yeah? 14:36:46 <ggainey> as a pointer 14:36:51 <dalley> yup 14:37:04 <dkliban> yeah ... dawalker is away right now. so it would be good to relay the discussion 14:37:22 <ggainey> zacly 14:37:56 <fabricioo0> #idea Proposed for #5511: ask for more specific reproduction steps 14:37:56 <fabricioo0> !propose other ask for more specific reproduction steps 14:37:57 <pulpbot> fabricioo0: Proposed for #5511: ask for more specific reproduction steps 14:38:03 <ggainey> concur 14:38:18 <fabricioo0> so, in this situation, I just have to skip this issue? 14:38:35 <ttereshc> yeah, I think we have !needinfo 14:38:55 <ttereshc> skip also works 14:38:55 <dalley> I commented on the issue 14:39:02 <fabricioo0> !needinfo 14:39:02 <pulpbot> fabricioo0: Error: "needinfo" is not a valid command. 14:39:11 <fabricioo0> !skip 14:39:12 <pulpbot> fabricioo0: No issues to triage. 14:39:20 <fabricioo0> Open floor! 14:40:42 <dkliban> https://pulp.plan.io/issues/5454 and https://pulp.plan.io/issues/5457 14:40:55 <fabricioo0> !issue 5457 14:40:56 <fabricioo0> #topic https://pulp.plan.io/issues/5457 14:40:56 <pulpbot> RM 5457 - dkliban@redhat.com - NEW - Prepend 'pulp' to fields on Model 14:40:57 <pulpbot> https://pulp.plan.io/issues/5457 14:41:53 <daviddavis> we can add this to the sprint I think 14:42:07 <dkliban> bmbouter: did you have some concerns? 14:42:15 <dkliban> ttereshc? 14:42:29 <ipanova> now i start to recall why we decided to go with _ on the fields 14:42:35 <fabricioo0> #idea Proposed for #5457: add to sprint 14:42:35 <fabricioo0> !propose other add to sprint 14:42:35 <pulpbot> fabricioo0: Proposed for #5457: add to sprint 14:42:47 <ipanova> because most of the people did not like to prepend 'pulp' 14:43:18 <ttereshc> +1 to 5457 14:43:31 <dkliban> ipanova: how do you feel about it now? 14:43:56 <ipanova> i guess as ttereshc and daviddavis on the issue 14:44:06 <ipanova> let's get is done 14:44:08 <dkliban> gotcha 14:44:08 <bmbouter> I couldn't think of anything better 14:44:15 <fabricioo0> #agreed add to sprint 14:44:15 <fabricioo0> !accept 14:44:15 <pulpbot> fabricioo0: Current proposal accepted: add to sprint 14:44:17 <pulpbot> fabricioo0: No issues to triage. 14:44:17 <bmbouter> but we should move away from _ I think 14:44:27 <fabricioo0> !issue 5454 14:44:27 <fabricioo0> #topic https://pulp.plan.io/issues/5454 14:44:28 <pulpbot> RM 5454 - dkliban@redhat.com - NEW - rename _type to resource_type 14:44:29 <pulpbot> https://pulp.plan.io/issues/5454 14:44:48 <dkliban> i was thinking that this should be 'pulp_type' instead of resource_type actually 14:44:58 <dkliban> so it matches the fields from the previous issue 14:45:06 <ggainey> consistency is good :) 14:45:08 <daviddavis> +1 14:45:40 <ttereshc> +1 to pulp_type 14:45:49 <bmbouter> +1 14:45:56 <ttereshc> -0.5 to resource_type, it's confusing to me 14:45:59 <dkliban> i'll update the ticket 14:46:15 <fabricioo0> #idea Proposed for #5454: change to pulp_type and add to sprint 14:46:15 <fabricioo0> !propose other change to pulp_type and add to sprint 14:46:15 <pulpbot> fabricioo0: Proposed for #5454: change to pulp_type and add to sprint 14:46:30 <ttereshc> +1 14:46:55 <fabricioo0> #agreed change to pulp_type and add to sprint 14:46:55 <fabricioo0> !accept 14:46:55 <pulpbot> fabricioo0: Current proposal accepted: change to pulp_type and add to sprint 14:46:56 <ggainey> +1 14:46:57 <pulpbot> fabricioo0: No issues to triage. 14:46:57 <ttereshc> also marked as a GA blocker 14:47:12 <ggainey> oh yes - if twere done, twere best done quickly 14:47:29 <fabricioo0> marked 14:48:01 <daviddavis> I have an issue 14:48:04 <daviddavis> 5511 14:48:17 <daviddavis> dalley is correct here https://pulp.plan.io/issues/5511#note-2 14:48:26 <fabricioo0> !issue 5511 14:48:26 <fabricioo0> #topic https://pulp.plan.io/issues/5511 14:48:26 <daviddavis> I reproduced it and I think we should add to sprint 14:48:26 <pulpbot> RM 5511 - CodeHeeler - NEW - Failed Task Returns 500 Error 14:48:28 <pulpbot> https://pulp.plan.io/issues/5511 14:49:11 <daviddavis> it's a dictfield trying to serialize a jsonfield 14:49:38 <daviddavis> anyway, we need to fix it asap 14:49:43 <dkliban> yep 14:49:45 <bmbouter> yep 14:49:46 <fabricioo0> #idea Proposed for #5511: accept and add to sprint 14:49:46 <fabricioo0> !propose other accept and add to sprint 14:49:46 <pulpbot> fabricioo0: Proposed for #5511: accept and add to sprint 14:49:47 <ggainey> concur 14:50:14 <ttereshc> +1 14:50:40 <fabricioo0> #agreed accept and add to sprint 14:50:40 <fabricioo0> !accept 14:50:40 <pulpbot> fabricioo0: Current proposal accepted: accept and add to sprint 14:50:41 <pulpbot> fabricioo0: No issues to triage. 14:50:57 <daviddavis> I'll add reproducer steps to 5511 14:51:30 <bmbouter> I want to check in on the pulp renaming PRs 14:52:12 <dkliban> sure 14:52:45 <ttereshc> Is it the ones lmjachky was working on? 14:52:50 <bmbouter> yes 14:52:55 <ttereshc> lmjachky, ^ 14:52:57 <bmbouter> and I think they are all posted and ready 14:53:29 <dkliban> yeah 14:53:35 <fabricioo0> #endmeeting 14:53:35 <fabricioo0> !end