| *** Xinran has quit IRC | 00:05 | |
| *** Xinran has joined #openstack-mogan | 00:09 | |
| *** wanghao has joined #openstack-mogan | 00:15 | |
| *** Xinran has quit IRC | 00:16 | |
| *** Xinran has joined #openstack-mogan | 00:16 | |
| *** Xinran has quit IRC | 00:26 | |
| *** Xinran has joined #openstack-mogan | 00:26 | |
| *** Xinran has quit IRC | 00:36 | |
| *** Xinran has joined #openstack-mogan | 00:36 | |
| zhenguo | morning mogan! | 00:41 |
|---|---|---|
| *** Xinran has quit IRC | 00:46 | |
| *** Xinran has joined #openstack-mogan | 00:47 | |
| *** Xinran has quit IRC | 00:57 | |
| *** Xinran has joined #openstack-mogan | 00:57 | |
| zhenguo | liusheng: mind to have a look at this https://review.openstack.org/#/c/455548/ when you got time, on my test it works well | 01:00 |
| liusheng | zhenguo: sure | 01:00 |
| zhenguo | liusheng: thanks | 01:00 |
| liusheng | zhenguo: np | 01:00 |
| *** Xinran has quit IRC | 01:07 | |
| *** Xinran has joined #openstack-mogan | 01:07 | |
| *** wanghao has quit IRC | 01:10 | |
| *** wanghao has joined #openstack-mogan | 01:11 | |
| liusheng | zhenguo: hi, 2 comments inline. | 01:11 |
| zhenguo | liusheng: ok, will check | 01:11 |
| zhenguo | liusheng: hah, seems the comments are made by harlowja https://review.openstack.org/#/c/455548/3/mogan/engine/api.py | 01:16 |
| liusheng | zhenguo: lol | 01:17 |
| *** Xinran has quit IRC | 01:17 | |
| *** Xinran has joined #openstack-mogan | 01:18 | |
| zhenguo | liusheng: do you know why nova put user data to instance? | 01:21 |
| zhenguo | liusheng: seems all metadata related infomation like this will save to instance db | 01:22 |
| liusheng | zhenguo: the instance already include most of the instance properties, I think the user_data don't need to be an exception | 01:26 |
| *** Xinran has quit IRC | 01:27 | |
| zhenguo | liusheng: as I understand, the reason why nova instance save with such properties is they have a metadata API | 01:27 |
| zhenguo | liusheng: otherwise, seems no need for this, and not only user data, all metadata API GET releated properties should be saved | 01:28 |
| liusheng | zhenguo: oh, I got it, you mean in mogan, we don't save user_data in db | 01:28 |
| zhenguo | but we only save sunch data to configdrive, and we don't have a metadata API | 01:28 |
| zhenguo | liusheng: yes | 01:29 |
| zhenguo | liusheng: I don't find the reason to save all such data | 01:29 |
| *** Xinran has joined #openstack-mogan | 01:29 | |
| zhenguo | liusheng: another question, can user update user_data through metadata API? | 01:30 |
| liusheng | zhenguo: let me ask others | 01:31 |
| zhenguo | liusheng: ok, thanks | 01:31 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add tool for adoptting baremetal servers from Nova https://review.openstack.org/455692 | 01:31 |
| liusheng | zhenguo: zhenyu is in a meeting, he will help to check | 01:31 |
| zhenguo | liusheng: ok, and do you think we need to put configdrive task to creating workflow? | 01:32 |
| liusheng | zhenguo: seems reasonable to me | 01:34 |
| zhenguo | liusheng: ok, will add that task later after user data finished | 01:35 |
| liusheng | zhenguo: will review your big patch :D, ok | 01:35 |
| zhenguo | liusheng: hah, thanks very much | 01:35 |
| liusheng | zhenguo: np, you are so capable | 01:35 |
| liusheng | zhenguo: lol | 01:36 |
| zhenguo | lol | 01:36 |
| *** Xinran has quit IRC | 01:38 | |
| zhenguo | liusheng: I think we need to put some time to our client, will compare with nova to see which fields should be presented for create/show/update/... | 01:38 |
| *** Xinran has joined #openstack-mogan | 01:38 | |
| liusheng | zhenguo: ok | 01:38 |
| *** wanghao has quit IRC | 01:43 | |
| *** wanghao has joined #openstack-mogan | 01:44 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add user_data support https://review.openstack.org/455548 | 01:47 |
| *** Xinran has quit IRC | 01:47 | |
| *** wanghao has quit IRC | 01:58 | |
| *** wanghao has joined #openstack-mogan | 02:15 | |
| *** wanghao has quit IRC | 02:25 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add user_data support https://review.openstack.org/455548 | 02:26 |
| *** Kevin_Zheng has joined #openstack-mogan | 02:54 | |
| *** luyao has joined #openstack-mogan | 03:00 | |
| liusheng | zhenguo: do you think if it is better to put the adoption tool under mogan/ and add an entry point to add an command like: mogan-nova-adoption ? | 03:19 |
| zhenguo | liusheng: maybe we can have a mogan-manage commands to handle all such operations, | 03:20 |
| liusheng | zhenguo: you mean also use mogan-namage to perform db upgrade ? | 03:21 |
| zhenguo | liusheng: but seems no rush, we can hold on for a while for the adopt API, just a simple tool for now is enough, wdyt? | 03:21 |
| zhenguo | liusheng: maybe for all management commands | 03:21 |
| liusheng | zhenguo: ok, but last night, I realized there maybe a challenge to clean the bms in Nova after adopting the bms to Mogan :( | 03:24 |
| zhenguo | liusheng: hah, yes, as it will triger real node deleting | 03:24 |
| liusheng | zhenguo: and will try to delete port in nova code process | 03:24 |
| zhenguo | liusheng: yes, and the port also have some nova related properties | 03:25 |
| liusheng | zhenguo: :( | 03:26 |
| zhenguo | liusheng: nova will set instance_uuid as port device_id | 03:26 |
| zhenguo | liusheng: we also set our instance_uuid to neutron port | 03:26 |
| liusheng | zhenguo: in the adoption tool, the uuid of instance is same with nova | 03:27 |
| zhenguo | liusheng: can we just adopt that uuid as well? | 03:27 |
| zhenguo | liusheng: oh, that's good | 03:27 |
| liusheng | zhenguo: yes, we reuse the instance properties and ports | 03:27 |
| zhenguo | liusheng: you also use flavor id for instance_type uuid? | 03:33 |
| liusheng | zhenguo: yes | 03:33 |
| zhenguo | liusheng: hah | 03:33 |
| zhenguo | liusheng: the tool will automatically handle that if the instance_type doesn't exist? | 03:33 |
| liusheng | zhenguo: bug create a mogan instance type, and when adopting multiple instances, will check if it existed fistly | 03:34 |
| liusheng | zhenguo: yes | 03:34 |
| zhenguo | liusheng: nice | 03:34 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add tool for adoptting baremetal servers from Nova https://review.openstack.org/455692 | 03:36 |
| openstackgerrit | liusheng proposed openstack/mogan master: Add tool for adoptting baremetal servers from Nova https://review.openstack.org/455692 | 03:37 |
| openstackgerrit | Merged openstack/mogan master: Add user_data support https://review.openstack.org/455548 | 03:46 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Ignore 404 exception when deleting port https://review.openstack.org/456057 | 05:33 |
| *** Xinran has joined #openstack-mogan | 05:56 | |
| *** Xinran has quit IRC | 06:04 | |
| *** Xinran has joined #openstack-mogan | 06:06 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Add tool for adoptting baremetal servers from Nova https://review.openstack.org/455692 | 06:07 |
| *** Xinran has quit IRC | 06:14 | |
| *** Xinran has joined #openstack-mogan | 06:14 | |
| *** Xinran has quit IRC | 06:24 | |
| *** Xinran has joined #openstack-mogan | 06:25 | |
| *** Xinran has quit IRC | 06:34 | |
| *** Xinran has joined #openstack-mogan | 06:35 | |
| *** Xinran has quit IRC | 06:45 | |
| *** Xinran has joined #openstack-mogan | 06:45 | |
| *** Xinran has quit IRC | 06:55 | |
| *** Xinran has joined #openstack-mogan | 06:55 | |
| *** Xinran has quit IRC | 07:01 | |
| openstackgerrit | Zhenguo Niu proposed openstack/python-moganclient master: Add more columns to server list https://review.openstack.org/456106 | 08:09 |
| zhenguo | liusheng: hi, do you think we need to return node_uuid with instance when list or get server? | 08:20 |
| liusheng | zhenguo: at least, I think get server api should return | 08:21 |
| zhenguo | liusheng: does nova instance expose such information to common users? | 08:22 |
| zhenguo | liusheng: or only admin | 08:22 |
| liusheng | zhenguo: the nova server includes the host name info | 08:22 |
| zhenguo | liusheng: so both admin and common users can see this, right? | 08:23 |
| liusheng | zhenguo: I am sure the admin can | 08:24 |
| zhenguo | liusheng: hah, how about common users? | 08:24 |
| liusheng | zhenguo: let me try with non-admin users | 08:25 |
| zhenguo | liusheng: ok, thanks | 08:25 |
| liusheng | zhenguo: just confirmed, for amin users, the host will be directly show in the instance info. for non-admin user, host don't be showed directly, but it will be encoded with project_id by hash method as the "hostId" field of instance | 08:39 |
| zhenguo | liusheng: oh, that's convoluted :( | 08:40 |
| liusheng | zhenguo: lol | 08:40 |
| liusheng | zhenguo: host_id = hashlib.sha224((project_id + self.conf.host).encode('utf-8')).hexdigest() | 08:40 |
| zhenguo | liusheng: hah, I would like to enable this for mogan as well | 08:40 |
| liusheng | zhenguo: "OS-EXT-SRV-ATTR:host": "liusheng", | 08:41 |
| liusheng | zhenguo: for admin users, there is this field | 08:41 |
| zhenguo | liusheng: do you want to do it :D | 08:41 |
| liusheng | zhenguo: sure, I can pick it if you have other tasks :D | 08:41 |
| zhenguo | liusheng: thanks very much! | 08:42 |
| * liusheng sometimes have to care about Telemetry... | 08:42 | |
| zhenguo | hah | 08:42 |
| liusheng | zhenguo: np :) | 08:43 |
| openstackgerrit | Zhenguo Niu proposed openstack/python-moganclient master: Add more columns to server list https://review.openstack.org/456106 | 09:04 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Following up patch of c6102131994d8b1ab7189ddb3b3210c83edc4cd7 https://review.openstack.org/456144 | 09:58 |
| openstackgerrit | Merged openstack/python-moganclient master: Add more columns to server list https://review.openstack.org/456106 | 11:07 |
| openstackgerrit | Merged openstack/mogan master: Ignore 404 exception when deleting port https://review.openstack.org/456057 | 11:38 |
| openstackgerrit | Merged openstack/mogan master: Following up patch of c6102131994d8b1ab7189ddb3b3210c83edc4cd7 https://review.openstack.org/456144 | 11:41 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/mogan master: Updated from global requirements https://review.openstack.org/455939 | 11:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-moganclient master: Updated from global requirements https://review.openstack.org/455999 | 11:53 |
| *** luyao_ has joined #openstack-mogan | 12:08 | |
| openstackgerrit | Zhenguo Niu proposed openstack/python-moganclient master: Add support to specify userdata when creating a server https://review.openstack.org/456199 | 12:23 |
| * zhenguo away | 12:26 | |
| *** luyao_ has quit IRC | 12:26 | |
| openstackgerrit | Merged openstack/python-moganclient master: Updated from global requirements https://review.openstack.org/455999 | 12:26 |
| *** luyao_ has joined #openstack-mogan | 12:27 | |
| *** luyao has quit IRC | 12:28 | |
| *** luyao_ is now known as luyao | 12:28 | |
| *** luyao has quit IRC | 12:33 | |
| *** luyao has joined #openstack-mogan | 12:33 | |
| luyao | shaohe_feng, ping | 12:37 |
| openstackgerrit | Merged openstack/mogan master: Updated from global requirements https://review.openstack.org/455939 | 13:02 |
| *** liusheng has quit IRC | 14:42 | |
| *** Xinran_ has joined #openstack-mogan | 14:59 | |
| Xinran_ | luyao, ping | 14:59 |
| *** Xinran_ has quit IRC | 15:04 | |
| *** Xinran has joined #openstack-mogan | 15:08 | |
| *** Xinran has quit IRC | 15:11 | |
| *** Xinran has joined #openstack-mogan | 15:11 | |
| Xinran | luyao, ping | 15:16 |
| *** Kevin_Zheng has quit IRC | 16:13 | |
| *** l4yerffeJ has joined #openstack-mogan | 16:21 | |
| *** l4yerffeJ has quit IRC | 16:23 | |
| -openstackstatus- NOTICE: Restarting Gerrit for our weekly memory leak cleanup. | 21:26 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!