Friday, 2017-02-03

*** wanghao has joined #openstack-mogan00:52
*** liusheng has joined #openstack-mogan00:56
*** wanghao_ has joined #openstack-mogan01:18
*** wanghao has quit IRC01:23
*** Kevin_Zheng has joined #openstack-mogan01:32
*** litao has joined #openstack-mogan01:34
zhenguomorning mogan!01:40
*** wanghao has joined #openstack-mogan01:54
*** litao has quit IRC01:54
*** wanghao_ has quit IRC01:55
*** openstackgerrit has joined #openstack-mogan02:06
openstackgerritwanghao proposed openstack/mogan master: Quota support in Mogan(part 1)  https://review.openstack.org/41732602:06
openstackgerritMerged openstack/python-moganclient master: Updated from global requirements  https://review.openstack.org/42368402:07
*** RuiChen has joined #openstack-mogan02:18
openstackgerritMerged openstack/mogan master: Add support for floating IPs association  https://review.openstack.org/41776002:53
openstackgerritZhenguo Niu proposed openstack/mogan master: Log pending task when instance is in unstable states  https://review.openstack.org/42855403:03
openstackgerritwanghao proposed openstack/mogan master: Quota support in Mogan(part 1)  https://review.openstack.org/41732603:04
openstackgerritwanghao proposed openstack/mogan master: Support quota in mogan(part two)  https://review.openstack.org/42855703:21
wanghaohi guys,  now quota patches have been committed all, pls review if you have time.03:25
zhenguowanghao: awesome,  thanks for the hard work, will review soon!03:27
wanghaozhenguo: thanks,03:28
*** Kevin_Zheng has quit IRC03:37
*** l4yerffeJ_ has quit IRC03:52
*** l4yerffeJ_ has joined #openstack-mogan03:53
openstackgerritliusheng proposed openstack/mogan master: Refactor policy: Support precision control of power action  https://review.openstack.org/42495104:17
* zhenguo join #openstack-stackalytics04:51
openstackgerritMerged openstack/mogan master: Log pending task when instance is in unstable states  https://review.openstack.org/42855405:42
*** Kevin_Zheng has joined #openstack-mogan06:33
openstackgerritZhenguo Niu proposed openstack/mogan master: Weigh nodes by their ports quantity  https://review.openstack.org/42860006:54
openstackgerritTao Li proposed openstack/mogan master: Support multi-instance creating (part 1)  https://review.openstack.org/42106407:19
*** litao has joined #openstack-mogan07:21
litaotest07:22
zhenguohi all, I just added a weigher for ports quantity which default to choose the node with less ports associated, do you think it makes sense?07:32
zhenguoliusheng, wanghao, litao: hi07:34
liushengzhenguo: hi07:34
zhenguoliusheng: do you like the new weigher :P07:34
zhenguoFYI, our stats in stacklytics has been broken after project renaming :(07:38
zhenguohope this can fix it, https://review.openstack.org/#/c/428572/07:38
liushengzhenguo: cool07:41
litaozhenguo: no problem08:03
openstackgerritliusheng proposed openstack/mogan master: Refactor policy: Support precision control of power action  https://review.openstack.org/42495108:04
openstackgerritliusheng proposed openstack/mogan master: Use policy-generator to generate policy sample file  https://review.openstack.org/42496208:05
zhenguoliusheng: regarding https://review.openstack.org/#/c/428600/1/mogan/engine/scheduler/weights/port.py , please find my reply inline, thanks08:23
liushengzhenguo: oh, sorry, seems it leaded my to Nova's code of "get_all_classes" with load class in a folder. lol08:35
zhenguoliusheng: hah,08:36
liushengzhenguo: do you think the namespace is a bit too long? I guess it may used for keep compatibility of the old extension loading mechanism08:37
liushengzhenguo: e.g. mogan.engine.scheduler.weights08:37
liushengzhenguo: the namespace of stevedore don't need to be the path of class08:38
zhenguoliusheng: seems yes, but this maybe more clear to use full path, wdyt?08:38
liushengzhenguo: I am ok with it, but just curious because I didn't found the usage like that in Ceilometer, the full patch can be indicated by the class path after '='. the namespace is just a symbol08:41
zhenguoliusheng: yes, anyway is ok, hah, I don't have strong opinion about this08:42
zhenguoliusheng: as we copied cinder's scheduler, so keep that way, the filter entry point and now the weight entry point08:43
liushengzhenguo: yes, I found that, the cinder may copied from Nova very early and then they guys switch to use stevedore, hah08:44
zhenguoliusheng: hah, yes08:44
zhenguoliusheng: I remember you suggested to rename network info to nics right?08:51
liushengzhenguo: yes08:51
zhenguoliusheng: seems nics is a good option08:51
zhenguoliusheng: I don't like nova's addresses field08:51
zhenguoliusheng: and ports also seems a bit confused,08:52
liushengzhenguo: hah, just my opinion08:53
zhenguoliusheng: hah, I like that08:53
zhenguoliusheng: when creating a server we need to specify --nic, so it should return a server with nics08:54
zhenguoliusheng: we can refactor that, but need to construct the fields in nics08:54
liushengzhenguo: looks more brief08:55
liushengzhenguo: yes08:55
zhenguoliusheng: like port_id, fixed_address, port_type, mac address, etc08:55
openstackgerritliusheng proposed openstack/mogan master: Use policy-generator to generate policy sample file  https://review.openstack.org/42496208:56
liushengzhenguo: agree, btw, we can support ipv6 ?08:57
zhenguoliusheng: I think yes,08:57
liushengzhenguo: cool08:58
zhenguoliusheng: hah08:58
*** liusheng has quit IRC09:07
*** liusheng has joined #openstack-mogan09:07
*** wanghao_ has joined #openstack-mogan09:23
openstackgerritZhenguo Niu proposed openstack/mogan master: Add power_state to InstancePayLoad  https://review.openstack.org/42866209:24
zhenguoliusheng: hi, I just put a patch to added power_state to InstancePayLoad, as we have a power_state attribute now09:26
*** wanghao has quit IRC09:26
liushengzhenguo: ok, thanks09:27
openstackgerritMerged openstack/mogan master: Weigh nodes by their ports quantity  https://review.openstack.org/42860009:28
*** wanghao_ has quit IRC09:29
zhenguoliusheng: but I didn't bump ther versions09:29
liushengzhenguo: may don't need as we do with db schema version, hah09:30
zhenguoliusheng: hah09:30
*** wanghao has joined #openstack-mogan09:30
*** wanghao has quit IRC09:30
zhenguoliusheng: hi, wrt https://review.openstack.org/#/c/424951/ , can we combine the tests with another class as my comments?09:40
liushengzhenguo: hmm, just thought this is more like unit tests09:42
liushengzhenguo: how about keep this and add other tests in that test file ?09:43
zhenguoliusheng: ok, but that class named InstanceAuthorization, it's a bit confused to have two classes for this09:44
*** litao has quit IRC09:44
zhenguoliusheng: but ok, we can refactor that later09:44
liushengzhenguo: the tests I added are specially for the authorize_wsgi method09:45
zhenguoliusheng: ha, yes09:46
openstackgerritZhenguo Niu proposed openstack/mogan master: Use policy-generator to generate policy sample file  https://review.openstack.org/42496209:55
openstackgerritMerged openstack/mogan master: Add power_state to InstancePayLoad  https://review.openstack.org/42866211:16
openstackgerritMerged openstack/mogan master: Refactor policy: Support precision control of power action  https://review.openstack.org/42495112:50
*** l4yerffeJ_ has quit IRC15:51
*** shaohe_feng has quit IRC17:56
*** shaohe_feng has joined #openstack-mogan18:03
*** Jeffrey4l has quit IRC21:35
*** Jeffrey4l has joined #openstack-mogan21:35

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