14:30:19 #startmeeting Pulp Triage 2019-09-20 14:30:19 !start 14:30:19 #info fabricioo0 has joined triage 14:30:19 Meeting started Fri Sep 20 14:30:19 2019 UTC. The chair is fabricioo0. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:30:19 Useful Commands: #action #agreed #help #info #idea #link #topic. 14:30:19 The meeting name has been set to 'pulp_triage_2019-09-20' 14:30:19 fabricioo0: fabricioo0 has joined triage 14:30:28 #info dkliban has joined triage 14:30:28 !here 14:30:28 dkliban: dkliban has joined triage 14:31:15 !next 14:31:16 fabricioo0: 4 issues left to triage: 5485, 5484, 5474, 5456 14:31:16 #topic https://pulp.plan.io/issues/5485 14:31:17 RM 5485 - ipanova@redhat.com - NEW - Migration plugin should filter only those repo types that belong to the plugin that is being migrated 14:31:18 https://pulp.plan.io/issues/5485 14:31:23 #info daviddavis has joined triage 14:31:23 !here 14:31:23 daviddavis: daviddavis has joined triage 14:31:50 #info ipanova has joined triage 14:31:50 !here 14:31:51 ipanova: ipanova has joined triage 14:32:04 #info dalley has joined triage 14:32:04 !here 14:32:04 dalley: dalley has joined triage 14:32:11 #info bmbouter has joined triage 14:32:11 !here 14:32:12 bmbouter: bmbouter has joined triage 14:32:12 we should accept it 14:32:21 +1 14:32:26 +1 14:32:26 #idea Proposed for #5485: Leave the issue as-is, accepting its current state. 14:32:26 !propose accept 14:32:26 fabricioo0: Proposed for #5485: Leave the issue as-is, accepting its current state. 14:32:41 #info mikedep333 has joined triage 14:32:41 !here 14:32:41 mikedep333: mikedep333 has joined triage 14:32:56 #agreed Leave the issue as-is, accepting its current state. 14:32:56 !accept 14:32:56 fabricioo0: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:32:57 #topic https://pulp.plan.io/issues/5484 14:32:57 fabricioo0: 3 issues left to triage: 5484, 5474, 5456 14:32:58 RM 5484 - mdellweg - NEW - plugin templates bootstrap section contains publisher code 14:32:59 https://pulp.plan.io/issues/5484 14:33:09 we should also accept 14:33:12 +1 14:33:18 #idea Proposed for #5484: Leave the issue as-is, accepting its current state. 14:33:18 !propose accept 14:33:18 fabricioo0: Proposed for #5484: Leave the issue as-is, accepting its current state. 14:33:33 #info ttereshc has joined triage 14:33:33 !here 14:33:33 ttereshc: ttereshc has joined triage 14:34:03 +1 14:34:05 +1 14:34:05 #agreed Leave the issue as-is, accepting its current state. 14:34:05 !accept 14:34:05 fabricioo0: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:34:06 fabricioo0: 2 issues left to triage: 5474, 5456 14:34:07 #topic https://pulp.plan.io/issues/5474 14:34:07 RM 5474 - bmbouter - POST - For all processes launched by systemd ensure the PATH includes the virtualenv if installed in a virtualenv 14:34:09 https://pulp.plan.io/issues/5474 14:34:38 thi sis already in POST 14:34:41 let's accept 14:34:54 agreed 14:34:54 +1 14:34:55 this POST is waiting on the installer CI to merge (it was an ansiblefest blocker) 14:35:07 #idea Proposed for #5474: Leave the issue as-is, accepting its current state. 14:35:07 !propose accept 14:35:07 fabricioo0: Proposed for #5474: Leave the issue as-is, accepting its current state. 14:35:12 #agreed Leave the issue as-is, accepting its current state. 14:35:12 !accept 14:35:13 #topic https://pulp.plan.io/issues/5456 14:35:13 fabricioo0: Current proposal accepted: Leave the issue as-is, accepting its current state. 14:35:13 bmbouter: I agree. We just need to figure out how to properly set it. I left a comment. 14:35:14 fabricioo0: 1 issues left to triage: 5456 14:35:15 RM 5456 - mdellweg - NEW - The current CI implementation makes it hard to discuss small changes to a bigger PR 14:35:16 https://pulp.plan.io/issues/5456 14:35:17 +1 14:36:00 for this one, are we talking about not failing fast? 14:36:12 I agree with mdellweg, but all those later squashed commits I address by just adding [noissue]. 14:36:15 or running the commit validation as a separate job (it says "separate test") 14:36:17 x9c4: ^ 14:36:29 In fact, I often put [noissue] on all my commits, then put the issue # when I squash. 14:36:44 daviddavis: That's a good idea. 14:36:56 The overall test suite would show up as failed, but the individual tests would work. 14:37:16 from the issue: "If commit message validation was extracted as a separate test, one could use for example the git commit --fixup aabbccdd" 14:37:20 if we turn off failing fast, then we could still see the test results 14:37:48 daviddavis: that's a good idea 14:37:58 that would be easy but it would tax our resources 14:38:03 want to comment on the issue w/ that workaround 14:38:27 he mentioned it on chat and I asked him to file 14:38:33 ok 14:39:08 I'm more worried ATM about our resources in Travis and any solution to this issue would add to that problem 14:39:43 let's send to parking log for now 14:39:54 backlog I guess is the right term for that 14:39:59 yep 14:40:13 I propose we accept and I'll comment 14:40:17 +1 14:40:25 #idea Proposed for #5456: Leave the issue as-is, accepting its current state. 14:40:25 !propose accept 14:40:25 fabricioo0: Proposed for #5456: Leave the issue as-is, accepting its current state. 14:41:13 #idea Proposed for #5456: accept and daviddavis comments on this 14:41:13 !propose other accept and daviddavis comments on this 14:41:14 fabricioo0: Proposed for #5456: accept and daviddavis comments on this 14:41:25 +1 14:41:48 +1 14:41:51 #agreed accept and daviddavis comments on this 14:41:51 !accept 14:41:51 fabricioo0: Current proposal accepted: accept and daviddavis comments on this 14:41:53 fabricioo0: No issues to triage. 14:42:05 Open floor! 14:42:26 dalley: can you tell me more about the migration squashing? 14:44:10 any issues? 14:45:10 #endmeeting 14:45:10 !end