01:01:06 <zhiyuan> #startmeeting tricircle
01:01:07 <openstack> Meeting started Wed Oct 18 01:01:06 2017 UTC and is due to finish in 60 minutes.  The chair is zhiyuan. Information about MeetBot at http://wiki.debian.org/MeetBot.
01:01:08 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
01:01:10 <openstack> The meeting name has been set to 'tricircle'
01:01:30 <song> hello I am song from szzt. last week our team xiaohan zhang have push the qos code. and we plan to
01:01:30 <song> improve the tricircle test case. is there some detail doc about this task. so we can know how many work
01:01:30 <song> to do and from where to start. thank you!
01:02:23 <RongHui11> hello
01:03:20 <zhiyuan> i see the patch. however only zuul pipeline ran and it voted -1. i try to recheck the patch but the pipeline doesn't run again
01:03:59 <zhiyuan> could you modify the commit message and submit again?
01:04:21 <song> that is ok.
01:04:38 <song> we will do it today.
01:05:15 <zhiyuan> simply follow the commit message of the original qos patch
01:06:12 <song> oh thank you.we will follow that.
01:06:51 <song> after this we want improve the test case for tricricle
01:07:11 <song> is there some detail doc about this?
01:07:24 <song> how many test case we need to add.
01:07:38 <zhiyuan> fine, for that, you can start from adding add/delete sub-port scenario for the trunk test
01:08:05 <RongHui11> good
01:08:08 <zhiyuan> currently only trunk port creation and using the port to boot a VM are covered
01:08:46 <RongHui11> so just add these two case in the trunk smoke test?
01:08:48 <song> good.
01:09:34 <zhiyuan> oh, I temporarily disable the trunk test because of a neutron bug. i notice that the bug fix has been merged, so i will re-enable the test today
01:10:27 <RongHui11> ok
01:10:53 <zhiyuan> to ronghui, yes, since we haven't finish the service chain updating feature
01:10:55 <RongHui11> so after we finish the qos function, we can quickly start this work
01:12:01 <RongHui11> you mean we miss the updating feature in the service chain smoke test?
01:12:46 <zhiyuan> i think i may spend some time reviewing the new patch. it's a huge patch and our previous review history is lost
01:13:08 <zxh> to zhiyuan,I submitted the tricircle qos patch,in openstack-tox-py27 stage,there are 'module' object has no attribute 'DB_INTEGER_MAX_VALUE' mistakes,but DB_INTEGER_MAX_VALUE constants is included in the neutron/common/constants.py,can you help me to look at the environment,thanks.https://review.openstack.org/#/c/512558/2
01:13:59 <zhiyuan> yes, this patch which implement the update feature is no longer active: https://review.openstack.org/#/c/480883/
01:16:00 <zhiyuan> could you check the latest neutron code? it's possible that the constant has been moved to neutron-lib
01:16:20 <RongHui11> i will check it
01:18:36 <joehuang> it's moved to neutron-lib https://github.com/openstack/neutron-lib/blob/51913aaee67f7eb54220dea5a5c7d785cf010768/neutron_lib/db/constants.py
01:19:08 <zhiyuan> thanks, joe
01:19:28 <song> thank you!
01:19:54 <RongHui11> thanks
01:21:10 <zhiyuan> ok, for our queens-1 tag. the document and script update for nova cellv2 has been merged, and there's still some improvement needed for lb
01:22:06 <zhiyuan> patch for default security group udpate has just passed the CI test
01:22:55 <zhiyuan> so i think we postpone the lb and default sg update to queens-2 and release queens-1 with nova cellv2 integration
01:23:00 <RongHui11> to zhiyuan, for the service chain, you mean we should finish this patch https://review.openstack.org/#/c/480883/  or we can submit a new one?
01:23:58 <Yipei> yes, i will fix the local network case, and improve get_subnets this week. hopefully lb can be delivered in queen-2
01:24:01 <zhiyuan> to joe, do we need to wait for neutron team to tag queens-1 first? since we depend on it
01:26:19 <joehuang> we need to wait for Neutron tagging.
01:26:44 <zhiyuan> to ronghui, i think you can ping the author and continue his work if possible
01:28:22 <zhiyuan> to joe, ok
01:31:12 <zhiyuan> to yipei, hopefully the problem we discussed yesterday can be solved by the "get_subnets" improvement :)
01:32:13 <Yipei> to zhiyuan, i hope so. thanks a lot for your help :)
01:33:14 <zhiyuan> you are welcome
01:36:35 <zhiyuan> any other topic? if not, we can finish our meeting earlier
01:36:53 <RongHui11> yes
01:37:24 <zhiyuan> please
01:37:24 <song> ok
01:38:24 <RongHui11> i see the trunk smoke test , we have two function add_subports and remove_subports in trunk.py  so  add/delete sub-port scenario  we haven't finish?
01:40:18 <joehuang> ok
01:41:30 <zhiyuan> these two functions are added in the network_sdk, but not used in the test definition
01:41:40 <RongHui11> got it
01:42:58 <RongHui11> i have no problem :)
01:45:08 <zhiyuan> other topics?
01:45:31 <Yipei> no
01:46:47 <joehuang> no
01:47:18 <zhiyuan> ok, thanks for attending the meeting
01:47:21 <zhiyuan> #endmeeting