14:31:22 #startmeeting Pulp Triage 2019-09-13 14:31:22 !start 14:31:22 #info fabricioo0 has joined triage 14:31:22 Meeting started Fri Sep 13 14:31:22 2019 UTC. The chair is fabricioo0. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:31:22 Useful Commands: #action #agreed #help #info #idea #link #topic. 14:31:22 The meeting name has been set to 'pulp_triage_2019-09-13' 14:31:22 fabricioo0: fabricioo0 has joined triage 14:31:23 !here 14:31:24 #info mikedep333 has joined triage 14:31:24 mikedep333: mikedep333 has joined triage 14:31:26 #info daviddavis has joined triage 14:31:26 !here 14:31:26 daviddavis: daviddavis has joined triage 14:31:27 it !here 14:31:39 #info asmacdo has joined triage 14:31:39 !here 14:31:39 asmacdo: asmacdo has joined triage 14:31:46 !next 14:31:47 fabricioo0: 9 issues left to triage: 5448, 5447, 5444, 5442, 5439, 5437, 5435, 5426, 5425 14:31:47 #topic https://pulp.plan.io/issues/5448 14:31:48 RM 5448 - dalley - NEW - If multiple source repos contain the same package, it can be chosen at random 14:31:49 https://pulp.plan.io/issues/5448 14:32:28 is this rpm? 14:32:39 yup 14:32:41 whoops 14:32:48 whale, dang 14:32:50 sprint 59 exists and remember next 2 weeks 14:33:00 I just changed the project 14:33:04 #idea Proposed for #5448: Skip this issue for this triage session. 14:33:04 #info dalley has joined triage 14:33:04 !propose skip 14:33:04 dalley: dalley has joined triage 14:33:05 dalley: Proposed for #5448: Skip this issue for this triage session. 14:33:21 +1 skip since it will be triaged by rpm folks 14:33:23 !accept 14:33:29 ;( 14:33:49 #agreed Skip this issue for this triage session. 14:33:49 !accept 14:33:49 fabricioo0: Current proposal accepted: Skip this issue for this triage session. 14:33:50 fabricioo0: 8 issues left to triage: 5447, 5444, 5442, 5439, 5437, 5435, 5426, 5425 14:33:50 #topic https://pulp.plan.io/issues/5447 14:33:51 RM 5447 - gmbnomis - NEW - /etc/pulp/server.conf still has owner "root:apache" after upgrade 14:33:52 https://pulp.plan.io/issues/5447 14:34:19 accept and add to sprint 14:34:44 #idea Proposed for #5447: accept and add to sprint 14:34:44 !propose other accept and add to sprint 14:34:45 asmacdo: Proposed for #5447: accept and add to sprint 14:34:52 #agreed accept and add to sprint 14:34:52 !accept 14:34:52 fabricioo0: Current proposal accepted: accept and add to sprint 14:34:53 #topic https://pulp.plan.io/issues/5444 14:34:53 fabricioo0: 7 issues left to triage: 5444, 5442, 5439, 5437, 5435, 5426, 5425 14:34:54 RM 5444 - bmbouter - NEW - Consolidate ProgressReporting into a single model so it's simpler for plugin writers 14:34:55 https://pulp.plan.io/issues/5444 14:35:03 refactor 14:35:19 or task maybe 14:35:24 story? 14:35:51 I don't know if this is new functionality 14:36:00 it's removal 14:36:04 ah, kk 14:36:15 let's go with task 14:36:29 +1 14:36:30 +1 14:36:31 #info bmbouter has joined triage 14:36:31 !here 14:36:31 bmbouter: bmbouter has joined triage 14:36:34 #info ttereshc has joined triage 14:36:34 !here 14:36:35 ttereshc: ttereshc has joined triage 14:36:55 #idea Proposed for #5444: convert to task 14:36:55 !propose other convert to task 14:36:55 asmacdo: Proposed for #5444: convert to task 14:37:05 #agreed convert to task 14:37:05 !accept 14:37:05 fabricioo0: Current proposal accepted: convert to task 14:37:06 fabricioo0: 6 issues left to triage: 5442, 5439, 5437, 5435, 5426, 5425 14:37:06 #topic https://pulp.plan.io/issues/5442 14:37:07 RM 5442 - bmbouter - NEW - Replace pulpcore.plugin.models Task with the actual Task from pulpcore 14:37:08 https://pulp.plan.io/issues/5442 14:37:25 this is one of the stub issues from our blockers review that was unfiled 14:37:35 this could become a story 14:37:43 +1 14:37:43 because the plugin writer would receive a more functional use case 14:38:03 +1 to a story or task 14:38:51 +1 story or task 14:38:59 !propose convert to a story 14:38:59 fabricioo0: Error: "propose" is not a valid command. 14:39:13 s/propose/propose other/ 14:39:30 #idea Proposed for #5442: convert to a story 14:39:30 !propose other convert to a story 14:39:30 fabricioo0: Proposed for #5442: convert to a story 14:39:33 ^ an error i've made once a week for years 14:39:37 heh 14:39:43 #agreed convert to a story 14:39:43 !accept 14:39:43 fabricioo0: Current proposal accepted: convert to a story 14:39:44 #topic https://pulp.plan.io/issues/5439 14:39:44 fabricioo0: 5 issues left to triage: 5439, 5437, 5435, 5426, 5425 14:39:45 RM 5439 - bmbouter - NEW - As a user, my remote's SSL settings use names similar to openssl 14:39:46 https://pulp.plan.io/issues/5439 14:40:07 could become a story, also it's still a stub 14:40:20 wait 14:40:47 nm 14:41:10 I thought I saw something similar to this the other day 14:41:20 yeah lets convert to a story. no problem that its still a stub since its not marked "groomed" 14:41:28 +1 14:41:29 daviddavis: yeah I suspect we have this filed somewhere 14:41:33 +1 14:41:34 +1 14:41:42 #idea Proposed for #5439: convert to story 14:41:42 !propose other convert to story 14:41:42 fabricioo0: Proposed for #5439: convert to story 14:41:47 but I couldn't find it so I refiled (this originally came from a community user) 14:41:52 #agreed convert to story 14:41:52 !accept 14:41:52 fabricioo0: Current proposal accepted: convert to story 14:41:53 fabricioo0: 4 issues left to triage: 5437, 5435, 5426, 5425 14:41:53 #topic https://pulp.plan.io/issues/5437 14:41:54 RM 5437 - dkliban@redhat.com - NEW - plugin_template config needs to allow 'post_install_script' and 'pre_install_script' 14:41:55 https://pulp.plan.io/issues/5437 14:42:07 story? 14:42:23 +1 14:42:50 So this is a legitimate need, but the solution I disagree with. 14:43:04 wait 14:43:10 sounds like a design-discussion to have in the issue, yeah? 14:43:11 dont we have before_install.sh? 14:43:23 +1 to ggainey 14:43:33 True, we do. But we are using Dockerfiles to build the images now. 14:43:49 And if we have plugins do this individually, then the all-in-one plugin build logic needs to do it. 14:44:04 *all-in-one "pulp" image with several plugins 14:44:04 #idea Proposed for #5437: convert to a story and discuss possible solutions on the ticket 14:44:04 !propose other convert to a story and discuss possible solutions on the ticket 14:44:04 ttereshc: Proposed for #5437: convert to a story and discuss possible solutions on the ticket 14:44:05 so this is not an issue 14:44:09 I want to avoid duplicate logic. 14:44:18 mikedep333: let's design this outside this meeting 14:44:23 Yes, agreed 14:44:23 so we have $POST_BEFORE_INSTALL, which would run at the same time as a preinstall 14:44:27 +1 later 14:44:32 #idea Proposed for #5437: Skip this issue for this triage session. 14:44:32 !propose skip 14:44:33 asmacdo: Proposed for #5437: Skip this issue for this triage session. 14:44:58 +1 14:45:06 this should not be listed as an issue 14:45:07 #agreed Skip this issue for this triage session. 14:45:07 !accept 14:45:07 fabricioo0: Current proposal accepted: Skip this issue for this triage session. 14:45:08 #topic https://pulp.plan.io/issues/5435 14:45:08 fabricioo0: 3 issues left to triage: 5435, 5426, 5425 14:45:09 it's a task 14:45:10 RM 5435 - amacdona@redhat.com - NEW - As a pulp-operator user, I can add arbitrary settings to settings.py 14:45:11 https://pulp.plan.io/issues/5435 14:45:47 I don't get why we want to talk about the issue next triage even if it's not a bug 14:46:10 yeah lets go back to 5437 and propose to convert to task 14:46:22 i am doing that either way 14:46:26 changing it to a task 14:46:28 ok, thanks 14:46:31 nm then 14:46:43 !issue 5437 14:46:44 #topic https://pulp.plan.io/issues/5437 14:46:44 RM 5437 - dkliban@redhat.com - NEW - plugin_template config needs to allow 'post_install_script' and 'pre_install_script' 14:46:46 https://pulp.plan.io/issues/5437 14:47:07 #idea Proposed for #5437: convert to a task 14:47:07 !propose other convert to a task 14:47:07 fabricioo0: Proposed for #5437: convert to a task 14:47:28 done! 14:47:29 +1 14:47:35 #agreed convert to a task 14:47:35 !accept 14:47:35 fabricioo0: Current proposal accepted: convert to a task 14:47:36 #topic https://pulp.plan.io/issues/5435 14:47:36 fabricioo0: 3 issues left to triage: 5435, 5426, 5425 14:47:37 RM 5435 - amacdona@redhat.com - NEW - As a pulp-operator user, I can add arbitrary settings to settings.py 14:47:38 https://pulp.plan.io/issues/5435 14:47:50 #idea Proposed for #5435: convert to story 14:47:50 !propose other convert to story 14:47:50 asmacdo: Proposed for #5435: convert to story 14:47:55 +1 14:48:01 +1 14:48:18 #agreed convert to story 14:48:18 !accept 14:48:18 fabricioo0: Current proposal accepted: convert to story 14:48:19 #topic https://pulp.plan.io/issues/5426 14:48:19 fabricioo0: 2 issues left to triage: 5426, 5425 14:48:20 RM 5426 - dkliban@redhat.com - NEW - orphan cleanup does not properly find orphaned content 14:48:21 https://pulp.plan.io/issues/5426 14:48:46 accept and add? 14:48:54 yes 14:48:54 i thought this was by-design 14:49:09 is it a blocker? 14:49:10 if we remove orphans that are in an old version, we cant roll back anymore 14:49:13 asmacdo: what do you mean? i think this implementation pre-dates repo versions 14:49:19 #idea Proposed for #5426: accept and add to sprint 14:49:19 !propose other accept and add to sprint 14:49:19 fabricioo0: Proposed for #5426: accept and add to sprint 14:49:30 I think asmacdo is right 14:49:50 dkliban: if we "fix" this users couldn't rollback 14:51:03 #idea Proposed for #5426: asmacdo will close as WONTFIX with comment 14:51:03 !propose other asmacdo will close as WONTFIX with comment 14:51:03 asmacdo: Proposed for #5426: asmacdo will close as WONTFIX with comment 14:51:10 +1 14:51:14 kk 14:51:27 #agreed asmacdo will close as WONTFIX with comment 14:51:27 !accept 14:51:27 fabricioo0: Current proposal accepted: asmacdo will close as WONTFIX with comment 14:51:28 #topic https://pulp.plan.io/issues/5425 14:51:28 fabricioo0: 1 issues left to triage: 5425 14:51:29 RM 5425 - bmbouter - NEW - As a user, I cannot provide settings that override defaults in pulpcore or plugins 14:51:30 https://pulp.plan.io/issues/5425 14:53:14 'who wins" problems are always fun :) 14:53:20 let's accept 14:53:27 yeah, concur 14:53:42 #idea Proposed for #5425: accept 14:53:42 !propose other accept 14:53:42 fabricioo0: Proposed for #5425: accept 14:53:47 I'm not sure if it's a story or a bug 14:53:55 kinda both :) 14:54:10 also maybe we ought to mrk this a related to 5435? 14:54:12 ttereshc: reads like a bug to me, depends on whether the order is written down 'officially' somewhere :) 14:54:34 ggainey, yeah, just the title is written in a story style 14:54:40 ah, true 14:55:12 +1 to accept 14:55:17 #idea Proposed for #5425: convert to story and relate to 5435 14:55:17 !propose other convert to story and relate to 5435 14:55:17 asmacdo: Proposed for #5425: convert to story and relate to 5435 14:55:24 change title to 'user additions to /etc/pulp/settings.py can be unexpectedly overwritten'? 14:55:38 or that :) 14:55:52 yup 14:55:55 eventual consistency, im fine with what yall said too 14:56:13 accept or convert to story? 14:56:23 retitle and accept? 14:56:42 #idea Proposed for #5425: retitle and accept 14:56:42 !propose other retitle and accept 14:56:42 fabricioo0: Proposed for #5425: retitle and accept 14:56:48 yeah, sounds good 14:57:00 +1 14:57:09 #agreed retitle and accept 14:57:09 !accept 14:57:09 fabricioo0: Current proposal accepted: retitle and accept 14:57:10 fabricioo0: No issues to triage. 14:57:39 open floor! 14:57:45 asmacdo: good call on the orphans observation 14:57:52 ty 14:59:17 asmacdo++ 14:59:17 dkliban: asmacdo's karma is now 142 14:59:38 any issue for open floor? 15:00:13 15:00:36 * mikedep333 listens 15:00:39 #endmeeting 15:00:39 !end