| *** liujiong has joined #openstack-mogan | 00:11 | |
| *** wanghao has joined #openstack-mogan | 00:46 | |
| zhenguo | morning mogan! | 00:47 |
|---|---|---|
| openstackgerrit | wanghao proposed openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 01:05 |
| wanghao | morning | 01:05 |
| zhenguo | shaohe_feng: around? | 01:22 |
| *** liusheng has joined #openstack-mogan | 01:23 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for console https://review.openstack.org/474498 | 01:28 |
| openstackgerrit | Merged openstack/mogan master: Updated from global requirements https://review.openstack.org/474656 | 01:28 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Adds `disabled` field for flavors. https://review.openstack.org/474441 | 01:33 |
| zhenguo | liusheng: maybe I can +2 my revert patch, as it's `revert` | 01:35 |
| liusheng | zhenguo: sure | 01:35 |
| zhenguo | liusheng: hah | 01:35 |
| wanghao | zhenguo: of course you can | 01:35 |
| zhenguo | wanghao: lol | 01:36 |
| liusheng | zhenguo: please ping me if you need another +2, lol | 01:36 |
| zhenguo | liusheng: sure | 01:36 |
| * zhenguo brb | 01:38 | |
| wanghao | shaohe_feng: I need your +2 again for specifying neutron port patch, since I rebase it to slove merge confilct. | 01:38 |
| liusheng | zhenguo: may you can help give a +A for https://review.openstack.org/#/c/473204/, it merged failed and I just rebased it. | 01:44 |
| liusheng | wanghao: will test your patch | 01:45 |
| wanghao | liusheng: thanks | 01:45 |
| zhenguo | liusheng: sure | 01:48 |
| liusheng | zhenguo: thanks | 01:48 |
| zhenguo | liusheng: just a quick look through the placement API, we can list all resouce providers and can PUT/GET aggregates info, seems well fit for us | 01:56 |
| liusheng | zhenguo: great, can we set the resource_class and filter query by the field ? | 01:57 |
| zhenguo | liusheng: not sure, seems it is | 01:57 |
| liusheng | zhenguo: ok, will learn it | 01:58 |
| zhenguo | liusheng: on our env, there's nothing in inventory table, not sure how to enable it | 01:58 |
| liusheng | zhenguo: hah,may need to report resources to it | 01:59 |
| zhenguo | liusheng: as I understand, nova resouce tracker will report resouces to inventories | 01:59 |
| liusheng | zhenguo: werid, I cannot create flaovr after updating code and run dbsync | 01:59 |
| liusheng | zhenguo: seems yes | 01:59 |
| zhenguo | liusheng: really? we do have a flavor tempest gate | 02:00 |
| liusheng | zhenguo: yes, so it is weird | 02:00 |
| zhenguo | liusheng: maybe because the client, there's also a patch to revert client stuff | 02:00 |
| liusheng | zhenguo: oh, forgot that, hah | 02:01 |
| zhenguo | liusheng: this one https://review.openstack.org/#/c/474496/ | 02:01 |
| openstackgerrit | Merged openstack/mogan master: Revert "Add requested neworks check with the specified flavor" https://review.openstack.org/474405 | 02:04 |
| liusheng | wanghao: you patch conflicted :) | 02:08 |
| wanghao | liusheng: okay, again | 02:09 |
| wanghao | will update it | 02:09 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Revert "New flavor" https://review.openstack.org/474433 | 02:20 |
| openstackgerrit | wanghao proposed openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 02:23 |
| openstackgerrit | Merged openstack/mogan master: Tempest: Add test for node listing api https://review.openstack.org/473204 | 02:25 |
| zhenguo | wanghao: FYI ironic will support boot from volume soon, we may need to interact with cinder as well, please keep an eye on that, thanks! | 02:32 |
| wanghao | zhenguo: okay | 02:32 |
| wanghao | need to create volume in CInder. | 02:33 |
| zhenguo | wanghao: yes | 02:33 |
| zhenguo | wanghao: with new v3 api? | 02:33 |
| wanghao | yes | 02:33 |
| wanghao | current API is V3 | 02:33 |
| zhenguo | wanghao: hah | 02:33 |
| zhenguo | wanghao: just pretend I'm familiar with Cinder :D | 02:34 |
| openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for console https://review.openstack.org/474498 | 02:34 |
| openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for server networks API https://review.openstack.org/474829 | 02:34 |
| wanghao | zhenguo: propably could consider to add cinder client in Mogan | 02:34 |
| zhenguo | wanghao: sure, but not in this release, hah | 02:35 |
| wanghao | zhenguo: yes, agree | 02:35 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Revert "New flavor" https://review.openstack.org/474433 | 02:39 |
| *** litao__ has joined #openstack-mogan | 02:44 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for server networks API https://review.openstack.org/474829 | 02:44 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Revert "Add more properties to default flavor" https://review.openstack.org/474495 | 03:11 |
| openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for server networks API https://review.openstack.org/474829 | 03:13 |
| openstackgerrit | liusheng proposed openstack/mogan master: Tempest: add tempest test cases for server interface attaching https://review.openstack.org/474835 | 03:13 |
| zhenguo | liusheng: this one https://review.openstack.org/#/c/474495/ needs to land first | 03:13 |
| liusheng | zhenguo: ok | 03:14 |
| openstackgerrit | Merged openstack/python-moganclient master: Revert "Add support for specifying new properties for flavor" https://review.openstack.org/474496 | 04:50 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-moganclient master: Updated from global requirements https://review.openstack.org/472917 | 05:33 |
| openstackgerrit | wanghao proposed openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 05:42 |
| zhenguo | gates broken? | 06:45 |
| openstackgerrit | Merged openstack/mogan master: Revert "Add more properties to default flavor" https://review.openstack.org/474495 | 06:47 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Revert "New flavor" https://review.openstack.org/474433 | 07:15 |
| zhenguo | liusheng: the gate seems broken with get_nodes_list | 07:57 |
| zhenguo | liusheng: https://review.openstack.org/#/c/473204/4/mogan/tests/tempest/api/test_nodes.py | 08:00 |
| zhenguo | liusheng: #L30 nodes[0] raise IndexError: list index out of range | 08:00 |
| zhenguo | liusheng: as nodes could be [] | 08:01 |
| liusheng | zhenguo: I just found patch run cannot pass the gate, but I am not sure if it because this | 08:01 |
| liusheng | zhenguo: oh, I see | 08:02 |
| zhenguo | liusheng: so can you add a quick fix like check before perfom nodes[0] or do we need to revert the patch :D | 08:03 |
| liusheng | zhenguo: don't need to revert, fix coming :D | 08:03 |
| zhenguo | liusheng: thanks | 08:04 |
| openstackgerrit | liusheng proposed openstack/mogan master: Fix gate broken by test_nodes_list https://review.openstack.org/474886 | 08:07 |
| openstackgerrit | wanghao proposed openstack/mogan master: Specify neutron port when creating servers https://review.openstack.org/470824 | 08:22 |
| zhenguo | liusheng: please have a look at this https://review.openstack.org/#/c/474433/ | 08:39 |
| openstackgerrit | Merged openstack/mogan master: Remove unneeded *encoding: utf-8* https://review.openstack.org/474028 | 08:39 |
| liusheng | zhenguo: +2ed | 08:42 |
| zhenguo | liusheng: thanks | 08:42 |
| zhenguo | shaohe_feng: mind to +A this https://review.openstack.org/#/c/474433/ , or I will do it myself, lol | 08:43 |
| *** liujiong has quit IRC | 08:59 | |
| zhenguo | liusheng: all placement stuff in nova_api databases; | 09:25 |
| liusheng | zhenguo: may placement will be separated out in the comming cycles | 09:26 |
| zhenguo | liusheng: seems not | 09:27 |
| liusheng | zhenguo: really ? | 09:27 |
| zhenguo | liusheng: but it's a separated service and with separated api stack | 09:27 |
| liusheng | zhenguo: yes | 09:27 |
| zhenguo | liusheng: even it's in nova repo, we can treat it as a separated service | 09:27 |
| liusheng | zhenguo: hah, yes | 09:28 |
| zhenguo | liusheng: not sure if we can only enable placement-api servcie | 09:28 |
| liusheng | zhenguo: that is a problem... | 09:28 |
| zhenguo | liusheng: seems they still track baremetal resouces as like other generic resouces not using node resource class | 09:29 |
| zhenguo | liusheng: but we can do it | 09:29 |
| liusheng | zhenguo: will try, the console tempest can pass in my env, but cannot pass the gate :( | 09:30 |
| zhenguo | liusheng: also found the resource_provider_aggregates table, for us, every node is a resouce provider | 09:30 |
| zhenguo | liusheng: hah | 09:30 |
| liusheng | zhenguo: so it can help to implementation node aggregates ? | 09:30 |
| zhenguo | liusheng: yes, I think so | 09:31 |
| zhenguo | liusheng: as I understand, we can move to placement | 09:31 |
| liusheng | zhenguo: hah, that is great! | 09:31 |
| zhenguo | liusheng: hah | 09:32 |
| zhenguo | liusheng: the node aggregates spec need to be rebased after placement get in, so time is tight | 09:32 |
| liusheng | zhenguo: hmm, yes, so many things need to do | 09:34 |
| zhenguo | liusheng: yes, hah | 09:34 |
| liusheng | zhenguo: your patch didn't pass py23, it is veird | 09:35 |
| liusheng | zhenguo: py27 | 09:35 |
| openstackgerrit | Merged openstack/mogan master: Fix gate broken by test_nodes_list https://review.openstack.org/474886 | 09:35 |
| zhenguo | liusheng: will update, as many reverts the obj version changed again and again :D | 09:36 |
| liusheng | zhenguo: lol | 09:36 |
| zhenguo | liusheng: for disabled flavors, I will just add a check when creating server and raise NotFound, but both common and admin users can list or show it, wdyt? | 09:40 |
| zhenguo | liusheng: maybe need filter for disabled later | 09:40 |
| liusheng | zhenguo: yes, make sense | 09:40 |
| zhenguo | liusheng: ok, so after the last revert patch landed, I will continue | 09:41 |
| liusheng | zhenguo: hah, go ahead | 09:41 |
| zhenguo | liusheng: sure, hah | 09:41 |
| zhenguo | liusheng: aha, just checked the flavor reverted patch, not because obj version but this mogan.tests.unit.db.test_compute_nodes.DbComputeNodeTestCase.test_compute_node_get_all | 09:44 |
| zhenguo | liusheng: I find it occationally failed, but not sure why | 09:44 |
| liusheng | zhenguo: yes, may need to check more | 09:45 |
| zhenguo | liusheng: yes, will dig | 09:46 |
| * zhenguo away | 09:51 | |
| *** wanghao has quit IRC | 12:43 | |
| *** luyao has quit IRC | 15:02 | |
| *** wanghao has joined #openstack-mogan | 15:56 | |
| *** wanghao has quit IRC | 19:00 | |
| *** wanghao has joined #openstack-mogan | 19:01 | |
| *** litao__ has quit IRC | 19:12 | |
| *** liusheng has quit IRC | 19:21 | |
| *** liusheng has joined #openstack-mogan | 19:22 | |
| *** wanghao_ has joined #openstack-mogan | 19:47 | |
| *** wanghao has quit IRC | 19:47 | |
| *** wanghao_ has quit IRC | 20:01 | |
| *** wanghao has joined #openstack-mogan | 20:02 | |
| *** wanghao has quit IRC | 20:16 | |
| *** wanghao has joined #openstack-mogan | 20:17 | |
| *** wanghao_ has joined #openstack-mogan | 20:44 | |
| *** wanghao has quit IRC | 20:48 | |
| *** wanghao_ has quit IRC | 20:49 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Revert "New flavor" https://review.openstack.org/474433 | 23:09 |
| *** luyao has joined #openstack-mogan | 23:29 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!