19:00:35 <dtruong> #startmeeting senlin
19:00:36 <openstack> Meeting started Thu Jan 31 19:00:35 2019 UTC and is due to finish in 60 minutes.  The chair is dtruong. Information about MeetBot at http://wiki.debian.org/MeetBot.
19:00:37 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
19:00:39 <openstack> The meeting name has been set to 'senlin'
19:00:51 <dtruong> anybody here to join today?
19:02:25 <problem_v> o/
19:02:42 <dtruong> hi problem_v
19:03:33 <dtruong> ok, let's get started
19:03:37 <dtruong> #topic announcements
19:04:19 <dtruong> There are two Senlin related talks proposed by us for the Denver summit
19:04:33 <dtruong> If you are interested, please vote them in the community voting
19:04:42 <dtruong> #link https://www.openstack.org/summit/denver-2019/vote-for-speakers#/23379
19:04:54 <dtruong> #link https://www.openstack.org/summit/denver-2019/vote-for-speakers#/23385
19:06:57 <dtruong> moving on
19:07:03 <dtruong> #topic blueprints
19:07:17 <dtruong> I don't think there are any updates on the existing blueprints
19:08:03 <dtruong> #topic Code Reviews
19:08:27 <dtruong> problem_v can you update us on the status of https://review.openstack.org/#/c/629952/
19:08:52 <problem_v> Kind of on hold for now.
19:09:24 <problem_v> I think I want to approach this issue differently.
19:09:31 <problem_v> Than what I have up for review right now.
19:09:46 <problem_v> As soon as I get some cycles I will get a new patch set up to address this proble,
19:09:49 <problem_v> problem*
19:10:01 <dtruong> Cool. Thanks for the update.
19:10:25 <dtruong> This one has been open for awhile: https://review.openstack.org/#/c/629087/
19:10:43 <dtruong> It looks good but just needs a unit test
19:10:57 <dtruong> If anybody has time, maybe we can help this person out and add the unit test
19:11:27 <problem_v> I will do it if I can find the time.
19:11:47 <dtruong> Thanks
19:11:59 <dtruong> This one needs another +2: https://review.openstack.org/#/c/633100/
19:12:15 <dtruong> eandersson can you review that one ^
19:12:56 <dtruong> problem_v this one has a comment: https://review.openstack.org/#/c/633161/
19:13:18 <dtruong> can you reply to the comment left in the review
19:13:32 <problem_v> Yep :)
19:15:44 <dtruong> These ones need another +2:  https://review.openstack.org/#/c/631365  https://review.openstack.org/#/c/633844/
19:15:56 <dtruong> eandersson ^ when you get a chance
19:17:09 <dtruong> This one is a workaround for memory leak with connections: https://review.openstack.org/#/c/627215/
19:17:18 <dtruong> I was not able to reproduce this issue in devstack
19:17:42 <dtruong> I will try to reproduce it with latest code from master in our environment
19:18:14 <dtruong> But I feel like this not something that should be done in Senlin.
19:18:22 <dtruong> Anyways, we'll see if I can reproduce it
19:19:14 <dtruong> Last one is a backport for a bug fix to rocky: https://review.openstack.org/#/c/634079/
19:19:30 <dtruong> For some reason all the tests are failing
19:19:35 <dtruong> I'll take a look at that one
19:20:36 <dtruong> That covers all the open code reviews
19:20:57 <dtruong> #topic open discussion
19:21:08 <dtruong> that's all i have for today
19:21:20 <dtruong> anybody else have anything to discuss?
19:21:28 <problem_v> I dont have anything else either.
19:23:39 <dtruong> ok, let's end the meeting
19:23:44 <dtruong> thanks for attending
19:24:19 <dtruong> next week's meeting will be on Friday at 530 UTC
19:24:23 <dtruong> #endmeeting