15:29:18 #startmeeting Pulp Triage 2016-12-09 15:29:18 #info bizhang has joined triage 15:29:19 Meeting started Fri Dec 9 15:29:18 2016 UTC and is due to finish in 60 minutes. The chair is bizhang. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:29:19 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:29:19 The meeting name has been set to 'pulp_triage_2016_12_09' 15:29:19 bizhang has joined triage 15:29:21 !here 15:29:21 #info daviddavis has joined triage 15:29:22 daviddavis has joined triage 15:29:33 !here 15:29:33 #info dkliban has joined triage 15:29:33 dkliban has joined triage 15:30:51 !here 15:30:51 #info pcreech has joined triage 15:30:52 pcreech has joined triage 15:30:54 !next 15:30:55 5 issues left to triage: 2437, 2464, 2465, 2473, 2478 15:30:56 #topic Upgrade from 2.8.0 to 2.10.2: Post upgrade publish creates invalid links for pre-upgrade files - http://pulp.plan.io/issues/2437 15:30:56 Pulp Issue #2437 [NEW] (unassigned) - Priority: Normal | Severity: High 15:30:57 Upgrade from 2.8.0 to 2.10.2: Post upgrade publish creates invalid links for pre-upgrade files - http://pulp.plan.io/issues/2437 15:31:07 we still have not heard from this user 15:31:10 let 15:31:14 !here 15:31:14 #info ipanova has joined triage 15:31:16 ipanova has joined triage 15:31:17 's skip it again 15:31:19 !here 15:31:19 #info bmbouter has joined triage 15:31:19 bmbouter has joined triage 15:31:26 !here 15:31:26 #info ttereshc has joined triage 15:31:26 ttereshc has joined triage 15:31:32 !here 15:31:32 #info preethi has joined triage 15:31:33 preethi has joined triage 15:31:39 dkliban: i propose if he does not reply till next triage, let's close it 15:31:52 !propose needsinfo 15:31:52 Error: "propose" is not a valid command. 15:31:54 ipanova: sounds good to me 15:31:54 !here 15:31:54 #info mhrivnak has joined triage 15:31:55 mhrivnak has joined triage 15:32:06 !triage propose needsinfo 15:32:06 https://pulp.plan.io/issues?query_id=30 15:32:19 uggg it's been a week and I forgot all the triage commands 15:32:30 !propose triage needsinfo 15:32:31 (propose triage [target_release]) -- Propose triage values including priority, severity, and an optional target release. 15:32:35 !here 15:32:35 #info jortel has joined triage 15:32:35 jortel has joined triage 15:32:38 !propose needsinfo 15:32:38 Error: "propose" is not a valid command. 15:33:51 needinfo 15:33:54 not needsinfo 15:34:20 ah oops 15:34:21 !propose needinfo 15:34:21 #idea Proposed for #2437: This issue cannot be triaged without more info. 15:34:22 Proposed for #2437: This issue cannot be triaged without more info. 15:34:45 +1 15:34:54 +1 15:35:14 !accept 15:35:14 #agreed This issue cannot be triaged without more info. 15:35:14 Current proposal accepted: This issue cannot be triaged without more info. 15:35:16 4 issues left to triage: 2464, 2465, 2473, 2478 15:35:16 #topic pulp-admin broken in vagrant environment on 2.y - http://pulp.plan.io/issues/2464 15:35:17 Pulp Issue #2464 [NEW] (unassigned) - Priority: High | Severity: Medium 15:35:18 pulp-admin broken in vagrant environment on 2.y - http://pulp.plan.io/issues/2464 15:36:09 !propose accept 15:36:09 #idea Proposed for #2464: Leave the issue as-is, accepting its current state. 15:36:09 #info smyers has joined triage 15:36:10 smyers has joined triage 15:36:11 Proposed for #2464: Leave the issue as-is, accepting its current state. 15:36:13 pin it and forget it 15:36:22 Sounds good. 15:36:26 cool 15:36:31 !accept 15:36:31 #agreed Leave the issue as-is, accepting its current state. 15:36:31 Current proposal accepted: Leave the issue as-is, accepting its current state. 15:36:32 3 issues left to triage: 2465, 2473, 2478 15:36:32 #topic Vagrant env does not install goferd systemd file - http://pulp.plan.io/issues/2465 15:36:33 Pulp Issue #2465 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:36:34 Vagrant env does not install goferd systemd file - http://pulp.plan.io/issues/2465 15:37:33 medium normal? 15:37:38 +1 15:37:41 how does the rpm get installed without the systemd service O.o 15:38:11 I'm not sure, but this is a 2.y only issue (no goferd) on 3.0-dev 15:38:39 this used to work also (iirc) 15:38:43 the systemd for goferd is installed with "gofer" rpm, not python-gofer 15:39:03 !propose accept 15:39:03 #idea Proposed for #2465: Leave the issue as-is, accepting its current state. 15:39:04 Proposed for #2465: Leave the issue as-is, accepting its current state. 15:39:08 +1 15:39:09 @jortel: can you comment on the issue with that 15:39:13 sure 15:39:23 ty 15:39:29 !propose other jortel to comment 15:39:29 #idea Proposed for #2465: jortel to comment 15:39:30 Proposed for #2465: jortel to comment 15:39:33 !accept 15:39:33 #agreed jortel to comment 15:39:33 Current proposal accepted: jortel to comment 15:39:34 2 issues left to triage: 2473, 2478 15:39:35 #topic mock: there is no such thing as assert_any_with - http://pulp.plan.io/issues/2473 15:39:35 Pulp Issue #2473 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:39:36 mock: there is no such thing as assert_any_with - http://pulp.plan.io/issues/2473 15:39:48 heh, misa++ 15:40:39 !propose accept 15:40:39 #idea Proposed for #2473: Leave the issue as-is, accepting its current state. 15:40:40 Proposed for #2473: Leave the issue as-is, accepting its current state. 15:40:47 ichimonji10: thank you 15:41:03 I did some testing with this, and I'm also not sure how those tests aren't failing right now. 15:41:15 dkliban: I will be able to work on it after finishing the upgrade testing for 2.11 15:41:19 That said, I'ma be that guy, and... 15:41:22 !propose low low 15:41:22 Error: "propose" is not a valid command. 15:41:29 !propose triage low low 15:41:29 #idea Proposed for #2473: Priority: Low, Severity: Low 15:41:30 Proposed for #2473: Priority: Low, Severity: Low 15:42:23 +1 15:42:43 +1 15:43:13 should we also figure out why the tests aren't failing as a part of this? 15:43:32 if tests are failing shouldn't this be higher priority? 15:43:38 Tests aren't failing 15:43:57 At least not in our PR tester? 15:44:40 If tests do start to fail, that will naturally bump the priority. :) 15:44:54 That code has been in place for a very long time, so despite assert_any_with not being a mock method, we've been happily living with it for a while. 15:44:55 makes sense 15:45:14 it must be that the tests aren't running 15:45:33 bmbouter: or an older mock library, as I was hinting 15:45:50 oh yes that could also be it 15:46:02 in that case are we all good with low/low? 15:46:51 i am 15:46:56 if somebody other than me (and I think smyers did) reproduce this, and only those two are failing, then low/low seems appropriate 15:47:12 !accept 15:47:12 #agreed Priority: Low, Severity: Low 15:47:13 Current proposal accepted: Priority: Low, Severity: Low 15:47:14 #topic Support more conservative dependency solving - http://pulp.plan.io/issues/2478 15:47:14 1 issues left to triage: 2478 15:47:15 RPM Support Issue #2478 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:47:16 Support more conservative dependency solving - http://pulp.plan.io/issues/2478 15:47:50 I only reproduced it in a shell, basically just ran Mock().assert_any_with 15:48:19 I was curious about #2478, because I thought we already filtered the dependency list to filter out old evrs with the same name 15:48:52 I'm not very sure about that, though, so I don't feel confident calling regression 15:49:27 #2478 is an RFE 15:49:48 agreed 15:50:01 right now, you can copy an erratum by itself, or copy it with "dep solving" on. 15:50:18 when dep solving is on, it grabs all the RPMs listed in the errata, plus all of their dependencies. 15:50:38 He's asking us to add an option to grab just the RPMs listed in the erratum. 15:51:13 !propose other change to story 15:51:13 #idea Proposed for #2478: change to story 15:51:14 Proposed for #2478: change to story 15:51:45 +1 15:51:45 That's not my understanding, rather it's to grab the RPMs listed in the erratum, and the latest known versions of the deps 15:52:01 that is not my understanding either. nothing about erratum in this issue 15:52:16 jsherrill: care to coomment on #2478? 15:52:21 oh crap, you're right. Wrong issue in my brain. :) 15:52:24 he specifically gives the example that a single package is requested, and only the latest version of the dep should come with it in the example 15:52:26 * mhrivnak re-reads 15:53:00 And I thought we actually did already do that, I just don't remember where (or in which) workflow we do it 15:53:01 Ok yes, still an RFE, but to be less greedy when dep solving. 15:53:14 More greedy 15:53:14 smyers: well the 2nd part was to not pull in the newest version if its already solved in the destination repo 15:53:27 yes :) 15:53:45 (I guess it depends on whether or not you interpret "conservative" to be "greedy" in this case) 15:54:29 jsherrill, *not* pull in the latest version? That seems odd 15:54:37 Gotcha. I was just thinking about "greedy" being that it matches more stuff. 15:54:47 regex greedy, got it 15:55:06 smyers, I think it is more about *needed* version 15:55:11 it may not be hte latest 15:55:21 All makes sense, +1 RFE 15:56:11 I think the particular example pointing to how yum update behaves is particularly helpful, because I also think that's a little odd in exactly the same way :) 15:57:00 heh 15:57:01 +1 for story 15:57:04 !accept 15:57:04 #agreed change to story 15:57:05 Current proposal accepted: change to story 15:57:06 No issues to triage. 15:57:10 !end 15:57:10 #endmeeting