| *** wanghao has joined #openstack-mogan | 00:27 | |
| *** wanghao has quit IRC | 00:28 | |
| *** wanghao has joined #openstack-mogan | 00:29 | |
| *** wanghao_ has joined #openstack-mogan | 00:54 | |
| *** wanghao has quit IRC | 00:57 | |
| *** litao__ has joined #openstack-mogan | 00:59 | |
| wanghao_ | morning mogan! | 01:00 |
|---|---|---|
| litao__ | morning | 01:02 |
| zhenguo | wanghao_, litao__: morning | 01:24 |
| zhenguo | shaohe_feng, wanghao_, litao__, Xinran, zhangjl, zhangyang: can you join the meeting later? | 01:55 |
| wanghao_ | yeah | 01:55 |
| *** liusheng_ has joined #openstack-mogan | 01:56 | |
| litao__ | o/ | 01:56 |
| Xinran | zhenguo, ok | 02:00 |
| zhenguo | ok, let's move to #openstack-meeting | 02:00 |
| shaohe_feng | zhenguo: morning | 02:01 |
| litao__ | Xinran: I leave a comment in your patch | 02:52 |
| Xinran | litao__, thanks :) | 02:53 |
| wanghao_ | guys, if you have time, plz review the unmanage spec, thanks. | 02:59 |
| zhenguo | wanghao: ok | 03:04 |
| *** wanghao_ has quit IRC | 03:07 | |
| *** wanghao has joined #openstack-mogan | 03:07 | |
| *** wanghao has quit IRC | 03:32 | |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Pagination support for server list API https://review.openstack.org/514506 | 04:04 |
| *** wanghao has joined #openstack-mogan | 04:17 | |
| *** wanghao has quit IRC | 04:35 | |
| *** wanghao has joined #openstack-mogan | 04:38 | |
| *** wanghao has quit IRC | 04:53 | |
| *** wanghao has joined #openstack-mogan | 05:37 | |
| *** liusheng has joined #openstack-mogan | 06:15 | |
| *** zhangjl has quit IRC | 06:28 | |
| *** zhangjl has joined #openstack-mogan | 06:28 | |
| zhenguo | wanghao: hi, please check litao__'s replies here https://review.openstack.org/#/c/513215/ | 06:45 |
| wanghao | okay | 06:46 |
| zhenguo | litao__: seems that patch follows my implementation in ironic, lol | 06:47 |
| zhenguo | wanghao, litao__: for managed servers, I think we also need to put some informaiton to server system metadata to indicate it's not deployed by mogan | 06:48 |
| wanghao | zhenguo: yes | 06:49 |
| wanghao | zhenguo: maybe we can use server tag? | 06:49 |
| zhenguo | wanghao: server tag can be seen by users | 06:49 |
| zhenguo | wanghao: I would like to add a system metadata for internal use | 06:50 |
| wanghao | zhenguo: so you didn't want user to see this is a managed server? | 06:50 |
| zhenguo | wanghao: like nova :D | 06:50 |
| wanghao | zhenguo: emmm, that works too. | 06:50 |
| zhenguo | wanghao: hah | 06:51 |
| zhenguo | wanghao: I left some comments on the unmanage specs | 06:51 |
| wanghao | zhenguo: okay, thanks. | 06:51 |
| zhenguo | wanghao: just want to know if we plan to only unmanage the server that managed by mogan | 06:51 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Add admin_password support https://review.openstack.org/511198 | 06:51 |
| wanghao | zhenguo: I talked this with litao__, we want to unmanage both server that managed and server that created by mogan. | 06:52 |
| zhenguo | wanghao: ok | 06:52 |
| wanghao | zhenguo: since Cinder can support to unmanage a created volume. | 06:52 |
| zhenguo | wanghao: yes, but cinder volume is alone | 06:53 |
| zhenguo | wanghao: we have port/volume associated with server | 06:53 |
| wanghao | zhenguo: yes, indeed. | 06:53 |
| zhenguo | wanghao: so when unmanaging it, we should handle that associated things gracefully | 06:53 |
| wanghao | zhenguo: so we will keep the port and volume | 06:53 |
| wanghao | zhenguo: since this node may be using by users. | 06:53 |
| zhenguo | wanghao: yes, but if we remove the association information, it can be used by other servers | 06:53 |
| zhenguo | wanghao: just like if a port doesn't have a device_id, it can be attached to other servers | 06:54 |
| zhenguo | wanghao: so t he unmanaged server got network lost | 06:54 |
| wanghao | zhenguo: we didn't remove the port from neutron, so I think it have the device_id. | 06:55 |
| wanghao | zhenguo: we just remove the information from mogan. | 06:55 |
| zhenguo | wanghao: you mean we still keep the removed server id there? | 06:56 |
| wanghao | zhenguo: no, we remove the server from mogan. but keep it in neutron and cinder. | 06:56 |
| zhenguo | wanghao: volume also has some thing like device_id? | 06:57 |
| zhenguo | wanghao: seems that can help keep the port/volume | 06:57 |
| wanghao | zhenguo: if a volume be attached to a server, it will have server id | 06:59 |
| zhenguo | wanghao: ok | 06:59 |
| wanghao | zhenguo: yes, since we didn't remove the resource from neutron and cinder. So the port still have device_id and volume have the attached instance id. | 07:00 |
| zhenguo | wanghao: how about replace that server id with node id after unmanagement? | 07:00 |
| zhenguo | wanghao: seems a bit weird if we keep something that can't be found anymore | 07:01 |
| zhenguo | wanghao: and when manage the server, replace it with the new server id | 07:01 |
| zhenguo | wanghao: the node id can be used to find the associated port/volume of unmanaged servers | 07:03 |
| wanghao | zhenguo: I see, but I think Neutron and Cinder can't support it yet | 07:04 |
| wanghao | zhenguo: I didn't find the api in cinder at least. | 07:05 |
| zhenguo | wanghao: jsut update it | 07:05 |
| zhenguo | wanghao: I'm sure that port can be updated with device_id | 07:05 |
| zhenguo | wanghao: not sure volume | 07:05 |
| wanghao | zhenguo: cinder can't | 07:05 |
| zhenguo | wanghao: sigh | 07:05 |
| zhenguo | wanghao: we may got probolems | 07:06 |
| zhenguo | wanghao: as when manage a server, we also need to update it | 07:06 |
| wanghao | zhenguo: but I think we may don't update it, just keep it. | 07:06 |
| zhenguo | wanghao: so when managing it again, we also keep the old server id? | 07:07 |
| zhenguo | wanghao: operators may complain with that | 07:08 |
| wanghao | zhenguo: for example, a port have a device_id before managing, it should be a node id in Ironic, right? | 07:08 |
| zhenguo | wanghao: I think it is | 07:09 |
| zhenguo | wanghao: and now, we already have some logic to replace that during managing process | 07:09 |
| wanghao | zhenguo: why not just record the mapping between service id and device_id? | 07:10 |
| zhenguo | wanghao: that got extra work, hah | 07:11 |
| wanghao | zhenguo: but if want to replace it in unmanaging process, mogan must record the old device_id anyway. | 07:11 |
| wanghao | zhenguo: so maybe just record the mapping | 07:12 |
| zhenguo | wanghao: no, we have the node id | 07:12 |
| wanghao | zhenguo: so the only problem is cinder can't update the attached instance id.... | 07:13 |
| zhenguo | wanghao: yes | 07:13 |
| zhenguo | wanghao: but for now, we can keep that | 07:13 |
| zhenguo | wanghao: for volume | 07:13 |
| wanghao | zhenguo: emm | 07:14 |
| zhenguo | wanghao: not sure if it's possible to add that support in volume update API :D | 07:14 |
| zhenguo | wanghao: anyway, you can add some details of the taskflow in the specs | 07:15 |
| wanghao | zhenguo: sure | 07:16 |
| zhenguo | wanghao: thanks | 07:16 |
| wanghao | zhenguo: I check this cinder, it can support to update the connection_info, but can't support to update instance_id... | 07:17 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Pagination support for server list API https://review.openstack.org/514506 | 07:18 |
| zhenguo | wanghao: sigh, maybe can add a patch to Cinder for that :D | 07:18 |
| zhenguo | wanghao: but we still got volume support yet, maybe after we finished that | 07:18 |
| zhenguo | wanghao: not got | 07:18 |
| zhenguo | shaohe_feng, liusheng: please have a look at this https://review.openstack.org/#/c/513215/ , when you got time, thanks! | 07:19 |
| liusheng | zhenguo: ok | 07:19 |
| zhenguo | liusheng: thanks! | 07:19 |
| liusheng | zhenguo: np | 07:19 |
| wanghao | zhenguo: yes | 07:21 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Pagination support for server list API https://review.openstack.org/514506 | 07:40 |
| * zhenguo brb | 07:51 | |
| *** zhangjl has quit IRC | 07:56 | |
| *** zhangjl has joined #openstack-mogan | 07:57 | |
| *** liusheng_ has quit IRC | 08:15 | |
| *** wanghao_ has joined #openstack-mogan | 08:17 | |
| *** wanghao has quit IRC | 08:20 | |
| *** wanghao_ has quit IRC | 09:08 | |
| *** wanghao has joined #openstack-mogan | 09:10 | |
| *** wanghao has quit IRC | 09:11 | |
| *** wanghao has joined #openstack-mogan | 09:11 | |
| Xinran | zhenguo, I have register a bp for system metadata, here is the link : https://blueprints.launchpad.net/mogan/+spec/system-metadata | 09:27 |
| zhenguo | Xinran: thanks! | 09:35 |
| *** wanghao has quit IRC | 09:36 | |
| *** zhangjl has quit IRC | 11:08 | |
| *** zhangjl has joined #openstack-mogan | 11:10 | |
| *** zhangjl has quit IRC | 11:18 | |
| *** zhangjl has joined #openstack-mogan | 11:26 | |
| *** zhangjl has quit IRC | 11:32 | |
| *** zhangjl has joined #openstack-mogan | 11:42 | |
| *** RuiChen has joined #openstack-mogan | 11:43 | |
| *** zhangjl has quit IRC | 11:50 | |
| *** zhangjl has joined #openstack-mogan | 11:52 | |
| *** zhangjl has quit IRC | 12:03 | |
| *** zhangjl has joined #openstack-mogan | 12:04 | |
| *** zhangjl has quit IRC | 12:50 | |
| *** zhangjl has joined #openstack-mogan | 12:51 | |
| *** litao__ has quit IRC | 12:52 | |
| *** harlowja has quit IRC | 19:46 | |
| *** harlowja has joined #openstack-mogan | 20:33 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!