*** wanghao has joined #openstack-mogan | 00:52 | |
*** liusheng has joined #openstack-mogan | 00:56 | |
*** wanghao_ has joined #openstack-mogan | 01:18 | |
*** wanghao has quit IRC | 01:23 | |
*** Kevin_Zheng has joined #openstack-mogan | 01:32 | |
*** litao has joined #openstack-mogan | 01:34 | |
zhenguo | morning mogan! | 01:40 |
---|---|---|
*** wanghao has joined #openstack-mogan | 01:54 | |
*** litao has quit IRC | 01:54 | |
*** wanghao_ has quit IRC | 01:55 | |
*** openstackgerrit has joined #openstack-mogan | 02:06 | |
openstackgerrit | wanghao proposed openstack/mogan master: Quota support in Mogan(part 1) https://review.openstack.org/417326 | 02:06 |
openstackgerrit | Merged openstack/python-moganclient master: Updated from global requirements https://review.openstack.org/423684 | 02:07 |
*** RuiChen has joined #openstack-mogan | 02:18 | |
openstackgerrit | Merged openstack/mogan master: Add support for floating IPs association https://review.openstack.org/417760 | 02:53 |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Log pending task when instance is in unstable states https://review.openstack.org/428554 | 03:03 |
openstackgerrit | wanghao proposed openstack/mogan master: Quota support in Mogan(part 1) https://review.openstack.org/417326 | 03:04 |
openstackgerrit | wanghao proposed openstack/mogan master: Support quota in mogan(part two) https://review.openstack.org/428557 | 03:21 |
wanghao | hi guys, now quota patches have been committed all, pls review if you have time. | 03:25 |
zhenguo | wanghao: awesome, thanks for the hard work, will review soon! | 03:27 |
wanghao | zhenguo: thanks, | 03:28 |
*** Kevin_Zheng has quit IRC | 03:37 | |
*** l4yerffeJ_ has quit IRC | 03:52 | |
*** l4yerffeJ_ has joined #openstack-mogan | 03:53 | |
openstackgerrit | liusheng proposed openstack/mogan master: Refactor policy: Support precision control of power action https://review.openstack.org/424951 | 04:17 |
* zhenguo join #openstack-stackalytics | 04:51 | |
openstackgerrit | Merged openstack/mogan master: Log pending task when instance is in unstable states https://review.openstack.org/428554 | 05:42 |
*** Kevin_Zheng has joined #openstack-mogan | 06:33 | |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Weigh nodes by their ports quantity https://review.openstack.org/428600 | 06:54 |
openstackgerrit | Tao Li proposed openstack/mogan master: Support multi-instance creating (part 1) https://review.openstack.org/421064 | 07:19 |
*** litao has joined #openstack-mogan | 07:21 | |
litao | test | 07:22 |
zhenguo | hi 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 |
zhenguo | liusheng, wanghao, litao: hi | 07:34 |
liusheng | zhenguo: hi | 07:34 |
zhenguo | liusheng: do you like the new weigher :P | 07:34 |
zhenguo | FYI, our stats in stacklytics has been broken after project renaming :( | 07:38 |
zhenguo | hope this can fix it, https://review.openstack.org/#/c/428572/ | 07:38 |
liusheng | zhenguo: cool | 07:41 |
litao | zhenguo: no problem | 08:03 |
openstackgerrit | liusheng proposed openstack/mogan master: Refactor policy: Support precision control of power action https://review.openstack.org/424951 | 08:04 |
openstackgerrit | liusheng proposed openstack/mogan master: Use policy-generator to generate policy sample file https://review.openstack.org/424962 | 08:05 |
zhenguo | liusheng: regarding https://review.openstack.org/#/c/428600/1/mogan/engine/scheduler/weights/port.py , please find my reply inline, thanks | 08:23 |
liusheng | zhenguo: oh, sorry, seems it leaded my to Nova's code of "get_all_classes" with load class in a folder. lol | 08:35 |
zhenguo | liusheng: hah, | 08:36 |
liusheng | zhenguo: do you think the namespace is a bit too long? I guess it may used for keep compatibility of the old extension loading mechanism | 08:37 |
liusheng | zhenguo: e.g. mogan.engine.scheduler.weights | 08:37 |
liusheng | zhenguo: the namespace of stevedore don't need to be the path of class | 08:38 |
zhenguo | liusheng: seems yes, but this maybe more clear to use full path, wdyt? | 08:38 |
liusheng | zhenguo: 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 symbol | 08:41 |
zhenguo | liusheng: yes, anyway is ok, hah, I don't have strong opinion about this | 08:42 |
zhenguo | liusheng: as we copied cinder's scheduler, so keep that way, the filter entry point and now the weight entry point | 08:43 |
liusheng | zhenguo: yes, I found that, the cinder may copied from Nova very early and then they guys switch to use stevedore, hah | 08:44 |
zhenguo | liusheng: hah, yes | 08:44 |
zhenguo | liusheng: I remember you suggested to rename network info to nics right? | 08:51 |
liusheng | zhenguo: yes | 08:51 |
zhenguo | liusheng: seems nics is a good option | 08:51 |
zhenguo | liusheng: I don't like nova's addresses field | 08:51 |
zhenguo | liusheng: and ports also seems a bit confused, | 08:52 |
liusheng | zhenguo: hah, just my opinion | 08:53 |
zhenguo | liusheng: hah, I like that | 08:53 |
zhenguo | liusheng: when creating a server we need to specify --nic, so it should return a server with nics | 08:54 |
zhenguo | liusheng: we can refactor that, but need to construct the fields in nics | 08:54 |
liusheng | zhenguo: looks more brief | 08:55 |
liusheng | zhenguo: yes | 08:55 |
zhenguo | liusheng: like port_id, fixed_address, port_type, mac address, etc | 08:55 |
openstackgerrit | liusheng proposed openstack/mogan master: Use policy-generator to generate policy sample file https://review.openstack.org/424962 | 08:56 |
liusheng | zhenguo: agree, btw, we can support ipv6 ? | 08:57 |
zhenguo | liusheng: I think yes, | 08:57 |
liusheng | zhenguo: cool | 08:58 |
zhenguo | liusheng: hah | 08:58 |
*** liusheng has quit IRC | 09:07 | |
*** liusheng has joined #openstack-mogan | 09:07 | |
*** wanghao_ has joined #openstack-mogan | 09:23 | |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add power_state to InstancePayLoad https://review.openstack.org/428662 | 09:24 |
zhenguo | liusheng: hi, I just put a patch to added power_state to InstancePayLoad, as we have a power_state attribute now | 09:26 |
*** wanghao has quit IRC | 09:26 | |
liusheng | zhenguo: ok, thanks | 09:27 |
openstackgerrit | Merged openstack/mogan master: Weigh nodes by their ports quantity https://review.openstack.org/428600 | 09:28 |
*** wanghao_ has quit IRC | 09:29 | |
zhenguo | liusheng: but I didn't bump ther versions | 09:29 |
liusheng | zhenguo: may don't need as we do with db schema version, hah | 09:30 |
zhenguo | liusheng: hah | 09:30 |
*** wanghao has joined #openstack-mogan | 09:30 | |
*** wanghao has quit IRC | 09:30 | |
zhenguo | liusheng: hi, wrt https://review.openstack.org/#/c/424951/ , can we combine the tests with another class as my comments? | 09:40 |
liusheng | zhenguo: hmm, just thought this is more like unit tests | 09:42 |
liusheng | zhenguo: how about keep this and add other tests in that test file ? | 09:43 |
zhenguo | liusheng: ok, but that class named InstanceAuthorization, it's a bit confused to have two classes for this | 09:44 |
*** litao has quit IRC | 09:44 | |
zhenguo | liusheng: but ok, we can refactor that later | 09:44 |
liusheng | zhenguo: the tests I added are specially for the authorize_wsgi method | 09:45 |
zhenguo | liusheng: ha, yes | 09:46 |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Use policy-generator to generate policy sample file https://review.openstack.org/424962 | 09:55 |
openstackgerrit | Merged openstack/mogan master: Add power_state to InstancePayLoad https://review.openstack.org/428662 | 11:16 |
openstackgerrit | Merged openstack/mogan master: Refactor policy: Support precision control of power action https://review.openstack.org/424951 | 12:50 |
*** l4yerffeJ_ has quit IRC | 15:51 | |
*** shaohe_feng has quit IRC | 17:56 | |
*** shaohe_feng has joined #openstack-mogan | 18:03 | |
*** Jeffrey4l has quit IRC | 21:35 | |
*** Jeffrey4l has joined #openstack-mogan | 21:35 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!