*** wanghao has joined #openstack-mogan | 00:14 | |
wanghao | morning mogan | 00:52 |
---|---|---|
liusheng | morning | 01:03 |
zhenguo | wanghao, liusheng: morning! | 01:03 |
zhenguo | liusheng: please have a look at this https://review.openstack.org/#/c/470897/ , seems a critical bug | 01:25 |
liusheng | zhenguo: lol | 01:26 |
zhenguo | liusheng: hah, we really need more reviews before landing patches :D | 01:27 |
liusheng | zhenguo: yes, quickly merged your patch | 01:27 |
zhenguo | liusheng: thanks :D | 01:28 |
liusheng | zhenguo: np | 01:28 |
*** litao__ has joined #openstack-mogan | 01:41 | |
openstackgerrit | Merged openstack/mogan master: Correct ironic driver sync power states interface spelling error https://review.openstack.org/470897 | 01:56 |
openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: Node aggregates https://review.openstack.org/470927 | 02:33 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/mogan master: Updated from global requirements https://review.openstack.org/470935 | 02:37 |
* zhenguo brb | 02:39 | |
shaohe_feng | liusheng: zhenguo morning | 02:39 |
liusheng | shaohe_feng: morning shaohe | 02:40 |
zhenguo | shaohe_feng: o/ | 02:51 |
openstackgerrit | liusheng proposed openstack/mogan master: Add support for scheduler_hints https://review.openstack.org/463534 | 02:52 |
zhenguo | Is there somebody want to take this https://bugs.launchpad.net/mogan/+bug/1696034 ? | 02:57 |
openstack | Launchpad bug 1696034 in Mogan "Missing JSON schema check for flavor extraspecs update" [Medium,New] | 02:57 |
zhenguo | liusheng, shaohe_feng: FYI https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/node-resource-class.html | 03:01 |
zhenguo | liusheng, shaohe_feng: I added a patch to replace node_type with resource_class | 03:02 |
zhenguo | zhangyang: not sure if this https://review.openstack.org/#/c/470860/ will affect cloudboot driver | 03:10 |
zhangyang | zhenguo: i'll review it~ | 03:11 |
zhenguo | zhangyang: thanks | 03:11 |
zhangyang | zhenguo: np:D | 03:11 |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [DOC] Move gmr to Advanced testing and guides section https://review.openstack.org/471192 | 03:41 |
litao__ | zhenguo: I will take this bug 1696034 | 04:42 |
openstack | bug 1696034 in Mogan "Missing JSON schema check for flavor extraspecs update" [Medium,New] https://launchpad.net/bugs/1696034 | 04:42 |
zhenguo | litao__: thanks | 04:59 |
openstackgerrit | wanghao proposed openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 06:50 |
openstackgerrit | Merged openstack/mogan master: Updated from global requirements https://review.openstack.org/470935 | 06:59 |
zhenguo | wanghao, 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 |
wanghao | zhenguo: cool, fine with me | 07:13 |
zhenguo | wanghao: thanks | 07:13 |
openstackgerrit | Van Hung Pham proposed openstack/mogan master: Optimize the link address https://review.openstack.org/471236 | 07:34 |
shaohe_feng | zhenguo: OK, let me check it. | 07:36 |
zhenguo | shaohe_feng: ok, thanks | 07:36 |
openstackgerrit | wanghao proposed openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 07:43 |
openstackgerrit | wanghao proposed openstack/mogan master: Fix some typos in Mogan doc https://review.openstack.org/471243 | 07:56 |
openstackgerrit | liusheng proposed openstack/mogan master: Split the tempest tests https://review.openstack.org/471246 | 08:16 |
zhenguo | wanghao: please find my reply here https://review.openstack.org/#/c/470860/ | 08:24 |
wanghao | zhenguo: sure, that makes sense to me. | 08:25 |
zhenguo | wanghao: thanks | 08:25 |
zhenguo | liusheng: can you access this http://10.82.8.1:8888/states.html? | 08:40 |
liusheng | zhenguo: yes, I can | 08:44 |
zhenguo | liusheng: really a weird diagram :0 | 08:45 |
liusheng | zhenguo: lol | 08:45 |
zhenguo | liusheng: see again, also seems weird, hah | 08:50 |
liusheng | zhenguo: how to draw this ? | 08:51 |
zhenguo | liusheng: graphviz, https://github.com/openstack/nova/blob/master/doc/source/vmstates.rst | 08:52 |
zhenguo | liusheng: but seems we should draw a diagram like ironic state machine instead of just this | 08:52 |
* zhenguo brb | 08:53 | |
zhenguo | liusheng: please update the tempest here https://etherpad.openstack.org/p/MoganWhiteBoard with https://review.openstack.org/#/c/471246/ | 09:10 |
liusheng | zhenguo: ok, thanks for reminding | 09:14 |
*** wanghao has quit IRC | 09:34 | |
openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [DOC] Add states and transitons diagram https://review.openstack.org/471293 | 09:37 |
shaohe_feng | zhenguo: are port_uuid and network_uuid | 09:38 |
shaohe_feng | exclusive? | 09:38 |
zhenguo | shaohe_feng: yes | 09:38 |
shaohe_feng | zhenguo: so our schema check "Oneof" means "exclusive" ? | 09:52 |
zhenguo | shaohe_feng: I'm not sure whether Oneof 'required' is legal | 09:54 |
zhenguo | shaohe_feng: 'required' includes properties must be specified, but not sure how to exclude | 09:56 |
shaohe_feng | zhenguo: I check the OK, for IP | 09:56 |
shaohe_feng | 'oneOf': [ | 09:56 |
shaohe_feng | {'format': 'ipv4'}, | 09:56 |
shaohe_feng | {'format': 'ipv6'} | 09:56 |
shaohe_feng | ] | 09:56 |
zhenguo | shaohe_feng: I will test the patch after dinner | 09:57 |
* zhenguo brb | 10:01 | |
openstackgerrit | Merged openstack/mogan master: Change to use node resource_class https://review.openstack.org/470860 | 10:05 |
openstackgerrit | Merged openstack/mogan master: Split the tempest tests https://review.openstack.org/471246 | 10:45 |
zhenguo | liusheng, shaohe_feng: mind to look at this https://review.openstack.org/#/c/471192/ ? | 10:58 |
* zhenguo away | 11:57 | |
*** litao__ has quit IRC | 11:58 | |
*** harlowja has quit IRC | 19:29 | |
*** harlowja has joined #openstack-mogan | 19:50 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!