Thursday, 2018-10-04

*** iyamahat__ has joined #openstack-meeting-501:32
*** yamahata has quit IRC01:35
*** iyamahat_ has quit IRC01:35
*** iyamahat__ has quit IRC01:38
*** iyamahat has joined #openstack-meeting-501:39
*** iyamahat has quit IRC01:44
*** iyamahat has joined #openstack-meeting-502:00
*** lujinluo has quit IRC02:34
*** roman_g_ has quit IRC02:47
*** ianychoi_ has joined #openstack-meeting-504:15
*** ianychoi has quit IRC04:17
*** skazi has quit IRC04:46
*** lemko has joined #openstack-meeting-504:47
*** slaweq has joined #openstack-meeting-505:11
*** slaweq has quit IRC05:16
*** lujinluo has joined #openstack-meeting-505:51
*** lujinluo has quit IRC05:55
*** lujinluo has joined #openstack-meeting-506:23
*** lujinluo has quit IRC06:27
*** iyamahat has quit IRC06:35
*** lujinluo has joined #openstack-meeting-506:54
*** slaweq has joined #openstack-meeting-506:57
*** lujinluo has quit IRC06:59
*** lemko has quit IRC07:07
*** lemko has joined #openstack-meeting-507:25
*** iyamahat has joined #openstack-meeting-507:53
*** derekh has joined #openstack-meeting-508:13
*** lujinluo has joined #openstack-meeting-508:29
*** lujinluo has quit IRC08:33
*** lujinluo has joined #openstack-meeting-508:45
*** lujinluo has quit IRC08:49
*** iyamahat has quit IRC09:02
*** lujinluo has joined #openstack-meeting-509:16
*** lujinluo has quit IRC09:20
*** lujinluo has joined #openstack-meeting-509:47
*** lujinluo has quit IRC09:52
*** lujinluo has joined #openstack-meeting-510:19
*** lujinluo has quit IRC10:23
*** roman_g has joined #openstack-meeting-510:25
*** slaweq_ has joined #openstack-meeting-510:38
*** slaweq has quit IRC10:40
*** lujinluo has joined #openstack-meeting-510:50
*** lujinluo has quit IRC10:55
*** slaweq__ has joined #openstack-meeting-510:56
*** slaweq_ has quit IRC10:57
*** slaweq__ is now known as slaweq13:36
*** mjturek has joined #openstack-meeting-513:37
*** evrardjp has quit IRC13:40
*** evrardjp has joined #openstack-meeting-513:42
*** lujinluo has joined #openstack-meeting-513:51
*** skazi has joined #openstack-meeting-513:54
*** TuanVu has joined #openstack-meeting-513:54
lujinluo#startmeeting neutron_upgrades14:00
openstackMeeting started Thu Oct  4 14:00:16 2018 UTC and is due to finish in 60 minutes.  The chair is lujinluo. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: neutron_upgrades)"14:00
openstackThe meeting name has been set to 'neutron_upgrades'14:00
lujinluoo/14:00
TuanVuHi Luo-san :D14:01
TuanVunice to see you14:01
lujinluohi TuanVu!14:01
njohnstono/14:01
lujinluook, let's get started14:01
lujinluo#topic OVO14:01
*** openstack changes topic to "OVO (Meeting topic: neutron_upgrades)"14:01
*** iyamahat has joined #openstack-meeting-514:02
lujinluo#link https://review.openstack.org/#/c/605365/ retry decorator in neutron-lib14:02
*** njohnston has quit IRC14:02
*** njohnston has joined #openstack-meeting-514:03
lujinluoMichel addressed a point that after engine facade transform, we would always have a "context" as parameters, thus the lookup maybe unnecessary14:03
njohnstonyes, I read that, it is an interesting point14:04
lujinluoyeah, but as we are still in the middle of the switch, i believe we still need the change14:05
lujinluoand i happen to look up the old engine facade in Neutron yesterday, and this is the current status14:05
lujinluo#link http://codesearch.openstack.org/?q=session.begin&i=nope&files=&repos=neutron14:05
*** haleyb has joined #openstack-meeting-514:06
*** ralonsoh has joined #openstack-meeting-514:07
njohnstonDo you think he would accept it if we put in a note that this would later be deprecated?14:08
njohnstononce the OVO transition is complete it will be rather moot in any event14:08
lujinluoi think this is what we need to do14:08
njohnstonok14:09
lujinluoi will put a comment explaining that, and let's see if he would take it14:11
lujinluo#link https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:bp/adopt-oslo-versioned-objects-for-db14:11
lujinluo#link https://review.openstack.org/#/c/544206/ Port Binding OVO14:11
*** annp has joined #openstack-meeting-514:11
lujinluomanjeet put some comments last week and I have addressed them14:11
lujinluoand I just found that there is another one coming from slawek14:12
lujinluooh, it is totally correct. i will address it very soon too14:12
annpHi14:13
annpSorry I'm late14:13
lujinluoafter that, please feel free to leave your comments14:13
lujinluohi annp, no worries14:13
TuanVuHi An-san :)14:13
lujinluonext is #link https://review.openstack.org/#/c/602362/ retry decorators in Tag14:13
annpHi lujinluo, Tuanvc Thanks.14:14
annp:-)14:14
lujinluoat first, I thought the failures come from the mistake in commit message, but it turns out not..14:14
lujinluothen i do not understand why it is failing almost all the tests14:14
lujinluoi downloaded the patch locally to test it, and the error i met in pep 8 is that it obviously does not understand keyword "context_attr_name"14:16
lujinluonjohnston: will you be available to have a look at it some time?14:16
njohnstonlooking at the py35 tests it looks like the error is "RuntimeError: Could not find position of var context"14:17
njohnstonlujinluo: yes I will take a look14:17
lujinluoyeah, i thought it was the dependency issue thus it could not locate the context14:17
*** hongbin has joined #openstack-meeting-514:17
lujinluobut if the "Depend-On" grammar is correct, it should work14:18
njohnstonso delete_objects is defined as: def delete_objects(cls, context, ...14:18
njohnstoncontext is not a kwarg14:18
njohnstonbut in tag_plugin.py it is being handled as a kwarg, not a positional parameter14:19
njohnston               tag_obj.Tag.delete_objects(14:19
njohnston                    context=context,14:19
njohnston                    standard_attr_id=res.standard_attr_id,14:19
lujinluooh, i see14:20
njohnstonso delete_objects would need to change to utilize context as a kwarg14:20
njohnstonI will comment in gerrit14:21
lujinluothanks!14:22
lujinluonext #link https://review.openstack.org/#/c/602497/ L3HARouterAgentPortBinding in l3_hamode14:22
lujinluothe codes are fine to me. it is a small change14:23
lujinluoso when you are available, please review and comment14:24
njohnstonIt's on my pile for today14:24
lujinluogreat!14:24
lujinluonext #link https://review.openstack.org/#/c/507772/ network OVO14:24
TuanVuThere are new comments from Hongbin and Luo.14:25
TuanVuThanks for your feedback, Luo-san :)14:25
lujinluoi have some concerns about NetworkRBAC14:25
TuanVuyes, I see14:25
lujinluocould you please have a look TuanVu?14:25
TuanVuI was busy with other tasks and haven’t had time to address them yet14:25
TuanVuHowever, I will work on these items tomorrow and next week14:26
lujinluoperfect!14:26
TuanVuthanks again for your help on this patch :D14:26
lujinluonope :)14:26
lujinluonext #link https://review.openstack.org/#/c/549168/ router OVO14:27
annpHi14:27
lujinluoi believe you are still working on it? annp14:27
annplujinluo, yes. I'm working on it14:27
annpThis week I don't have time for this due to internal task.14:27
annpSo I'm planing continue update this one in next week.14:28
annpSo sorry about that. :-)14:28
lujinluono problem! thanks!14:28
annplujinluo, thanks you too.14:29
lujinluothe rest two patches are not updated and we may give them more time14:29
lujinluo#topic engine facade14:29
*** openstack changes topic to "engine facade (Meeting topic: neutron_upgrades)"14:29
lujinluo#link https://review.openstack.org/#/q/topic:bp/enginefacade-switch+status:open14:30
lujinluoas i mentioned earlier, i was looking up the current old engine facades, and the result if here #link http://codesearch.openstack.org/?q=session.begin&i=nope&files=&repos=neutron14:30
lujinluoif you feel you have more bandwidth, you are more than welcome to take the small pieces! :)14:31
lujinluoi will try to upload a small one later this week14:31
lujinluo#topic open discussion14:31
*** openstack changes topic to "open discussion (Meeting topic: neutron_upgrades)"14:32
njohnstonthanks, that is great14:32
lujinluoDoes anyone have anything else to share?14:32
TuanVunothing else from me14:33
njohnstonnothing for me.  I have somewhat limited bandwidth right now, mostly because my top priority is the bulk port optimization bp, but after S-1 I should have more time14:33
lujinluogot it. njohnston14:33
lujinluook then, let's give the rest time back to us14:34
njohnstonthanks all!14:34
lujinluothank you guys and see you next week14:34
lujinluo#endmeeting14:34
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"14:34
openstackMeeting ended Thu Oct  4 14:34:35 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:34
openstackMinutes:        http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-10-04-14.00.html14:34
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-10-04-14.00.txt14:34
openstackLog:            http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-10-04-14.00.log.html14:34
*** lujinluo has quit IRC14:44
*** iyamahat has quit IRC15:08
*** annp has quit IRC15:16
*** lujinluo has joined #openstack-meeting-515:32
*** lujinluo has quit IRC15:36
*** iyamahat has joined #openstack-meeting-516:14
*** TuanVu has quit IRC16:24
*** iyamahat has quit IRC16:51
*** derekh has quit IRC17:04
*** iyamahat has joined #openstack-meeting-517:04
*** yamahata has joined #openstack-meeting-517:04
*** iyamahat_ has joined #openstack-meeting-518:02
*** iyamahat has quit IRC18:05
*** lemko has quit IRC18:27
*** ralonsoh has quit IRC18:52
*** mjturek has quit IRC19:17
*** mjturek has joined #openstack-meeting-519:49
*** mjturek has quit IRC20:00
*** mjturek has joined #openstack-meeting-520:12
*** mjturek has quit IRC20:17
*** mjturek has joined #openstack-meeting-520:19
*** mjturek has quit IRC20:23
*** mjturek has joined #openstack-meeting-520:48
*** mjturek has quit IRC20:54
*** hongbin has quit IRC22:57

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!