| *** wanghao has joined #openstack-mogan | 00:39 | |
| *** liusheng has quit IRC | 00:52 | |
| *** litao__ has joined #openstack-mogan | 00:57 | |
| *** liusheng has joined #openstack-mogan | 00:57 | |
| *** wanghao has quit IRC | 00:59 | |
| *** wanghao has joined #openstack-mogan | 01:00 | |
| zhenguo | morning mogan! | 01:05 |
|---|---|---|
| liusheng | morning | 01:06 |
| zhenguo | liusheng: seems we misused rp aggregate, set will replace it and set [] just means remove all aggregates | 01:08 |
| *** harlowja has quit IRC | 01:10 | |
| liusheng | zhenguo: so that is a good news ? | 01:10 |
| liusheng | zhenguo: hah | 01:10 |
| zhenguo | liusheng: yes, I have updated the patch | 01:10 |
| zhenguo | liusheng: it works now | 01:10 |
| liusheng | zhenguo: cool | 01:10 |
| litao__ | zhenguo,liusheng ,shaohe_feng ,wanghao pls review patch: https://review.openstack.org/#/c/481544/ | 01:48 |
| wanghao | litao__: cool | 01:48 |
| zhenguo | litao__: ok | 01:49 |
| *** wanghao_ has joined #openstack-mogan | 01:58 | |
| *** wanghao has quit IRC | 02:01 | |
| liusheng | litao__: just left some comments in your patch | 02:08 |
| * zhenguo brb | 02:11 | |
| litao__ | liusheng: ok, thanks for your -1 :) | 02:26 |
| liusheng | litao__: lol | 02:26 |
| zhenguo | liusheng: maybe you can re+A the aggregates patches if no objections, I will add a following up tests patch | 02:35 |
| liusheng | zhenguo: ok | 02:36 |
| zhenguo | liusheng: thanks | 02:36 |
| *** wanghao_ has quit IRC | 02:39 | |
| openstackgerrit | Merged openstack/mogan master: Updated from global requirements https://review.openstack.org/488015 | 02:40 |
| *** wanghao has joined #openstack-mogan | 02:44 | |
| *** wanghao has quit IRC | 02:51 | |
| *** wanghao has joined #openstack-mogan | 02:51 | |
| *** wanghao has quit IRC | 03:00 | |
| litao__ | liusheng: could we set multiple resource classes in flavor? | 03:03 |
| liusheng | litao__: that is possible | 03:09 |
| litao__ | liusheng: what do you mean 'fully match the resource_class and flavor['resources']'? | 03:12 |
| liusheng | litao__: when we create a server, we can specify a flavor with resources field. scheduler will select a node that must match all the condictons in flavor['resources']. so I think we should apply that in server atoptation | 03:13 |
| liusheng | litao__: I found you use a "in" in your patch | 03:14 |
| openstackgerrit | Merged openstack/mogan master: Adds aggregates DB model and API https://review.openstack.org/482786 | 03:14 |
| openstackgerrit | Merged openstack/mogan master: Add aggregate object https://review.openstack.org/484630 | 03:15 |
| litao__ | liusheng: But adoption is a little different from creating servers. The baremetal node is user-specific， so we just need to ensure the baremetal node's resource class is in flavor resources. | 03:23 |
| liusheng | litao__: why ? if so you need to chose a flavor that can match the node | 03:23 |
| litao__ | liusheng: Because the flavor is created by user and specified in adopting nodes. | 03:26 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Clean up some unused codes https://review.openstack.org/487701 | 03:26 |
| liusheng | zhenguo: so when create the flavor, you need to keep the resources field can match the nodes' | 03:27 |
| liusheng | litao__: ^^ sorry , sent to wrong person | 03:27 |
| zhenguo | lol | 03:27 |
| liusheng | zhenguo: may zhenguo can also give a suggestion, hah | 03:27 |
| zhenguo | liusheng: I need to read the patch first :D | 03:28 |
| litao__ | liusheng: Currently yes | 03:28 |
| litao__ | zhenguo: Give us advice | 03:28 |
| liusheng | litao__: we'd better to avoid different behaviors about matching flavor and nodes between creating and adoptation | 03:29 |
| litao__ | liusheng: yes, need more discussion | 03:30 |
| litao__ | liusheng: have a lunch first. | 03:31 |
| zhenguo | liusheng, litao__: currently, how do we handle the match if we specify flavor resources like GOLD:1, SILVER:1? | 03:31 |
| liusheng | zhenguo: that looks a bad flavor, hah | 03:31 |
| zhenguo | liusheng: hah | 03:31 |
| zhenguo | liusheng: but that could be possible as we don't have limitations | 03:32 |
| liusheng | zhenguo: as we are Mogan, for Nova, the resources can be like: RAM_MB:1024, DISK_GB:100 | 03:32 |
| liusheng | zhenguo: in Nova, it need to fully match the flavor and compute node | 03:32 |
| liusheng | zhenguo: but when request to placement, it will try to match all the conditions | 03:33 |
| zhenguo | liusheng: yes, can that mean both GOLD and SILVER are ok? | 03:33 |
| liusheng | zhenguo: unless we add the logical in our code, the placement cannot perform that | 03:34 |
| zhenguo | liusheng: if more than one resources specified, we only get the first one now? | 03:34 |
| *** wanghao has joined #openstack-mogan | 03:35 | |
| liusheng | zhenguo: no, placement will try to select a node with "and" all the conditions in flavor['resources'] | 03:35 |
| zhenguo | liusheng: so we just pass down the flavor resources to placement now, right? | 03:36 |
| liusheng | zhenguo: I am not sure if Mogan will have the siatuation, but maybe in future, the resources can be putted into other resources than node ? | 03:36 |
| liusheng | zhenguo: yes | 03:36 |
| zhenguo | liusheng: that's ok | 03:36 |
| zhenguo | liusheng: as we may use nested resource in future | 03:36 |
| liusheng | zhenguo: ok | 03:37 |
| * zhenguo brb | 03:38 | |
| liusheng | zhenguo: is there possible any shared hardware can be freely allocated for node provision ? | 03:38 |
| *** wanghao has quit IRC | 03:39 | |
| *** wanghao has joined #openstack-mogan | 03:41 | |
| zhenguo | liusheng: you mean someting like accelerators provided by Cyborg? | 03:50 |
| liusheng | zhenguo: not sure, but like some hardware that isn't belong to a node, but can be allocated to any node | 04:00 |
| *** harlowja has joined #openstack-mogan | 04:32 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Retrieve availability zone from aggregate https://review.openstack.org/485506 | 04:54 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add aggregate nodes API https://review.openstack.org/487284 | 04:54 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add node list support https://review.openstack.org/486016 | 04:54 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add aggregate API https://review.openstack.org/484690 | 04:54 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add aggregates tests https://review.openstack.org/488296 | 04:54 |
| openstackgerrit | liusheng proposed openstack/mogan master: WIP: add server groups db models and api https://review.openstack.org/488298 | 05:07 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add aggregate nodes API https://review.openstack.org/487284 | 05:21 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add aggregates tests https://review.openstack.org/488296 | 05:22 |
| *** harlowja has quit IRC | 05:28 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: Node aggregates https://review.openstack.org/470927 | 05:35 |
| *** harlowja has joined #openstack-mogan | 05:41 | |
| openstackgerrit | wanghao proposed openstack/mogan master: Support quota for keypairs in Mogan(part-three) https://review.openstack.org/485461 | 05:43 |
| litao__ | liusheng: so what your suggestion for matching the flavor resource classes. | 06:14 |
| liusheng | litao__: just ensure fully match the flavor | 06:16 |
| litao__ | liusheng: Also comparing the values? | 06:17 |
| liusheng | litao__: sure | 06:17 |
| * litao__ liusheng: OK, I will try | 06:18 | |
| zhenguo | liusheng: can you help to check whether this work https://review.openstack.org/#/c/487738/ ? thanks | 06:36 |
| liusheng | zhenguo: ok | 06:36 |
| * zhenguo brb | 06:38 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Add server groups db models and api https://review.openstack.org/488298 | 06:39 |
| *** harlowja has quit IRC | 06:48 | |
| *** wanghao_ has joined #openstack-mogan | 06:48 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Fix uwsgi doc to avoid misunderstanding https://review.openstack.org/487757 | 06:49 |
| *** wanghao has quit IRC | 06:52 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add aggregate nodes API https://review.openstack.org/487284 | 07:34 |
| litao__ | liusheng: For ironic, the flavor resource class value must be 1， right？ | 07:48 |
| liusheng | litao__: as discussed above, for now, it is, but not sure in future | 07:49 |
| litao__ | liusheng: OK | 07:50 |
| *** wanghao_ has quit IRC | 07:57 | |
| zhenguo | litao__, liusheng: for node resource_class is must be 1, as we just want 1 node | 08:00 |
| zhenguo | litao__, liusheng: but we may introduce nics resource_class later, then it can be more than 1 | 08:00 |
| liusheng | zhenguo: yes, so I said for now. hah | 08:01 |
| zhenguo | liusheng: hah, yes, and we can't claim more than 1 node for a server | 08:01 |
| liusheng | zhenguo: sure | 08:01 |
| liusheng | zhenguo: btw, about the resource updating process | 08:01 |
| liusheng | zhenguo: we disscussed yestoday, it seems impossible a rp in placement, but the node doesn't exist in ironic | 08:02 |
| zhenguo | liusheng: not exist in ironic means with bad status? | 08:03 |
| liusheng | zhenguo: oh sorry, I mean my previous concern about when rescheduling, we need to keep allocations but delete rp | 08:03 |
| liusheng | zhenguo: so maybe re-reconsider the ps1 of patch https://review.openstack.org/#/c/487351/ ? lol | 08:04 |
| zhenguo | liusheng: seems placement doesn't support to delete rp but left allocations, right? | 08:05 |
| liusheng | zhenguo: yes, so must delete allocations before deleting rp | 08:06 |
| litao__ | thank you two :) | 08:07 |
| zhenguo | liusheng: then when it come back, what will happen? | 08:08 |
| zhenguo | liusheng: seems there may be a dead server still associate with the node, right? | 08:10 |
| liusheng | zhenguo: hmm.. yes, seems need to consider again | 08:10 |
| *** wanghao has joined #openstack-mogan | 08:12 | |
| liusheng | zhenguo: how aobut just skip this rp, if there is a server associated with it ? | 08:12 |
| zhenguo | liusheng: how? | 08:16 |
| liusheng | zhenguo: before delete rp in placement, query server by the rp, if there is a server associated with the rp, just skip | 08:17 |
| liusheng | zhenguo: wdyt ? | 08:17 |
| zhenguo | liusheng: skip means not delete it? | 08:17 |
| liusheng | zhenguo: yes, | 08:17 |
| *** openstackgerrit has quit IRC | 08:18 | |
| zhenguo | liusheng: and keep the allocations | 08:18 |
| liusheng | zhenguo: yes | 08:18 |
| zhenguo | liusheng: make sense to me | 08:18 |
| liusheng | zhenguo: ok, will update the patch | 08:19 |
| zhenguo | liusheng: okay | 08:20 |
| zhenguo | liusheng: how's the tempest without nova going? | 08:22 |
| liusheng | zhenguo: I need to re-stack my env to test | 08:25 |
| zhenguo | liusheng: ok, and please help to test the aggregates when you got time :D | 08:25 |
| liusheng | zhenguo: ok | 08:25 |
| liusheng | zhenguo: the first server group patch just uploaded :) | 08:26 |
| zhenguo | liusheng: hah, then the folowing up patches maybe related to aggregates | 08:26 |
| liusheng | zhenguo: seems not yet, hah | 08:27 |
| zhenguo | liusheng: lol | 08:27 |
| zhenguo | liusheng: before you can specify server groups | 08:27 |
| liusheng | zhenguo: yes | 08:27 |
| *** openstackgerrit has joined #openstack-mogan | 09:15 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add aggregate nodes API https://review.openstack.org/487284 | 09:15 |
| *** wanghao has quit IRC | 09:29 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Improve resources updating process https://review.openstack.org/487351 | 09:39 |
| *** wanghao has joined #openstack-mogan | 11:57 | |
| *** litao__ has quit IRC | 11:59 | |
| *** wanghao has quit IRC | 11:59 | |
| *** wanghao has joined #openstack-mogan | 12:08 | |
| *** wanghao has quit IRC | 12:09 | |
| openstackgerrit | Merged openstack/mogan master: Clean up some unused codes https://review.openstack.org/487701 | 14:42 |
| *** harlowja has joined #openstack-mogan | 14:51 | |
| openstackgerrit | Merged openstack/mogan master: Fix uwsgi doc to avoid misunderstanding https://review.openstack.org/487757 | 14:56 |
| *** harlowja has quit IRC | 15:51 | |
| *** openstackstatus has quit IRC | 16:41 | |
| *** openstack has joined #openstack-mogan | 16:45 | |
| *** harlowja has joined #openstack-mogan | 16:54 | |
| *** harlowja_ has joined #openstack-mogan | 16:55 | |
| *** harlowja has quit IRC | 16:59 | |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being taken offline for roughly 5 minutes to perform a database backup and reconfiguration | 17:13 | |
| *** openstackgerrit has quit IRC | 17:33 | |
| *** openstackstatus has quit IRC | 18:11 | |
| *** openstack has joined #openstack-mogan | 18:15 | |
| *** openstackstatus has quit IRC | 18:42 | |
| *** openstack has joined #openstack-mogan | 18:46 | |
| *** liusheng has quit IRC | 19:45 | |
| *** liusheng has joined #openstack-mogan | 19:46 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!