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