14:30:18 #startmeeting Pulp Triage 2017-10-27 14:30:18 #info dalley has joined triage 14:30:19 Meeting started Fri Oct 27 14:30:18 2017 UTC and is due to finish in 60 minutes. The chair is dalley. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:30:19 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:30:19 The meeting name has been set to 'pulp_triage_2017_10_27' 14:30:19 dalley: dalley has joined triage 14:30:33 !here 14:30:33 #info bizhang has joined triage 14:30:33 bizhang: bizhang has joined triage 14:30:40 !here 14:30:40 #info asmacdo has joined triage 14:30:40 asmacdo: asmacdo has joined triage 14:30:41 !here 14:30:41 #info mhrivnak has joined triage 14:30:42 mhrivnak: mhrivnak has joined triage 14:30:43 !here 14:30:43 #info mansari has joined triage 14:30:43 mansari: mansari has joined triage 14:30:57 !next 14:30:58 dalley: 5 issues left to triage: 3093, 3098, 3100, 3101, 3103 14:30:59 #topic api schema includes the same params for all request types - http://pulp.plan.io/issues/3093 14:30:59 Issue #3093 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:31:00 api schema includes the same params for all request types - http://pulp.plan.io/issues/3093 14:31:04 !here 14:31:04 #info bmbouter has joined triage 14:31:04 bmbouter: bmbouter has joined triage 14:31:12 !propose accept 14:31:12 #idea Proposed for #3093: Leave the issue as-is, accepting its current state. 14:31:12 asmacdo: Proposed for #3093: Leave the issue as-is, accepting its current state. 14:31:16 I believe we skipped this one last time pending more discussion 14:31:21 I think this is an issue with drf 3.7.0 14:31:41 I don't know how we should handle it, but I think it is more of a documentation problem than a code problem 14:31:44 Did we resolve whatever it was we decided to wait on? (I don't recall what that was.) 14:31:56 I looked into it 14:31:57 yeah I think we got new comments 14:32:33 !here 14:32:33 #info fdobrovo has joined triage 14:32:33 fdobrovo: fdobrovo has joined triage 14:33:04 This brings up an interesting side-note: maybe we should comment on an issue when we skip it, listing exactly why it was skipped and what needs to happen for it to be triagable. 14:33:12 But we can discuss that post-triage. :) 14:33:14 Yeah, probably a good practice 14:33:19 anyone object to accepting? 14:33:23 we should open this as an issue with upstream drf, since it's working as expected in 3.6.4 14:33:33 +1 accept 14:33:38 bizhang: that is surprising to me 14:33:55 +1 14:33:59 should this go on the sprint? 14:34:10 asmacdo, If I revert to 3.6.4 the only params on DELETE is the name 14:34:14 I can comment on issue 14:34:39 bizhang: oh wow, ok. I think i got it wrong then. Lets accept it, and put it on the sprint. 14:35:12 !propose other accept and add to sprint 14:35:12 #idea Proposed for #3093: accept and add to sprint 14:35:12 dalley: Proposed for #3093: accept and add to sprint 14:35:35 +1 14:36:16 !accept 14:36:16 #agreed accept and add to sprint 14:36:16 dalley: Current proposal accepted: accept and add to sprint 14:36:17 dalley: 4 issues left to triage: 3098, 3100, 3101, 3103 14:36:18 #topic Docker publish may fail with "OSError: [Errno 17] File exists" if two publishes triggered at same time - http://pulp.plan.io/issues/3098 14:36:18 Issue #3098 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:36:19 Docker publish may fail with "OSError: [Errno 17] File exists" if two publishes triggered at same time - http://pulp.plan.io/issues/3098 14:38:05 interesting. 14:38:16 :*( 14:38:20 Behavior shows in pulp_docker, but looks like the problem is in platform. 14:38:53 In any case, I suspect this will be relatively easy to solve. 14:39:11 yeah, if it exists, problem solved, keep going 14:39:30 jortel: is this the problem you were working on in pulp3? 14:39:46 * jortel looks 14:40:08 we might be able to steal some of pulp3 code :) 14:40:17 !propose triage high medium 14:40:17 #idea Proposed for #3098: Priority: High, Severity: Medium 14:40:17 mhrivnak: Proposed for #3098: Priority: High, Severity: Medium 14:41:23 sounds reasonable to me 14:41:56 asmacdo: hard to say. which pulp3 problem do you mean? 14:42:27 the general filesystem race condition 14:42:40 I suggest continuing that discussion later unless it's relevant to triaging? 14:43:16 i'm fine with that, mostly wanted jortel to have a glance before we triage 14:43:17 agreed 14:43:45 !accept 14:43:45 #agreed Priority: High, Severity: Medium 14:43:45 dalley: Current proposal accepted: Priority: High, Severity: Medium 14:43:47 dalley: 3 issues left to triage: 3100, 3101, 3103 14:43:47 #topic Removal of existing iso units doesn't work if there are multiple iso files - http://pulp.plan.io/issues/3100 14:43:48 Issue #3100 [POST] (daviddavis@redhat.com) - Priority: Normal | Severity: Medium 14:43:49 Removal of existing iso units doesn't work if there are multiple iso files - http://pulp.plan.io/issues/3100 14:44:22 POST 14:44:26 #idea Proposed for #3100: Leave the issue as-is, accepting its current state. 14:44:26 !propose accept 14:44:26 asmacdo: Proposed for #3100: Leave the issue as-is, accepting its current state. 14:44:32 !propose other accept and add to sprint 14:44:32 #idea Proposed for #3100: accept and add to sprint 14:44:32 mhrivnak: Proposed for #3100: accept and add to sprint 14:44:45 daviddavis++ 14:44:45 asmacdo: daviddavis's karma is now 17 14:44:48 !accept 14:44:48 #agreed accept and add to sprint 14:44:48 dalley: Current proposal accepted: accept and add to sprint 14:44:49 dalley: 2 issues left to triage: 3101, 3103 14:44:50 #topic Changing a repository's name changes its URI - http://pulp.plan.io/issues/3101 14:44:50 Issue #3101 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:44:51 Changing a repository's name changes its URI - http://pulp.plan.io/issues/3101 14:45:38 i'm tempted to add this one to the sprint too 14:46:06 +1 14:46:27 I think we ought to groom it though 14:46:31 Should we expand the scope of this to all natural keys? 14:46:34 +1 14:46:39 I dont think we do mhrivnak 14:46:56 All natural keys when used to form URL paths? :) 14:46:56 !propose other accept, groom, add to sprint 14:46:56 #idea Proposed for #3101: accept, groom, add to sprint 14:46:56 dalley: Proposed for #3101: accept, groom, add to sprint 14:47:01 some resources need to change their name 14:47:06 wait dalley 14:47:07 yes we should 14:47:19 if we want more discussion we could skip 14:47:37 or continue discussion now b/c I do want it on the sprint soon 14:48:03 asmacdo if a name needs to change, then perhaps the name is not the natural key? 14:48:20 I guess we could just do this for all natural keys, and fix it later if there is an exception 14:48:28 +1 accept 14:48:32 we should have no natural keys in urls 14:48:34 +1 14:48:52 +1 14:49:08 +1 accept, and let's discuss natural keys in URLs separately. 14:49:30 sorry to hold up 14:49:30 just accept, or accept & add to sprint? 14:49:36 but that is different than i was thinking 14:49:41 i just noticed that last comment 14:49:55 i was thinking that the natural keys would be used in urls 14:49:59 but be immutable 14:50:36 That is the current intent. There's a suggestion to change that, but it needs discussion. 14:50:40 agreed 14:51:15 ok, then i'm ok to move on 14:51:35 !accept 14:51:35 #agreed accept, groom, add to sprint 14:51:35 dalley: Current proposal accepted: accept, groom, add to sprint 14:51:36 dalley: 1 issues left to triage: 3103 14:51:36 #topic Missing representation of spawned_tasks in response - http://pulp.plan.io/issues/3103 14:51:37 Issue #3103 [NEW] (unassigned) - Priority: Normal | Severity: Medium 14:51:38 Missing representation of spawned_tasks in response - http://pulp.plan.io/issues/3103 14:52:14 kersom discovered this while implementing the "follow a task and all of its children" code in pulp smash 14:52:29 I think the viewset is just not including that field for some reason 14:52:59 I'm +1 on accepting this, but I'm not sure it's worth adding to the current sprint. 14:53:00 s/viewset/serializer/ 14:53:22 based on what we talked yesterday, the same concept is applied for workers as well. but I did not find a good example to file an issue. it is on my todo list. 14:54:04 for them to port that code I think they need this field which is the sprint motivation 14:54:17 kersom another option is to make the following of child tasks a separate issue 14:54:19 if this is blocking QE +1 add t osprint 14:54:31 Ah. If it's blocking QE I'm good for adding it to the sprint. 14:54:31 in practice there should be few of those since pulp3 doesn't even have auto publish 14:54:44 it is blocking QE if they continue to port the way they planned to 14:54:56 kersom are you +1 to add this to the sprint or? 14:54:58 !propose other accept and add to sprint 14:54:58 #idea Proposed for #3103: accept and add to sprint 14:54:58 dalley: Proposed for #3103: accept and add to sprint 14:55:10 I'm +1 on adding it to the sprint (I think) 14:55:20 it could very helpful for QE 14:55:22 +1 14:55:30 be* 14:56:48 then we do not need to create a code that will be changed in a near future, and it will be very close to what we have in place for pulp2, and it has been used for a while. 14:58:09 any objections? 14:58:10 +1 14:58:22 +1 14:58:25 !accept 14:58:25 #agreed accept and add to sprint 14:58:25 dalley: Current proposal accepted: accept and add to sprint 14:58:27 dalley: No issues to triage. 14:58:35 \o/ 14:59:16 asmacdo, you mentioned that you thought 3101 should be groomed before putting it on the sprint 14:59:27 should we let the person who grooms it add to the sprint? 15:00:05 i was still talking about 3100 15:00:34 oh. no sorry 15:00:52 no, we don't need to groom. I think we should make all natural keys immutable, at least for now. 15:00:53 3100 was the one already in POST 15:01:03 ack 15:01:19 !end 15:01:19 #endmeeting