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