14:30:41 <asmacdo> #startmeeting Pulp Triage 2019-06-25
14:30:41 <asmacdo> #info asmacdo has joined triage
14:30:41 <asmacdo> !start
14:30:41 <pulpbot> Meeting started Tue Jun 25 14:30:41 2019 UTC.  The chair is asmacdo. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:30:41 <pulpbot> Useful Commands: #action #agreed #help #info #idea #link #topic.
14:30:41 <pulpbot> The meeting name has been set to 'pulp_triage_2019-06-25'
14:30:41 <pulpbot> asmacdo: asmacdo has joined triage
14:30:42 <ppicka> !here
14:30:42 <ppicka> #info ppicka has joined triage
14:30:43 <pulpbot> ppicka: ppicka has joined triage
14:30:44 <daviddavis> #info daviddavis has joined triage
14:30:44 <daviddavis> !here
14:30:44 <pulpbot> daviddavis: daviddavis has joined triage
14:30:47 <bmbouter> asmacdo: the names I made are epic confusing. more after 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:52 <ttereshc> #info ttereshc has joined triage
14:30:52 <ttereshc> !here
14:30:52 <pulpbot> ttereshc: ttereshc has joined triage
14:31:03 <dawalker> #info dawalker has joined triage
14:31:03 <dawalker> !here
14:31:04 <pulpbot> dawalker: dawalker has joined triage
14:31:25 <asmacdo> #info asmacdo has joined triage
14:31:25 <asmacdo> !start
14:31:25 <pulpbot> asmacdo: Error: Can't start another meeting, one is in progress.
14:31:26 <pulpbot> asmacdo: asmacdo has joined triage
14:31:30 <asmacdo> !next
14:31:31 <pulpbot> asmacdo: 20 issues left to triage: 4939, 4947, 4959, 4970, 4979, 4989, 4990, 4992, 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:31:31 <asmacdo> #topic https://pulp.plan.io/issues/4939
14:31:32 <pulpbot> RM 4939 - kersom - NEW - Docs - Collections upload workflows is using role endpoints
14:31:33 <pulpbot> https://pulp.plan.io/issues/4939
14:31:36 <daviddavis> skip
14:31:40 <asmacdo> !skip
14:31:41 <pulpbot> asmacdo: 19 issues left to triage: 4947, 4959, 4970, 4979, 4989, 4990, 4992, 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:31:41 <asmacdo> #topic https://pulp.plan.io/issues/4947
14:31:42 <pulpbot> RM 4947 - amacdona@redhat.com - NEW -  As a user I can add tags to a repository by name.
14:31:43 <pulpbot> https://pulp.plan.io/issues/4947
14:31:46 <daviddavis> skip
14:31:48 <asmacdo> !skip
14:31:49 <pulpbot> asmacdo: 18 issues left to triage: 4959, 4970, 4979, 4989, 4990, 4992, 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:31:49 <asmacdo> #topic https://pulp.plan.io/issues/4959
14:31:50 <pulpbot> RM 4959 - amacdona@redhat.com - NEW - Do not log that a download fails with 401 (INFO) unless it fails again after token refresh
14:31:51 <pulpbot> https://pulp.plan.io/issues/4959
14:32:00 <bmbouter> this is so confusing
14:32:08 <daviddavis> skip
14:32:15 <asmacdo> yeah pulpbot is struggling
14:32:24 <asmacdo> dkliban: lets chat after
14:32:27 <asmacdo> !skip
14:32:28 <pulpbot> asmacdo: 17 issues left to triage: 4970, 4979, 4989, 4990, 4992, 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:32:28 <asmacdo> #topic https://pulp.plan.io/issues/4970
14:32:29 <pulpbot> RM 4970 - kersom - NEW - Collection - RepositoryDistribution does not provide url to consume to cosume content from Pulp
14:32:30 <pulpbot> https://pulp.plan.io/issues/4970
14:32:37 <daviddavis> one solution is to switch queries
14:32:44 <daviddavis> skip
14:32:45 <asmacdo> we did, but it didnt take?
14:32:46 <bmbouter> daviddavis: that sounds so easy
14:32:48 <asmacdo> !skip
14:32:49 <pulpbot> asmacdo: 16 issues left to triage: 4979, 4989, 4990, 4992, 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:32:49 <asmacdo> #topic https://pulp.plan.io/issues/4979
14:32:50 <pulpbot> RM 4979 - dgoetz - NEW - Katello complains during reposync about being unable to update existing errate
14:32:51 <pulpbot> https://pulp.plan.io/issues/4979
14:33:04 <asmacdo> !skip
14:33:05 <pulpbot> asmacdo: 15 issues left to triage: 4989, 4990, 4992, 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:33:05 <asmacdo> #topic https://pulp.plan.io/issues/4989
14:33:06 <pulpbot> RM 4989 - mdellweg - ASSIGNED - Api bindings for python use unnecessary verbose action names
14:33:07 <pulpbot> https://pulp.plan.io/issues/4989
14:33:15 <daviddavis> dkliban: add this to the sprint?
14:33:30 <daviddavis> I think this work is done
14:33:34 <asmacdo> this has an associated revision, MODIFIED?
14:33:41 <bmbouter> I think so also
14:33:47 <daviddavis> let's ask dkliban to update it
14:33:48 <dalley> #info dalley has joined triage
14:33:48 <dalley> !here
14:33:48 <pulpbot> dalley: dalley has joined triage
14:33:53 <bmbouter> +1
14:33:58 <ggainey> and add to sprint, yeah?
14:34:04 <daviddavis> +1
14:34:13 <asmacdo> #idea Proposed for #4989: dkliban will update, and add to sprint
14:34:13 <asmacdo> !propose other dkliban will update, and add to sprint
14:34:13 <pulpbot> asmacdo: Proposed for #4989: dkliban will update, and add to sprint
14:34:29 <asmacdo> #agreed dkliban will update, and add to sprint
14:34:29 <asmacdo> !accept
14:34:29 <pulpbot> asmacdo: Current proposal accepted: dkliban will update, and add to sprint
14:34:30 <pulpbot> asmacdo: 14 issues left to triage: 4990, 4992, 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:34:30 <asmacdo> #topic https://pulp.plan.io/issues/4990
14:34:31 <pulpbot> RM 4990 - kersom - NEW - Core's serializer should only validate when policy='immediate' (the default)
14:34:32 <pulpbot> https://pulp.plan.io/issues/4990
14:34:39 <daviddavis> this is on the sprint
14:34:51 <ggainey> yupper
14:34:57 <asmacdo> #idea Proposed for #4990: Leave the issue as-is, accepting its current state.
14:34:57 <asmacdo> !propose accept
14:34:57 <pulpbot> asmacdo: Proposed for #4990: Leave the issue as-is, accepting its current state.
14:35:43 <bherring> #info bherring has joined triage
14:35:43 <bherring> !here
14:35:43 <pulpbot> bherring: bherring has joined triage
14:35:50 <asmacdo> #agreed Leave the issue as-is, accepting its current state.
14:35:50 <asmacdo> !accept
14:35:50 <pulpbot> asmacdo: Current proposal accepted: Leave the issue as-is, accepting its current state.
14:35:51 <pulpbot> asmacdo: 13 issues left to triage: 4992, 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:35:52 <asmacdo> #topic https://pulp.plan.io/issues/4992
14:35:52 <pulpbot> RM 4992 - jsherril@redhat.com - NEW - 'fields'  parameter is not available in api docs/bindings
14:35:53 <pulpbot> https://pulp.plan.io/issues/4992
14:36:27 <bmbouter> this will be an issue w/ pulp's openAPI schema
14:36:36 <daviddavis> yea
14:36:54 <daviddavis> there's no P tag on it so hard to know the priority
14:36:54 <ttereshc> yeah, I thought we triaged this one
14:36:57 <daviddavis> me too
14:37:20 <bmbouter> I'm asking katello devs to set them so we can bring that up at tomorrow's integration meeting also
14:37:27 <daviddavis> +1
14:37:28 <bmbouter> s/them/the P tags/
14:37:29 <ggainey> freenode-#pulp-dev.log:Jun 21 10:40:19 <pulpbot>	ttereshc: Proposed for #4992: accept, add to sprint, add bindings tag
14:37:32 <dkliban> this is a P1
14:37:38 <ttereshc> heh
14:37:49 <jsherrill> sorry, i will set that, for that one its a p3 i'd say
14:38:03 <daviddavis> accept and add to 3.0 milestone then
14:38:08 <ggainey> +1
14:38:15 <bmbouter> +1
14:38:17 <ttereshc> and + bindings tag
14:38:20 <daviddavis> +1
14:38:25 <asmacdo> #idea Proposed for #4992: accept, add to sprint, +3.0 miletone, +bindings tag, +p3, remind austin to hit save this time
14:38:25 <asmacdo> !propose other accept, add to sprint, +3.0 miletone, +bindings tag, +p3, remind austin to hit save this time
14:38:25 <pulpbot> asmacdo: Proposed for #4992: accept, add to sprint, +3.0 miletone, +bindings tag, +p3, remind austin to hit save this time
14:38:31 <ggainey> heh
14:38:32 <daviddavis> heh
14:38:45 <ggainey> asmacdo: hit save :)
14:38:49 <dawalker> +1
14:39:11 <asmacdo> #agreed accept, add to sprint, +3.0 miletone, +bindings tag, +p3, remind austin to hit save this time
14:39:11 <asmacdo> !accept
14:39:11 <pulpbot> asmacdo: Current proposal accepted: accept, add to sprint, +3.0 miletone, +bindings tag, +p3, remind austin to hit save this time
14:39:12 <pulpbot> asmacdo: 12 issues left to triage: 4994, 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:39:12 <asmacdo> #topic https://pulp.plan.io/issues/4994
14:39:13 <pulpbot> RM 4994 - iballou - POST - Combine manifest-list-tag and manifest-tag models in Docker plugin
14:39:14 <pulpbot> https://pulp.plan.io/issues/4994
14:39:18 <daviddavis> skip
14:39:22 <asmacdo> !skip
14:39:23 <pulpbot> asmacdo: 11 issues left to triage: 4996, 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:39:23 <asmacdo> #topic https://pulp.plan.io/issues/4996
14:39:24 <pulpbot> RM 4996 - sajha - NEW - Asible collection remote doesn't handle trailing / in sync url
14:39:25 <pulpbot> https://pulp.plan.io/issues/4996
14:39:28 <daviddavis> skip
14:39:35 <daviddavis> man we got to get this fixed
14:39:40 <asmacdo> !skip
14:39:41 <pulpbot> asmacdo: 10 issues left to triage: 4998, 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:39:41 <asmacdo> #topic https://pulp.plan.io/issues/4998
14:39:42 <pulpbot> RM 4998 - daviddavis - NEW - Artifact size is limited to 2 GB
14:39:43 <pulpbot> https://pulp.plan.io/issues/4998
14:40:07 <daviddavis> we discussed this before but I don't remember what the outcome was? did we triage it already?
14:40:08 <asmacdo> hmm did i miss this one too?
14:40:15 <daviddavis> maybe I was suppose to triage it
14:40:24 <daviddavis> I'll handle it
14:40:31 <daviddavis> it's a one line fix
14:40:39 <ggainey> we talked about it on the 21st
14:40:41 <ggainey> yeah
14:40:59 <ggainey> Integer to BigInteger, yeah?
14:41:06 <asmacdo> #idea Proposed for #4998: Leave the issue as-is, accepting its current state.
14:41:06 <asmacdo> !propose accept
14:41:06 <pulpbot> asmacdo: Proposed for #4998: Leave the issue as-is, accepting its current state.
14:41:08 <daviddavis> yea
14:41:13 <dawalker> I tried to keep a list of AI's after each triage and follow up with that person before the next triage to make sure they remembered.
14:41:38 <daviddavis> that's a good idea
14:41:52 <asmacdo> keeping up with the wrong query is distracting me
14:41:58 <daviddavis> ha yea
14:41:59 <ttereshc> +1 to accept
14:42:03 <ggainey> +1
14:42:08 <asmacdo> #agreed Leave the issue as-is, accepting its current state.
14:42:08 <asmacdo> !accept
14:42:08 <pulpbot> asmacdo: Current proposal accepted: Leave the issue as-is, accepting its current state.
14:42:10 <asmacdo> #topic https://pulp.plan.io/issues/5001
14:42:10 <pulpbot> asmacdo: 9 issues left to triage: 5001, 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:42:12 <pulpbot> RM 5001 - sajha - NEW - Provide content_ansible_collection_read method for ansible collections
14:42:12 <ggainey> dawalker++
14:42:13 <pulpbot> https://pulp.plan.io/issues/5001
14:42:14 <pulpbot> ggainey: dawalker's karma is now 41
14:42:26 <daviddavis> skip
14:42:32 <asmacdo> dawalker: pulpbot proposals can be used for that
14:42:38 <asmacdo> !skip
14:42:39 <pulpbot> asmacdo: 8 issues left to triage: 5002, 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:42:40 <asmacdo> #topic https://pulp.plan.io/issues/5002
14:42:40 <pulpbot> RM 5002 - horakmar - NEW - rpm_remote ignores proxy_url
14:42:41 <pulpbot> https://pulp.plan.io/issues/5002
14:42:48 <ttereshc> skip
14:42:54 <dkliban> it's fixe dalready
14:42:57 <dalley> are the rest all plugin issues?
14:43:03 * daviddavis skips to my lou
14:43:11 <asmacdo> !skip
14:43:11 <daviddavis> I think I filed some non-plugin issues
14:43:12 <ttereshc> dkliban, I commented on the one which I was sure that is pulp 3
14:43:12 <asmacdo> #topic https://pulp.plan.io/issues/5005
14:43:13 <pulpbot> asmacdo: 7 issues left to triage: 5005, 5006, 5008, 5009, 5010, 5012, 5028
14:43:14 <pulpbot> RM 5005 - horakmar - NEW - RPM remote sync ignores "proxy_url" settings
14:43:15 <pulpbot> https://pulp.plan.io/issues/5005
14:43:27 <ttereshc> skip
14:43:34 <asmacdo> !skip
14:43:35 <pulpbot> asmacdo: 6 issues left to triage: 5006, 5008, 5009, 5010, 5012, 5028
14:43:36 <asmacdo> #topic https://pulp.plan.io/issues/5006
14:43:36 <pulpbot> RM 5006 - ttereshc - NEW -  A publication can't be created immediately after kicking off a sync task
14:43:37 <pulpbot> https://pulp.plan.io/issues/5006
14:43:47 <dawalker> daviddavis, so help me if I have skip to my lou stuck in my head the rest of the day...
14:43:57 <daviddavis> lol
14:44:04 <asmacdo> !skiptomylou
14:44:04 <pulpbot> asmacdo: Error: "skiptomylou" is not a valid command.
14:44:07 <asmacdo> !skip
14:44:08 <pulpbot> asmacdo: 5 issues left to triage: 5008, 5009, 5010, 5012, 5028
14:44:08 <asmacdo> #topic https://pulp.plan.io/issues/5008
14:44:09 <pulpbot> RM 5008 - ttereshc - NEW - No way to enforce content specific uniqueness constaints in a repo version
14:44:09 <ggainey> dawalker: we'll all just call daviddavis "Mah Darlin'" :)
14:44:10 <pulpbot> https://pulp.plan.io/issues/5008
14:44:28 <mikedep333> #info mikedep333 has joined triage
14:44:28 <mikedep333> !here
14:44:28 <pulpbot> mikedep333: mikedep333 has joined triage
14:44:31 <daviddavis> do we have support for auto-publish in pulp 3?
14:44:52 <asmacdo> daviddavis: do you want to go back to 5006?
14:45:03 <daviddavis> oh
14:45:12 <daviddavis> no I am good
14:45:22 <daviddavis> sorry
14:45:34 <bmbouter> daviddavis: we don't have auto-publish we do have distributions that can serve the 'latest' publicaiton automatically
14:45:48 <daviddavis> I see
14:45:49 <bmbouter> which is effectively auto-publish only without waiting for an operation to occur
14:45:58 <bmbouter> same for auto-distribute of a repo version
14:45:59 <dawalker> ggainey, *snort*
14:46:07 <daviddavis> 5008 = story?
14:46:21 <daviddavis> or task
14:46:32 <asmacdo> lets skip 5008. ive got a proposal on the list, and bmbouter is working up another idea
14:46:44 <daviddavis> +1
14:46:49 <ttereshc> +1 to skip
14:46:56 <asmacdo> actually, can we just triage it
14:47:14 <asmacdo> 1 less to skip again :(
14:47:40 <asmacdo> its a real issue, without an agreed upon solution, which "triaged" is ok for
14:47:40 <daviddavis> sure
14:47:53 <bmbouter> I think I can reply on that thread today
14:47:56 <asmacdo> #idea Proposed for #5008: Leave the issue as-is, accepting its current state.
14:47:56 <asmacdo> !propose accept
14:47:56 <pulpbot> asmacdo: Proposed for #5008: Leave the issue as-is, accepting its current state.
14:47:58 <asmacdo> #agreed Leave the issue as-is, accepting its current state.
14:47:58 <asmacdo> !accept
14:47:59 <pulpbot> asmacdo: Current proposal accepted: Leave the issue as-is, accepting its current state.
14:47:59 <asmacdo> #topic https://pulp.plan.io/issues/5009
14:48:00 <pulpbot> asmacdo: 4 issues left to triage: 5009, 5010, 5012, 5028
14:48:01 <pulpbot> RM 5009 - daviddavis - NEW - HTML in our json api schema
14:48:02 <pulpbot> https://pulp.plan.io/issues/5009
14:48:14 <dkliban> i can fix this one
14:48:17 <daviddavis> this one is a blocker for me as a pulp user
14:48:33 <dkliban> daviddavis: which aspect is the blocker for you?
14:48:38 <asmacdo> dkliban: how do you propose fixing it?
14:48:57 <bmbouter> I thought html was normal as it provides docs content about the calls
14:48:57 <daviddavis> the pulp schema is difficult to read in json/yml/etc
14:49:27 <dkliban> oh ... i don't think it is meant to be viewed as JSON by a human
14:49:28 <asmacdo> dkliban: IIRC you said something about being able to strip the html from those alternate formats?
14:49:29 <bmbouter> that's true but it will make the html rendering plaintext
14:49:47 <dkliban> we can render it without HTML in the bindings
14:49:57 <asmacdo> we already established that daviddavis is a machine
14:50:02 <daviddavis> :)
14:50:04 <dkliban> lol
14:50:08 <daviddavis> json and yaml are human readable formats!
14:50:27 <bmbouter> that's just what a machine would say!
14:50:31 <dkliban> lol
14:50:34 <daviddavis> haha
14:50:36 <asmacdo> LOLOL
14:50:51 <ggainey> :)
14:50:58 <dkliban> but in all seriousness, where would you like to find the JSON schema without HTML?
14:51:15 <daviddavis> in my command line
14:51:30 <dkliban> what about a query argument 'strip_html'
14:51:41 <daviddavis> I think some pulp users like me will want to read the docs from the command line and not have to render them to html, open a browser, etc
14:51:44 <dkliban> and if you pass it in, html is stripped out of the schema
14:52:04 <asmacdo> how about query argument  include_html instead
14:52:04 <daviddavis> sure
14:52:09 <daviddavis> asmacdo: +1
14:52:33 <dkliban> yeah ... i can try that
14:52:43 <asmacdo> if that doesnt work skip is cool too
14:53:01 <asmacdo> dkliban: do you mind updating the issue?
14:53:04 <dkliban> yeah ... but i understand why skipping by default would be desired
14:53:09 <dkliban> i'll update it
14:53:19 <asmacdo> #idea Proposed for #5009: dkliban will update, accept and add to sprint
14:53:19 <asmacdo> !propose other dkliban will update, accept and add to sprint
14:53:19 <pulpbot> asmacdo: Proposed for #5009: dkliban will update, accept and add to sprint
14:53:27 <daviddavis> thanks
14:53:34 <asmacdo> #agreed dkliban will update, accept and add to sprint
14:53:34 <asmacdo> !accept
14:53:34 <pulpbot> asmacdo: Current proposal accepted: dkliban will update, accept and add to sprint
14:53:36 <asmacdo> #topic https://pulp.plan.io/issues/5010
14:53:36 <pulpbot> asmacdo: 3 issues left to triage: 5010, 5012, 5028
14:53:37 <pulpbot> RM 5010 - daviddavis - NEW - Travis fails when PRs are rebased and merged
14:53:38 <pulpbot> https://pulp.plan.io/issues/5010
14:53:57 <daviddavis> this is not urgent. it just causes our tests to fail sometimes.
14:54:03 <asmacdo> i wouldnt mind this so much if rebase and merge werent the default button on pulp3
14:54:04 <bmbouter> I think this is best resolved in the Travis Configuration tool
14:54:33 <dkliban> https://pulp.plan.io/issues/5009#note-5
14:54:33 <asmacdo> +1 bmbouter
14:54:51 <asmacdo> dkliban++
14:54:51 <pulpbot> asmacdo: dkliban's karma is now 299
14:55:07 <bmbouter> here comes that perfect game!
14:55:18 <daviddavis> I don't think we can fix it in the travis config tool along
14:55:21 <daviddavis> alone
14:55:22 <asmacdo> and after that, moved permanently
14:55:53 <bmbouter> daviddavis: for each repo theTravis Config tool would be up to date w/ how that repo merges and if the repo changes their approach they need to update the tool so future deployers will get it correctly
14:55:56 <bmbouter> is what I imagined
14:56:02 <asmacdo> same
14:56:18 <daviddavis> bmbouter: some repos have pre_before_install.sh that check out code. we need to confirm that they aren't using HEAD^2
14:56:29 <ttereshc> dkliban, is 5009 still an issue or more like a story?
14:56:51 <bmbouter> daviddavis: true. I figure we would converge at some point on a steady state where those behaviors are reflected in the travis config tool
14:57:05 * bmbouter has a meeting in 4 minutes and wants to highlight https://pulp.plan.io/issues/5031
14:57:18 <dkliban> ttereshc: it's a story
14:57:50 <bmbouter> I can highlight https://pulp.plan.io/issues/5031 on pulp-dev actually
14:57:54 <asmacdo> #idea Proposed for #5010: switch to template, change to story
14:57:54 <asmacdo> !propose other switch to template, change to story
14:57:54 <pulpbot> asmacdo: Proposed for #5010: switch to template, change to story
14:58:19 <asmacdo> +1 5031 to the list
14:58:26 <daviddavis> 5010 is a story?
14:58:50 <daviddavis> I agree moving it to template but not sure about it being a story
14:59:13 <ggainey> daviddavis: thought 5009 was the subject of "template and story"?
14:59:27 <daviddavis> ggainey: look at the last pulpbot output
14:59:50 <ggainey> oooh, I see - sorry, clearly havin trouble keeping up today, thanks :)
14:59:54 <asmacdo> "as a commiter, I can rebase and merge a PR"
15:00:03 <daviddavis> eh
15:00:07 <asmacdo> but i dont feel strongly about it
15:00:12 <asmacdo> would you prefer a task?
15:00:18 <daviddavis> +1
15:00:21 <asmacdo> #idea Proposed for #5010: switch to template, change to task
15:00:21 <asmacdo> !propose other switch to template, change to task
15:00:21 <pulpbot> asmacdo: Proposed for #5010: switch to template, change to task
15:00:23 <daviddavis> I gotta meeting
15:00:30 <asmacdo> #agreed switch to template, change to task
15:00:30 <asmacdo> !accept
15:00:31 <pulpbot> asmacdo: Current proposal accepted: switch to template, change to task
15:00:31 <asmacdo> #topic https://pulp.plan.io/issues/5012
15:00:32 <pulpbot> asmacdo: 2 issues left to triage: 5012, 5028
15:00:33 <pulpbot> RM 5012 - dkliban@redhat.com - NEW - ssl_client_key printed in the logs
15:00:34 <pulpbot> https://pulp.plan.io/issues/5012
15:00:49 <ggainey> ew
15:00:53 <ttereshc> ha! daviddavis has a meeting with bmbouter :P
15:01:12 <daviddavis> I do :)
15:01:18 <ggainey> logging keys is Bad - let's fix this please
15:01:27 <dkliban> let's add to the sprint
15:01:30 <ggainey> +1
15:01:48 <ttereshc> #idea Proposed for #5012: accept, add to the sprint set to 3.0
15:01:48 <ttereshc> #info ttereshc has joined triage
15:01:48 <ttereshc> !propose other accept, add to the sprint set to 3.0
15:01:48 <pulpbot> ttereshc: ttereshc has joined triage
15:01:49 <pulpbot> ttereshc: Proposed for #5012: accept, add to the sprint set to 3.0
15:01:56 <asmacdo> +1
15:02:08 <dkliban> +1
15:02:17 <asmacdo> #agreed accept, add to the sprint set to 3.0
15:02:17 <asmacdo> !accept
15:02:17 <pulpbot> asmacdo: Current proposal accepted: accept, add to the sprint set to 3.0
15:02:18 <asmacdo> #topic https://pulp.plan.io/issues/5028
15:02:19 <pulpbot> asmacdo: 1 issues left to triage: 5028
15:02:20 <pulpbot> RM 5028 - dkliban@redhat.com - NEW - RepositoryVersion.create() can't be called outside of a task
15:02:21 <pulpbot> https://pulp.plan.io/issues/5028
15:02:30 <asmacdo> dkliban: this is necessary
15:02:38 <dkliban> yeah ... we need it for docker
15:02:45 <asmacdo> no i mean the current behavior
15:02:57 <dkliban> and i have a PR ready if the proposed solution is acceptable
15:03:09 <asmacdo> docker push cant be modifying repos that should be locked because they are currently syncing
15:03:36 <ttereshc> it looks like a story to me
15:04:11 <dkliban> it's a story from the plugin writer's perspective
15:04:49 <dkliban> asmacdo: if a task has to be dispatched to create a repository version, then docker push is not going to work
15:05:12 <asmacdo> :(
15:05:13 <daviddavis> dkliban: this story sounds good to me. what about using current() though https://git.io/fjr0a
15:05:17 <dkliban> so it's really up to the user to not sync repos that they use for pushing to
15:06:25 <dkliban> daviddavis: can't use it ... it will return None
15:06:42 <daviddavis> dkliban: right but can you check it before creating a created resource?
15:06:43 <dkliban> and the DB doesn't let you create a CreatedResource with task=None
15:07:02 <dkliban> daviddavis: you can, but i want the plugin writer to make this decision intentionally
15:07:23 <daviddavis> I see. in that case, an optional parameter works for me.
15:07:56 <dkliban> asmacdo: we should discuss this tomorrow in our docker meeting
15:08:05 <partha> dkliban: I was trying to sync a  pulp3 file repo from pulp2
15:08:27 <asmacdo> yes, lets do that. i'm really concerned that we've made a lot of assumptions about adding/removing only ever taking place in tasks
15:08:42 <daviddavis> yup
15:08:54 <asmacdo> #idea Proposed for #5028: Skip this issue for this triage session.
15:08:54 <asmacdo> !propose skip
15:08:54 <pulpbot> asmacdo: Proposed for #5028: Skip this issue for this triage session.
15:09:06 <daviddavis> s/only ever taking place in tasks//
15:09:12 <partha> dkliban: and hit upon the following error -> https://paste.fedoraproject.org/paste/IRBej9cICuj3ovqSUMG74w
15:09:38 <partha> "Downloading <3.iso> failed checksum validation. The manifest specified the checksum to be  1d7e85343d88da30fa418c129b8bb50eb2ef9d386c425ed005ded01c0c1197da, but it was 1d7e85343d88da30fa418c129b8bb50eb2ef9d386c425ed005ded01c0c1197da."
15:09:43 <partha> does it ring a bell
15:09:53 <asmacdo> !skip
15:09:54 <pulpbot> asmacdo: No issues to triage.
15:09:58 <asmacdo> #endmeeting
15:09:58 <asmacdo> !end