14:30:20 #startmeeting Pulp Triage 2019-03-29 14:30:20 #info dawalker has joined triage 14:30:21 Meeting started Fri Mar 29 14:30:20 2019 UTC and is due to finish in 60 minutes. The chair is dawalker. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:30:21 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:30:21 The meeting name has been set to 'pulp_triage_2019_03_29' 14:30:21 dawalker: dawalker has joined triage 14:30:21 * bmbouter calls upon the lorax 14:30:52 lol 14:30:57 !here 14:30:57 #info daviddavis has joined triage 14:30:58 daviddavis: daviddavis has joined triage 14:31:04 #info ipanova has joined triage 14:31:04 !here 14:31:04 ipanova: ipanova has joined triage 14:31:09 !care 4585 14:31:21 #info ttereshc has joined triage 14:31:21 !here 14:31:21 ttereshc: ttereshc has joined triage 14:31:56 !next 14:31:57 #topic Task Cancellation doesn't interrupt/cancel a running task with Celery 4+ - http://pulp.plan.io/issues/4581 14:31:58 dawalker: 10 issues left to triage: 4581, 4585, 4588, 4589, 4590, 4597, 4600, 4602, 4603, 4604 14:31:59 Issue #4581 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:32:00 Task Cancellation doesn't interrupt/cancel a running task with Celery 4+ - http://pulp.plan.io/issues/4581 14:32:09 accept and add to sprint 14:32:18 #idea Proposed for #4581: accept and add to sprint 14:32:18 !propose other accept and add to sprint 14:32:18 dawalker: Proposed for #4581: accept and add to sprint 14:32:37 #info ppicka has joined triage 14:32:37 !here 14:32:37 ppicka: ppicka has joined triage 14:32:39 #info dkliban has joined triage 14:32:39 !here 14:32:39 dkliban: dkliban has joined triage 14:33:03 #agreed accept and add to sprint 14:33:03 !accept 14:33:03 dawalker: Current proposal accepted: accept and add to sprint 14:33:05 #topic `make` should be installed by the pulp-devel role - http://pulp.plan.io/issues/4585 14:33:05 dawalker: 9 issues left to triage: 4585, 4588, 4589, 4590, 4597, 4600, 4602, 4603, 4604 14:33:06 Issue #4585 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:33:07 `make` should be installed by the pulp-devel role - http://pulp.plan.io/issues/4585 14:33:08 dawalker: asmacdo: Issue 4585 is currently being discussed. 14:33:23 #idea Proposed for #4585: accept and add to the sprint 14:33:23 #info asmacdo has joined triage 14:33:23 !propose other accept and add to the sprint 14:33:24 asmacdo: asmacdo has joined triage 14:33:24 accept and add to sprint 14:33:25 asmacdo: Proposed for #4585: accept and add to the sprint 14:33:31 +1 14:33:32 +1 14:33:37 +1 14:33:52 !care 4600 14:34:15 #agreed accept and add to the sprint 14:34:15 !accept 14:34:15 dawalker: Current proposal accepted: accept and add to the sprint 14:34:17 #topic Ostree repo sync fails - http://pulp.plan.io/issues/4588 14:34:17 dawalker: 8 issues left to triage: 4588, 4589, 4590, 4597, 4600, 4602, 4603, 4604 14:34:18 Issue #4588 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:34:19 Ostree repo sync fails - http://pulp.plan.io/issues/4588 14:34:28 skip, ostree bug 14:34:32 !care ttereshc 14:34:32 ipanova: Error: 'ttereshc' is not a valid positive integer. 14:34:36 dammit 14:34:36 !skip 14:34:38 #topic No published metadata is available on a Pubication via REST API - http://pulp.plan.io/issues/4589 14:34:38 dawalker: 7 issues left to triage: 4589, 4590, 4597, 4600, 4602, 4603, 4604 14:34:39 Issue #4589 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:34:40 No published metadata is available on a Pubication via REST API - http://pulp.plan.io/issues/4589 14:34:47 accept and add 3.0 milestone? 14:35:04 +1 14:35:08 #idea Proposed for #4589: accept and add 3.0 milestone 14:35:08 !propose other accept and add 3.0 milestone 14:35:09 dawalker: Proposed for #4589: accept and add 3.0 milestone 14:35:22 +1 14:35:28 +1 14:35:31 !accept 14:35:31 #agreed accept and add 3.0 milestone 14:35:31 dawalker: Current proposal accepted: accept and add 3.0 milestone 14:35:32 #topic Return relative URI for distributions - http://pulp.plan.io/issues/4590 14:35:33 dawalker: 6 issues left to triage: 4590, 4597, 4600, 4602, 4603, 4604 14:35:34 Issue #4590 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:35:35 Return relative URI for distributions - http://pulp.plan.io/issues/4590 14:35:37 I think there's a pr for this? 14:35:40 yeah 14:35:50 !propose accept 14:35:50 #idea Proposed for #4590: Leave the issue as-is, accepting its current state. 14:35:51 dawalker: Proposed for #4590: Leave the issue as-is, accepting its current state. 14:36:12 accept and add to sprint. also remind dkliban to update issue. 14:36:19 +1 14:36:21 +1 14:36:23 +1 14:36:26 should we add to 50 or 51 if there's a pr already up? 14:36:44 51 i think 14:36:53 I think when 50 is done its done so 51 14:37:07 #idea Proposed for #4590: accept and add to sprint 14:37:07 !propose other accept and add to sprint 14:37:08 dawalker: Proposed for #4590: accept and add to sprint 14:37:11 doesn't matter 14:37:15 !accept 14:37:15 #agreed accept and add to sprint 14:37:15 dawalker: Current proposal accepted: accept and add to sprint 14:37:16 #topic Deprecate docker v1 support - http://pulp.plan.io/issues/4597 14:37:17 dawalker: 5 issues left to triage: 4597, 4600, 4602, 4603, 4604 14:37:18 Issue #4597 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:37:19 Deprecate docker v1 support - http://pulp.plan.io/issues/4597 14:37:19 rchan, will move everything on Monday anyway 14:37:26 rchan: will move all items to the next sprint i.e 51 so it does not really matter 14:37:32 dawalker: ^ 14:37:39 skip docker issue 14:37:40 skip, docker issue 14:37:41 ah, good point, thanks 14:37:44 !skip 14:37:46 #topic DockerDistributions are not validated correctly - http://pulp.plan.io/issues/4600 14:37:46 dawalker: 4 issues left to triage: 4600, 4602, 4603, 4604 14:37:47 Issue #4600 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:37:48 DockerDistributions are not validated correctly - http://pulp.plan.io/issues/4600 14:37:49 dawalker: asmacdo: Issue 4600 is currently being discussed. 14:37:58 skip docker issue 14:38:01 !skip 14:38:03 #topic /var/lib/pulp/tmp/ seems to violate the FHS - http://pulp.plan.io/issues/4602 14:38:03 dawalker: 3 issues left to triage: 4602, 4603, 4604 14:38:04 Issue #4602 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:38:05 /var/lib/pulp/tmp/ seems to violate the FHS - http://pulp.plan.io/issues/4602 14:38:25 dkliban: we need to comment on ^ about it crossing the filesystem 14:38:31 yeah 14:38:34 i'll leave a comment 14:38:34 skip for now 14:38:38 we don't want the downloads and final storage to cross the filesystem volume 14:38:39 +1 14:38:49 +1 14:39:05 !skip 14:39:06 dawalker: 2 issues left to triage: 4603, 4604 14:39:07 #topic pulp_streamer fails to properly stream response - http://pulp.plan.io/issues/4603 14:39:08 Issue #4603 [ASSIGNED] (dkliban@redhat.com) - Priority: Normal | Severity: Medium 14:39:09 pulp_streamer fails to properly stream response - http://pulp.plan.io/issues/4603 14:39:13 accept and add to sprint 14:39:17 +1 14:39:18 ? 14:39:24 +1 14:39:26 +1 14:39:35 #idea Proposed for #4603: accept and add to sprint 14:39:35 !propose other accept and add to sprint 14:39:35 dawalker: Proposed for #4603: accept and add to sprint 14:39:38 !accept 14:39:38 #agreed accept and add to sprint 14:39:38 dawalker: Current proposal accepted: accept and add to sprint 14:39:39 dawalker: 1 issues left to triage: 4604 14:39:40 #topic Remove duplicates does not work properly if there is multiple content types in the repo - http://pulp.plan.io/issues/4604 14:39:40 Issue #4604 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:39:41 Remove duplicates does not work properly if there is multiple content types in the repo - http://pulp.plan.io/issues/4604 14:39:50 ugg 14:40:01 i will a pr later 14:40:04 submit 14:40:12 accept and add to sprint? 14:40:12 add to sprint 14:40:15 #idea Proposed for #4604: accept and add to sprint 14:40:15 !propose other accept and add to sprint 14:40:15 dawalker: Proposed for #4604: accept and add to sprint 14:40:24 +1 14:40:35 _1 14:40:35 !accept 14:40:35 #agreed accept and add to sprint 14:40:35 dawalker: Current proposal accepted: accept and add to sprint 14:40:37 dawalker: No issues to triage. 14:40:39 !dance 14:40:39 ♪┏(°.°)┛┗(°.°)┓┗(°.°)┛┏(°.°)┓ ♪ 14:40:48 !end 14:40:55 oh no wait 14:40:57 Anyone have any issues to bring up? 14:41:01 don't listen to me! 14:41:18 ipanova, I suggest to rename 4604- "...if there is multiple content types in a batch" 14:41:24 I have issues but they're not pulp related 14:41:24 We received an email a few hours ago re: documentation on ansible-pulp of systemd settings? 14:41:29 daviddavis: not sure if you can explain it to bot :) 14:41:39 hehe 14:42:17 ttereshc, +1 on the rename for clarity 14:42:52 dawalker: I saw that email and agree we need to fix the docs 14:42:59 +1 14:43:03 looks like travis is still not working for the pulp_file repo 14:43:15 *sigh* 14:43:27 dkliban: yup, I think we should just merge your pr. it seems like the cron job last night did run though 14:43:39 dkliban, reminder to update the issue 4590 14:43:55 ttereshc: it happens also when there is a 1 dc in a batch, just different one from the RemoveD model 14:44:42 dawalker: already did 14:44:48 dkliban, thanks! 14:44:53 I see 14:44:59 wasn't sure you were around earlier 14:45:05 travis buids seem to run well for me 14:45:27 I'm trying to figure out why there is a shadow one for pulp-cerguard though. (dkliban the new pr and new branch gave the same result) https://github.com/pulp/pulp-certguard/pull/15 14:45:30 bmbouter: me as well except for pulp_file 14:46:02 Do we want to end triage and continue this discussion after? 14:46:33 and I think I figured it out! 14:46:43 +1 to ending triage 14:46:56 so in the branch protection github is still thinking that the 'Travis CI' is a required check 14:47:04 so I think githu is adding it and expecting travis will keep up 14:47:06 but something changed 14:47:07 #endmeeting 14:47:07 !end