| zhenguo | morning nimble! | 00:56 |
|---|---|---|
| liusheng | morning! | 01:15 |
| *** Jeffrey4l has quit IRC | 01:39 | |
| *** Jeffrey4l has joined #openstack-nimble | 01:39 | |
| zhenguo | liusheng: moring | 01:58 |
| liusheng | zhenguo: morning :) | 01:58 |
| zhenguo | liusheng: so many internal meeting today, will be alive this afternoon :P | 01:59 |
| liusheng | zhenguo: hah | 01:59 |
| liusheng | zhenguo: you are so busy | 01:59 |
| silenceli | morning:) | 02:23 |
| *** lambert has joined #openstack-nimble | 02:42 | |
| silenceli | Hello everyone, lambert is my colleague. He is also from china mobile, we both work on ironic and anything it's related to. | 02:51 |
| shaohe_feng | silenceli: lambert: welcome | 02:52 |
| silenceli | thx :-) | 02:57 |
| lambert | Hello everyone, i'm glad to have this opportunity to discuss something interesting with all of you, thanks! | 03:00 |
| shaohe_feng | lambert: yes. here is good to discuss interesting | 03:37 |
| shaohe_feng | zhenguo: liusheng: can you check your glance code? | 03:37 |
| shaohe_feng | zhenguo: liusheng: $ git show 5601f7f | 03:37 |
| shaohe_feng | zhenguo: liusheng: I have see this patch, it add a "img_config_drive" attr | 03:38 |
| liusheng | zhenguo: ok, let my update the code firstly | 03:38 |
| shaohe_feng | zhenguo: liusheng: but http://developer.openstack.org/api-ref/image/v2/?expanded=show-image-details-detail | 03:39 |
| shaohe_feng | does not tell me how to get the "img_config_drive" | 03:39 |
| shaohe_feng | zhenguo: liusheng: we need to check the "img_config_drive" of a image. | 03:39 |
| shaohe_feng | zhenguo: liusheng: if the "img_config_drive" is mandatory, nimble need to generate the config_drive | 03:40 |
| liusheng | shaohe_feng: seems a incompleted feature ? | 03:46 |
| liusheng | zhenguo: the "img_config_drive" was only found once when i search the whole glance code | 03:47 |
| shaohe_feng | liusheng: I "git show 5601f7f", seems it support it. | 03:51 |
| shaohe_feng | liusheng: do we need to check this img_config_drive attr? | 03:52 |
| shaohe_feng | zhenguo: ^ | 03:52 |
| liusheng | zhenguo: yes, I mean I cannot found where glance use this metadata | 03:52 |
| zhenguo | shaohe_feng,liusheng: sorry, I have some internal stuff to deal with, will ping you back | 03:53 |
| * zhenguo away | 03:53 | |
| shaohe_feng | liusheng: yes. and I can not get the "img_config_drive" from the image docs. | 03:54 |
| liusheng | shaohe_feng: so I doubt the patch is a part of the feature, and the feature is incompleted, not sure :) | 03:55 |
| *** Jeffrey4l has quit IRC | 04:57 | |
| *** Jeffrey4l has joined #openstack-nimble | 04:58 | |
| *** l4yerffeJ has joined #openstack-nimble | 04:58 | |
| *** luyao has quit IRC | 07:02 | |
| zhenguo | shaohe_feng, liusheng: hi, I'm back | 07:03 |
| zhenguo | lambert: welcome! | 07:03 |
| zhenguo | shaohe_feng, liusheng: wrt config drive, do we need image_config_dirve? | 07:04 |
| zhenguo | oh, seems nobody is here... | 07:06 |
| *** zhangjl has joined #openstack-nimble | 07:06 | |
| zhenguo | silenceli, lambert: in fact, you can try to play with nimble as we have already implemented the basic functionalities and it's really easy to setup an env with our devstack plugin :P | 07:09 |
| *** l4yerffeJ has quit IRC | 07:11 | |
| *** l4yerffeJ has joined #openstack-nimble | 07:12 | |
| zhenguo | liusheng: regarding https://review.openstack.org/#/c/414801/ , can we change to use net-id instead of net_id like nova? | 07:27 |
| *** l4yerffeJ has quit IRC | 07:32 | |
| *** l4yerffeJ has joined #openstack-nimble | 07:33 | |
| *** l4yerffeJ has quit IRC | 07:45 | |
| liusheng | zhenguo: you mean in nimble code or in CLI commands ? | 07:52 |
| liusheng | zhenguo: I prefer underline than strike in code | 07:53 |
| zhenguo | liusheng: in CLI, but I just found it's net-id indeed, hah but the commit message said it's net_id | 07:53 |
| liusheng | zhenguo: lol | 07:53 |
| zhenguo | liusheng: agree, but in CLI seems better to use what like nova | 07:53 |
| zhenguo | liusheng: and I left a comment there, need to update that as well | 07:54 |
| liusheng | zhenguo: yes, in CLI, it should be net-id | 07:54 |
| zhenguo | liusheng: yes, thanks | 07:55 |
| openstackgerrit | liusheng proposed openstack/python-nimbleclient: Change "uuid" in instance post request body to "net_id" https://review.openstack.org/414801 | 07:58 |
| openstackgerrit | Zhenguo Niu proposed openstack/nimble: Refactor delete_instance https://review.openstack.org/413677 | 08:44 |
| zhenguo | Kevin_Zheng: are you around? | 08:46 |
| Kevin_Zheng | yes whats up | 08:47 |
| zhenguo | I have a question, is az a mandatory arg when creating instance? | 08:48 |
| Kevin_Zheng | no but a default value will be assigned | 08:48 |
| zhenguo | so if you don't specify an az, it will assgin a default az | 08:49 |
| zhenguo | ok, thanks | 08:49 |
| Kevin_Zheng | NP | 08:49 |
| zhenguo | liusheng: so for us, do you think we should work like nova? | 08:50 |
| liusheng | zhenguo: does ironic has default az ? | 08:51 |
| zhenguo | liusheng: no, | 08:52 |
| zhenguo | liusheng: we need to set az to ironic node properties | 08:52 |
| zhenguo | liusheng: currently if there's no az in ironic node, and users don't specify az, we just match them together | 08:53 |
| zhenguo | liusheng: seems like it's reasonable, hah | 08:53 |
| liusheng | zhenguo: yes, I think we should | 08:54 |
| zhenguo | liusheng: but when show the instance, there's no any az information present | 08:55 |
| liusheng | zhenguo: the az is a property of ironic node, not a property of nimble server, right ? | 08:55 |
| zhenguo | liusheng: no | 08:55 |
| zhenguo | liusheng: oh, yes, it's a property of ironic node, | 08:56 |
| zhenguo | liusheng: but instance need to include az information. as you may want to list instances by az | 08:56 |
| liusheng | zhenguo: yes | 08:56 |
| liusheng | zhenguo: agree | 08:56 |
| liusheng | zhenguo: the az of a node is configured in ironic or set by ironic api ? | 08:57 |
| zhenguo | liusheng: by ironic api | 08:57 |
| liusheng | zhenguo: if so, may we need to call ironic api to set the az info | 08:58 |
| zhenguo | liusheng: seems not, we just need to modify our node cache | 08:58 |
| zhenguo | liusheng: if the ironic node doens't have an az, we set the default az in our cache. | 08:58 |
| zhenguo | liusheng: no need to waste api call | 08:59 |
| liusheng | zhenguo: but the node cache isn't persistent ? | 08:59 |
| zhenguo | liusheng: yes, and we dont need az persistent, we just need to check whether it has an az | 09:00 |
| liusheng | zhenguo: so you propose the default az don't need to be set to the ironic nodes ? | 09:01 |
| zhenguo | liusheng: seems yes | 09:01 |
| zhenguo | liusheng: or we may also don't need to set in our node cache | 09:02 |
| liusheng | zhenguo: hmm, i am not sure is there any problem about inconsistency | 09:02 |
| zhenguo | liusheng: as az seems just used for scheduling | 09:02 |
| liusheng | zhenguo: seems reasonalbe | 09:03 |
| zhenguo | liusheng: you mean someone may want to call ironic api directly to see it's in which az? | 09:03 |
| liusheng | zhenguo: yes | 09:03 |
| zhenguo | liusheng: at least, tenants can't do taht | 09:04 |
| zhenguo | s/taht/that | 09:04 |
| zhenguo | liusheng: If operators want to do that, he should set it correctly | 09:04 |
| liusheng | zhenguo: just because the ironic api is not for end users, right ? | 09:04 |
| zhenguo | liusheng: yes, it's a black box for end users | 09:05 |
| liusheng | zhenguo: ok, so if the node in ironic has been set an az, we can specify the az to create sever, if we dont' specify an az, a default az will be assigned to the sever | 09:06 |
| zhenguo | liusheng: yes, | 09:06 |
| liusheng | zhenguo: don't we need to expose the "set az" api in nimble ? | 09:06 |
| liusheng | zhenguo: just pass the request to ironic api | 09:07 |
| zhenguo | liusheng: I don't think we should expose that to users | 09:07 |
| liusheng | zhenguo: but nova provide that api | 09:08 |
| zhenguo | liusheng: really? users can set their az? | 09:08 |
| liusheng | zhenguo: seems only for admin users | 09:08 |
| zhenguo | liusheng: hah, not sure if we need that, but now I think we don't | 09:09 |
| liusheng | zhenguo: yes, agree, at least for now | 09:09 |
| zhenguo | liusheng: yes, and we have a set of ironic api for admin, so seems no need to add that | 09:10 |
| liusheng | zhenguo: yes | 09:10 |
| shaohe_feng | zhenguo: I'm back | 09:11 |
| zhenguo | shaohe_feng: hah, welcome back | 09:11 |
| shaohe_feng | zhenguo: that are 3 ways in nova to require configdrive | 09:13 |
| shaohe_feng | zhenguo: let me show you. | 09:14 |
| zhenguo | shaohe_feng: seems we only need to care about the way for ironic | 09:14 |
| zhenguo | shaohe_feng: we only need to pass data to ironic, and ironic will save them to swift | 09:16 |
| zhenguo | shaohe_feng: during deployment, IPA will pull that data from swift, and add an extra partition to save it | 09:16 |
| shaohe_feng | zhenguo: https://github.com/openstack/nova/blob/master/nova/virt/configdrive.py#L158 | 09:16 |
| shaohe_feng | zhenguo: instance.config_drive means pass it by API | 09:17 |
| shaohe_feng | zhenguo: API require to use configdrive | 09:17 |
| shaohe_feng | zhenguo: CONF.force_config_drive , config the require in conf file | 09:18 |
| shaohe_feng | zhenguo: instance.image_meta.properties.get | 09:18 |
| zhenguo | shaohe_feng: and we don't need to set config drive, or specify config drive with API, as it's the only way for us | 09:18 |
| shaohe_feng | zhenguo: the image glance need config_drive, or the OS on the node will not works well after we install the image. | 09:19 |
| zhenguo | shaohe_feng: why glance image need that? | 09:20 |
| zhenguo | shaohe_feng: oh, yes, I remember when building image seems we need to specify configdrive | 09:21 |
| shaohe_feng | zhenguo: 1. API enable instance.config_drive 2. conf file enable CONF.force_config_drive 3. glance image require config_drive. instance.image_meta.properties.get("img_config_drive") | 09:21 |
| zhenguo | shaohe_feng: these are the 3 ways or all required? | 09:22 |
| shaohe_feng | zhenguo: yes. Then we can inject the key or passwork or network config to the image when install node by config_drive | 09:22 |
| shaohe_feng | zhenguo: all means kvm, ironic and vmware? | 09:22 |
| shaohe_feng | zhenguo: you can seen the code is "or" https://github.com/openstack/nova/blob/master/nova/virt/configdrive.py#L164 | 09:23 |
| zhenguo | shaohe_feng: ok, I will dig | 09:23 |
| zhenguo | shaohe_feng: but seems we don't need image support that | 09:24 |
| shaohe_feng | zhenguo: any of these 3 ways we config, then ironic will use config_drive | 09:24 |
| shaohe_feng | zhenguo: call you by wechat. | 09:24 |
| zhenguo | shaohe_feng: ok | 09:24 |
| shaohe_feng | zhenguo: git show 5601f7f | 09:27 |
| shaohe_feng | zhenguo: search img_config_drive | 09:27 |
| shaohe_feng | zhenguo: https://github.com/openstack/nova/blob/master/nova/virt/configdrive.py#L164 | 09:28 |
| shaohe_feng | liusheng: has discuss with zhenguo, nimble will always enable config_drive | 09:40 |
| zhenguo | shaohe_feng, liusheng: wechat is really an effective communication tool :P | 09:41 |
| shaohe_feng | liusheng: so we do not care whether image need it. | 09:41 |
| shaohe_feng | zhenguo: yes, than IRC. | 09:41 |
| zhenguo | shaohe_feng: hah | 09:41 |
| *** liusheng has quit IRC | 09:59 | |
| *** liusheng has joined #openstack-nimble | 10:00 | |
| zhenguo | liusheng, shaohe_feng: do you think we need to create another webchat group just for contributors | 11:00 |
| zhenguo | s/webchat/wechat | 11:01 |
| liusheng | zhenguo: it is not good to talk technology in "大宝剑" group lol | 11:04 |
| zhenguo | liusheng: yes, hah | 11:04 |
| liusheng | zhenguo: I am OK | 11:04 |
| zhenguo | liusheng: ok | 11:05 |
| zhenguo | zhangjl: hi, welcome here, are you also from china mobile? | 11:09 |
| zhangjl | no | 11:09 |
| zhangjl | from awcloud | 11:09 |
| zhenguo | zhangjl: oh, welcome | 11:09 |
| zhangjl | a new of nimble | 11:09 |
| liusheng | zhangjl: welcome! | 11:09 |
| zhangjl | zhenguo,liusheng:Thanks | 11:10 |
| zhenguo | zhangjl: I'm curious about how do you know this project? | 11:10 |
| zhangjl | enn, from github and openstack | 11:10 |
| zhenguo | zhangjl: hah, please feel free to ping us if needed | 11:11 |
| zhangjl | zhenguo:That`s great!!!! | 11:11 |
| zhenguo | zhangjl: you can first setup an env with our devstack plugin | 11:12 |
| zhangjl | zhenguo:yes | 11:12 |
| zhenguo | zhangjl: ok, thanks | 11:13 |
| zhangjl | :-D | 11:14 |
| zhangjl | zhenguo:bye! | 11:16 |
| zhenguo | zhangjl: see you! | 11:16 |
| *** zhangjl has quit IRC | 11:16 | |
| zhenguo | liusheng: still one thing left here https://review.openstack.org/#/c/414801/ | 11:25 |
| liusheng | zhenguo: ok, I am so careless | 11:26 |
| zhenguo | liusheng: hah | 11:26 |
| openstackgerrit | liusheng proposed openstack/python-nimbleclient: Change "uuid" in instance post request body to "net_id" https://review.openstack.org/414801 | 11:28 |
| openstackgerrit | Zhenguo Niu proposed openstack/nimble: Add az check before instance create https://review.openstack.org/414949 | 12:07 |
| openstackgerrit | liusheng proposed openstack/nimble: Add versioned notification related objects fields https://review.openstack.org/415214 | 12:26 |
| openstackgerrit | liusheng proposed openstack/nimble: Add versioned object notification base(WIP) https://review.openstack.org/415215 | 12:26 |
| openstackgerrit | liusheng proposed openstack/python-nimbleclient: Change "uuid" in instance post request body to "net-id" https://review.openstack.org/414801 | 12:30 |
| openstackgerrit | liusheng proposed openstack/nimble: Add versioned notification related objects fields https://review.openstack.org/415214 | 12:35 |
| openstackgerrit | liusheng proposed openstack/nimble: Add versioned object notification base(WIP) https://review.openstack.org/415215 | 12:35 |
| openstackgerrit | liusheng proposed openstack/python-nimbleclient: Change "uuid" in instance post request body to "net-id" https://review.openstack.org/414801 | 12:43 |
| openstackgerrit | Zhenguo Niu proposed openstack/nimble: Change "uuid" in instance post request body to "net_id" https://review.openstack.org/414775 | 12:49 |
| openstackgerrit | liusheng proposed openstack/nimble: Add versioned object notification base(WIP) https://review.openstack.org/415215 | 14:06 |
| openstackgerrit | liusheng proposed openstack/nimble: Add versioned object notification base https://review.openstack.org/415215 | 14:07 |
| *** l4yerffeJ has joined #openstack-nimble | 14:40 | |
| *** Jeffrey4l has quit IRC | 15:02 | |
| *** Jeffrey4l has joined #openstack-nimble | 15:09 | |
| *** Jeffrey4l has quit IRC | 15:12 | |
| *** Jeffrey4l has joined #openstack-nimble | 15:13 | |
| *** Jeffrey4l has quit IRC | 20:30 | |
| *** l4yerffeJ has quit IRC | 20:30 | |
| *** Jeffrey4l_ has joined #openstack-nimble | 20:31 | |
| *** l4yerffeJ_ has joined #openstack-nimble | 20:31 | |
| *** l4yerffeJ_ has quit IRC | 20:31 | |
| *** Jeffrey4l_ has quit IRC | 21:35 | |
| *** Jeffrey4l_ has joined #openstack-nimble | 21:36 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!