Thursday, 2018-05-31

*** yamamoto has joined #openstack-meeting-501:14
*** markvoelker has joined #openstack-meeting-501:54
*** markvoelker has quit IRC02:02
*** markvoelker has joined #openstack-meeting-502:08
*** markvoelker has quit IRC02:14
*** markvoelker has joined #openstack-meeting-502:15
*** markvoelker has quit IRC02:19
*** slaweq has joined #openstack-meeting-502:26
*** slaweq has quit IRC02:31
*** yamahata has quit IRC02:35
*** slaweq has joined #openstack-meeting-503:11
*** slaweq has quit IRC03:16
*** ianychoi has quit IRC03:51
*** slaweq has joined #openstack-meeting-503:58
*** slaweq has quit IRC04:03
*** radeks__ has joined #openstack-meeting-504:11
*** slaweq has joined #openstack-meeting-504:26
*** slaweq has quit IRC04:31
*** skazi has joined #openstack-meeting-504:36
*** slaweq has joined #openstack-meeting-504:58
*** slaweq has quit IRC05:02
*** slaweq has joined #openstack-meeting-505:30
*** ricolin has joined #openstack-meeting-505:33
*** slaweq has quit IRC05:33
*** slaweq has joined #openstack-meeting-505:34
*** skazi has quit IRC05:37
*** slaweq has quit IRC05:39
*** markvoelker has joined #openstack-meeting-506:05
*** slaweq has joined #openstack-meeting-506:42
*** radeks__ has quit IRC06:43
*** slaweq has quit IRC06:46
*** radeks__ has joined #openstack-meeting-507:24
*** ianychoi has joined #openstack-meeting-507:32
*** markvoelker has quit IRC08:15
*** radeks__ has quit IRC08:19
*** radeks__ has joined #openstack-meeting-508:36
*** radeks__ has quit IRC08:41
*** derekh has joined #openstack-meeting-508:50
*** markvoelker has joined #openstack-meeting-509:04
*** slaweq has joined #openstack-meeting-509:06
*** slaweq has quit IRC09:10
*** ttx has quit IRC09:52
*** slaweq has joined #openstack-meeting-510:11
*** ttx has joined #openstack-meeting-510:11
*** radeks__ has joined #openstack-meeting-510:11
*** ttx has quit IRC10:14
*** ttx has joined #openstack-meeting-510:14
*** slaweq has quit IRC10:15
*** radeks__ has quit IRC10:17
*** radeks__ has joined #openstack-meeting-510:30
*** radeks_ has joined #openstack-meeting-510:32
*** radeks__ has quit IRC10:35
*** radeks_ has quit IRC10:55
*** markvoelker_ has joined #openstack-meeting-511:04
*** markvoelker has quit IRC11:06
*** ttx has quit IRC11:07
*** ttx has joined #openstack-meeting-511:08
*** markvoelker has joined #openstack-meeting-511:14
*** markvoelker_ has quit IRC11:15
*** ttx has quit IRC11:34
*** ttx has joined #openstack-meeting-511:35
*** yamamoto has quit IRC11:41
*** ttx has quit IRC11:47
*** ttx has joined #openstack-meeting-511:47
*** ttx has quit IRC11:51
*** ttx has joined #openstack-meeting-511:51
*** yamamoto has joined #openstack-meeting-512:41
*** yamamoto has quit IRC12:46
*** slaweq has joined #openstack-meeting-512:54
*** yamamoto has joined #openstack-meeting-512:56
*** slaweq has quit IRC12:58
*** slaweq has joined #openstack-meeting-513:10
*** slaweq has quit IRC13:15
*** markvoelker_ has joined #openstack-meeting-513:35
*** markvoelker has quit IRC13:36
*** markvoelker has joined #openstack-meeting-513:38
*** roman_g has joined #openstack-meeting-513:39
*** markvoelker_ has quit IRC13:40
*** lujinluo has joined #openstack-meeting-513:46
*** markvoelker has quit IRC13:52
*** slaweq has joined #openstack-meeting-513:53
*** hongbin has joined #openstack-meeting-513:56
*** slaweq has quit IRC13:57
*** TuanVu has joined #openstack-meeting-513:58
*** hungpv has joined #openstack-meeting-513:59
lujinluo#startmeeting neutron_upgrades14:00
openstackMeeting started Thu May 31 14:00:07 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-san14:00
lujinluohi TuanVu !14:00
lujinluoWe do not have any AIs from last meeting14:01
lujinluoso jump into OVO patches14:01
lujinluohttps://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:bp/adopt-oslo-versioned-objects-for-db14:01
lujinluo#topic OVO14:01
*** openstack changes topic to "OVO (Meeting topic: neutron_upgrades)"14:01
TuanVuok14:01
lujinluohttps://review.openstack.org/#/c/570244/14:01
lujinluothis one is from Nate14:01
njohnstono/14:01
lujinluoThe codes look fine to me. but Zuul failed last time14:02
lujinluohi njohnston14:02
njohnstonThere is an issue with the sphinx docs build job14:02
lujinluolet's wait for the result of Zuul and see how it goes14:02
lujinluonjohnston: does the latest patch solve it?14:02
njohnstonI have https://review.openstack.org/#/c/571301/ to fix it, but I saw there was a discussion between boden and haleyb about whether it was the right approach.14:02
njohnstonI rebased 570244 onto 571301 and the build-sphinx-docs job has already succeeded so I believe it will build14:03
lujinluoi see. from a rough review, i think it will fix it14:05
njohnstonhopefully so :-)14:05
lujinluoyeah14:05
lujinluonext is https://review.openstack.org/#/c/507772/14:06
TuanVuI’ve already left a comment about latest status on the patch14:06
lujinluoi think you have already fixed the RBAC issue? TuanVu14:06
TuanVuyes :)14:06
lujinluogood! i see. tempest is still failing14:07
TuanVuthanks to the help from both you and Ihar :)14:07
TuanVuyeah, I'm not sure how to fix it at this moment14:07
TuanVuso any suggestions are welcome14:07
TuanVuit looks like it's related to attach qos14:08
lujinluoare the failed tests related to qos policies?14:08
TuanVuyes, I think so14:08
lujinluoneutron_tempest_plugin.api.test_qos.RbacSharedQosPoliciesTest.test_network_presence_prevents_policy_rbac_policy_deletion [5.378070s] ... FAILED14:08
lujinluoalso, slawek pointed potential failures of mtu14:10
lujinluohave you checked that as well?14:10
TuanVuyes, I've checked that14:10
TuanVuit looks like there's problem with getting mtu in:14:11
TuanVuhttps://review.openstack.org/#/c/507772/54/neutron/plugins/ml2/plugin.py14:11
lujinluois it one of the 5 failed tests?14:11
TuanVuyes14:12
TuanVuneutron_tempest_plugin.api.test_trunk_negative.TrunkTestMtusJSON. test_add_subport_with_mtu_greater_than_trunk14:12
TuanVu&14:12
TuanVutest_create_trunk_with_mtu_smaller_than_subport14:12
lujinluook, i think i need a bit more time to catch up with the latest patch. will take a closer look rmt. but i understood the current status14:13
TuanVuthank you in advance, Luo-san :)14:13
lujinluonext https://review.openstack.org/#/c/565358/14:14
lujinluothis one is from Ihar14:14
lujinluoand it is blocked by sphinx docs issue14:14
lujinluowe may need to recheck it after Nate's patch got merged14:15
TuanVuyeah, I think so14:16
lujinluoi have added more core reviewers to 571301. hopefully they will take time to review it14:16
lujinluohttps://review.openstack.org/#/c/544206/ this one is on me14:16
lujinluoi have two remaining AIs for this patch: 1) reabse it on Migule's patch; 2) resolve the postgresql issue exposed last time14:17
lujinluofor 1) I plan to wait for Miguel's patch to get merged, https://review.openstack.org/#/c/414251/. and it is very close to be merged now14:18
lujinluofor 2) i need to talk to Ihar, because i could not find which test is deployed with postgresql as backend14:18
lujinluoif you know which one it is, or you know how i can find it out, please help me out14:19
lujinluook, i will try to catch ihar or Miguel or slawek tmr..14:20
lujinluohttps://review.openstack.org/#/c/537320/ this one is still on me..14:21
lujinluoi rebased it to accommodate auto detect engine facade14:21
lujinluobut i have one tempest failure, which i believe is related to my change14:22
lujinluoi will work it out tmr14:22
lujinluo(tmr seems to be a tough day for me)14:22
TuanVuyeah14:22
TuanVubut please take it easy, Luo14:22
lujinluoTuanVu: haha, thanks!14:23
TuanVu:)14:23
lujinluowe do not have any updates of the remaining patches14:23
lujinluodoes anyone want to address any of their patches?14:23
hungpvyeah14:23
lujinluoshoot14:23
hungpvi have a few questions, maybe basic ones, but you know I'm still noob :D14:24
lujinluono problem at all. will try my best14:24
hungpvhttps://review.openstack.org/#/c/549168/8/neutron/tests/unit/extensions/test_l3_ext_gw_mode.py14:24
hungpvL149    self.context.session.flush()14:25
hungpvi'm not sure is this one necessary or not14:25
lujinluoafter converting to OVO, we do not need it anymore. .flush() commits all the pending commits to the sessions, because after calling .add() all the commits are not actually commited, they are in pending state14:26
hungpvthat's great!14:26
lujinluowhile in OVO world, a create() is a commit to db14:26
lujinluogood. anything else you want to discuss?14:27
hungpvyes, and about create() function14:27
lujinluosure14:27
hungpvi saw some patches rewrite it, not use the built-in one in base.py14:28
hungpvcan you give some ideas about it?14:28
hungpvwhat situations we should do this one or other?14:28
lujinluoyeah, normally create() in base.py does the basic stuff. but for some OVO objects, they have special fields, which are not defined in base.py, then it'd better to rewrite14:29
lujinluolet me try to find an example here14:29
lujinluohttps://github.com/openstack/neutron/blob/master/neutron/objects/ports.py#L34414:30
lujinluoPort has QoS and security groups as extensions attaching to it14:31
lujinluobut QoS and security groups are loaded as synthetic field, thus you need to add additional methods to attach them14:31
hungpvi see14:32
lujinluosorry QoS and security groups are NOT loaded as synthetic field14:32
lujinluoso it really depends on what object you are dealing with14:33
hungpvi've got your point14:34
lujinluogreat!14:34
lujinluoanything else you want to discuss?14:35
hungpvyeah, one final question14:35
lujinluosure14:35
hungpvL267 router.gw_port.mac_address)14:36
hungpvif I understand it, I see router db model calls mac _address of gw_port14:37
hungpvjust sure how to convert it when dealing with router object14:37
hungpvcan you shed me some light on it? thank you14:38
lujinluohmm, i do not quit understand your question14:38
lujinluodo you mean you want to know how to deal with gw_port and router objects?14:38
hungpvyes, that's where I'm stuck right now14:39
hungpvthank you for quick understanding14:39
lujinluoas far as i recall, we may need to add port as a synthetic field of router's in order to proceed. but this is a bit tricky. can you try if we can call router.db_obj.gw_port to request gw_port from db model14:41
lujinluojust an idea though14:41
lujinluocan you point me to the patch? i can add one more task to my to-do list for tmr..14:41
hungpv@luo-san: https://review.openstack.org/#/c/549168/14:42
njohnstonA reference to router.gw_port works on L269, so I would expect "router.gw_port['mac_address']" might have a chance of success.14:42
hungpvThank you Nate, gonna try it out14:43
lujinluoyeah, worth a try14:43
lujinluook, i will review the patch tmr14:45
hungpvso nothing more from me. thank you14:45
lujinluo#topic open discussion14:45
*** openstack changes topic to "open discussion (Meeting topic: neutron_upgrades)"14:45
hungpvI'm prepare a new patch, and will update tmr for review14:45
lujinluounderstood, please add us as reviewers14:45
hungpvyes, surely14:46
lujinluodoes anyone have anything to talk about?14:46
TuanVunone from me14:46
lujinluocool! now let us call it a day!14:47
lujinluohave a good night/day everyone14:47
lujinluosee you next week14:47
lujinluo#endmeeting14:47
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/"14:47
openstackMeeting ended Thu May 31 14:47:54 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:47
openstackMinutes:        http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-05-31-14.00.html14:47
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-05-31-14.00.txt14:47
openstackLog:            http://eavesdrop.openstack.org/meetings/neutron_upgrades/2018/neutron_upgrades.2018-05-31-14.00.log.html14:47
TuanVusee you later14:48
TuanVubye :)14:48
njohnstono/14:48
hungpvbye all14:48
*** TuanVu has quit IRC14:48
*** vkmc_ has joined #openstack-meeting-515:06
*** lujinluo has quit IRC15:11
*** skazi has joined #openstack-meeting-515:13
*** skazi has quit IRC15:24
*** hungpv has quit IRC15:43
*** markvoelker has joined #openstack-meeting-516:11
*** ricolin has quit IRC16:12
*** derekh has quit IRC16:13
*** markvoelker has quit IRC16:24
*** amotoki has quit IRC16:35
*** amotoki has joined #openstack-meeting-516:37
*** slaweq has joined #openstack-meeting-517:11
*** vkmc_ has quit IRC17:15
*** felipemonteiro has joined #openstack-meeting-517:15
*** slaweq has quit IRC17:16
*** felipemonteiro has quit IRC17:37
*** felipemonteiro has joined #openstack-meeting-517:37
*** markvoelker has joined #openstack-meeting-517:39
*** yamahata has joined #openstack-meeting-517:50
*** markvoelker has quit IRC17:51
*** radeks_ has joined #openstack-meeting-518:25
*** slaweq has joined #openstack-meeting-518:50
*** slaweq has quit IRC18:54
*** radeks_ has quit IRC18:54
*** slaweq has joined #openstack-meeting-518:54
*** radeks_ has joined #openstack-meeting-518:55
*** radeks_ has quit IRC19:04
*** radeks_ has joined #openstack-meeting-519:24
*** radeks__ has joined #openstack-meeting-519:28
*** radeks_ has quit IRC19:30
*** mjturek has quit IRC20:15
*** skazi has joined #openstack-meeting-520:36
*** radeks__ has quit IRC20:39
*** skazi has quit IRC20:41
*** markvoelker has joined #openstack-meeting-520:51
*** slaweq has quit IRC21:19
*** slaweq has joined #openstack-meeting-521:20
*** slaweq has quit IRC21:24
*** markvoelker has quit IRC21:26
*** markvoelker has joined #openstack-meeting-521:26
*** markvoelker has quit IRC21:31
*** slaweq has joined #openstack-meeting-522:11
*** slaweq has quit IRC22:15
*** felipemonteiro has quit IRC22:33
*** ricolin has joined #openstack-meeting-522:39
*** hongbin has quit IRC22:41

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