14:32:16 <daviddavis> #startmeeting Pulp Triage 2018-08-31
14:32:16 <daviddavis> #info daviddavis has joined triage
14:32:17 <pulpbot> Meeting started Fri Aug 31 14:32:16 2018 UTC and is due to finish in 60 minutes.  The chair is daviddavis. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:32:17 <pulpbot> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:32:17 <pulpbot> The meeting name has been set to 'pulp_triage_2018_08_31'
14:32:17 <pulpbot> daviddavis: daviddavis has joined triage
14:32:23 <bmbouter> #info bmbouter has joined triage
14:32:23 <bmbouter> !here
14:32:24 <pulpbot> bmbouter: bmbouter has joined triage
14:32:24 <ipanova> #info ipanova has joined triage
14:32:24 <ipanova> !here
14:32:25 <pulpbot> ipanova: ipanova has joined triage
14:33:22 <daviddavis> one more person?
14:33:27 <ttereshc> #info ttereshc has joined triage
14:33:27 <ttereshc> !here
14:33:27 <pulpbot> ttereshc: ttereshc has joined triage
14:33:36 <daviddavis> ttereshc: welcome back :)
14:33:39 <daviddavis> !next
14:33:40 <daviddavis> #topic The ArtifactFileField is deleting source files. - http://pulp.plan.io/issues/3915
14:33:41 <pulpbot> daviddavis: 10 issues left to triage: 3915, 3943, 3949, 3951, 3952, 3955, 3956, 3957, 3958, 3960
14:33:42 <pulpbot> Issue #3915 [NEW] (unassigned) - Priority: High | Severity: High
14:33:43 <ttereshc> :)
14:33:44 <pulpbot> The ArtifactFileField is deleting source files. - http://pulp.plan.io/issues/3915
14:33:44 <bmbouter> ttereshc is baaaack!
14:33:49 <daviddavis> lol
14:34:09 <daviddavis> jortel is out, should we skip this one?
14:34:35 <bmbouter> same as last time, I think this needs to be connected to the django issue for more info
14:34:36 <bmbouter> +1 skip
14:34:42 <daviddavis> !skip
14:34:43 <daviddavis> #topic Cannot drive conf file from environment variables - http://pulp.plan.io/issues/3943
14:34:44 <pulpbot> daviddavis: 9 issues left to triage: 3943, 3949, 3951, 3952, 3955, 3956, 3957, 3958, 3960
14:34:45 <pulpbot> Issue #3943 [NEW] (unassigned) - Priority: Normal | Severity: Medium
14:34:46 <pulpbot> Cannot drive conf file from environment variables - http://pulp.plan.io/issues/3943
14:34:57 <dalley> #info dalley has joined triage
14:34:57 <dalley> !here
14:34:57 <pulpbot> dalley: dalley has joined triage
14:35:13 <daviddavis> should this be a story?
14:35:19 <bmbouter> yeah I think it should be
14:35:20 <dkliban> i think so
14:35:25 <dalley> should I paste my email there too?
14:35:38 <bmbouter> also mhrivnak's option (1) I think is good so I want to comment w/ that
14:35:42 <bmbouter> +1 story
14:35:44 <ttereshc> +1 to a story
14:36:12 <daviddavis> ok, I'll convert to a story. maybe we can get that groomed for next sprint if needed.
14:36:19 <daviddavis> #idea Proposed for #3943: convert to story
14:36:19 <daviddavis> !propose other convert to story
14:36:20 <pulpbot> daviddavis: Proposed for #3943: convert to story
14:36:24 <ttereshc> +1
14:36:34 <daviddavis> #agreed convert to story
14:36:34 <daviddavis> !accept
14:36:34 <pulpbot> daviddavis: Current proposal accepted: convert to story
14:36:36 <daviddavis> #topic The same package can have two (or more?) assigned metadata definitions - http://pulp.plan.io/issues/3949
14:36:36 <pulpbot> daviddavis: 8 issues left to triage: 3949, 3951, 3952, 3955, 3956, 3957, 3958, 3960
14:36:37 <pulpbot> Issue #3949 [NEW] (unassigned) - Priority: Normal | Severity: Medium
14:36:38 <pulpbot> The same package can have two (or more?) assigned metadata definitions - http://pulp.plan.io/issues/3949
14:36:54 <daviddavis> I need to research this one
14:37:07 <daviddavis> and try to reproduce. so let's skip for now
14:37:36 <bmbouter> +1
14:37:48 <daviddavis> !skip
14:37:49 <daviddavis> #topic Cannot dissociate modulemd-defaults - http://pulp.plan.io/issues/3951
14:37:50 <pulpbot> daviddavis: 7 issues left to triage: 3951, 3952, 3955, 3956, 3957, 3958, 3960
14:37:51 <pulpbot> Issue #3951 [NEW] (jortel@redhat.com) - Priority: Normal | Severity: Medium
14:37:52 <pulpbot> Cannot dissociate modulemd-defaults - http://pulp.plan.io/issues/3951
14:37:53 <ttereshc> daviddavis, I haven't read all the comments yet but i htink it's easy to have duplicated records in primary during incremental publish
14:38:14 <daviddavis> ttereshc: there's code to remove duplicates though I thought
14:38:15 <bmbouter> ohh interesting
14:38:26 <daviddavis> oh incremental publish
14:38:31 <ttereshc> daviddavis, yes, but during normal publish
14:38:39 <ttereshc> so you can tryif force_full helps
14:38:49 <daviddavis> ttereshc: ok, thanks. will look into that
14:38:53 <ttereshc> anyway, +1 to skip for now, just wanted to let you know
14:38:58 <daviddavis> +1, thanks
14:39:22 <daviddavis> I'll ping partha to follow up on this modulemd defaults issue
14:39:27 <daviddavis> sounds like we can't reproduce
14:39:28 <ttereshc> it seems like it should be in the assigned state
14:39:44 <ttereshc> ah ok
14:39:56 <ipanova> daviddavis: i was able to reproduce this only with backslashes actually
14:40:06 <ipanova> but i have not looked into that more
14:40:21 <daviddavis> oh interesting
14:40:31 <ipanova> maybe the mongo syntax has something to do with this? the '$in' operator
14:41:27 <ttereshc> daviddavis, I think jortel is on this issue, just because it was created by copying the other one he as assigned to
14:41:42 <ipanova> yeah
14:42:28 <daviddavis> ok, I can unassign him. I'll ping partha to see if he can try to reproduce if he removes slashes.
14:42:32 <daviddavis> #idea Proposed for #3951: Skip this issue for this triage session.
14:42:32 <daviddavis> !propose skip
14:42:32 <pulpbot> daviddavis: Proposed for #3951: Skip this issue for this triage session.
14:42:56 <ttereshc> +1
14:43:27 <daviddavis> !skip
14:43:27 <bmbouter> like in this:  https://github.com/pulp/pulp_rpm/pull/1175#pullrequestreview-151425212
14:43:28 <pulpbot> daviddavis: 6 issues left to triage: 3952, 3955, 3956, 3957, 3958, 3960
14:43:29 <daviddavis> #topic Using the ErrataRelatedModelSaver can result in creating incomplete content. - http://pulp.plan.io/issues/3952
14:43:29 <pulpbot> Issue #3952 [NEW] (unassigned) - Priority: Normal | Severity: Medium
14:43:30 <pulpbot> Using the ErrataRelatedModelSaver can result in creating incomplete content. - http://pulp.plan.io/issues/3952
14:43:32 <bmbouter> whoops wrong chan
14:43:33 <ipanova> daviddavis: also please ask why they use the backslashes
14:43:46 <daviddavis> +1
14:45:34 <daviddavis> this could maybe be a story
14:45:45 <bmbouter> yeah +1 story
14:45:56 <bmbouter> and I need to comment on it to collab w/ jortel on design
14:46:01 <daviddavis> +1
14:46:10 <daviddavis> mind changing this to a story bmbouter?
14:46:21 <bmbouter> I will
14:46:28 <daviddavis> #idea Proposed for #3952: bmbouter to comment and convert to story
14:46:28 <daviddavis> !propose other bmbouter to comment and convert to story
14:46:28 <pulpbot> daviddavis: Proposed for #3952: bmbouter to comment and convert to story
14:47:03 <daviddavis> #agreed bmbouter to comment and convert to story
14:47:03 <daviddavis> !accept
14:47:03 <pulpbot> daviddavis: Current proposal accepted: bmbouter to comment and convert to story
14:47:03 <ttereshc> I don't mind changing it toa story if the resolution is not postponed
14:47:05 <pulpbot> daviddavis: 5 issues left to triage: 3955, 3956, 3957, 3958, 3960
14:47:05 <daviddavis> #topic Pulp unit tests breaking on F27 - http://pulp.plan.io/issues/3955
14:47:06 <pulpbot> Issue #3955 [NEW] (unassigned) - Priority: High | Severity: Medium
14:47:07 <pulpbot> Pulp unit tests breaking on F27 - http://pulp.plan.io/issues/3955
14:47:11 <ttereshc> looks like a bug to me
14:47:11 <daviddavis> ttereshc: agreed
14:47:17 <partha> daviddavis:reading ....
14:47:48 <daviddavis> partha: we think the problem might be backslashes but we couldn't reproduce
14:47:59 <partha> lemme see
14:48:11 <daviddavis> for 3955, I think we should accept and add to sprint
14:48:56 <ttereshc> is it a blocker?
14:49:35 <daviddavis> PR tests against 2-master in pulp are failing
14:49:40 <daviddavis> so I think yes
14:49:59 <dkliban> yeah ... we need to fix this asap
14:50:00 <bmbouter> +1
14:50:05 <partha> ttereshc: 3955 will affect cv publis, if its a bug in the first place
14:50:20 <daviddavis> #idea Proposed for #3955: accept and add to sprint
14:50:20 <daviddavis> !propose other accept and add to sprint
14:50:20 <pulpbot> daviddavis: Proposed for #3955: accept and add to sprint
14:50:24 <ttereshc> +1
14:50:32 <ipanova> +1
14:50:36 <daviddavis> #agreed accept and add to sprint
14:50:36 <daviddavis> !accept
14:50:36 <pulpbot> daviddavis: Current proposal accepted: accept and add to sprint
14:50:38 <pulpbot> daviddavis: 4 issues left to triage: 3956, 3957, 3958, 3960
14:50:38 <daviddavis> #topic on_demand repositories should not allow custom checksum values - http://pulp.plan.io/issues/3956
14:50:39 <pulpbot> Issue #3956 [NEW] (unassigned) - Priority: Normal | Severity: Medium
14:50:40 <pulpbot> on_demand repositories should not allow custom checksum values - http://pulp.plan.io/issues/3956
14:51:42 <ttereshc> it's probably as story and some discussion will be helpful
14:51:47 <dkliban> yeah
14:51:55 <ttereshc> I recognise the problem
14:51:57 <dkliban> let's convert to a story and discuss on th eissue
14:52:05 <ttereshc> +1
14:52:12 <daviddavis> #idea Proposed for #3956: convert to story
14:52:12 <daviddavis> !propose other convert to story
14:52:13 <pulpbot> daviddavis: Proposed for #3956: convert to story
14:52:20 <dkliban> +1
14:52:30 <daviddavis> #agreed convert to story
14:52:30 <daviddavis> !accept
14:52:31 <pulpbot> daviddavis: Current proposal accepted: convert to story
14:52:32 <pulpbot> daviddavis: 3 issues left to triage: 3957, 3958, 3960
14:52:32 <daviddavis> #topic  on_demand repositories should not allow remove_missing to be false - http://pulp.plan.io/issues/3957
14:52:33 <pulpbot> Issue #3957 [NEW] (unassigned) - Priority: Normal | Severity: Medium
14:52:34 <pulpbot> on_demand repositories should not allow remove_missing to be false - http://pulp.plan.io/issues/3957
14:52:59 <daviddavis> this one also makes sense to me
14:53:16 <ipanova> +1
14:53:41 <ttereshc> yeah but again with all the switches between policies...
14:53:56 <ttereshc> if user switches to on_demand temporarily
14:53:58 <dkliban> yep
14:54:10 <dkliban> let's discuss on the issue
14:54:10 <ttereshc> that's what katello does today if I'm not mistaken
14:54:16 <ttereshc> +1
14:54:26 <daviddavis> can someone be responsible for leading these two issues?
14:54:32 <daviddavis> I just want to make sure we follow up
14:54:34 <ttereshc> I can be
14:54:39 <daviddavis> ttereshc++
14:54:39 <pulpbot> daviddavis: ttereshc's karma is now 40
14:54:47 <daviddavis> #idea Proposed for #3957: Skip this issue for this triage session.
14:54:47 <daviddavis> !propose skip
14:54:47 <pulpbot> daviddavis: Proposed for #3957: Skip this issue for this triage session.
14:54:58 <daviddavis> #idea Proposed for #3957: ttereshc to follow up
14:54:58 <daviddavis> !propose other ttereshc to follow up
14:54:58 <pulpbot> daviddavis: Proposed for #3957: ttereshc to follow up
14:55:05 <ttereshc> +1
14:55:09 <daviddavis> !accept
14:55:09 <daviddavis> #agreed ttereshc to follow up
14:55:09 <pulpbot> daviddavis: Current proposal accepted: ttereshc to follow up
14:55:11 <daviddavis> #topic Viewset docstring written from developer perspective - http://pulp.plan.io/issues/3958
14:55:11 <pulpbot> daviddavis: 2 issues left to triage: 3958, 3960
14:55:12 <pulpbot> Issue #3958 [NEW] (unassigned) - Priority: Normal | Severity: Medium
14:55:13 <pulpbot> Viewset docstring written from developer perspective - http://pulp.plan.io/issues/3958
14:56:34 <dkliban> maybe this should be a story to fix this up?>
14:57:09 <daviddavis> what would the story be?
14:57:20 <daviddavis> I was thinking maybe more of an issue or task
14:57:26 <dkliban> task!
14:57:35 <ttereshc> +1 to a task
14:57:43 <dkliban> we need to decide how teh plugin writers will document their APIs
14:57:48 <ttereshc> #idea Proposed for #3958: convert to a task
14:57:48 <ttereshc> !propose other convert to a task
14:57:48 <pulpbot> ttereshc: Proposed for #3958: convert to a task
14:57:51 <daviddavis> +1
14:58:07 <daviddavis> #agreed convert to a task
14:58:07 <daviddavis> !accept
14:58:07 <pulpbot> daviddavis: Current proposal accepted: convert to a task
14:58:08 <pulpbot> daviddavis: 1 issues left to triage: 3960
14:58:08 <daviddavis> #topic Add support to handle pagination for tags/list endpoint - http://pulp.plan.io/issues/3960
14:58:09 <pulpbot> Issue #3960 [NEW] (unassigned) - Priority: Normal | Severity: Medium
14:58:10 <pulpbot> Add support to handle pagination for tags/list endpoint - http://pulp.plan.io/issues/3960
14:58:29 <ttereshc> #idea Proposed for #3960: convert to a story
14:58:29 <ttereshc> !propose other convert to a story
14:58:29 <pulpbot> ttereshc: Proposed for #3960: convert to a story
14:59:11 <ipanova> i have written this in a way it is treated as story but in reality it is a gap
14:59:26 <daviddavis> I kind of expect endpoints like this to have pagination
14:59:51 <daviddavis> therefore it seems like a bug to me
15:00:05 <ipanova> daviddavis: you'd be surprised but until now i have not seen a registry that supports it, heh
15:00:14 <daviddavis> ha
15:00:17 <dkliban> yeah
15:00:53 <daviddavis> ipanova: is there a stakeholder waiting on this?
15:01:00 <ttereshc> I'm fine with a bug, it's just that we never supported pagination before
15:01:02 <ipanova> i suggest to add on sprint as well
15:01:11 <ipanova> daviddavis: yes
15:01:20 <dkliban> +1 to adding to sprint
15:01:20 <ipanova> sat is not able to sync from  quay
15:01:32 <ipanova> io mean fully sync
15:01:38 <ipanova> it syncs just 50 tags
15:01:45 <daviddavis> !propose other accept and add to sprint
15:01:45 <daviddavis> #idea Proposed for #3960: accept and add to sprint
15:01:45 <pulpbot> daviddavis: Proposed for #3960: accept and add to sprint
15:01:49 <bmbouter> we should be able to sync from quay.io
15:02:05 <ttereshc> +1
15:02:15 <asmacdo> +1 story +1 sprint
15:02:20 * asmacdo is back online
15:02:27 <ipanova> asmacdo: is baaaaack
15:02:31 <daviddavis> ha
15:02:48 <daviddavis> I'll let whoever fight over issue vs story
15:03:00 <daviddavis> I'll just add it to the sprint
15:03:02 <asmacdo> -1 fight over it, either way is fine
15:03:10 <daviddavis> cool
15:03:13 <daviddavis> #agreed accept and add to sprint
15:03:13 <daviddavis> !accept
15:03:13 <pulpbot> daviddavis: Current proposal accepted: accept and add to sprint
15:03:15 <pulpbot> daviddavis: No issues to triage.
15:03:21 <daviddavis> #endmeeting
15:03:21 <daviddavis> !end