14:32:18 <asmacdo> #startmeeting Pulp Triage 2019-05-17
14:32:18 <asmacdo> #info asmacdo has joined triage
14:32:19 <pulpbot> Meeting started Fri May 17 14:32:18 2019 UTC.  The chair is asmacdo. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:32:19 <pulpbot> Useful Commands: #action #agreed #help #info #idea #link #topic.
14:32:19 <pulpbot> The meeting name has been set to 'pulp_triage_2019-05-17'
14:32:19 <pulpbot> asmacdo: asmacdo has joined triage
14:32:23 <bmbouter> #info bmbouter has joined triage
14:32:23 <bmbouter> !here
14:32:23 <pulpbot> bmbouter: bmbouter has joined triage
14:32:25 <dalley> #info dalley has joined triage
14:32:25 <dalley> !here
14:32:25 <pulpbot> dalley: dalley has joined triage
14:32:27 <dkliban> #info dkliban has joined triage
14:32:27 <dkliban> !here
14:32:27 <pulpbot> dkliban: dkliban has joined triage
14:32:32 <daviddavis> #info daviddavis has joined triage
14:32:32 <daviddavis> !here
14:32:34 <pulpbot> daviddavis: daviddavis has joined triage
14:32:42 <asmacdo> flake8 makes /me unaware of time :(
14:32:46 <asmacdo> !next
14:32:47 <asmacdo> #topic https://pulp.plan.io/issues/4795
14:32:47 <pulpbot> asmacdo: 11 issues left to triage: 4795, 4825, 4827, 4831, 4833, 4834, 4835, 4836, 4837, 4840, 4841
14:32:48 <pulpbot> RM 4795 - bherring - NEW - Pulp2 task details information returns incorrect values when polled too quickly
14:32:49 <pulpbot> https://pulp.plan.io/issues/4795
14:33:06 <dkliban> we are going to close as wont fix
14:33:20 <dkliban> we discussed in RPM meeting yesterday
14:33:28 <daviddavis> is qe fixing it on their side?
14:33:35 <dkliban> yes, they already did
14:33:38 <daviddavis> great
14:33:58 <asmacdo> even so, what a kickass bug report
14:34:04 <daviddavis> bherring++
14:34:04 <pulpbot> daviddavis: bherring's karma is now 18
14:34:04 <asmacdo> bherring++
14:34:06 <pulpbot> asmacdo: bherring's karma is now 19
14:34:35 <asmacdo> #idea Proposed for #4795: dkliban will close as wontfix
14:34:35 <asmacdo> !propose other dkliban will close as wontfix
14:34:37 <pulpbot> asmacdo: Proposed for #4795: dkliban will close as wontfix
14:34:41 <asmacdo> work for you dkliban?
14:35:00 <dkliban> yep
14:35:47 <asmacdo> #agreed dkliban will close as wontfix
14:35:47 <asmacdo> !accept
14:35:47 <pulpbot> asmacdo: Current proposal accepted: dkliban will close as wontfix
14:35:48 <pulpbot> asmacdo: 10 issues left to triage: 4825, 4827, 4831, 4833, 4834, 4835, 4836, 4837, 4840, 4841
14:35:48 <asmacdo> #topic https://pulp.plan.io/issues/4825
14:35:49 <pulpbot> RM 4825 - jsherril@redhat.com - NEW - syncing with ssl client certs and ca cert fail
14:35:50 <pulpbot> https://pulp.plan.io/issues/4825
14:35:52 <daviddavis> accept and add to sprint
14:36:00 <asmacdo> #idea Proposed for #4825: accept and add to sprint
14:36:00 <asmacdo> !propose other accept and add to sprint
14:36:00 <pulpbot> asmacdo: Proposed for #4825: accept and add to sprint
14:36:57 <dkliban> +1
14:37:01 <asmacdo> is the ssl stuff actually in the file plugin, or is this a pulpcore problem?
14:37:02 <ggainey> #info ggainey has joined triage
14:37:02 <ggainey> !here
14:37:02 <pulpbot> ggainey: ggainey has joined triage
14:38:10 <dkliban> it's in pulpcore
14:38:11 <asmacdo> if this is a pulpcore problem, should it block rc2?
14:38:19 <asmacdo> imo yes?
14:38:27 <dkliban> asmacdo: no
14:38:32 <dkliban> it's been broken
14:38:32 <daviddavis> is it a regression?
14:38:42 <daviddavis> +1 to not blocking
14:39:07 <asmacdo> #idea Proposed for #4825: accept, add to sprint, switch to pulp
14:39:07 <asmacdo> !propose other accept, add to sprint, switch to pulp
14:39:07 <pulpbot> asmacdo: Proposed for #4825: accept, add to sprint, switch to pulp
14:39:42 <dawalker> #info dawalker has joined triage
14:39:42 <dawalker> !here
14:39:42 <pulpbot> dawalker: dawalker has joined triage
14:40:00 <dkliban> daviddavis: it actually worked for me when i tested with RPM plugin ... so i need to talk to jsherrill and see what he is doing
14:40:06 <dkliban> different
14:40:08 <daviddavis> ah ok
14:40:21 <asmacdo> so accept?
14:40:46 <daviddavis> accept and add to sprint
14:40:56 <asmacdo> #agreed accept, add to sprint, switch to pulp
14:40:56 <asmacdo> !accept
14:40:56 <pulpbot> asmacdo: Current proposal accepted: accept, add to sprint, switch to pulp
14:40:57 <pulpbot> asmacdo: 9 issues left to triage: 4827, 4831, 4833, 4834, 4835, 4836, 4837, 4840, 4841
14:40:57 <asmacdo> #topic https://pulp.plan.io/issues/4827
14:40:58 <pulpbot> RM 4827 - sajha - NEW - Add artifact sha details to the content API
14:40:59 <pulpbot> https://pulp.plan.io/issues/4827
14:41:25 <daviddavis> convert to task and add to sprint?
14:41:36 <dkliban> seems like a story
14:41:47 <daviddavis> story works
14:41:50 <asmacdo> #idea Proposed for #4827: convert to story, switch to pulp, add to sprint
14:41:50 <asmacdo> !propose other convert to story, switch to pulp, add to sprint
14:41:50 <pulpbot> asmacdo: Proposed for #4827: convert to story, switch to pulp, add to sprint
14:41:55 <dkliban> +1
14:41:56 <bmbouter> +1
14:42:01 <daviddavis> switch to pulp?
14:42:13 <bmbouter> +1
14:42:18 <daviddavis> I was thinking this was file specific
14:42:45 <dkliban> i think what is being suggested is that all content return digests for all its artifacts
14:43:03 <asmacdo> imo, this will be necessary for all plugins
14:43:07 <daviddavis> there may be content units with multiple or no artifacts. I supopse we could do it for all single artifact content units but I would err on the side of letting plugin authors decide.
14:43:19 <bmbouter> also who provides the serializers?
14:43:29 <asmacdo> the content serializers are master/detail
14:43:31 <dkliban> both pulpcore and plugin
14:43:48 <dkliban> so there is an opportunity to add this to pulpcore so all plugins get it
14:43:49 <bmbouter> ic so we could add to the master
14:43:54 <daviddavis> we could always add these fields to pulpcore later. much harder to remove them later though.
14:44:02 <dkliban> fair enough
14:44:05 <bmbouter> true
14:44:05 <dkliban> let's start with file
14:44:10 <asmacdo> +1
14:44:13 <bmbouter> +1
14:44:15 <bmbouter> it's safe
14:44:18 <asmacdo> #idea Proposed for #4827: convert to story add to sprint
14:44:18 <asmacdo> !propose other convert to story add to sprint
14:44:18 <pulpbot> asmacdo: Proposed for #4827: convert to story add to sprint
14:44:26 <dkliban> +1
14:44:37 <asmacdo> #agreed convert to story add to sprint
14:44:37 <asmacdo> !accept
14:44:37 <pulpbot> asmacdo: Current proposal accepted: convert to story add to sprint
14:44:38 <asmacdo> #topic https://pulp.plan.io/issues/4831
14:44:38 <pulpbot> asmacdo: 8 issues left to triage: 4831, 4833, 4834, 4835, 4836, 4837, 4840, 4841
14:44:39 <pulpbot> RM 4831 - daviddavis - NEW - As a user, I have docs on how to delete content
14:44:40 <pulpbot> https://pulp.plan.io/issues/4831
14:45:00 <daviddavis> convert to story or task
14:45:10 <dkliban> +1
14:45:13 <bmbouter> +1
14:45:21 <asmacdo> +1, and this will need grooming before we can act on it
14:45:43 <asmacdo> #idea Proposed for #4831: convert to story
14:45:43 <asmacdo> !propose other convert to story
14:45:43 <pulpbot> asmacdo: Proposed for #4831: convert to story
14:46:18 <dkliban> +1
14:46:43 <asmacdo> i am also marking it as a sprint candidate so we dont lose it
14:47:04 <asmacdo> #agreed convert to story
14:47:04 <asmacdo> !accept
14:47:04 <pulpbot> asmacdo: Current proposal accepted: convert to story
14:47:05 <pulpbot> asmacdo: 7 issues left to triage: 4833, 4834, 4835, 4836, 4837, 4840, 4841
14:47:06 <asmacdo> #topic https://pulp.plan.io/issues/4833
14:47:06 <pulpbot> RM 4833 - both0 - NEW - roles not found when running example-use/playbook.yml
14:47:08 <pulpbot> https://pulp.plan.io/issues/4833
14:47:31 <asmacdo> playbooks need a relative path to the roles if the roles aren't in the same dir as the playbook
14:47:39 <asmacdo> #idea Proposed for #4833: accept and add to sprint
14:47:39 <asmacdo> !propose other accept and add to sprint
14:47:39 <pulpbot> asmacdo: Proposed for #4833: accept and add to sprint
14:48:10 <bmbouter> +1
14:48:40 <dkliban> +1
14:48:40 <asmacdo> #agreed accept and add to sprint
14:48:40 <asmacdo> !accept
14:48:40 <pulpbot> asmacdo: Current proposal accepted: accept and add to sprint
14:48:41 <pulpbot> asmacdo: 6 issues left to triage: 4834, 4835, 4836, 4837, 4840, 4841
14:48:42 <asmacdo> #topic https://pulp.plan.io/issues/4834
14:48:43 <pulpbot> RM 4834 - daviddavis - NEW - File directories between pulp 2 and pulp 3 conflict
14:48:44 <pulpbot> https://pulp.plan.io/issues/4834
14:48:54 <dkliban> accept and add to sprint
14:49:41 <asmacdo> this should be a story, and we probably need to get agreement on the solution before we can go
14:50:26 <daviddavis> I don't think of this as a story but agreed that we need a solution
14:50:59 <asmacdo> "As a user, I can configure published directory"
14:51:21 <daviddavis> hmm, is that the solution here though?
14:51:24 <daviddavis> it's an option
14:51:34 <dkliban> let's accept and go to the next one
14:51:43 <daviddavis> ok
14:51:47 <dkliban> we don't have to add to sprint now
14:51:53 <dkliban> but someone needs to drive this forward
14:51:54 <bmbouter> I'll add a question/comment too
14:52:01 <asmacdo> sounds good to me
14:52:03 <daviddavis> cool
14:52:04 <asmacdo> !accept
14:52:04 <pulpbot> asmacdo: No action proposed, nothing to accept.
14:52:09 <daviddavis> heh
14:52:18 <asmacdo> #idea Proposed for #4834: Leave the issue as-is, accepting its current state.
14:52:18 <asmacdo> !propose accept
14:52:19 <pulpbot> asmacdo: Proposed for #4834: Leave the issue as-is, accepting its current state.
14:52:20 <asmacdo> #agreed Leave the issue as-is, accepting its current state.
14:52:20 <asmacdo> !accept
14:52:20 <pulpbot> asmacdo: Current proposal accepted: Leave the issue as-is, accepting its current state.
14:52:21 <asmacdo> #topic https://pulp.plan.io/issues/4835
14:52:21 <pulpbot> asmacdo: 5 issues left to triage: 4835, 4836, 4837, 4840, 4841
14:52:22 <pulpbot> RM 4835 - dkliban@redhat.com - NEW - RPM package can be created without specifying the 'relative_path'
14:52:23 <pulpbot> https://pulp.plan.io/issues/4835
14:52:30 <asmacdo> !skip
14:52:31 <pulpbot> asmacdo: 4 issues left to triage: 4836, 4837, 4840, 4841
14:52:32 <asmacdo> #topic https://pulp.plan.io/issues/4836
14:52:32 <pulpbot> RM 4836 - dkliban@redhat.com - ASSIGNED - Docs are incomplete for the one shot upload API
14:52:34 <pulpbot> https://pulp.plan.io/issues/4836
14:52:49 <asmacdo> !skip
14:52:49 <asmacdo> #topic https://pulp.plan.io/issues/4837
14:52:50 <pulpbot> asmacdo: 3 issues left to triage: 4837, 4840, 4841
14:52:51 <pulpbot> RM 4837 - daviddavis - NEW - Prevent users from creating repo version distributions without repo and without repo versions
14:52:52 <pulpbot> https://pulp.plan.io/issues/4837
14:53:12 <daviddavis> easyfix
14:53:23 <asmacdo> daviddavis: isnt it ok to create a distribution before it has anything to distribute?
14:53:53 <daviddavis> hmm
14:54:05 <dkliban> yeah
14:54:07 <daviddavis> what's the use case?
14:54:45 <dkliban> i agree ... it only confuses the user
14:54:48 <asmacdo> to stake out the namespace i guees
14:55:00 <daviddavis> ah ok, I could be convinced either way I suppose
14:55:03 <dkliban> haha
14:55:11 <bmbouter> hmmmm
14:55:22 <asmacdo> maybe a user wants to create dev, testing, prod, but they don't want to promote to testing or prod yet
14:55:43 <bmbouter> asmacdo: I see what you're saying
14:55:51 <bmbouter> or maybe needs to take something offline quickly...
14:55:57 <dkliban> yeah
14:56:05 <dkliban> and not delete the distribution
14:56:13 <daviddavis> that makes sense to me. let's close this as wontfix.
14:56:16 <daviddavis> I can comment
14:56:20 <dkliban> thank you daviddavis
14:56:27 <asmacdo> #idea Proposed for #4837: daviddavis will close-wontfix and comment
14:56:27 <asmacdo> !propose other daviddavis will close-wontfix and comment
14:56:27 <pulpbot> asmacdo: Proposed for #4837: daviddavis will close-wontfix and comment
14:56:44 <asmacdo> #agreed daviddavis will close-wontfix and comment
14:56:44 <asmacdo> !accept
14:56:44 <pulpbot> asmacdo: Current proposal accepted: daviddavis will close-wontfix and comment
14:56:45 <pulpbot> asmacdo: 2 issues left to triage: 4840, 4841
14:56:46 <asmacdo> #topic https://pulp.plan.io/issues/4840
14:56:46 <pulpbot> RM 4840 - ipanova@redhat.com - NEW -  Prevent users from creating publication distributions without a publication
14:56:47 <pulpbot> https://pulp.plan.io/issues/4840
14:56:54 <daviddavis> same
14:56:56 <bmbouter> same
14:56:57 <asmacdo> #idea Proposed for #4840: daviddavis will close-wontfix and comment
14:56:57 <asmacdo> !propose other daviddavis will close-wontfix and comment
14:56:57 <pulpbot> asmacdo: Proposed for #4840: daviddavis will close-wontfix and comment
14:56:59 <asmacdo> #agreed daviddavis will close-wontfix and comment
14:56:59 <asmacdo> !accept
14:56:59 <pulpbot> asmacdo: Current proposal accepted: daviddavis will close-wontfix and comment
14:57:00 <asmacdo> #topic https://pulp.plan.io/issues/4841
14:57:00 <pulpbot> asmacdo: 1 issues left to triage: 4841
14:57:01 <pulpbot> RM 4841 - amacdona@redhat.com - NEW - Deprecation warnings for Ansible installer
14:57:02 <pulpbot> https://pulp.plan.io/issues/4841
14:57:33 <asmacdo> i haven't looked into this closely, but our travis for ansible-pulp is spammed to hell
14:57:42 <dkliban> accept and add to sprint
14:57:42 <asmacdo> #idea Proposed for #4841: accept and add to sprint
14:57:42 <asmacdo> !propose other accept and add to sprint
14:57:42 <pulpbot> asmacdo: Proposed for #4841: accept and add to sprint
14:57:45 <bmbouter> we should fix
14:57:47 <bmbouter> +1
14:57:49 <dkliban> +1
14:57:52 <asmacdo> #agreed accept and add to sprint
14:57:52 <asmacdo> !accept
14:57:52 <pulpbot> asmacdo: Current proposal accepted: accept and add to sprint
14:57:53 <pulpbot> asmacdo: No issues to triage.
14:58:08 <asmacdo> open floor, if anyone has something they'd like to discuss
14:58:14 <daviddavis> 4832
14:58:33 <asmacdo> !4832
14:58:33 <pulpbot> asmacdo: Error: "4832" is not a valid command.
14:58:38 <asmacdo> !issue 4832
14:58:38 <asmacdo> #topic https://pulp.plan.io/issues/4832
14:58:39 <pulpbot> RM 4832 - daviddavis - NEW - As a user, I can filter all repo versions by content unit
14:58:40 <pulpbot> https://pulp.plan.io/issues/4832
14:58:45 <daviddavis> this is a usability problem. suppose you have a content unit you want to delete, there's no way to see what repo versions its in currently
14:59:00 <daviddavis> well you can filter repo versions by repo but it's nested
14:59:14 <bmbouter> this is a big usability problem
14:59:18 <bmbouter> I agree
14:59:37 <daviddavis> my first thought was to create a repo version endpoint (not nested)
14:59:56 <bmbouter> true
15:00:03 <daviddavis> or we could create a /content/.../<id>/repository_versions/ endpoint?
15:00:05 <asmacdo> daviddavis: lets say that you can find all versions with a content unit in it
15:00:58 <asmacdo> what do you do next? deleting those version will squash. you could create a new version without the content and squash, but that whole thing feels very awkward
15:01:27 <daviddavis> it does. do you have an alternate idea?
15:01:36 <asmacdo> sort of
15:01:55 <asmacdo> maybe you shouldn't need to delete a content unit
15:02:06 <asmacdo> if we can disable the distribution of it
15:02:37 <dkliban> this needs to be converted to a story
15:02:41 <dkliban> and we need to discuss this some more
15:02:45 <daviddavis> it is a story
15:02:54 <daviddavis> definitely need to discuss more
15:02:59 <bmbouter> agreed
15:03:08 <asmacdo> i figure, its better to move forward to fix a problem, rather than having to rewrite history
15:03:40 <daviddavis> yea
15:03:45 <daviddavis> we can discuss on the issue
15:03:48 <bmbouter> yeah
15:03:59 <asmacdo> cool, ill make a note to bring up the idea.
15:04:22 <bmbouter> this is a good file, it's great we can discuss it
15:04:55 <asmacdo> any more on that one, or just a "please comment on this one"
15:04:55 <bmbouter> I have a few random/related ideas to comment on the issue w/ also
15:05:05 <daviddavis> +1
15:05:16 <daviddavis> I think that's all for 4832
15:06:08 <bmbouter> are all plugins rc2 compat ready for monday?
15:06:29 <bmbouter> I know ansible, rpm, deb, and file are
15:07:09 <dkliban> maven is not
15:07:22 <dkliban> but i plan to do that on tuesday
15:07:26 <dalley> python?
15:07:26 <asmacdo> python is ready
15:07:42 <daviddavis> docker?
15:07:43 <gmbnomis> pulp_cookbook as well :-)
15:07:44 <asmacdo> i think docker has some distribution stuff left to go
15:08:02 <dkliban> docker should be ready by end of monday
15:08:38 <bmbouter> gmbnomis, dkliban: we have the lingering issue of the "what is configurable" in the Handler I think that is going to be a small breaking change
15:08:57 <bmbouter> but we need to plan it so I think we will need to do that as part of rc3 and only cookbook and docker would be affected
15:09:09 <dkliban> ype
15:09:12 <bmbouter> I need to file a story for that issue so we can track it (I will today)
15:09:12 <dkliban> and i am fine with that
15:09:20 <gmbnomis> me too
15:09:33 <bmbouter> ok I'll link you all to it and we won't plan to include in rc2
15:09:35 <bmbouter> ty!
15:10:01 <daviddavis> bmbouter: https://pulp.plan.io/issues/4834#note-4
15:11:38 <asmacdo> if theres nothing else, we can call it
15:11:46 <bmbouter> daviddavis: https://pulp.plan.io/issues/4834#note-5
15:11:49 <bmbouter> +1 al it
15:11:50 <bmbouter> call
15:11:52 <asmacdo> #endmeeting
15:11:52 <asmacdo> !end