14:30:58 #startmeeting Pulp Triage 2018-04-03 14:30:58 #info dalley has joined triage 14:30:59 Meeting started Tue Apr 3 14:30:58 2018 UTC and is due to finish in 60 minutes. The chair is dalley. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:30:59 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:30:59 The meeting name has been set to 'pulp_triage_2018_04_03' 14:30:59 dalley: dalley has joined triage 14:31:01 * daviddavis puts his shirt back on 14:31:02 !here 14:31:02 #info asmacdo has joined triage 14:31:02 asmacdo: asmacdo has joined triage 14:31:04 !here 14:31:04 #info daviddavis has joined triage 14:31:04 daviddavis: daviddavis has joined triage 14:31:10 !here 14:31:10 #info dkliban has joined triage 14:31:10 dkliban: dkliban has joined triage 14:31:12 !here 14:31:12 #info dawalker has joined triage 14:31:12 dawalker: dawalker has joined triage 14:31:15 !next 14:31:17 #topic /api/v3/ and /content/ could collide with a Live API - http://pulp.plan.io/issues/3532 14:31:17 dalley: 9 issues left to triage: 3532, 3534, 3535, 3536, 3537, 3539, 3540, 3541, 3546 14:31:17 daviddavis: so thats why no video :) 14:31:18 Issue #3532 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:31:19 /api/v3/ and /content/ could collide with a Live API - http://pulp.plan.io/issues/3532 14:31:24 lol 14:31:27 !here 14:31:27 #info bmbouter has joined triage 14:31:27 bmbouter: bmbouter has joined triage 14:31:58 lol 14:32:08 I'm recommending we put this on the sprint because it's a significant but easy change 14:32:09 just occurred to me that this one would need some changes to pulp/devel also 14:32:12 yup 14:32:21 +1 add to sprint 14:32:23 I thought we dropped pulp/ from the url for some reason w/r/t live apis to begin with 14:32:26 I don't think it's easy but +1 to add to sprint 14:32:34 we'll need to update pulp-smash, docs, etc 14:32:48 a lot of easy work 14:32:56 !propose other accept and add to sprint 14:32:56 #idea Proposed for #3532: accept and add to sprint 14:32:56 dalley: Proposed for #3532: accept and add to sprint 14:32:59 it's true. low complexity but not wasy 14:33:03 easy 14:33:10 wasy? 14:33:17 somewhat easy = wasy 14:33:38 +1 new tag: wasy 14:33:46 can someone comment on the issue w/ all of the projects that need changes? 14:33:51 checklist, probably 14:34:10 they are all different projects, so ill file issues and relate them to this one 14:34:18 asmacdo++ 14:34:18 dalley: asmacdo's karma is now 39 14:34:24 !accept 14:34:24 #agreed accept and add to sprint 14:34:24 dalley: Current proposal accepted: accept and add to sprint 14:34:26 #topic Error syncing OSTree Repository - http://pulp.plan.io/issues/3534 14:34:26 dalley: 8 issues left to triage: 3534, 3535, 3536, 3537, 3539, 3540, 3541, 3546 14:34:27 Issue #3534 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:34:28 Error syncing OSTree Repository - http://pulp.plan.io/issues/3534 14:34:59 jortel: ^ 14:35:16 !propose skip 14:35:16 #idea Proposed for #3534: Skip this issue for this triage session. 14:35:16 asmacdo: Proposed for #3534: Skip this issue for this triage session. 14:35:19 who else is on the ostree team? 14:35:20 hes in scrum of scrums 14:35:23 ah 14:35:27 that's still going on? 14:35:31 lol 14:35:32 jeez, long one today 14:35:33 maybe not 14:35:40 * asmacdo is making things up 14:35:47 +1 to skppign 14:35:58 !skip 14:36:00 #topic Error syncing Oracle EPEL repository - http://pulp.plan.io/issues/3535 14:36:00 dalley: 7 issues left to triage: 3535, 3536, 3537, 3539, 3540, 3541, 3546 14:36:01 Issue #3535 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:36:01 i'll ping jortel about it later 14:36:02 Error syncing Oracle EPEL repository - http://pulp.plan.io/issues/3535 14:36:12 we can come back to it if jortel materializes 14:37:18 we should look into this issue 14:37:39 at least try to reproduce. i would say that it's worth adding to the sprint to at least try reproducing 14:37:46 the Oracle answers in its own way 14:38:30 ... 14:38:35 heh 14:38:37 accept and add to sprint? 14:38:46 !propose accept and add to sprint 14:38:46 dalley: propose accept Propose accepting the current issue in its current state. 14:38:46 +1 14:38:50 +1 14:38:54 #agreed Skip this issue for this triage session. 14:38:54 !accept 14:38:54 dalley: Current proposal accepted: Skip this issue for this triage session. 14:38:55 #topic Can't get date filters to work - http://pulp.plan.io/issues/3536 14:38:56 dalley: 6 issues left to triage: 3536, 3537, 3539, 3540, 3541, 3546 14:38:57 Issue #3536 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:38:58 Can't get date filters to work - http://pulp.plan.io/issues/3536 14:39:11 ? 14:39:21 current proposal accepted skip? 14:39:47 bmbouter: not sure I understand 14:39:48 nice catch bmbouter 14:40:00 dalley> !accept 14:40:00 10:39 dalley: Current proposal accepted: Skip this issue for this triage session. 14:40:00 lol 14:40:00 oh 14:40:19 well, I did the right thing anyway 14:40:24 moving on :P 14:40:30 for #3536 lets skip for now 14:40:34 (marked accept and added) 14:40:45 ill take an AI to check it and comment 14:40:47 cool thx 14:41:21 with this one, I'm working on filtering lately anyway so i can check it, it would take 30 seconds 14:41:35 I think asmacdo beat you 14:41:46 * asmacdo hands the torch to dalley 14:41:50 !here 14:41:50 #info jortel has joined triage 14:41:50 jortel: jortel has joined triage 14:42:12 !propose other dalley will comment on the issue after investigation 14:42:12 #idea Proposed for #3536: dalley will comment on the issue after investigation 14:42:13 dalley: Proposed for #3536: dalley will comment on the issue after investigation 14:42:24 accept or skip, as far as triage goes? 14:42:30 dalley: im curious so ping me when you finish 14:42:39 skip 14:42:45 !skip 14:42:46 #topic Filters should not exist for /api/v3/repositories//versions// - http://pulp.plan.io/issues/3537 14:42:47 dalley: 5 issues left to triage: 3537, 3539, 3540, 3541, 3546 14:42:48 Issue #3537 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:42:49 Filters should not exist for /api/v3/repositories//versions// - http://pulp.plan.io/issues/3537 14:43:03 asmacdo, date filtering worked for me with worker.last_heartbeat 14:43:08 fwiw 14:43:13 interesting 14:43:18 dalley: it might just be a doc bug in that case 14:43:22 so this is that same issue right? 14:43:31 that all params/filters exist for a whole viewset? 14:43:41 whether list or detail 14:43:49 yea 14:44:20 I don't think this exists outside repo versions though 14:44:21 not sure 14:44:31 i don't think so eitehr 14:45:01 I forgot to tag this issue 14:45:24 I think our api schema is a mess and we should address these ASAP. I vote to add this to the sprint. 14:45:38 !propose other accept and add to sprint 14:45:38 #idea Proposed for #3537: accept and add to sprint 14:45:38 dalley: Proposed for #3537: accept and add to sprint 14:46:02 seems reasonable. lets make sure this isnt widespread 14:46:18 I'll take an AI to add a note 14:46:24 +1 14:46:28 +1 14:46:43 +1 14:46:50 !accept 14:46:50 #agreed accept and add to sprint 14:46:50 dalley: Current proposal accepted: accept and add to sprint 14:46:52 dalley: 4 issues left to triage: 3539, 3540, 3541, 3546 14:46:52 #topic Install drf_openapi when installing pulp3 - http://pulp.plan.io/issues/3539 14:46:53 Issue #3539 [NEW] (unassigned) - Priority: Normal | Severity: Low 14:46:54 Install drf_openapi when installing pulp3 - http://pulp.plan.io/issues/3539 14:47:14 i asked kersom to file this issue 14:47:35 i was hoping that he would add a test that makes sure that the /api/v3/docs/ endpoint is working 14:47:44 +1 14:47:46 !propose other accept, add to sprint, easy fix 14:47:46 #idea Proposed for #3539: accept, add to sprint, easy fix 14:47:46 dalley: Proposed for #3539: accept, add to sprint, easy fix 14:47:51 +1 14:47:53 but it's hard to add a test like that because drf_openapi is an optional dependency 14:48:06 yea, I like bizhang's solution 14:48:16 I also do that is what other projects do too 14:48:18 optional deps 14:48:19 +1 bizhang's solution 14:48:32 should we not include it in 100% of installs? 14:48:44 I think that is the only question still 14:48:57 i dont think its necessary 14:49:03 i think it is 14:49:10 every Foreman install comes with the docs 14:49:18 and i like that about Foreman a lot 14:49:27 that's true 14:49:28 core API is documented in core docs, plugin API is documented in plugin docs 14:49:37 everything serves with this package though 14:49:47 and even if plugins don't core would 14:49:57 yeah, but i bet a lot of users will use the hosted docs 14:50:06 so they won't care about that extra dep 14:50:22 ok so diverse opinions, let's skip and comment on the issue 14:50:24 I'll post the question 14:50:29 !propose skip 14:50:29 #idea Proposed for #3539: Skip this issue for this triage session. 14:50:29 dalley: Proposed for #3539: Skip this issue for this triage session. 14:50:29 ok 14:50:48 im ok to add it, -0 14:51:02 if everyone else agrees, lets not spend more time on it 14:51:05 well, for the purposes of triage,moving on 14:51:19 sounds good 14:51:26 k 14:51:31 adding it or discussion? 14:51:43 moving on 14:51:50 !accept 14:51:50 #agreed Skip this issue for this triage session. 14:51:50 dalley: Current proposal accepted: Skip this issue for this triage session. 14:51:51 dalley: 3 issues left to triage: 3540, 3541, 3546 14:51:51 #topic When pulp_workers are restarted, /etc/default/pulp_workers might be ignored - http://pulp.plan.io/issues/3540 14:51:52 Issue #3540 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:51:53 When pulp_workers are restarted, /etc/default/pulp_workers might be ignored - http://pulp.plan.io/issues/3540 14:51:53 !issue 3534 14:51:53 #topic Error syncing OSTree Repository - http://pulp.plan.io/issues/3534 14:51:54 Issue #3534 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:51:55 Error syncing OSTree Repository - http://pulp.plan.io/issues/3534 14:52:03 switching back to the ostree one 14:52:06 jortel? 14:52:36 I think #3534 should be high prio and sev but not added to the sprint since no bz. 14:52:54 I'm not sure this is ostree specific 14:53:00 !propose triage high high 14:53:00 #idea Proposed for #3534: Priority: High, Severity: High 14:53:00 dalley: Proposed for #3534: Priority: High, Severity: High 14:53:14 the code seems to be in pulp/server/controllers/repository.py 14:53:20 jortel: i told thomasmckay that we will try to reproduce ... if it reproduces he will open a BZ 14:53:40 okay. that sounds good. 14:53:54 !propose other accept and add to sprint, high/high 14:53:54 #idea Proposed for #3534: accept and add to sprint, high/high 14:53:54 dalley: Proposed for #3534: accept and add to sprint, high/high 14:54:10 if we're trying to reproduce it 14:54:42 objections? 14:55:15 or just leave it off the sprint still 14:55:18 ? 14:55:44 on the sprint is fine. 14:55:54 !accept 14:55:54 #agreed accept and add to sprint, high/high 14:55:54 dalley: Current proposal accepted: accept and add to sprint, high/high 14:55:56 * daviddavis does the 4 min warning dance 14:55:56 dalley: 3 issues left to triage: 3540, 3541, 3546 14:55:56 #topic When pulp_workers are restarted, /etc/default/pulp_workers might be ignored - http://pulp.plan.io/issues/3540 14:55:57 Issue #3540 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:55:58 When pulp_workers are restarted, /etc/default/pulp_workers might be ignored - http://pulp.plan.io/issues/3540 14:56:02 but thinking it could be deferred til after the beta 14:58:42 yeah the processes fail to restart 14:58:47 dkliban: I'd be glad to reproduce #3534 as well. lemme know. 14:58:51 so the settings don't take effect 14:59:29 jortel: let's talk to thomasmckay and see how urgent tthis is 14:59:57 alright it's 11:00 15:00:07 should we cut it off? 15:00:34 yes we have another meeting... 15:00:44 * bmbouter does the marathon meeting dance 15:00:46 !end 15:00:46 #endmeeting