Wednesday, 2016-12-28

openstackgerritliusheng proposed openstack/nimble: Add versioned object notification base  https://review.openstack.org/41521500:55
liushengmorning nimble01:07
*** zhangjl has joined #openstack-nimble01:43
zhenguoliusheng: morning01:46
liushengzhenguo: we may have a new big custorm lol01:47
zhenguoliusheng: hah01:47
silenceliwelcome~01:49
*** wanghao has joined #openstack-nimble01:56
zhenguosilenceli: hi, morning!01:56
wanghaohi everyone, I'm a new guy here.01:57
zhenguoshaohe_feng: are you around? please find my reply here https://review.openstack.org/#/c/413677/ , thanks for the review01:57
zhenguowanghao: hi, welcome here :P01:58
zhenguoliusheng: you mean wanghao is our new big customer? lol02:00
liushengzhenguo: no liudong02:00
liushengwanghao: hah, welcome!02:00
liushengzhenguo: you can see the wechat group02:00
zhenguoliusheng: hah,02:01
liushengzhenguo: liudong is a bank customer :)02:01
zhenguoliusheng: oh, really a good news for us02:01
wanghaozhenguo: liusheng: I may be a customer too, hahaha.02:02
liushengwanghao: hah, cool!02:02
zhenguowanghao: hah, another good news!02:02
wanghaozhenguo: liusheng: it's really possible, seriously.02:02
liushengwanghao: very welcome02:02
zhenguowanghao: thanks02:03
liushengzhenguo: liudong has talked to me, that they want to choose a baremetal compute project, and he is intereted in Nimble02:03
zhenguoliusheng: hah, we can invite him here02:04
wanghaozhenguo: +102:04
liushengzhenguo: I have toled him the IRC chennal02:04
liushengwanghao: will you also contribute to nimble ? lol02:05
wanghaoliusheng: zhenguo: I'd like to see we are customer as well as developer.02:05
wanghaoliusheng: yes, I'd like to.02:05
liushengwanghao: hah, cool02:05
zhenguowanghao: hah, heavily lack of developers now :P02:06
zhenguowanghao, liusheng, silenceli, lambert, zhangjl: seems tomorrow's IRC meeting will be very exicting with all of you02:07
zhangjlzhenguo:it`s my pleasure02:07
liushengzhenguo: sure, we can invite all of them02:07
wanghaozhenguo: yeah02:08
Kevin_Zhengso many customer02:08
zhenguolol02:08
wanghaoI'd like to invite one more developer to us, he is familiar with Ironic too.02:09
zhenguowanghao: cool02:09
*** liudong has joined #openstack-nimble02:28
zhenguooh, the big bank customer is coming!!!!!02:28
zhenguowelcome liudong02:29
liudongthx02:31
zhenguoliudong: kong_, RuiChen, wanghao, liusheng, Kevin_zheng are also here, and hanzhang will be here later :P02:33
*** liudong has quit IRC02:42
*** liudong has joined #openstack-nimble02:43
*** liudong_ has joined #openstack-nimble02:44
zhenguols02:45
*** liudong has quit IRC02:46
RuiChen\o/02:54
*** shaohe_feng has quit IRC02:57
*** shaohe_feng has joined #openstack-nimble02:58
*** shaohe_feng is now known as shaohe_fe02:59
openstackgerritliusheng proposed openstack/python-nimbleclient: Add unit tests for listing servers  https://review.openstack.org/41532303:00
*** Jeffrey4l_ has quit IRC03:03
*** Jeffrey4l_ has joined #openstack-nimble03:03
*** Jeffrey4l_ has quit IRC03:03
*** Jeffrey4l_ has joined #openstack-nimble03:04
*** l4yerffeJ_ has joined #openstack-nimble03:04
*** Jeffrey4l_ has quit IRC03:04
*** Jeffrey4l has joined #openstack-nimble03:04
openstackgerritMerged openstack/nimble: Add az check before instance create  https://review.openstack.org/41494903:04
*** Jeffrey4l has quit IRC03:07
*** Jeffrey4l has joined #openstack-nimble03:07
*** shaohe_fe has quit IRC03:07
*** zly has joined #openstack-nimble03:16
*** liusheng has quit IRC03:17
*** liusheng has joined #openstack-nimble03:18
openstackgerritMerged openstack/nimble: Change "uuid" in instance post request body to "net_id"  https://review.openstack.org/41477503:29
*** wanghao has quit IRC03:30
openstackgerritZhenguo Niu proposed openstack/nimble: Add tempest tests for instance creation  https://review.openstack.org/40455203:35
zhenguoliusheng: for default az, seems we don't even need to set to our node cache, just using the current way to match, and set it to instance is enough :P03:46
liushengzhenguo: you mean set it to instance, don't set to the node ?03:56
*** wanghao has joined #openstack-nimble04:46
*** liudong_ has quit IRC04:47
*** liudong has joined #openstack-nimble04:47
*** liudong has quit IRC04:53
*** liudong has joined #openstack-nimble04:54
zhenguoliusheng: yes, seems no need to set to node04:58
*** liudong has quit IRC05:00
*** liudong has joined #openstack-nimble05:01
*** wanghao has quit IRC05:03
*** wanghao has joined #openstack-nimble05:03
*** wanghao has quit IRC05:04
*** wanghao has joined #openstack-nimble05:04
*** wanghao has quit IRC05:05
*** wanghao has joined #openstack-nimble05:05
*** wanghao has quit IRC05:05
*** wanghao has joined #openstack-nimble05:06
*** wanghao has joined #openstack-nimble05:06
*** wanghao has quit IRC05:07
*** wanghao has joined #openstack-nimble05:07
*** wanghao has quit IRC05:08
*** wanghao has joined #openstack-nimble05:08
*** wanghao has quit IRC05:09
*** wanghao has joined #openstack-nimble05:38
*** wanghao has quit IRC05:38
*** wanghao has joined #openstack-nimble05:39
*** wanghao_ has joined #openstack-nimble05:41
*** wanghao has quit IRC05:41
*** wanghao has joined #openstack-nimble05:41
*** wanghao_ has joined #openstack-nimble05:42
*** wanghao has quit IRC05:42
*** liudong has quit IRC05:55
*** liudong has joined #openstack-nimble05:55
*** liudong has quit IRC05:57
*** liudong has joined #openstack-nimble05:58
*** yuntongjin has joined #openstack-nimble06:03
*** liudong has quit IRC06:06
*** liudong has joined #openstack-nimble06:06
*** liudong has quit IRC06:10
*** liudong has joined #openstack-nimble06:10
*** liudong has quit IRC06:21
*** liudong has joined #openstack-nimble06:22
zhenguoliusheng: sorry, I will set the default az here https://github.com/openstack/nimble/blob/master/nimble/engine/scheduler/node_manager.py#L38 , in case of user specify the default az.06:22
*** liudong has quit IRC06:25
*** liudong has joined #openstack-nimble06:25
*** liudong has quit IRC06:27
*** liudong has joined #openstack-nimble06:27
*** liudong has quit IRC06:29
*** liudong has joined #openstack-nimble06:30
*** liudong has quit IRC06:33
*** liudong has joined #openstack-nimble06:33
*** liudong has quit IRC06:35
*** liudong has joined #openstack-nimble06:36
*** liudong has quit IRC06:38
*** liudong has joined #openstack-nimble06:38
*** liudong has quit IRC06:39
*** liudong has joined #openstack-nimble06:40
*** liudong has quit IRC06:43
*** liudong has joined #openstack-nimble06:43
*** liudong has quit IRC06:45
*** liudong has joined #openstack-nimble06:46
*** liudong has quit IRC06:49
*** liudong has joined #openstack-nimble06:49
*** l4yerffeJ_ has quit IRC06:49
*** l4yerffeJ_ has joined #openstack-nimble06:50
*** liudong has quit IRC06:51
*** liudong has joined #openstack-nimble06:52
*** liudong has quit IRC06:53
*** liudong has joined #openstack-nimble06:53
*** liudong has quit IRC06:54
*** liudong has joined #openstack-nimble06:55
*** liudong has quit IRC06:56
*** liudong has joined #openstack-nimble06:56
*** liudong has quit IRC06:58
*** liudong has joined #openstack-nimble06:58
*** liudong has quit IRC07:02
*** liudong has joined #openstack-nimble07:03
*** liudong has quit IRC07:05
*** liudong has joined #openstack-nimble07:05
*** liudong has quit IRC07:06
*** liudong has joined #openstack-nimble07:07
*** liudong has quit IRC07:09
*** liudong has joined #openstack-nimble07:09
*** liudong has quit IRC07:12
*** liudong has joined #openstack-nimble07:12
*** liudong has quit IRC07:14
*** liudong has joined #openstack-nimble07:14
*** liudong has quit IRC07:17
*** liudong has joined #openstack-nimble07:17
*** liudong has quit IRC07:18
*** liudong has joined #openstack-nimble07:18
*** liudong has quit IRC07:19
*** liudong has joined #openstack-nimble07:25
openstackgerritliusheng proposed openstack/python-nimbleclient: Change "uuid" in instance post request body to "net-id"  https://review.openstack.org/41480108:11
*** liudong has quit IRC08:11
liushengzhenguo: it seems, the above patch cannot be merged, so I remove the 'Depends-On', can you please +2+A again ?08:12
zhenguoliusheng: sure, but that's weird08:15
liushengzhenguo: seems someting wrong about the 'Depends-On'08:16
zhenguoliusheng: hah08:16
*** liudong has joined #openstack-nimble08:17
openstackgerritliusheng proposed openstack/nimble: Don't init engine_api separately in controllers  https://review.openstack.org/41539308:23
liushengzhenguo: oh, it is not because the 'depends-on' lol08:23
liushengzhenguo: it is because the patch https://review.openstack.org/#/c/414787/108:24
zhenguoliusheng: hah08:25
zhenguoliusheng: so you also changed port-type?08:25
liushengzhenguo: yes, actually, the current --port-type parameter cannot be used08:26
liushengzhenguo: because client just pass the 'port-type' directly to nimble-api08:26
zhenguoliusheng: you mean we use port_type in nimble, but client will pass port-type there?08:26
liushengzhenguo: but nimble-api need port_type08:26
liushengzhenguo: yes08:27
zhenguoliusheng: oh, thanks for fixing that, in fact, I didn't test port_type before, lol08:27
liushengzhenguo: lol, I also don't test the really functionality of --port-type, just try and found the error, and I debug in nimble-api08:28
liushengzhenguo: not sure how  to use port-type and not sure we can support port-type currently08:29
zhenguoliusheng: hah, we need to test that more, and maybe also need to refactor that part08:29
zhenguoliusheng: port type is used to assign different type of interfaces to different network08:30
liushengzhenguo: yes, but that should be need to confirm what port ironic support firstly, right ?08:31
zhenguoliusheng: exactly08:31
zhenguoliusheng: currently we assume the flavor can help to do this08:32
liushengzhenguo: since I don't know ironic, so I only debug it in nimble-api side, I am lazy, hah08:32
zhenguoliusheng: hah, nimble is enough!08:32
zhenguoliusheng: apparently we need to do something more to handle the port scheduling08:34
liushengzhenguo: agree08:35
openstackgerritZhong Luyao proposed openstack/nimble: bug_fix in create_instance taskflow  https://review.openstack.org/41540008:35
zhenguozly: oh, you are back, hah08:36
zlyzhenguo, yes,:)08:36
zhenguozly: hah, so you have tested the taskflow?08:37
zlyyes08:37
openstackgerritliusheng proposed openstack/nimble: Avoid to init db api object in every requests  https://review.openstack.org/41540208:39
zhenguozly: hi, I left some comments on your patch, I think we don't need to duplicate code in every revert method08:50
zlyzhenguo, OK ,thanks08:52
zhenguozly: thanks for fixing that08:52
openstackgerritzhangjl proposed openstack/nimble: Fix translations usage of _  https://review.openstack.org/41540608:54
openstackgerritzhangjl proposed openstack/nimble: Fix translations usage of _  https://review.openstack.org/41540609:02
zhenguozhangjl: thanks for the first patch :P09:03
zhangjlzhenguo:Thanks09:03
zhenguozhangjl: hah09:03
zhangjli updated the commit message to add the close bug09:04
openstackgerritliusheng proposed openstack/nimble: Don't init engine_api separately in controllers  https://review.openstack.org/41539309:04
zhangjlzhenguo:pls review my patch again09:04
zhenguozhangjl: ok09:04
*** shaohe_feng has joined #openstack-nimble09:05
zhangjlzhenguo:Thanks09:05
zhenguozhangjl: np09:05
zhangjl:P09:05
openstackgerritliusheng proposed openstack/nimble: Add instance CURD related functional tests  https://review.openstack.org/39806809:05
zhenguoliusheng: wrt https://review.openstack.org/#/c/415402/ , not sure if we really init db api object on every request, seems get_instance just retrun a IMPL?09:07
liushengzhenguo: oh, yes, my mistak, the IMPL is a global variable09:08
zhenguoliusheng: seems yes09:08
liushengzhenguo: abandon this :)09:09
zhenguoliusheng: thanks09:09
zhenguoshaohe_feng: your irc is ok now?09:09
shaohe_fengzhenguo: yes. I'm OK.09:14
zhenguoshaohe_feng: hah09:15
shaohe_fengzhenguo: how do you know, I'm OK?09:15
zhenguoshaohe_feng: as you just joined09:15
shaohe_fengzhenguo: good, But I though I'm always in join.09:15
*** l4yerffeJ_ has quit IRC09:16
zhenguoshaohe_feng: yes, you are always in, lol09:16
*** l4yerffeJ_ has joined #openstack-nimble09:16
shaohe_fenghah09:18
zhenguohi all, just a reminder, tomorrow's team meeting will start at 10 am, please feel free to add agenda here https://wiki.openstack.org/wiki/Meetings/Nimble#Agenda_for_next_meeting09:22
*** wanghao_ has quit IRC09:32
*** wanghao has joined #openstack-nimble09:33
*** liudong has joined #openstack-nimble09:38
openstackgerritZhenguo Niu proposed openstack/nimble: Trivial-Fix: Set patch directly on unset_instance_info  https://review.openstack.org/41541709:40
openstackgerritliusheng proposed openstack/nimble: Add instance CURD related functional tests  https://review.openstack.org/39806809:43
*** yuntongjin has quit IRC09:47
*** liudong has quit IRC09:47
*** liudong has joined #openstack-nimble09:48
*** liudong_ has joined #openstack-nimble09:50
*** liudong has quit IRC09:51
*** l4yerffeJ_ has quit IRC09:51
*** l4yerffeJ_ has joined #openstack-nimble09:51
*** wanghao has quit IRC09:52
*** liudong has joined #openstack-nimble09:54
*** liudong_ has quit IRC09:54
openstackgerritZhenguo Niu proposed openstack/nimble: Refactor delete_instance  https://review.openstack.org/41367709:56
*** liudong has quit IRC09:58
*** liudong has joined #openstack-nimble09:59
openstackgerritZhenguo Niu proposed openstack/nimble: Add default az support  https://review.openstack.org/41542610:05
openstackgerritZhong Luyao proposed openstack/nimble: bug_fix in create_instance taskflow  https://review.openstack.org/41540010:05
*** liudong has quit IRC10:10
*** liudong has joined #openstack-nimble10:11
*** liudong_ has joined #openstack-nimble10:14
*** liudong has quit IRC10:14
*** liudong has joined #openstack-nimble10:16
*** liudong_ has quit IRC10:20
*** zhangjl has quit IRC10:34
openstackgerritliusheng proposed openstack/python-nimbleclient: Add unit tests for server deletion  https://review.openstack.org/41543410:34
*** liudong has quit IRC12:03
openstackgerritliusheng proposed openstack/python-nimbleclient: Add unit tests for showing server  https://review.openstack.org/41545412:31
*** liudong has joined #openstack-nimble13:23
openstackgerritMerged openstack/python-nimbleclient: Change 'port-type' to 'port_type' when pass the parameter to API  https://review.openstack.org/41478714:12
openstackgerritMerged openstack/python-nimbleclient: Change "uuid" in instance post request body to "net-id"  https://review.openstack.org/41480114:12
openstackgerritMerged openstack/nimble: Fix translations usage of _  https://review.openstack.org/41540614:16
openstackgerritMerged openstack/nimble: Refactor delete_instance  https://review.openstack.org/41367714:28
openstackgerritMerged openstack/nimble: Trivial-Fix: Set patch directly on unset_instance_info  https://review.openstack.org/41541714:32
openstackgerritZhenguo Niu proposed openstack/nimble: Add default az support  https://review.openstack.org/41542615:57
*** liudong_ has joined #openstack-nimble16:15
*** liudong has quit IRC16:17
*** liudong_ has quit IRC19:57
*** l4yerffeJ has joined #openstack-nimble21:32
*** Jeffrey4l has quit IRC21:34
*** l4yerffeJ_ has quit IRC21:35
*** Jeffrey4l has joined #openstack-nimble21:47

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