14:31:15 #startmeeting Pulp Triage 2020-06-02 14:31:15 #info fao89 has joined triage 14:31:15 !start 14:31:16 Meeting started Tue Jun 2 14:31:15 2020 UTC. The chair is fao89. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:31:16 Useful Commands: #action #agreed #help #info #idea #link #topic. 14:31:16 The meeting name has been set to 'pulp_triage_2020-06-02' 14:31:16 fao89: fao89 has joined triage 14:31:18 that was a long triage 14:31:20 #info daviddavis has joined triage 14:31:20 !here 14:31:20 daviddavis: daviddavis has joined triage 14:31:24 ha yup 14:31:27 #info ppicka has joined triage 14:31:27 !here 14:31:29 ppicka: ppicka has joined triage 14:31:29 !next 14:31:30 fao89: No issues to triage. 14:31:36 Open floor! 14:31:38 #info ipanova has joined triage 14:31:38 !here 14:31:38 ipanova: ipanova has joined triage 14:31:44 https://hackmd.io/SVCMjpwXTfOMqF2OeyyLRw 14:31:54 topic: how to deduplicate pulpcore Documentation tag and Documentation category? 14:32:47 I believe we don't need a category for that 14:32:51 i think documentation category is not available across all projects 14:33:14 we need to use either one or another 14:33:35 #info ggainey has joined triage 14:33:35 !here 14:33:44 on installer for example, it is a category, and we have docs issues, so the tag works for us, the category wouldn't 14:34:03 fao89: I agree that's a great point in favor of using the tag 14:34:08 unless it is possible to set 2 categories on the same issue 14:34:23 fao89: it is not 14:34:27 why would you need 2 categories? 14:34:37 installer + documentation 14:34:38 Installer is a category 14:34:42 aha 14:34:51 tag is the winner 14:34:52 +1 to using tag then 14:35:05 i think in any case lately we have been using the tag 14:35:09 I agree 14:35:33 I can make this change and I will replace the categort w/ the tag so nothing will be lost 14:35:40 next topic? 14:35:50 I will make this change today 14:36:04 ok w/ me 14:36:04 sounds good, thank you 14:36:12 topic: announcing https://fixtures.pulpproject.org/fixtures/ 14:36:29 awesomee 14:36:48 yup so this is online! I'm going to send an announcement to pulp-dev also and ask mcorr to add to the website if she thinks that makes sense 14:36:58 so just an fyi here 14:37:03 great, will it be updated with every merge? 14:37:04 ok 14:37:18 it will be, there is a 5 min delay though after the container is published 14:37:40 bmbouter: what about tests that use link for the old fixtures 14:38:13 ipanova: great question. we have an epic tracking each plugin's removal of fedorapeople fixtures and they will not be taken down yet 14:38:21 ack 14:38:23 https://pulp.plan.io/issues/6638 14:38:43 great! 14:38:45 file maven python and rpm all have open items in this area ^ 14:38:45 #info dkliban has joined triage 14:38:45 !here 14:38:51 and ansible 14:38:57 !here 14:39:08 pulpbot is struggling 14:39:08 bmbouter: Error: "is" is not a valid command. 14:39:25 pulpbot does not like pulp-meeting channel 14:39:25 fao89: Error: "does" is not a valid command. 14:39:26 yeah ... it didn't send a message but it told me privately that i already joined 14:39:42 pulpbot-- 14:39:42 fao89: pulpbot's karma is now -1 14:39:45 lol 14:39:54 next topic? 14:39:56 ppicka: would you be able to look at the rpm tests issue https://pulp.plan.io/issues/6656 14:39:58 I'm trying to think about how we can start testing https://fixtures.pulpproject.org/fixtures/ 14:40:09 * ppicka just checking it 14:40:15 and gerrod for python? 14:40:23 https://pulp.plan.io/issues/6656 14:40:52 bmbouter: I will assign it to me for RPM, do we have some time plan how fast do we want to move from fedorapeople to our new? 14:40:53 daviddavis: so the current plan (up for discussion) is that when running in CI the container is locally hosted 14:41:04 ppicka: we don't have a timeline 14:41:10 bmbouter: that's what we do today 14:41:27 at least in travis 14:41:29 daviddavis: great, and when running off CI it uses fixtures.fedorapeople.org 14:41:41 yea 14:41:47 you mean fixtures.pulpproject.org ? 14:41:54 lol yeah... 14:42:06 we need to update the docs too 14:42:10 dkliban: I think the fixtures change you are making may roll that out for all plugins? 14:42:21 s/fixtures change/pulp smash change/ ? 14:42:34 bmbouter: i am not making a change ... i was just doing it locally on my box to test that it's working 14:42:41 ack ty 14:43:05 daviddavis: so I'm thinking that actually all of these plugin issues are really audits to make sure they're using fixtures from the container exclusively 14:43:23 and that we only need to change pulp-smash itself to switch the default location? 14:43:29 or maybe I'm confused, wdyt? 14:43:58 pulp-smahs also needs a change 14:45:03 git grep showed me only 1 result ^ 14:46:03 so do I need to make a sub-task for the epic to update pulp-smash? 14:46:10 bmbouter: in case of rpm it is defined in rpm tests 14:46:19 bmbouter: yes 14:46:31 * bmbouter makes it now 14:47:15 ppicka: every plugin will need to make an audit and change docs + tests 14:47:48 can we put 6638 on the sprint or can someone quick PR it? 14:47:51 ipanova: I agree, it was more 'fyi' when the topic was about pulp-smash too 14:48:14 +1 to add to sprint 14:48:16 I agree this is way more than an fyi 14:48:22 +1 14:48:36 I'll add to sprint now per this convo 14:49:07 ok 14:49:13 https://pulp.plan.io/issues/6881 specifically 14:49:38 next topic: what is the purpose of the ‘backlog’ Tag? 14:50:00 this confused me because all things are backlog effectively 14:50:03 this is for this query https://pulp.plan.io/projects/pulp/issues?query_id=152 14:50:21 mmmm 14:50:22 issues that are good candidates for interns are marked as backlog 14:50:40 this query always makes me laugh 14:50:43 well that answers my question 14:50:48 the query is veyr useful 14:50:50 the tag not so much 14:51:03 agree about the tag name 14:51:04 we could do this differently, we could have the query refer to specific issue numbers 14:51:14 s/backlog/love 14:51:15 but at the times we did not come up with a better one 14:51:52 bmbouter: i would prefer to keep a tag 14:52:15 I'm ok with that, but does this make sense for users to see though? 14:53:48 i can see it being confusing for the uers 14:54:07 i've already heard that we have too many fields that dont' make sense to users filing bugs 14:54:15 i am not sure what to answer, imo this kind of tag should not create any kind of feelings 14:54:22 yup that's my highlest level goal is to simplify redmine 14:55:18 our redmine is confusing for users 14:55:25 yep 14:55:45 if that's a concern then we can point query to specific issues number until we come up with a proper tag name 14:56:15 I was looking at how to do that and I didn't exactly see how 14:56:22 but that would resolve my concern completely 14:56:33 the goal was to mark issue that would be good being worked on for interns and not create more cinfusion 14:56:41 confusion* 14:57:02 oic how 14:57:24 if you want to make a change now or wait I'm ok either way 14:57:31 I don't want to take folks somewhere they don't want to go 14:57:35 issue 'is' 14:57:41 yup I agree 14:57:47 bmbouter: https://pulp.plan.io/issues?set_filter=1&status_id=*&issue_id=5707,5894,5896,5941,5955,5955,5833,5867,5870,5873,5706,5941 14:58:06 so we could remove 'backlog' and have the query use issue is instead 14:58:31 but only if folks are comfortable w/ this change 14:58:52 yes we can, if that will improve user experience i am in favour 14:59:00 i can do that 14:59:53 ipanova: ty! 15:00:39 last topic: removing the ‘Demo Requested’ tag 15:01:02 yup same reasoning here, the goal is to simplify redmine and this tag has no life/process/place 15:01:12 +1 15:01:19 +1 15:01:36 I can remove this one 15:02:26 #endmeeting 15:02:26 !end