15:30:37 #startmeeting Pulp Triage 2018-01-12 15:30:37 #info dalley has joined triage 15:30:37 Meeting started Fri Jan 12 15:30:37 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:37 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:30:37 The meeting name has been set to 'pulp_triage_2018_01_12' 15:30:37 dalley: dalley has joined triage 15:30:42 !here 15:30:42 #info daviddavis has joined triage 15:30:42 daviddavis: daviddavis has joined triage 15:30:43 doing it this way allows you to name the manifest file whatever 15:30:44 it seemed unnecessarily restrictive to require the file have a special name. 15:30:44 #info milan_ has joined triage 15:30:44 !here 15:30:44 milan_: milan_ has joined triage 15:30:45 !here 15:30:45 #info dkliban has joined triage 15:30:46 dkliban: dkliban has joined triage 15:30:53 !here 15:30:53 #info asmacdo has joined triage 15:30:53 asmacdo: asmacdo has joined triage 15:30:55 !next 15:30:56 dalley: 7 issues left to triage: 3255, 3256, 3257, 3258, 3259, 3270, 3277 15:30:56 !here 15:30:56 #topic pulp-consumer does not remove binding if repo is no longer available on server - http://pulp.plan.io/issues/3255 15:30:56 #info bmbouter has joined triage 15:30:57 Issue #3255 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:30:58 pulp-consumer does not remove binding if repo is no longer available on server - http://pulp.plan.io/issues/3255 15:30:59 bmbouter: bmbouter has joined triage 15:31:06 daviddavis: let's discuss that when we do planning for the file plugin. 15:31:07 !propose skip 15:31:07 #idea Proposed for #3255: Skip this issue for this triage session. 15:31:07 asmacdo: Proposed for #3255: Skip this issue for this triage session. 15:31:14 jortel: sounds good 15:31:28 !here 15:31:28 #info jortel has joined triage 15:31:28 jortel: jortel has joined triage 15:32:02 !accept 15:32:02 #agreed Skip this issue for this triage session. 15:32:02 dalley: Current proposal accepted: Skip this issue for this triage session. 15:32:03 dalley: 6 issues left to triage: 3256, 3257, 3258, 3259, 3270, 3277 15:32:03 !here 15:32:03 #topic all sslclientcert in pulp.repo are called client.crt - http://pulp.plan.io/issues/3256 15:32:03 #info ipanova has joined triage 15:32:04 Issue #3256 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:32:05 all sslclientcert in pulp.repo are called client.crt - http://pulp.plan.io/issues/3256 15:32:06 ipanova: ipanova has joined triage 15:32:34 this seems like a reasonable request, would it be backwards compatible? 15:33:02 !here 15:33:02 #info ttereshc has joined triage 15:33:02 ttereshc: ttereshc has joined triage 15:34:42 thomasmckay: sorry i was in a meeting i hope you got your answer 15:35:04 since it's a bug semver allows a backwards incompatible change 15:35:08 I think it would be backwards incompatible 15:35:38 which should be ok because it's a bug. we should add a backwards incompatible section to the release notes though 15:35:53 agreed 15:35:56 I don't recommend adding to the sprint 15:36:32 ipanova: yes, thanks 15:36:47 +1 not added to sprint 15:36:50 !propose accept 15:36:50 #idea Proposed for #3256: Leave the issue as-is, accepting its current state. 15:36:51 +1 15:36:51 asmacdo: Proposed for #3256: Leave the issue as-is, accepting its current state. 15:36:52 I can comment w/ all of this on the ticket if others think we should accept 15:37:00 +1 15:37:04 bmbouter +1 15:37:07 ack 15:37:08 +1 15:37:09 will do 15:37:19 !accept 15:37:19 #agreed Leave the issue as-is, accepting its current state. 15:37:19 dalley: Current proposal accepted: Leave the issue as-is, accepting its current state. 15:37:20 dalley: 5 issues left to triage: 3257, 3258, 3259, 3270, 3277 15:37:20 #topic pulp does not ask, provide or configure entitlement certificate key - http://pulp.plan.io/issues/3257 15:37:21 Issue #3257 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:37:22 pulp does not ask, provide or configure entitlement certificate key - http://pulp.plan.io/issues/3257 15:37:51 bmbouter, just to avoid edit conflicts, could you also mark as triaged? 15:38:51 dalley: yes 15:39:22 this one sounds like a story 15:39:36 i'm surprised this isn't in a CONFIG somewhere 15:40:14 i thought it's the options to use like --auth-cert 15:40:23 which are available via cli 15:40:52 * asmacdo misses randy :( 15:40:59 --auth-cert - full path to the entitlement certificate that will be given to 15:40:59 bound consumers to grant access to this repository 15:41:21 why would pulp need the key? 15:41:34 oh 15:41:46 agreed, it should just need the cert 15:42:37 let's skip this one for now and leave some comments on the issue 15:42:44 i think we all have some questions 15:42:50 +1 15:43:25 !propose skip 15:43:25 #idea Proposed for #3257: Skip this issue for this triage session. 15:43:25 dalley: Proposed for #3257: Skip this issue for this triage session. 15:43:44 +1 15:44:44 oh, I know why .. it's because in pulp2 the agent will create the .repo file on the consumer as part of bind. 15:46:53 as a work around, the user can put the key in the cert file. yum will work with that. 15:47:09 I'll update the issue. 15:47:13 jortel: thank you 15:47:16 er, comment 15:47:20 ty 15:47:35 so is this still a skip? 15:47:42 or an accept + eddit 15:47:46 *edit 15:47:51 +1 to accept and edit 15:48:17 !propose other accept and jortel will comment 15:48:17 #idea Proposed for #3257: accept and jortel will comment 15:48:17 dalley: Proposed for #3257: accept and jortel will comment 15:48:26 +1 15:49:06 !accept 15:49:06 #agreed accept and jortel will comment 15:49:06 dalley: Current proposal accepted: accept and jortel will comment 15:49:08 #topic Update with new recipe for v2s1 manifest upload - http://pulp.plan.io/issues/3258 15:49:08 dalley: 4 issues left to triage: 3258, 3259, 3270, 3277 15:49:09 Issue #3258 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:49:10 Update with new recipe for v2s1 manifest upload - http://pulp.plan.io/issues/3258 15:49:31 accept as a documentation bug 15:49:41 +1 15:49:48 ipanova: should we add to sprint? 15:49:54 it should have the docs tag 15:50:16 i got some input from dkliban and we decided to accept this as a docs bug 15:50:20 !propose other accept and add docs tag 15:50:20 #idea Proposed for #3258: accept and add docs tag 15:50:20 dalley: Proposed for #3258: accept and add docs tag 15:50:41 and in addition to open a new one, which will deal with last paragraph issues in the description 15:50:50 unless there will be other feedback in the team 15:51:45 on sprint? 15:51:52 1) accept as docs bug, add to sprint 2) ipanova will open an new issue and add to sprint as well 15:51:57 thx 15:52:00 +1 15:52:08 +1 15:52:20 i will comment about 2) so i don't forget 15:52:43 +1 15:52:46 !propose other accept + add to sprint + mark as docs bug, ipanova will split part into a new issue 15:52:46 #idea Proposed for #3258: accept + add to sprint + mark as docs bug, ipanova will split part into a new issue 15:52:46 dalley: Proposed for #3258: accept + add to sprint + mark as docs bug, ipanova will split part into a new issue 15:52:48 !accept 15:52:48 #agreed accept + add to sprint + mark as docs bug, ipanova will split part into a new issue 15:52:48 dalley: Current proposal accepted: accept + add to sprint + mark as docs bug, ipanova will split part into a new issue 15:52:49 dalley: 3 issues left to triage: 3259, 3270, 3277 15:52:49 #topic MongoDB 3 support - http://pulp.plan.io/issues/3259 15:52:50 Issue #3259 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:52:51 MongoDB 3 support - http://pulp.plan.io/issues/3259 15:53:09 I'm tempted to close this out. no one's asking for it and our time could be better spent elsewhere. 15:53:20 fedora is already on mongo 3 too 15:53:46 daviddavis: we already support mongo 3, right? 15:54:04 rchan: we do ... this is more about dropping support for 2 15:54:05 rchan: that's correct. we just don't have it as a dependency for rhel/centos users 15:54:05 this was just to drop mongo 2 support and update our min to >= 3.0 15:54:15 yea pretty much 15:54:20 Can we convert to an ungroomed task and leave open? 15:54:31 I think we will do this at some point 15:54:42 but i agree we have no driver at this time 15:55:11 accepting it and leaving it open/ungroomed for now seems like a good idea to me 15:55:23 and technically, upstream will need to support running with Mongo 2 until all of our stakeholders are Ok with not doing that 15:55:57 yea 15:56:12 !propose accept 15:56:12 #idea Proposed for #3259: Leave the issue as-is, accepting its current state. 15:56:13 dalley: Proposed for #3259: Leave the issue as-is, accepting its current state. 15:56:29 !propose other convert to task and accept 15:56:29 #idea Proposed for #3259: convert to task and accept 15:56:29 asmacdo: Proposed for #3259: convert to task and accept 15:56:33 +1 15:56:34 +1 15:56:38 +1 15:56:43 +1 15:57:02 !accept 15:57:02 #agreed convert to task and accept 15:57:02 dalley: Current proposal accepted: convert to task and accept 15:57:03 dalley: 2 issues left to triage: 3270, 3277 15:57:04 #topic Improve exception handling for incorrect feed_url - http://pulp.plan.io/issues/3270 15:57:04 Issue #3270 [NEW] (unassigned) - Priority: Normal | Severity: Medium 15:57:05 Improve exception handling for incorrect feed_url - http://pulp.plan.io/issues/3270 15:58:05 * ttereshc thought we had a similar issue filed, but she was not able to find it 16:08:18 welcome back pulpbot 16:08:37 pulpbot: NullPointerException 16:08:37 dalley: Error: "NullPointerException" is not a valid command. 16:28:03 re: pulpbot, networking issues in it's hosting infrastructure again :( 16:29:11 asmacdo: the CreatedResource should be cascade deleted when either the task or the resource is deleted. 16:30:18 jortel, i was thinking it was marginally better to keep the record 16:30:27 the task did create an object, but now its gone 16:31:46 The information is only useful when the references are valid. 16:35:56 though, not sure how to get django GenericForeignKey to cascade delete 16:36:54 looks like on_delete is not supported. 16:37:10 but havent looked very hard 16:43:48 jortel: how about https://github.com/pulp/pulp/pull/3265 at least for now 16:44:25 * milan_ calls it a day 16:44:34 asmacdo: yeah, seems fine as a short term fix 16:44:38 have a nice weekend folks! 16:44:44 c'ya, milan_ 16:44:59 o/ 16:46:14 asmacdo: approved 16:47:39 \o 17:48:41 do you sometimes get connection refused from repos.fedorapeople.org? 17:49:19 bmbouter: yeah I have 17:49:51 bmbouter: might be a dns problem 17:50:07 try setting /etc/resolve.conf to use 8.8.8.8 17:50:18 and then restart your workers also 17:50:24 ty 17:51:39 fwiw, I have the same problem too sometimes 17:52:03 that was it I think 18:13:06 bmbouter, yeah I see that fairly often 19:08:34 pcreech: hi, when is 2.15.0 (GA) expected to go out? 19:09:27 misa: he is out today 19:09:52 misa: but i think it should be next wednesday 19:13:14 I think this page is out of date https://pulp.plan.io/projects/pulp/wiki/2150_Release_Status 19:13:51 just a little 19:40:04 dkliban: ouch. I am scheduled to push it out to production on Friday. Hopefully I can push final builds to our test systems on Wed. 19:49:06 misa: did you try out the RC? that is expected to be behaviorally identical 19:53:40 bmbouter: not yet, I am still running the beta 19:53:52 * misa goes to install it 19:54:15 I would probably still do a final test run on Wed after GA is announced, but in terms of what to expect that RC should be it 19:55:25 ah ok, still the beta url. I'll just run an update then 20:13:46 asmacdo: Looks like https://github.com/pulp/pulp/pull/3259 added another tasks.py. there was a tasking.py already and I think the imports should have been added there. 20:39:10 bmbouter: question about asyncio when you can hop on a chat. no hurry, can wait til next week. 20:52:21 jortel: cool, a time on monday would be ideal 20:52:25 I'm dealing w/ a QMF bug 20:52:31 reporting/working aroun 20:52:34 around 20:52:36 lucky you 20:52:57 it's good, it'll be easy for them to fix 20:53:05 I think it's a Python3 issue in their library 20:55:22 ah, interesting 20:55:52 yup 20:56:07 jortel: my calendar is up to date for monday if you want to pick a time it's totally open 20:56:13 we could do first thing for whatever time that is 20:56:15 if you wanted 20:56:38 okay. I'll pulp something on the cal. 20:56:45 er, put :) 20:58:12 haha, ok thanks 21:14:36 jortel: for the file plugin, it's just me and you. do we need to plan anything? 21:15:30 daviddavis: been thinking about that today. Let's setup a planning meeting (you and me) for next week. We should write up all the use cases. should not take long. 21:15:42 jortel: ok, I'll send an invite 21:15:54 daviddavis: thanks.