15:30:42 #startmeeting Pulp Triage 2018-01-05 15:30:42 Meeting started Fri Jan 5 15:30:42 2018 UTC and is due to finish in 60 minutes. The chair is dalley. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:30:42 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:30:42 The meeting name has been set to 'pulp_triage_2018_01_05' 15:30:42 #info dalley has joined triage 15:30:42 dalley: dalley has joined triage 15:30:46 oh 15:30:48 !here 15:30:48 #info asmacdo has joined triage 15:30:48 asmacdo: asmacdo has joined triage 15:30:49 !here 15:30:49 #info ttereshc has joined triage 15:30:50 ttereshc: ttereshc has joined triage 15:30:52 !here 15:30:52 #info daviddavis has joined triage 15:30:52 daviddavis: daviddavis has joined triage 15:31:08 !here 15:31:08 #info bizhang has joined triage 15:31:08 bizhang: bizhang has joined triage 15:31:10 !here 15:31:10 #info ipanova has joined triage 15:31:11 ipanova: ipanova has joined triage 15:31:23 !next 15:31:25 #topic Rewrite redirect URL - http://pulp.plan.io/issues/3227 15:31:25 dalley: 9 issues left to triage: 3227, 3231, 3232, 3239, 3240, 3241, 3242, 3245, 3246 15:31:26 Issue #3227 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:31:26 !here 15:31:26 #info dkliban has joined triage 15:31:27 Rewrite redirect URL - http://pulp.plan.io/issues/3227 15:31:28 dkliban: dkliban has joined triage 15:31:33 i think that's a story 15:31:49 !here 15:31:49 #info milan has joined triage 15:31:49 milan: milan has joined triage 15:32:01 yes, I think last triage we skipped so that we could take note of it before making it a story 15:32:08 !propose other convert to story 15:32:08 #idea Proposed for #3227: convert to story 15:32:08 dalley: Proposed for #3227: convert to story 15:32:18 i'd like to hear more implementation details before accepting this? 15:33:08 !here 15:33:08 #info jortel has joined triage 15:33:08 jortel: jortel has joined triage 15:33:11 initial purpose of crane was doing dumb redirects and nothing more, now it would need to dig into each url and chew on it 15:33:38 ipanova: let's discuss next week 15:33:38 +1 skip 15:33:48 agreed 15:33:49 !propose skip 15:33:49 #idea Proposed for #3227: Skip this issue for this triage session. 15:33:49 dalley: Proposed for #3227: Skip this issue for this triage session. 15:34:01 +1 15:34:07 +1 15:34:14 !skip 15:34:15 dalley: 8 issues left to triage: 3231, 3232, 3239, 3240, 3241, 3242, 3245, 3246 15:34:15 #topic Docker image upload can be more efficient - http://pulp.plan.io/issues/3231 15:34:16 Issue #3231 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:34:17 Docker image upload can be more efficient - http://pulp.plan.io/issues/3231 15:34:28 triage is not for accepting stories ... i think we should convert it to a story so it stops showing up in triage 15:34:39 same for this one 15:34:49 i will leave a note on this one 15:35:00 mind converting it to a story when you do so? 15:35:00 there is an effort ongoing with skopeo 15:35:08 will do 15:35:14 ipanova++ 15:35:14 daviddavis: ipanova's karma is now 14 15:35:15 dkliban, I agree with you but it seems generally useful just to get visibility for them 15:35:45 agree with dalley 15:36:01 adding that to retro etherpad 15:36:05 +1 15:36:06 !here 15:36:06 #info bmbouter has joined triage 15:36:06 bmbouter: bmbouter has joined triage 15:36:09 +1 15:36:36 asmacdo, heh, we discussed it once I believe :) but only that 15:36:43 !propose other ipanova will comment and convert to story 15:36:43 #idea Proposed for #3231: ipanova will comment and convert to story 15:36:43 dalley: Proposed for #3231: ipanova will comment and convert to story 15:36:59 does this apply to the last one as well? 15:37:34 ok, i will convert both o fthem 15:37:51 and leave proper comments 15:38:02 +1 15:38:10 !accept 15:38:10 #agreed ipanova will comment and convert to story 15:38:10 dalley: Current proposal accepted: ipanova will comment and convert to story 15:38:11 dalley: 7 issues left to triage: 3232, 3239, 3240, 3241, 3242, 3245, 3246 15:38:11 #topic As a user, I should be told what I have just uploaded - http://pulp.plan.io/issues/3232 15:38:12 Issue #3232 [POST] (unassigned) - Priority: Normal | Severity: Medium 15:38:13 As a user, I should be told what I have just uploaded - http://pulp.plan.io/issues/3232 15:39:00 i was not sure if we should consider this a bug or a story 15:39:07 i see this more like a bug, because details are empty 15:39:26 i tend to agree with that also 15:39:29 so i suggest to make this an issue and accept as it is 15:39:34 +1 15:39:41 !propose accept 15:39:41 #idea Proposed for #3232: Leave the issue as-is, accepting its current state. 15:39:41 dalley: Proposed for #3232: Leave the issue as-is, accepting its current state. 15:39:42 we have a PR already, i started to test it today 15:39:45 is this backwards incompatible? 15:39:49 one sec 15:40:02 doesn't this change the return data format in a backwards incompatible way? 15:40:07 or is it additive? 15:40:27 no, it just populates empty dict with info form details 15:40:36 ok great, ty 15:40:39 +1 15:40:40 until now details:{} where empty all the time 15:40:44 perfect 15:40:46 +1 15:40:52 +1 15:41:10 !accept 15:41:10 #agreed Leave the issue as-is, accepting its current state. 15:41:10 dalley: Current proposal accepted: Leave the issue as-is, accepting its current state. 15:41:12 #topic POST /api/v3/users/{user_id}/jwt_reset/ returns HTTP 500 - http://pulp.plan.io/issues/3239 15:41:12 dalley: 6 issues left to triage: 3239, 3240, 3241, 3242, 3245, 3246 15:41:13 Issue #3239 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:41:14 POST /api/v3/users/{user_id}/jwt_reset/ returns HTTP 500 - http://pulp.plan.io/issues/3239 15:41:29 I think we can close this out 15:41:32 ipanova, if you're already testing the PR, should I add it to the sprint also? 15:41:51 yes JWT is no longer in the MVP and it needs to be removed 15:42:01 so + 1 to closing somehow 15:42:11 !propose close 15:42:11 dalley: Error: "propose" is not a valid command. 15:42:21 !propose other close 15:42:21 #idea Proposed for #3239: close 15:42:21 dalley: Proposed for #3239: close 15:42:32 +1 15:42:34 close or just remove MVP tag? 15:42:38 dalley: i am not sure, are we ok to put on sprint contributions? 15:43:00 I think close 15:43:02 our new jwt code will probably be created from scratch so this route may not even exist 15:43:07 a bug on code that doesnt' exit doesnt' make sense 15:43:11 +1 15:43:20 !accept 15:43:20 #agreed close 15:43:20 dalley: Current proposal accepted: close 15:43:20 do we have story to bring jwt back? 15:43:21 dalley: 5 issues left to triage: 3240, 3241, 3242, 3245, 3246 15:43:21 #topic Add href_in_list as a filter for models - http://pulp.plan.io/issues/3240 15:43:22 Issue #3240 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:43:23 Add href_in_list as a filter for models - http://pulp.plan.io/issues/3240 15:43:33 ttereshc: no 15:43:44 ty 15:43:44 we could create one but it'll be post-MVP 15:44:00 yeah, we will need to create a bunch of stories anyway 15:44:06 yea 15:44:12 I'm just asking if there's already one, thanks 15:44:18 np 15:44:27 we kind of punted on this id_in_list and href_in_list 15:44:27 this should be a story 15:44:30 +1 15:44:41 !propose other convert to story 15:44:41 #idea Proposed for #3240: convert to story 15:44:41 dalley: Proposed for #3240: convert to story 15:44:45 +1 15:44:46 also, something we need to think about is query string size 15:45:04 katello wants to filter by 100s of ids/hrefs for example 15:45:11 so we may need to figure out filtering by POST 15:45:18 also, should this be in the mvp? 15:45:43 anyway, i can update the story 15:45:53 let's keep it in the mvp for now 15:46:41 +1 to move on and continue discussion on the issue 15:46:41 all sounds good, we can discuss on the issue 15:46:46 :) 15:46:50 +1 15:46:52 +1 15:47:10 !accept 15:47:10 #agreed convert to story 15:47:10 dalley: Current proposal accepted: convert to story 15:47:11 dalley: 4 issues left to triage: 3241, 3242, 3245, 3246 15:47:12 #topic docker_distributor_web does not advertise supporting docker_manifest, docker_tag, docker_blob (any v2 unit types) - http://pulp.plan.io/issues/3241 15:47:12 Issue #3241 [POST] (unassigned) - Priority: Normal | Severity: High 15:47:13 docker_distributor_web does not advertise supporting docker_manifest, docker_tag, docker_blob (any v2 unit types) - http://pulp.plan.io/issues/3241 15:47:49 !propose accept 15:47:49 #idea Proposed for #3241: Leave the issue as-is, accepting its current state. 15:47:49 asmacdo: Proposed for #3241: Leave the issue as-is, accepting its current state. 15:47:51 +1 15:47:56 +1 15:48:13 misa++ 15:48:13 asmacdo: misa's karma is now 4 15:48:38 !accept 15:48:38 #agreed Leave the issue as-is, accepting its current state. 15:48:38 dalley: Current proposal accepted: Leave the issue as-is, accepting its current state. 15:48:39 dalley: 3 issues left to triage: 3242, 3245, 3246 15:48:39 #topic copying a tag does not preserve its pulp_user_metadata - http://pulp.plan.io/issues/3242 15:48:40 Issue #3242 [NEW] (unassigned) - Priority: Normal | Severity: High 15:48:41 copying a tag does not preserve its pulp_user_metadata - http://pulp.plan.io/issues/3242 15:49:06 !propose accept 15:49:06 #idea Proposed for #3242: Leave the issue as-is, accepting its current state. 15:49:06 ipanova: Proposed for #3242: Leave the issue as-is, accepting its current state. 15:49:07 !propose accept 15:49:13 asmacdo: ha! 15:49:15 +1 :) 15:49:55 +1 15:50:08 !accept 15:50:08 #agreed Leave the issue as-is, accepting its current state. 15:50:08 dalley: Current proposal accepted: Leave the issue as-is, accepting its current state. 15:50:09 dalley: 2 issues left to triage: 3245, 3246 15:50:09 #topic Broken link to protected-branches.py - http://pulp.plan.io/issues/3245 15:50:10 Issue #3245 [POST] (milan) - Priority: Low | Severity: Low 15:50:11 Broken link to protected-branches.py - http://pulp.plan.io/issues/3245 15:50:21 !propose accept 15:50:21 #idea Proposed for #3245: Leave the issue as-is, accepting its current state. 15:50:21 dalley: Proposed for #3245: Leave the issue as-is, accepting its current state. 15:50:32 fwiw a oneliner docs update 15:50:39 +1 15:50:44 +1 15:51:05 !accept 15:51:05 #agreed Leave the issue as-is, accepting its current state. 15:51:05 dalley: Current proposal accepted: Leave the issue as-is, accepting its current state. 15:51:07 #topic Update branching.rst to match pup-0003 - http://pulp.plan.io/issues/3246 15:51:07 dalley: 1 issues left to triage: 3246 15:51:08 Issue #3246 [POST] (milan) - Priority: Normal | Severity: Medium 15:51:09 Update branching.rst to match pup-0003 - http://pulp.plan.io/issues/3246 15:51:20 should this be merged to 2.15-dev? 15:51:33 15:51:36 asmacdo: 3245 or 3246? 15:51:38 ha 15:51:42 :) 15:51:46 this one I believe there was some questions over whether it should wait 15:51:57 yea, we should talk to the build team 15:51:57 since the process may change when it gets implemented 15:52:09 I think we can triage this though and follow up with them 15:52:14 agreed 15:52:15 our docs def need to be updated 15:52:17 the merging changes are already inplace 15:52:26 as in "always merge to master" 15:52:35 !propose other accept and continue discussion on issue 15:52:35 #idea Proposed for #3246: accept and continue discussion on issue 15:52:35 dalley: Proposed for #3246: accept and continue discussion on issue 15:52:36 !propose accept 15:52:44 +1 15:52:45 argh, no luck today 15:52:49 +1 15:52:49 lol asmacdo 15:53:05 milan: I want to ping pcreech on monday about 3246 and then we can merge (if that's ok). he's on PTO until then. 15:53:20 slow on the draw :) break has diminished your reflexes xD 15:53:25 daviddavis, sure, thx! 15:53:32 my hands are too cold to type fast 15:53:42 !accept 15:53:42 #agreed accept and continue discussion on issue 15:53:42 dalley: Current proposal accepted: accept and continue discussion on issue 15:53:43 asmacdo: you have no idea what cold is :) 15:53:44 dalley: No issues to triage. 15:53:48 !end 15:53:48 #endmeeting