| *** wanghao has joined #openstack-mogan | 00:29 | |
| *** litao__ has joined #openstack-mogan | 00:52 | |
| zhenguo | morning mogan! | 00:54 |
|---|---|---|
| liusheng | morning | 01:02 |
| zhenguo | shaohe_feng: please have a look at this https://review.openstack.org/#/c/478406/ | 01:11 |
| zhenguo | wanghao, litao__: would be appreciated if you can add more quotas support before Pike:D seems every resource should have a quota, like keypair, metadata, network, etc. | 01:16 |
| wanghao | zhenguo: yes, I have create a branch yesterday | 01:17 |
| zhenguo | wanghao: thanks | 01:17 |
| wanghao | zhenguo: I plan to support flavor first, do you think it's okay. | 01:17 |
| zhenguo | wanghao: sure | 01:17 |
| wanghao | zhenguo: thanks, I think this week can commit the code patch. | 01:18 |
| zhenguo | wanghao: cool | 01:18 |
| zhenguo | wanghao: btw, does nova have a quota for flavors? | 01:19 |
| wanghao | zhenguo: well not sure yet, let me check it. | 01:20 |
| zhenguo | wanghao: ok, as flavor is for admins, so maybe not need a quota, need to check nova | 01:20 |
| wanghao | zhenguo: sure | 01:20 |
| wanghao | zhenguo: yeah, nova didn't check quota for flavor. | 01:21 |
| zhenguo | wanghao: hah | 01:21 |
| wanghao | zhenguo: so we also didn't need it. | 01:21 |
| zhenguo | wanghao: yes | 01:22 |
| wanghao | zhenguo: well, then it should be keypair and something else. | 01:22 |
| zhenguo | wanghao: yes, keypair, and maybe just something else like metadata and injected files | 01:22 |
| zhenguo | wanghao: network related stuff seems have it's own quota from neutron | 01:23 |
| wanghao | zhenguo: yes, neutron have quota for network | 01:23 |
| wanghao | zhenguo: okay, I got it. | 01:23 |
| zhenguo | wanghao: hah, so it's easy for us, not much to do | 01:23 |
| wanghao | zhenguo: :) haha | 01:24 |
| *** harlowja has quit IRC | 01:34 | |
| litao__ | zhenguo: If i have spare time , I will assist wanghao | 01:44 |
| zhenguo | litao__: thanks :D | 01:45 |
| zhenguo | te | 01:48 |
| *** litao___ has joined #openstack-mogan | 02:06 | |
| *** mattcburns_ has joined #openstack-mogan | 02:06 | |
| *** wanghao_ has joined #openstack-mogan | 02:10 | |
| *** litao__ has quit IRC | 02:13 | |
| *** wanghao has quit IRC | 02:13 | |
| *** mattcburns has quit IRC | 02:13 | |
| *** litao___ is now known as litao__ | 02:13 | |
| *** mattcburns_ is now known as mattcburns | 02:13 | |
| liusheng | zhenguo: hi, for this: https://review.openstack.org/#/c/483203, I just given a -1 for the strange mac address, please correct me if there is a reason :) | 03:01 |
| zhenguo | liusheng: hah, also not sure why I added that :D | 03:03 |
| liusheng | zhenguo: hah | 03:03 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add indexes to tables https://review.openstack.org/483203 | 03:13 |
| *** harlowja has joined #openstack-mogan | 03:40 | |
| *** harlowja has quit IRC | 04:54 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Adds aggregates DB model and API https://review.openstack.org/482786 | 05:22 |
| openstackgerrit | Merged openstack/mogan master: Clean the compute_node and compute_port objects and db interfaces https://review.openstack.org/478406 | 05:42 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [WIP]Add aggregate object https://review.openstack.org/484630 | 05:50 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add indexes to tables https://review.openstack.org/483203 | 05:51 |
| openstackgerrit | Merged openstack/mogan master: Updated from global requirements https://review.openstack.org/484559 | 05:51 |
| zhenguo | liusheng: with the last placement patch landed, we have totally migrated to placement :D | 06:10 |
| liusheng | zhenguo: hah | 06:11 |
| zhenguo | liusheng: maybe you can +A this again https://review.openstack.org/#/c/483203/ :D | 06:48 |
| liusheng | zhenguo: done | 06:49 |
| zhenguo | liusheng: thanks | 06:49 |
| liusheng | zhenguo: np :) | 06:49 |
| liusheng | zhenguo: please take a look https://review.openstack.org/#/c/482057/ if you have time :) | 06:50 |
| zhenguo | liusheng: sure | 06:50 |
| *** liusheng has quit IRC | 06:59 | |
| *** liusheng has joined #openstack-mogan | 06:59 | |
| zhenguo | liusheng: just left a couple of comments on the uwsgi patch although I'm not quite familiar with that :D | 07:06 |
| liusheng | zhenguo: thanks, let me check | 07:07 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [WIP]Add aggregate object https://review.openstack.org/484630 | 07:13 |
| openstackgerrit | Merged openstack/mogan master: Add indexes to tables https://review.openstack.org/483203 | 07:28 |
| liusheng | zhenguo: hi, I have replied you about your last comment inline | 07:31 |
| zhenguo | liusheng: ok, I'm fine with it :D | 07:46 |
| liusheng | zhenguo: thanks | 07:46 |
| zhenguo | liusheng: hah | 07:47 |
| openstackgerrit | liusheng proposed openstack/mogan master: Support running api under uWSGI https://review.openstack.org/482057 | 07:47 |
| litao__ | liusheng: Does the mogan client support creating flavor with resource classes? | 07:55 |
| liusheng | litao__: yes | 07:55 |
| liusheng | liusheng: --resources | 07:56 |
| liusheng | litao__: --resources | 07:56 |
| litao__ | liusheng: Ok, maybe I need to pull the latest codes. | 07:56 |
| liusheng | litao__: hah | 07:57 |
| zhenguo | liusheng: does placement support an API to create aggregates | 08:29 |
| liusheng | zhenguo: yes | 08:29 |
| zhenguo | liusheng: just POST /aggregates? | 08:29 |
| liusheng | zhenguo: sorry, let me confirm | 08:30 |
| zhenguo | liusheng: I can't find 'aggregates' in ROUTE | 08:32 |
| liusheng | zhenguo: sorry, seems there is only get aggregates by rp and put aggregates for a rp interfaces | 08:36 |
| zhenguo | liusheng: so, how do we know the aggregate id to put :( | 08:37 |
| liusheng | zhenguo: GET/PUT /resource_providers/{uuid}/aggregates | 08:37 |
| zhenguo | liusheng: or anything is ok? | 08:37 |
| liusheng | zhenguo: let me test. | 08:37 |
| zhenguo | liusheng: ok ,thanks | 08:37 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Support running api under uWSGI https://review.openstack.org/482057 | 08:38 |
| zhenguo | liushen: just asked alex, the uuid can be created automatically | 08:45 |
| zhenguo | liusheng: if the specified uuid doens't exist in placement | 08:45 |
| liusheng | zhenguo: it is strange in my testing, the id is a integer than uuid | 08:46 |
| zhenguo | liusheng: ? | 08:47 |
| zhenguo | liusheng: you specify a uuid then get an integer | 08:47 |
| liusheng | zhenguo: curl -g -i -X PUT http://10.229.40.107/placement/resource_providers/69d7aba4-ccfd-4a7c-aba0-0e34c91ca1dd/aggregates -H "Accept: application/json" -H "Content-Type: application/json" -H "X-Auth-Token: $(openstack token issue -f value -c id)" -H "OpenStack-API-Version: placement latest" -d '["42896e0d-205d-4fe3-bd1e-100924931787","5e08ea53-c4c6-448e-9334-ac4953de3cfa"]' | 08:47 |
| zhenguo | liusheng: the placement_aggregate table will save them? | 08:48 |
| liusheng | zhenguo: no the in the resource_provider_aggregates table, there are two items, each one have a column named resoruce_provider_id, and another is "id" | 08:49 |
| liusheng | zhenguo: the "id" column is a integer | 08:49 |
| zhenguo | liusheng: that's ok | 08:50 |
| zhenguo | liusheng: nothing in placement_aggregates table? | 08:50 |
| zhenguo | liusheng: the resource_provider_aggregates just save the id field instead of uuid | 08:51 |
| liusheng | zhenguo: oh, yes, it is | 08:51 |
| liusheng | zhenguo: thanks for reminding me, hah | 08:51 |
| zhenguo | liusheng: so, it's easy for us, we can add our node aggregates, not need to sync with placement aggregate uuid :D | 08:52 |
| liusheng | zhenguo: but the uuid isn't generated automatically | 08:52 |
| liusheng | zhenguo: it is the inputted two items | 08:52 |
| zhenguo | liusheng: we create aggregate object on mogan side then got the uuid | 08:53 |
| zhenguo | liusheng: when add nodes to the aggregates, we call PUT rps/uuid/aggregates with the aggregate uuid | 08:53 |
| liusheng | zhenguo: yes, maybe a aggregate uuid is enough. the aggregate object including nothing else.hah | 08:54 |
| zhenguo | liusheng: name and metadata as well, hah | 08:54 |
| liusheng | zhenguo: oh, yes, for us | 08:54 |
| zhenguo | liusheng: yes, seems placement aggregate is just designed for our use case | 08:55 |
| liusheng | zhenguo: hah | 08:55 |
| liusheng | zhenguo for the PUT interface, the inputted items must be uuid format | 08:55 |
| zhenguo | liusheng: hah, so things become clear, I will update the spec, and add patches soon | 08:55 |
| zhenguo | liusheng: that's ok | 08:56 |
| zhenguo | liusheng: another thing is the rp cache in report client, if it's singleton, I think we can leverage that | 08:56 |
| liusheng | zhenguo: yes, actually in Nova, there is anothor related cache mechanism, for the resource class, usaully, it exposed as name, but for internal usages, it used by id. there is a exchanging cache | 08:58 |
| zhenguo | liusheng: yes, I also noticed tthat | 08:58 |
| zhenguo | liusheng: if you like, you can work on the aggregate cache part, hah | 08:59 |
| liusheng | zhenguo: ok, will try. hah | 09:01 |
| zhenguo | liusheng: hah, thanks | 09:01 |
| zhenguo | liusheng: so, before adding nodes to aggregates, the mogan aggregate will not interact with placement | 09:02 |
| liusheng | zhenguo: got it | 09:02 |
| zhenguo | liusheng: hah, now we really learned how to use placement aggregates, so it's totally different with nova host aggregates | 09:03 |
| liusheng | zhenguo: yes | 09:03 |
| liusheng | zhenguo: actually nova also has aggregates and aggregate_metadata | 09:04 |
| zhenguo | liusheng: yes, | 09:04 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Adds aggregates DB model and API https://review.openstack.org/482786 | 09:25 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [WIP]Add aggregate object https://review.openstack.org/484630 | 09:25 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [WIP]Add aggregate API https://review.openstack.org/484690 | 09:25 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: Node aggregates https://review.openstack.org/470927 | 09:32 |
| *** wanghao_ has quit IRC | 09:37 | |
| openstackgerrit | Merged openstack/mogan master: Support running api under uWSGI https://review.openstack.org/482057 | 09:55 |
| *** zhuli_ has quit IRC | 11:42 | |
| *** zhuli has joined #openstack-mogan | 11:42 | |
| *** zhuli has quit IRC | 11:43 | |
| *** zhuli has joined #openstack-mogan | 11:43 | |
| *** litao__ has quit IRC | 12:06 | |
| zhenguo | liusheng: how to restart service with the current devstack | 12:16 |
| liusheng | zhenguo: same way, service devstack@mogan-api restart | 12:16 |
| zhenguo | liusheng: ok, thanks | 12:16 |
| liusheng | zhenguo: np | 12:18 |
| *** harlowja has joined #openstack-mogan | 17:24 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!