14:29:46 #startmeeting Pulp Triage 2016-10-18 14:29:46 #info asmacdo has joined triage 14:29:46 Meeting started Tue Oct 18 14:29:46 2016 UTC and is due to finish in 60 minutes. The chair is asmacdo. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:29:46 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:29:46 The meeting name has been set to 'pulp_triage_2016_10_18' 14:29:46 asmacdo has joined triage 14:29:51 !here 14:29:51 #info dkliban has joined triage 14:29:52 dkliban has joined triage 14:29:52 !next 14:29:54 #topic Python egg uploads fail - http://pulp.plan.io/issues/2334 14:29:54 5 issues left to triage: 2334, 2339, 2340, 2342, 2346 14:29:55 Python Support Issue #2334 [NEW] (unassigned) - Priority: Normal | Severity: Medium | Severity: Medium 14:29:56 !here 14:29:56 #info pcreech has joined triage 14:29:56 Python egg uploads fail - http://pulp.plan.io/issues/2334 14:29:57 pcreech has joined triage 14:30:08 !here 14:30:08 #info daviddavis has joined triage 14:30:08 daviddavis has joined triage 14:30:13 !here 14:30:13 #info bizhang has joined triage 14:30:14 bizhang has joined triage 14:30:21 !here 14:30:21 #info jortel has joined triage 14:30:22 jortel has joined triage 14:30:54 I looked into this a bit, and I am really not sure which way to go. it brings up valid concerns, but I am not sure what needs to be done. I think we should accept the bug, it should block 2.12 platform. 14:31:01 !here 14:31:01 #info ttereshc has joined triage 14:31:01 ttereshc has joined triage 14:31:02 !propose triage high medium 14:31:02 #idea Proposed for #2334: Priority: High, Severity: Medium 14:31:02 Proposed for #2334: Priority: High, Severity: Medium 14:31:09 Isn't it broken in 2.11? 14:31:31 smyers, this is in python plugin master 14:31:43 which we decided not to release with 2.11 yesterday 14:32:37 so assuming that current python master goes out with 2.12 it should block 2.12. 14:32:45 !here 14:32:45 #info bmbouter has joined triage 14:32:45 bmbouter has joined triage 14:33:31 is this not a regression in 2.11, since the issue clearly states what he was doing has worked in 2.8-2.10 ?? 14:33:44 or am i way off base? 14:34:20 it is a regression, but it lives in the python master branch, which will become pulp_python 2.0 14:34:35 2.11 will contain pulp_python 1.1 14:34:50 so, werid mismatching of versions then :) 14:35:22 !here 14:35:22 #info mhrivnak has joined triage 14:35:23 mhrivnak has joined triage 14:35:33 yes, we had planned on releasing pulp_python 2.0 with 2.11, but it wasn't worth holding platform for these features 14:35:51 if it is a blocker, then maybe we can set it as Urgent and set Target release for Python plugin as 2.0 and figure out platform target release later. 14:36:06 ^ 14:36:12 ttereshc++ 14:36:12 ttereshc's karma is now 10 14:36:25 !propose triage urgent medium 14:36:25 #idea Proposed for #2334: Priority: Urgent, Severity: Medium 14:36:26 Proposed for #2334: Priority: Urgent, Severity: Medium 14:36:33 +1 14:36:51 Something exploded outside, and now I have no power 14:36:55 So that's fun 14:37:09 O.O 14:37:15 !accept 14:37:15 #agreed Priority: Urgent, Severity: Medium 14:37:16 Current proposal accepted: Priority: Urgent, Severity: Medium 14:37:17 #topic httpd fails to start if AMQP bus for consumers is unavailable - http://pulp.plan.io/issues/2339 14:37:17 4 issues left to triage: 2339, 2340, 2342, 2346 14:37:18 Pulp Issue #2339 [NEW] (unassigned) - Priority: Normal | Severity: Medium | Severity: Medium 14:37:19 httpd fails to start if AMQP bus for consumers is unavailable - http://pulp.plan.io/issues/2339 14:37:29 * mhrivnak notes that we should consider something besides "urgent" to block future releases 14:38:02 mhrivnak, I was thinking that 14:38:10 I've been thinking it'd be good to have a "Blocks Platform Release" field, specifically 14:38:15 yep 14:38:21 ooh, I like that 14:38:27 It's one of the RFCs I've got lined up, but I'm trying to not send more than two or so a week. :) 14:38:35 so this is a bug that ichimonji10 and I discovered on friday 14:38:48 it's an availability problem with Pulp 14:38:54 is "rabbitmq" an invalid value for that setting? 14:39:00 I think normal/med is what we submitted 14:39:11 rabbitmq is valid 14:39:40 Ok. I read the bug report as saying it's invalid, so that would be good to clarify. 14:40:00 "edit the [messaging] section, and set transport to something invalid. For example, try this:" 14:40:13 !propose triage high low 14:40:13 #idea Proposed for #2339: Priority: High, Severity: Low 14:40:13 Proposed for #2339: Priority: High, Severity: Low 14:40:15 It might have been referencing invalid for the current setup 14:40:26 so, while a valid value... it's invalid on that server 14:40:28 that would just be incorrect. 14:41:00 this is the main part: " httpd will not start if the consumer messaging bus is unavailable." 14:41:04 ^ 14:41:13 You could set "transport: foo" and the bug report would read the same. 14:41:31 I'm not good at making this clearer because it's already clear to me 14:41:55 I'm unable to edit bug reports. If someone else with the right priveliges wants to make that change, please do. 14:41:59 *privileges 14:42:06 * bmbouter fixes ichi's perms 14:42:09 ty 14:42:22 in terms of triage: I think normal / med or high / med 14:42:49 For me, it comes down to this: If the bug is "pulp won't start if the transport is set to "foo", that's a reasonable behavior. If the bug is "pulp won't start if it can't connect to a message broker", that's a problem. 14:43:04 So I think focusing on the latter part, and not focusing on validity of a setting, would be helpful. 14:43:46 It's not a big deal, but would be a small improvement. 14:44:08 !propose triage high medium 14:44:08 #idea Proposed for #2339: Priority: High, Severity: Medium 14:44:09 Proposed for #2339: Priority: High, Severity: Medium 14:44:28 any other thoughts on prio before I accept? 14:44:43 I feel like maybe it's too high, but not sure why. 14:45:03 i think its important that it is easy to switch to rabbitmq 14:45:14 so this isn't really about the broker type 14:45:34 the issue is that consumer features are optional, but httpd requires you to correctly configure them in order to start 14:45:36 right, but ichimonji10 mentioned users would hit this when switching? 14:46:19 I'm fine with either norm or high prio 14:46:21 That's when I hit this issue: when switching from qpid to rabbitmq. 14:46:32 actually let's go with normal 14:46:39 !propose normal medium 14:46:39 Error: "propose" is not a valid command. 14:46:42 !propose triage normal medium 14:46:42 #idea Proposed for #2339: Priority: Normal, Severity: Medium 14:46:43 Proposed for #2339: Priority: Normal, Severity: Medium 14:46:50 !propose triage accept 14:46:51 (propose triage [target_release]) -- Propose triage values including priority, severity, and an optional target release. 14:46:55 +1 14:46:56 !propose accept 14:46:56 #idea Proposed for #2339: Leave the issue as-is, accepting its current state. 14:46:57 Proposed for #2339: Leave the issue as-is, accepting its current state. 14:47:00 !accept 14:47:00 #agreed Leave the issue as-is, accepting its current state. 14:47:00 Current proposal accepted: Leave the issue as-is, accepting its current state. 14:47:02 3 issues left to triage: 2340, 2342, 2346 14:47:02 #topic `pulp-admin status` tells the health of the wrong AMQP message bus - http://pulp.plan.io/issues/2340 14:47:03 Pulp Issue #2340 [NEW] (unassigned) - Priority: Normal | Severity: Medium | Severity: Medium 14:47:04 `pulp-admin status` tells the health of the wrong AMQP message bus - http://pulp.plan.io/issues/2340 14:47:13 heh 14:47:13 oops... didn't mean to do that there :) 14:47:22 although, outcome is the same i belive :) 14:47:38 this is another that we discovered on friday 14:48:16 ichimonji10's all up in the consumer business 14:48:24 heh 14:48:48 so this would be to switch which connection settings we use when checking the message bus 14:48:53 That one seems pretty severe, assuming folks use the status view as a main method for monitoring pulp. 14:49:07 agreed. 14:49:17 I would be fine with high / med 14:49:27 Also sounds like not a blocker, not a regression, etc 14:49:29 and putting it on a sprint if we want 14:49:36 it's not a blocker, correct 14:49:48 So it's a big deal, by my hair has not caught fire 14:49:52 it's worked this way since /status/ was created 14:50:02 !propose triage high med 14:50:02 #idea Proposed for #2340: Priority: High, Severity: Medium 14:50:02 #info smyers-nopower has joined triage 14:50:03 smyers-nopower has joined triage 14:50:04 Proposed for #2340: Priority: High, Severity: Medium 14:50:14 +1 14:50:22 +1 14:50:43 +1 14:50:47 !accept 14:50:47 #agreed Priority: High, Severity: Medium 14:50:47 Current proposal accepted: Priority: High, Severity: Medium 14:50:48 2 issues left to triage: 2342, 2346 14:50:48 #topic Vagrant setup does not allow to checkout the code in anywhere except $HOME/devel - http://pulp.plan.io/issues/2342 14:50:49 Pulp Issue #2342 [NEW] (unassigned) - Priority: Normal | Severity: Medium | Severity: Medium 14:50:50 Vagrant setup does not allow to checkout the code in anywhere except $HOME/devel - http://pulp.plan.io/issues/2342 14:51:04 this isnt true 14:51:09 ina and I learned this 14:51:15 https://github.com/pulp/devel/blob/master/Vagrantfile.example#L13 14:51:16 Title: devel/Vagrantfile.example at master · pulp/devel · GitHub (at github.com) 14:51:25 it goes up one level from wherever the code lives 14:51:26 her setup would not work until we made it ~/devel/ 14:51:35 my code on my host lives in ~/code 14:51:53 but the docs say to use ~/devel 14:52:04 the docs check it out into devel on the vagrant machine 14:52:12 but it doens't actually matter 14:52:23 asmacdo: we couldn't get it to work for us 14:52:34 how should we proceed? 14:52:42 hmm, interesting. 14:52:52 I don't think it matters if some can get ~/code to work or not...only ~/devel is what I'd consider supported 14:53:03 If you don't put the code there and it doesn't work...not a bug 14:53:14 hm, it's always worked for me living in a directory not called "devel". 14:53:16 fair enough. 14:53:26 !propose other close-notabug 14:53:26 #idea Proposed for #2342: close-notabug 14:53:26 which folder are we talking about, host os or guest os? 14:53:26 Proposed for #2342: close-notabug 14:53:42 I'm talking about ~/devel on the vagrant guest 14:53:50 oh on the guest? 14:53:54 you can change that 14:53:57 can we not close this and instead discuss more on the bug 14:54:04 we obviously have things to discuss 14:54:27 before we move on, bmbouter do you the answer to pcreech 's question? 14:54:33 sure 14:54:36 host 14:54:50 Agree with bmbouter. I think this will be quickly resolved on the bug, and we can probably close after that discussion 14:54:55 !propose skip 14:54:55 #idea Proposed for #2342: Skip this issue for this triage session. 14:54:56 Proposed for #2342: Skip this issue for this triage session. 14:55:00 Thanks. 14:55:00 +1 14:55:02 until her checkouts lived on the host machine at ~/devel/ it would not work 14:55:11 I'm ok skipping. 14:55:24 cool 14:55:28 !accept 14:55:28 #agreed Skip this issue for this triage session. 14:55:29 Current proposal accepted: Skip this issue for this triage session. 14:55:29 #topic repoview decode errors - http://pulp.plan.io/issues/2346 14:55:30 1 issues left to triage: 2346 14:55:31 Pulp Issue #2346 [NEW] (unassigned) - Priority: Normal | Severity: Medium | Severity: Medium 14:55:32 repoview decode errors - http://pulp.plan.io/issues/2346 14:55:51 Yeah, it sounds like I interpreted it incorrectly, maybe some clarification about htis happening on the host machine would help there because wat that shouldn't matter at all 14:57:08 This one's fun, the error comes when trying to encode the exception message. So maybe something *else* is failing, but it's being stomped by an inability to log the actual exception? 14:57:28 That's my reading as well. 14:57:48 this guy had a weird problem before, related to attempting to log an exception 14:57:48 It's calling str(e) on an exception, whose message apparently contains a non-ascii character. Which is reasonable. 14:57:55 I think hes on a weird linux 14:58:17 he's syncing scientific linux FWIW. 14:58:26 But the bug is still legit I suspect. 15:00:02 !propose accept 15:00:02 #idea Proposed for #2346: Leave the issue as-is, accepting its current state. 15:00:02 Proposed for #2346: Leave the issue as-is, accepting its current state. 15:00:20 works for me 15:00:22 could it be that we are assuming an incorrect encoding? 15:00:31 asmacdo, yes. We're assuming ascii 15:00:34 location_href contains a lot of weird stuff but it's strange that we do not handle it well, for some reason I thought we had such bug in hte past 15:00:58 +1 to accept it as-is 15:01:09 !accept 15:01:09 #agreed Leave the issue as-is, accepting its current state. 15:01:09 Current proposal accepted: Leave the issue as-is, accepting its current state. 15:01:11 No issues to triage. 15:01:27 !end 15:01:27 #endmeeting