Tuesday, 2017-06-06

« Monday, 2017-06-05 Index Wednesday, 2017-06-07 »
*** wanghao has joined #openstack-mogan00:14
wanghaomorning mogan00:52
liushengmorning01:03
zhenguowanghao, liusheng: morning!01:03
zhenguoliusheng: please have a look at this https://review.openstack.org/#/c/470897/ , seems a critical bug01:25
liushengzhenguo: lol01:26
zhenguoliusheng: hah, we really need more reviews before landing patches :D01:27
liushengzhenguo: yes, quickly merged your patch01:27
zhenguoliusheng: thanks :D01:28
liushengzhenguo: np01:28
*** litao__ has joined #openstack-mogan01:41
openstackgerritMerged openstack/mogan master: Correct ironic driver sync power states interface spelling error  https://review.openstack.org/47089701:56
openstackgerritZhenguo Niu proposed openstack/mogan-specs master: Node aggregates  https://review.openstack.org/47092702:33
openstackgerritOpenStack Proposal Bot proposed openstack/mogan master: Updated from global requirements  https://review.openstack.org/47093502:37
* zhenguo brb02:39
shaohe_fengliusheng: zhenguo morning02:39
liushengshaohe_feng: morning shaohe02:40
zhenguoshaohe_feng: o/02:51
openstackgerritliusheng proposed openstack/mogan master: Add support for scheduler_hints  https://review.openstack.org/46353402:52
zhenguoIs there somebody want to take this https://bugs.launchpad.net/mogan/+bug/1696034 ?02:57
openstackLaunchpad bug 1696034 in Mogan "Missing JSON schema check for flavor extraspecs update" [Medium,New]02:57
zhenguoliusheng, shaohe_feng: FYI https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/node-resource-class.html03:01
zhenguoliusheng, shaohe_feng: I added a patch to replace node_type with resource_class03:02
zhenguozhangyang: not sure if this https://review.openstack.org/#/c/470860/ will affect cloudboot driver03:10
zhangyangzhenguo: i'll review it~03:11
zhenguozhangyang: thanks03:11
zhangyangzhenguo: np:D03:11
openstackgerritZhenguo Niu proposed openstack/mogan master: [DOC] Move gmr to Advanced testing and guides section  https://review.openstack.org/47119203:41
litao__zhenguo: I will take this bug 169603404:42
openstackbug 1696034 in Mogan "Missing JSON schema check for flavor extraspecs update" [Medium,New] https://launchpad.net/bugs/169603404:42
zhenguolitao__: thanks04:59
openstackgerritwanghao proposed openstack/mogan master: Specify neutron port when creating servers  https://review.openstack.org/47082406:50
openstackgerritMerged openstack/mogan master: Updated from global requirements  https://review.openstack.org/47093506:59
zhenguowanghao, liusheng, shaohe_feng: hey, I moved gmr docs under Advanced testing and guides section like nova as it seems not proper to occupy a first level section, and will enrich this section later https://review.openstack.org/#/c/471192/07:11
wanghaozhenguo: cool,  fine with me07:13
zhenguowanghao: thanks07:13
openstackgerritVan Hung Pham proposed openstack/mogan master: Optimize the link address  https://review.openstack.org/47123607:34
shaohe_fengzhenguo: OK, let me check it.07:36
zhenguoshaohe_feng: ok, thanks07:36
openstackgerritwanghao proposed openstack/mogan master: Specify neutron port when creating servers  https://review.openstack.org/47082407:43
openstackgerritwanghao proposed openstack/mogan master: Fix some typos in Mogan doc  https://review.openstack.org/47124307:56
openstackgerritliusheng proposed openstack/mogan master: Split the tempest tests  https://review.openstack.org/47124608:16
zhenguowanghao: please find my reply here https://review.openstack.org/#/c/470860/08:24
wanghaozhenguo: sure, that makes sense to me.08:25
zhenguowanghao: thanks08:25
zhenguoliusheng: can you access this http://10.82.8.1:8888/states.html?08:40
liushengzhenguo: yes, I can08:44
zhenguoliusheng: really a weird diagram :008:45
liushengzhenguo: lol08:45
zhenguoliusheng: see again, also seems weird, hah08:50
liushengzhenguo: how to draw this ?08:51
zhenguoliusheng: graphviz, https://github.com/openstack/nova/blob/master/doc/source/vmstates.rst08:52
zhenguoliusheng: but seems we should draw a diagram like ironic state machine instead of just this08:52
* zhenguo brb08:53
zhenguoliusheng: please update the tempest here https://etherpad.openstack.org/p/MoganWhiteBoard with https://review.openstack.org/#/c/471246/09:10
liushengzhenguo: ok, thanks for reminding09:14
*** wanghao has quit IRC09:34
openstackgerritZhenguo Niu proposed openstack/mogan master: [DOC] Add states and transitons diagram  https://review.openstack.org/47129309:37
shaohe_fengzhenguo: are port_uuid and network_uuid09:38
shaohe_fengexclusive?09:38
zhenguoshaohe_feng: yes09:38
shaohe_fengzhenguo: so our schema check "Oneof" means "exclusive" ?09:52
zhenguoshaohe_feng: I'm not sure whether Oneof 'required' is legal09:54
zhenguoshaohe_feng: 'required' includes properties must be specified, but not sure how to exclude09:56
shaohe_fengzhenguo: I check the OK, for IP09:56
shaohe_feng    'oneOf': [09:56
shaohe_feng        {'format': 'ipv4'},09:56
shaohe_feng        {'format': 'ipv6'}09:56
shaohe_feng    ]09:56
zhenguoshaohe_feng: I will test the patch after dinner09:57
* zhenguo brb10:01
openstackgerritMerged openstack/mogan master: Change to use node resource_class  https://review.openstack.org/47086010:05
openstackgerritMerged openstack/mogan master: Split the tempest tests  https://review.openstack.org/47124610:45
zhenguoliusheng, shaohe_feng: mind to look at this https://review.openstack.org/#/c/471192/ ?10:58
* zhenguo away11:57
*** litao__ has quit IRC11:58
*** harlowja has quit IRC19:29
*** harlowja has joined #openstack-mogan19:50
« Monday, 2017-06-05 Index Wednesday, 2017-06-07 »

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