| *** wanghao has joined #openstack-mogan | 00:36 | |
| *** kevinz has joined #openstack-mogan | 01:20 | |
| zhenguo | morning mogan! | 01:55 |
|---|---|---|
| liusheng | morning | 02:15 |
| *** openstackgerrit has joined #openstack-mogan | 02:15 | |
| openstackgerrit | liusheng proposed openstack/mogan master: WIP: Split out network_info from Instance object https://review.openstack.org/430139 | 02:15 |
| openstackgerrit | Tao Li proposed openstack/mogan master: Support multi-instance creating (part 1) https://review.openstack.org/421064 | 02:19 |
| openstackgerrit | Tao Li proposed openstack/mogan master: Support multi-instance creating (part 1) https://review.openstack.org/421064 | 02:47 |
| zhenguo | liusheng: hi, can availability zone be None? | 03:09 |
| liusheng | zhenguo: not sure | 03:10 |
| liusheng | zhenguo: how it will be None ? | 03:10 |
| zhenguo | liusheng: currently for mogan, if we don't specify the option, it will be None | 03:11 |
| zhenguo | liusheng: but not sure for Nova | 03:11 |
| zhenguo | liusheng: I remember we follow nova's way, but just found it's not | 03:11 |
| liusheng | zhenguo: Nova has a default value | 03:12 |
| zhenguo | liusheng: let me check | 03:12 |
| liusheng | zhenguo: https://github.com/openstack/nova/blob/master/nova/conf/availability_zone.py#L32-L37 | 03:13 |
| zhenguo | liusheng: ah, yes, thanks | 03:13 |
| zhenguo | liusheng: Is cinder az default value same with nova's | 03:14 |
| liusheng | zhenguo: seems yes | 03:15 |
| liusheng | zhenguo: both the default values are 'Nova' | 03:15 |
| zhenguo | liusheng: nova? | 03:15 |
| zhenguo | liusheng: what default value is proper for us | 03:16 |
| liusheng | zhenguo: 'nova' | 03:16 |
| liusheng | zhenguo: currently we get az list from ironic | 03:16 |
| liusheng | zhenguo: how ironic set the az value ? | 03:16 |
| *** zhangjl has joined #openstack-mogan | 03:16 | |
| zhenguo | liusheng: as a node property | 03:16 |
| zhenguo | liusheng: if it's not set, how do we handle it | 03:17 |
| liusheng | zhenguo: maybe 'baremetal' as default value ? | 03:17 |
| zhenguo | liusheng: if we have different default value with other projects, maybe will introduce some problems | 03:22 |
| zhenguo | liusheng: like if we don't explicitly given the default az for both mogan and cinder, it will break if we attach a volume to the instance | 03:23 |
| liusheng | zhenguo: just talked with zhenyu, there is a config option named cross_az_attach in nova that allow users to attach volumes cross az | 03:27 |
| zhenguo | seems we have both default_schedule_zone and default_availability_zone | 03:27 |
| liusheng | zhenguo: hmm.. yes | 03:27 |
| zhenguo | liusheng: seems that option has been deprecated https://bugs.launchpad.net/nova/+bug/1497253 | 03:28 |
| openstack | Launchpad bug 1497253 in OpenStack Compute (nova) "different availability zone for nova and cinder when AZ is not explicitly given" [Low,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | 03:28 |
| * zhenguo brb | 03:29 | |
| zhenguo | zhangjl: sorry, I remember you commented to the patch for default_schedule_zone about the default value, but it doesn't have a default value indeed, not noticed that nova also have a defalt_availability_zone option | 03:41 |
| zhangjl | zhenguo::) | 03:42 |
| zhenguo | zhangjl: will introduce that soon | 03:42 |
| zhangjl | zhenguo:nice | 03:42 |
| zhenguo | zhangjl: for the production perspective, do you have concerns with different default value with other projects like nova, cinder... | 03:43 |
| zhangjl | zhenguo:no, maybe. our customers seems do not care this....orz. while, IMO, same with nova maybe good | 03:46 |
| zhenguo | zhangjl: ok thanks | 03:46 |
| zhangjl | zhenguo:np | 03:46 |
| zhenguo | liusheng: wdyt if we use a 'nova' for default value? | 03:47 |
| liusheng | zhenguo: I just thought about it, maybe we can keep same with Nova | 03:48 |
| zhenguo | liusheng: ok | 03:48 |
| openstackgerrit | ShaoHe Feng proposed openstack/mogan master: support configdrive https://review.openstack.org/416434 | 04:15 |
| openstackgerrit | ShaoHe Feng proposed openstack/mogan master: support configdrive https://review.openstack.org/416434 | 04:25 |
| zhenguo | I'm a bit confused with the default_schedule_zone and default_availability_zone | 04:48 |
| zhenguo | seems default_availability_zone is for compute node, and if we don't sepecify a default_schedule_zone and it's not in the request body, any node can pass. | 04:55 |
| zhenguo | should we follow that manner? when ironic node doesn't have a az propertiey, we use the default_availability_zone | 04:55 |
| openstackgerrit | ShaoHe Feng proposed openstack/mogan master: support configdrive https://review.openstack.org/416434 | 07:01 |
| zhenguo | liusheng: do you know why nova need to set device_owner to neutron port? | 07:59 |
| liusheng | zhenguo: maybe just indicate the ower of the port | 08:00 |
| zhenguo | liusheng: but in fact, the device id is the instance uuid | 08:01 |
| zhenguo | liusheng: device owner is something like 'compute:zone' | 08:01 |
| liusheng | zhenguo: really ? I remember the divice owner is instance uuid | 08:01 |
| zhenguo | liusheng: let me check | 08:02 |
| zhenguo | liusheng: https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L938-L940 | 08:03 |
| *** yuntongjin has joined #openstack-mogan | 08:04 | |
| liusheng | zhenguo: really weird, I will ask other people | 08:04 |
| zhenguo | liusheng: thanks | 08:05 |
| zhenguo | wanghao, litao: Is hanzhang still busy these days? hope he can join this channel again, then we can get network related answers from a neutron guy :D | 08:07 |
| zhenguo | Kevin_Zheng: are you around? | 08:16 |
| Kevin_Zheng | yeah im checking | 08:16 |
| zhenguo | thanks | 08:18 |
| zhenguo | Kevin_Zheng: And I have another question about default_schedule_zone, when it is not set and availability_zone not specified in the create request, any node can pass the az schedule filter, right? | 08:19 |
| Kevin_Zheng | yeas | 08:19 |
| Kevin_Zheng | yes | 08:20 |
| zhenguo | ok, thanks but I didn't find when the instance.availability_zone set with the schedule node az | 08:20 |
| *** yuntongjin has quit IRC | 08:21 | |
| Kevin_Zheng | I think we should understand what device_id means and what device_owner means | 08:23 |
| zhenguo | the concept is only for nova? | 08:24 |
| openstackgerrit | zhangjl proposed openstack/mogan master: Lock instances support in Mogan(part 1) https://review.openstack.org/430189 | 08:24 |
| Kevin_Zheng | why nova, I think it is concept in neutron | 08:24 |
| zhenguo | seems yes, but no neutron guy here :( | 08:25 |
| Kevin_Zheng | And, I think, as we discussed before, seems Neutron and Cinder AZs are designed to be sync with nova AZs | 08:26 |
| Kevin_Zheng | if I remembered correctly | 08:27 |
| zhenguo | yes, I read the discussion | 08:27 |
| zhenguo | but for ports associated with nova instances, the device_id and devcie_owner is only used by nova, right? | 08:27 |
| Kevin_Zheng | hm... not sure | 08:28 |
| zhenguo | currently we don't set these, but it works | 08:28 |
| zhenguo | luyao: a suggestion for instance fault info, do you think it's better to follow nova's manner to use a seperate table to save the fault? | 08:41 |
| openstackgerrit | liusheng proposed openstack/mogan master: WIP: Split out network_info from Instance object https://review.openstack.org/430139 | 08:43 |
| luyao | zhenguo, I think the dict is enough,I just ask your opinion | 08:45 |
| zhenguo | luyao: seems yes, but do you plan to save all operations fault info to that field? | 08:46 |
| zhenguo | luyao: maybe like power on? | 08:46 |
| luyao | zhenguo, sorry,I don't catch your meaning. | 08:48 |
| zhenguo | luyao: I mean, when power on failed, or update instance faild, do you plan to save the exception to fault_info? | 08:49 |
| luyao | zhenguo, yes | 08:52 |
| zhenguo | luyao: ok | 08:53 |
| zhenguo | luyao: I just read the nova api doc about the fault | 08:53 |
| zhenguo | luyao: A fault object. Only displayed in the failed response. Default keys are code, created, and message (response code, created time, and message respectively). In addition, the key details (stack trace) is available if you have the administrator privilege. | 08:53 |
| zhenguo | luyao: But as the first step, I agree that we don't have to be that convoluted like nova | 08:54 |
| zhenguo | luyao: have you test it? | 08:55 |
| luyao | zhenguo, yes,I think the fault in nova is a little different from the fault_info in mogan | 08:56 |
| luyao | zhenguo, no | 08:56 |
| zhenguo | luyao: aha, ok, you can test it in the devstack env if got time, I think the code is almost good now | 08:57 |
| luyao | zhenguo,ok | 08:57 |
| zhenguo | luyao: thanks | 08:57 |
| luyao | zhenguo, no thanks. :) | 08:58 |
| openstackgerrit | ShaoHe Feng proposed openstack/mogan master: support configdrive https://review.openstack.org/416434 | 08:59 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Add default_availability_zone config option https://review.openstack.org/430674 | 09:18 |
| * zhenguo brb | 09:19 | |
| *** openstackgerrit has quit IRC | 09:34 | |
| zhenguo | zhangjl: the first patch of instance lock seems ok, thanks! | 09:36 |
| zhenguo | shaohe_feng: seems the config drive patch passes all tests | 09:37 |
| zhangjl | zhenguo: my pleasure | 09:41 |
| *** wanghao has quit IRC | 09:41 | |
| *** litao has quit IRC | 09:45 | |
| *** litao has joined #openstack-mogan | 09:46 | |
| *** zhangjl1 has joined #openstack-mogan | 09:51 | |
| *** zhangjl has quit IRC | 09:52 | |
| *** openstackgerrit has joined #openstack-mogan | 10:01 | |
| openstackgerrit | Merged openstack/mogan master: Lock instances support in Mogan(part 1) https://review.openstack.org/430189 | 10:01 |
| *** kevinz has quit IRC | 10:17 | |
| *** zhangjl1 has quit IRC | 10:38 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Set device_id to neutron port https://review.openstack.org/430860 | 11:56 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: [WIP]Add disassociate floating ip support https://review.openstack.org/430862 | 12:01 |
| openstackgerrit | Merged openstack/mogan master: Add default_availability_zone config option https://review.openstack.org/430674 | 12:04 |
| *** kevinz has joined #openstack-mogan | 13:47 | |
| *** kevinz has quit IRC | 14:04 | |
| *** kevinz has joined #openstack-mogan | 14:05 | |
| *** kevinz has quit IRC | 14:18 | |
| *** Kevin_Zheng has quit IRC | 18:35 | |
| *** RuiChen has quit IRC | 18:35 | |
| *** lin_yang has quit IRC | 18:35 | |
| *** Kevin_Zheng has joined #openstack-mogan | 19:43 | |
| *** RuiChen has joined #openstack-mogan | 19:47 | |
| *** lin_yang has joined #openstack-mogan | 19:47 | |
| *** litao has quit IRC | 20:13 | |
| *** litao has joined #openstack-mogan | 20:14 | |
| *** Jeffrey4l has quit IRC | 21:34 | |
| *** Jeffrey4l has joined #openstack-mogan | 21:34 | |
| *** RuiChen has quit IRC | 21:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!