openstackgerrit | liusheng proposed openstack/nimble: Add versioned object notification base https://review.openstack.org/415215 | 00:55 |
---|---|---|
liusheng | morning nimble | 01:07 |
*** zhangjl has joined #openstack-nimble | 01:43 | |
zhenguo | liusheng: morning | 01:46 |
liusheng | zhenguo: we may have a new big custorm lol | 01:47 |
zhenguo | liusheng: hah | 01:47 |
silenceli | welcome~ | 01:49 |
*** wanghao has joined #openstack-nimble | 01:56 | |
zhenguo | silenceli: hi, morning! | 01:56 |
wanghao | hi everyone, I'm a new guy here. | 01:57 |
zhenguo | shaohe_feng: are you around? please find my reply here https://review.openstack.org/#/c/413677/ , thanks for the review | 01:57 |
zhenguo | wanghao: hi, welcome here :P | 01:58 |
zhenguo | liusheng: you mean wanghao is our new big customer? lol | 02:00 |
liusheng | zhenguo: no liudong | 02:00 |
liusheng | wanghao: hah, welcome! | 02:00 |
liusheng | zhenguo: you can see the wechat group | 02:00 |
zhenguo | liusheng: hah, | 02:01 |
liusheng | zhenguo: liudong is a bank customer :) | 02:01 |
zhenguo | liusheng: oh, really a good news for us | 02:01 |
wanghao | zhenguo: liusheng: I may be a customer too, hahaha. | 02:02 |
liusheng | wanghao: hah, cool! | 02:02 |
zhenguo | wanghao: hah, another good news! | 02:02 |
wanghao | zhenguo: liusheng: it's really possible, seriously. | 02:02 |
liusheng | wanghao: very welcome | 02:02 |
zhenguo | wanghao: thanks | 02:03 |
liusheng | zhenguo: liudong has talked to me, that they want to choose a baremetal compute project, and he is intereted in Nimble | 02:03 |
zhenguo | liusheng: hah, we can invite him here | 02:04 |
wanghao | zhenguo: +1 | 02:04 |
liusheng | zhenguo: I have toled him the IRC chennal | 02:04 |
liusheng | wanghao: will you also contribute to nimble ? lol | 02:05 |
wanghao | liusheng: zhenguo: I'd like to see we are customer as well as developer. | 02:05 |
wanghao | liusheng: yes, I'd like to. | 02:05 |
liusheng | wanghao: hah, cool | 02:05 |
zhenguo | wanghao: hah, heavily lack of developers now :P | 02:06 |
zhenguo | wanghao, liusheng, silenceli, lambert, zhangjl: seems tomorrow's IRC meeting will be very exicting with all of you | 02:07 |
zhangjl | zhenguo:it`s my pleasure | 02:07 |
liusheng | zhenguo: sure, we can invite all of them | 02:07 |
wanghao | zhenguo: yeah | 02:08 |
Kevin_Zheng | so many customer | 02:08 |
zhenguo | lol | 02:08 |
wanghao | I'd like to invite one more developer to us, he is familiar with Ironic too. | 02:09 |
zhenguo | wanghao: cool | 02:09 |
*** liudong has joined #openstack-nimble | 02:28 | |
zhenguo | oh, the big bank customer is coming!!!!! | 02:28 |
zhenguo | welcome liudong | 02:29 |
liudong | thx | 02:31 |
zhenguo | liudong: kong_, RuiChen, wanghao, liusheng, Kevin_zheng are also here, and hanzhang will be here later :P | 02:33 |
*** liudong has quit IRC | 02:42 | |
*** liudong has joined #openstack-nimble | 02:43 | |
*** liudong_ has joined #openstack-nimble | 02:44 | |
zhenguo | ls | 02:45 |
*** liudong has quit IRC | 02:46 | |
RuiChen | \o/ | 02:54 |
*** shaohe_feng has quit IRC | 02:57 | |
*** shaohe_feng has joined #openstack-nimble | 02:58 | |
*** shaohe_feng is now known as shaohe_fe | 02:59 | |
openstackgerrit | liusheng proposed openstack/python-nimbleclient: Add unit tests for listing servers https://review.openstack.org/415323 | 03:00 |
*** Jeffrey4l_ has quit IRC | 03:03 | |
*** Jeffrey4l_ has joined #openstack-nimble | 03:03 | |
*** Jeffrey4l_ has quit IRC | 03:03 | |
*** Jeffrey4l_ has joined #openstack-nimble | 03:04 | |
*** l4yerffeJ_ has joined #openstack-nimble | 03:04 | |
*** Jeffrey4l_ has quit IRC | 03:04 | |
*** Jeffrey4l has joined #openstack-nimble | 03:04 | |
openstackgerrit | Merged openstack/nimble: Add az check before instance create https://review.openstack.org/414949 | 03:04 |
*** Jeffrey4l has quit IRC | 03:07 | |
*** Jeffrey4l has joined #openstack-nimble | 03:07 | |
*** shaohe_fe has quit IRC | 03:07 | |
*** zly has joined #openstack-nimble | 03:16 | |
*** liusheng has quit IRC | 03:17 | |
*** liusheng has joined #openstack-nimble | 03:18 | |
openstackgerrit | Merged openstack/nimble: Change "uuid" in instance post request body to "net_id" https://review.openstack.org/414775 | 03:29 |
*** wanghao has quit IRC | 03:30 | |
openstackgerrit | Zhenguo Niu proposed openstack/nimble: Add tempest tests for instance creation https://review.openstack.org/404552 | 03:35 |
zhenguo | liusheng: 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 :P | 03:46 |
liusheng | zhenguo: you mean set it to instance, don't set to the node ? | 03:56 |
*** wanghao has joined #openstack-nimble | 04:46 | |
*** liudong_ has quit IRC | 04:47 | |
*** liudong has joined #openstack-nimble | 04:47 | |
*** liudong has quit IRC | 04:53 | |
*** liudong has joined #openstack-nimble | 04:54 | |
zhenguo | liusheng: yes, seems no need to set to node | 04:58 |
*** liudong has quit IRC | 05:00 | |
*** liudong has joined #openstack-nimble | 05:01 | |
*** wanghao has quit IRC | 05:03 | |
*** wanghao has joined #openstack-nimble | 05:03 | |
*** wanghao has quit IRC | 05:04 | |
*** wanghao has joined #openstack-nimble | 05:04 | |
*** wanghao has quit IRC | 05:05 | |
*** wanghao has joined #openstack-nimble | 05:05 | |
*** wanghao has quit IRC | 05:05 | |
*** wanghao has joined #openstack-nimble | 05:06 | |
*** wanghao has joined #openstack-nimble | 05:06 | |
*** wanghao has quit IRC | 05:07 | |
*** wanghao has joined #openstack-nimble | 05:07 | |
*** wanghao has quit IRC | 05:08 | |
*** wanghao has joined #openstack-nimble | 05:08 | |
*** wanghao has quit IRC | 05:09 | |
*** wanghao has joined #openstack-nimble | 05:38 | |
*** wanghao has quit IRC | 05:38 | |
*** wanghao has joined #openstack-nimble | 05:39 | |
*** wanghao_ has joined #openstack-nimble | 05:41 | |
*** wanghao has quit IRC | 05:41 | |
*** wanghao has joined #openstack-nimble | 05:41 | |
*** wanghao_ has joined #openstack-nimble | 05:42 | |
*** wanghao has quit IRC | 05:42 | |
*** liudong has quit IRC | 05:55 | |
*** liudong has joined #openstack-nimble | 05:55 | |
*** liudong has quit IRC | 05:57 | |
*** liudong has joined #openstack-nimble | 05:58 | |
*** yuntongjin has joined #openstack-nimble | 06:03 | |
*** liudong has quit IRC | 06:06 | |
*** liudong has joined #openstack-nimble | 06:06 | |
*** liudong has quit IRC | 06:10 | |
*** liudong has joined #openstack-nimble | 06:10 | |
*** liudong has quit IRC | 06:21 | |
*** liudong has joined #openstack-nimble | 06:22 | |
zhenguo | liusheng: 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 IRC | 06:25 | |
*** liudong has joined #openstack-nimble | 06:25 | |
*** liudong has quit IRC | 06:27 | |
*** liudong has joined #openstack-nimble | 06:27 | |
*** liudong has quit IRC | 06:29 | |
*** liudong has joined #openstack-nimble | 06:30 | |
*** liudong has quit IRC | 06:33 | |
*** liudong has joined #openstack-nimble | 06:33 | |
*** liudong has quit IRC | 06:35 | |
*** liudong has joined #openstack-nimble | 06:36 | |
*** liudong has quit IRC | 06:38 | |
*** liudong has joined #openstack-nimble | 06:38 | |
*** liudong has quit IRC | 06:39 | |
*** liudong has joined #openstack-nimble | 06:40 | |
*** liudong has quit IRC | 06:43 | |
*** liudong has joined #openstack-nimble | 06:43 | |
*** liudong has quit IRC | 06:45 | |
*** liudong has joined #openstack-nimble | 06:46 | |
*** liudong has quit IRC | 06:49 | |
*** liudong has joined #openstack-nimble | 06:49 | |
*** l4yerffeJ_ has quit IRC | 06:49 | |
*** l4yerffeJ_ has joined #openstack-nimble | 06:50 | |
*** liudong has quit IRC | 06:51 | |
*** liudong has joined #openstack-nimble | 06:52 | |
*** liudong has quit IRC | 06:53 | |
*** liudong has joined #openstack-nimble | 06:53 | |
*** liudong has quit IRC | 06:54 | |
*** liudong has joined #openstack-nimble | 06:55 | |
*** liudong has quit IRC | 06:56 | |
*** liudong has joined #openstack-nimble | 06:56 | |
*** liudong has quit IRC | 06:58 | |
*** liudong has joined #openstack-nimble | 06:58 | |
*** liudong has quit IRC | 07:02 | |
*** liudong has joined #openstack-nimble | 07:03 | |
*** liudong has quit IRC | 07:05 | |
*** liudong has joined #openstack-nimble | 07:05 | |
*** liudong has quit IRC | 07:06 | |
*** liudong has joined #openstack-nimble | 07:07 | |
*** liudong has quit IRC | 07:09 | |
*** liudong has joined #openstack-nimble | 07:09 | |
*** liudong has quit IRC | 07:12 | |
*** liudong has joined #openstack-nimble | 07:12 | |
*** liudong has quit IRC | 07:14 | |
*** liudong has joined #openstack-nimble | 07:14 | |
*** liudong has quit IRC | 07:17 | |
*** liudong has joined #openstack-nimble | 07:17 | |
*** liudong has quit IRC | 07:18 | |
*** liudong has joined #openstack-nimble | 07:18 | |
*** liudong has quit IRC | 07:19 | |
*** liudong has joined #openstack-nimble | 07:25 | |
openstackgerrit | liusheng proposed openstack/python-nimbleclient: Change "uuid" in instance post request body to "net-id" https://review.openstack.org/414801 | 08:11 |
*** liudong has quit IRC | 08:11 | |
liusheng | zhenguo: it seems, the above patch cannot be merged, so I remove the 'Depends-On', can you please +2+A again ? | 08:12 |
zhenguo | liusheng: sure, but that's weird | 08:15 |
liusheng | zhenguo: seems someting wrong about the 'Depends-On' | 08:16 |
zhenguo | liusheng: hah | 08:16 |
*** liudong has joined #openstack-nimble | 08:17 | |
openstackgerrit | liusheng proposed openstack/nimble: Don't init engine_api separately in controllers https://review.openstack.org/415393 | 08:23 |
liusheng | zhenguo: oh, it is not because the 'depends-on' lol | 08:23 |
liusheng | zhenguo: it is because the patch https://review.openstack.org/#/c/414787/1 | 08:24 |
zhenguo | liusheng: hah | 08:25 |
zhenguo | liusheng: so you also changed port-type? | 08:25 |
liusheng | zhenguo: yes, actually, the current --port-type parameter cannot be used | 08:26 |
liusheng | zhenguo: because client just pass the 'port-type' directly to nimble-api | 08:26 |
zhenguo | liusheng: you mean we use port_type in nimble, but client will pass port-type there? | 08:26 |
liusheng | zhenguo: but nimble-api need port_type | 08:26 |
liusheng | zhenguo: yes | 08:27 |
zhenguo | liusheng: oh, thanks for fixing that, in fact, I didn't test port_type before, lol | 08:27 |
liusheng | zhenguo: lol, I also don't test the really functionality of --port-type, just try and found the error, and I debug in nimble-api | 08:28 |
liusheng | zhenguo: not sure how to use port-type and not sure we can support port-type currently | 08:29 |
zhenguo | liusheng: hah, we need to test that more, and maybe also need to refactor that part | 08:29 |
zhenguo | liusheng: port type is used to assign different type of interfaces to different network | 08:30 |
liusheng | zhenguo: yes, but that should be need to confirm what port ironic support firstly, right ? | 08:31 |
zhenguo | liusheng: exactly | 08:31 |
zhenguo | liusheng: currently we assume the flavor can help to do this | 08:32 |
liusheng | zhenguo: since I don't know ironic, so I only debug it in nimble-api side, I am lazy, hah | 08:32 |
zhenguo | liusheng: hah, nimble is enough! | 08:32 |
zhenguo | liusheng: apparently we need to do something more to handle the port scheduling | 08:34 |
liusheng | zhenguo: agree | 08:35 |
openstackgerrit | Zhong Luyao proposed openstack/nimble: bug_fix in create_instance taskflow https://review.openstack.org/415400 | 08:35 |
zhenguo | zly: oh, you are back, hah | 08:36 |
zly | zhenguo, yes,:) | 08:36 |
zhenguo | zly: hah, so you have tested the taskflow? | 08:37 |
zly | yes | 08:37 |
openstackgerrit | liusheng proposed openstack/nimble: Avoid to init db api object in every requests https://review.openstack.org/415402 | 08:39 |
zhenguo | zly: hi, I left some comments on your patch, I think we don't need to duplicate code in every revert method | 08:50 |
zly | zhenguo, OK ,thanks | 08:52 |
zhenguo | zly: thanks for fixing that | 08:52 |
openstackgerrit | zhangjl proposed openstack/nimble: Fix translations usage of _ https://review.openstack.org/415406 | 08:54 |
openstackgerrit | zhangjl proposed openstack/nimble: Fix translations usage of _ https://review.openstack.org/415406 | 09:02 |
zhenguo | zhangjl: thanks for the first patch :P | 09:03 |
zhangjl | zhenguo:Thanks | 09:03 |
zhenguo | zhangjl: hah | 09:03 |
zhangjl | i updated the commit message to add the close bug | 09:04 |
openstackgerrit | liusheng proposed openstack/nimble: Don't init engine_api separately in controllers https://review.openstack.org/415393 | 09:04 |
zhangjl | zhenguo:pls review my patch again | 09:04 |
zhenguo | zhangjl: ok | 09:04 |
*** shaohe_feng has joined #openstack-nimble | 09:05 | |
zhangjl | zhenguo:Thanks | 09:05 |
zhenguo | zhangjl: np | 09:05 |
zhangjl | :P | 09:05 |
openstackgerrit | liusheng proposed openstack/nimble: Add instance CURD related functional tests https://review.openstack.org/398068 | 09:05 |
zhenguo | liusheng: 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 |
liusheng | zhenguo: oh, yes, my mistak, the IMPL is a global variable | 09:08 |
zhenguo | liusheng: seems yes | 09:08 |
liusheng | zhenguo: abandon this :) | 09:09 |
zhenguo | liusheng: thanks | 09:09 |
zhenguo | shaohe_feng: your irc is ok now? | 09:09 |
shaohe_feng | zhenguo: yes. I'm OK. | 09:14 |
zhenguo | shaohe_feng: hah | 09:15 |
shaohe_feng | zhenguo: how do you know, I'm OK? | 09:15 |
zhenguo | shaohe_feng: as you just joined | 09:15 |
shaohe_feng | zhenguo: good, But I though I'm always in join. | 09:15 |
*** l4yerffeJ_ has quit IRC | 09:16 | |
zhenguo | shaohe_feng: yes, you are always in, lol | 09:16 |
*** l4yerffeJ_ has joined #openstack-nimble | 09:16 | |
shaohe_feng | hah | 09:18 |
zhenguo | hi 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_meeting | 09:22 |
*** wanghao_ has quit IRC | 09:32 | |
*** wanghao has joined #openstack-nimble | 09:33 | |
*** liudong has joined #openstack-nimble | 09:38 | |
openstackgerrit | Zhenguo Niu proposed openstack/nimble: Trivial-Fix: Set patch directly on unset_instance_info https://review.openstack.org/415417 | 09:40 |
openstackgerrit | liusheng proposed openstack/nimble: Add instance CURD related functional tests https://review.openstack.org/398068 | 09:43 |
*** yuntongjin has quit IRC | 09:47 | |
*** liudong has quit IRC | 09:47 | |
*** liudong has joined #openstack-nimble | 09:48 | |
*** liudong_ has joined #openstack-nimble | 09:50 | |
*** liudong has quit IRC | 09:51 | |
*** l4yerffeJ_ has quit IRC | 09:51 | |
*** l4yerffeJ_ has joined #openstack-nimble | 09:51 | |
*** wanghao has quit IRC | 09:52 | |
*** liudong has joined #openstack-nimble | 09:54 | |
*** liudong_ has quit IRC | 09:54 | |
openstackgerrit | Zhenguo Niu proposed openstack/nimble: Refactor delete_instance https://review.openstack.org/413677 | 09:56 |
*** liudong has quit IRC | 09:58 | |
*** liudong has joined #openstack-nimble | 09:59 | |
openstackgerrit | Zhenguo Niu proposed openstack/nimble: Add default az support https://review.openstack.org/415426 | 10:05 |
openstackgerrit | Zhong Luyao proposed openstack/nimble: bug_fix in create_instance taskflow https://review.openstack.org/415400 | 10:05 |
*** liudong has quit IRC | 10:10 | |
*** liudong has joined #openstack-nimble | 10:11 | |
*** liudong_ has joined #openstack-nimble | 10:14 | |
*** liudong has quit IRC | 10:14 | |
*** liudong has joined #openstack-nimble | 10:16 | |
*** liudong_ has quit IRC | 10:20 | |
*** zhangjl has quit IRC | 10:34 | |
openstackgerrit | liusheng proposed openstack/python-nimbleclient: Add unit tests for server deletion https://review.openstack.org/415434 | 10:34 |
*** liudong has quit IRC | 12:03 | |
openstackgerrit | liusheng proposed openstack/python-nimbleclient: Add unit tests for showing server https://review.openstack.org/415454 | 12:31 |
*** liudong has joined #openstack-nimble | 13:23 | |
openstackgerrit | Merged openstack/python-nimbleclient: Change 'port-type' to 'port_type' when pass the parameter to API https://review.openstack.org/414787 | 14:12 |
openstackgerrit | Merged openstack/python-nimbleclient: Change "uuid" in instance post request body to "net-id" https://review.openstack.org/414801 | 14:12 |
openstackgerrit | Merged openstack/nimble: Fix translations usage of _ https://review.openstack.org/415406 | 14:16 |
openstackgerrit | Merged openstack/nimble: Refactor delete_instance https://review.openstack.org/413677 | 14:28 |
openstackgerrit | Merged openstack/nimble: Trivial-Fix: Set patch directly on unset_instance_info https://review.openstack.org/415417 | 14:32 |
openstackgerrit | Zhenguo Niu proposed openstack/nimble: Add default az support https://review.openstack.org/415426 | 15:57 |
*** liudong_ has joined #openstack-nimble | 16:15 | |
*** liudong has quit IRC | 16:17 | |
*** liudong_ has quit IRC | 19:57 | |
*** l4yerffeJ has joined #openstack-nimble | 21:32 | |
*** Jeffrey4l has quit IRC | 21:34 | |
*** l4yerffeJ_ has quit IRC | 21:35 | |
*** Jeffrey4l has joined #openstack-nimble | 21:47 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!