Wednesday, 2017-03-01

*** wxy has joined #openstack-mogan00:42
*** wanghao has joined #openstack-mogan00:51
*** litao has joined #openstack-mogan00:52
*** kevinz has joined #openstack-mogan01:13
*** zhangjl has joined #openstack-mogan01:19
wanghaozhenguo: qutoa patch has been updated, pls have a review.01:22
zhenguowanghao: thanks01:39
wanghaozhenguo: np01:40
*** wanghao has quit IRC01:48
*** wanghao has joined #openstack-mogan01:48
openstackgerritliusheng proposed openstack/mogan master: WIP: test support console -- ignore me
openstackgerritliusheng proposed openstack/mogan master: Split out network_info from Instance object
openstackgerritliusheng proposed openstack/mogan master: Split out network_info from Instance object
*** litao has quit IRC03:27
*** litao has joined #openstack-mogan03:28
*** wanghao has quit IRC03:28
*** wanghao has joined #openstack-mogan03:28
*** l4yerffeJ has joined #openstack-mogan03:38
*** l4yerffeJ has quit IRC03:42
*** l4yerffeJ has joined #openstack-mogan03:43
*** yuntongjin has joined #openstack-mogan04:20
*** yuntongjin has quit IRC04:25
openstackgerritZhenguo Niu proposed openstack/mogan master: Include lock state in InstanceStates API object
openstackgerritMerged openstack/python-moganclient master: Updated from global requirements
openstackgerritMerged openstack/mogan master: Updated from global requirements
openstackgerritMerged openstack/mogan master: Move api samples to correct directories
*** yuntongjin has joined #openstack-mogan05:49
openstackgerritliusheng proposed openstack/mogan master: Split out network_info from Instance object
*** yuntongjin has quit IRC06:28
*** yuntongjin has joined #openstack-mogan06:37
openstackgerritMerged openstack/mogan master: Include lock state in InstanceStates API object
zhenguoliusheng: hi, after refactoring to use instance.nics, should we change the API response as well? in fact, currently the instance.network_info is ports07:18
*** luyao has joined #openstack-mogan07:19
liushengzhenguo: yes, I want to also refactor the api, and want to ask your opinion :)07:22
zhenguoliusheng: hah, I think the current patch is ok, we can add another path for API side07:22
liushengzhenguo: yes, how do you think the "nics" field in instance body is like a list of dict, e.g. [{'port_id'xxx, 'port_type':xx, 'floatingip': xx, 'ip':xx}, .. another port] ?07:24
liushengzhenguo: since currently it is a dict and the keys is the port uuids.07:25
zhenguoliusheng: I think is ok, btw, do we have a uuid for nic?07:25
liushengzhenguo: it is implicit the port uuid07:25
liushengzhenguo: no, the primary key of nic is also port_id07:26
zhenguoliusheng: should we consider to add a uuid for nic, as it's a separate object now, like when detaching a interface, we should use it's uuid?07:27
zhenguoliusheng: or does it make sense that we have a nic which not associate neutron port id?07:28
zhenguoliusheng: I remember nova now support to create instance with nics that set net-id=none07:28
liushengzhenguo: I don't think so, the nic is still a port in fact, but include some extra info07:29
liushengzhenguo: yes07:29
zhenguoliusheng: let's try to make is clear, does nic equals to port?07:31
liushengzhenguo: I think the "port_id" field in a nic object is always not null, so the nic must include a port's identity. and also it can include port_type, floatingip. ip_address .. fields07:32
liushengzhenguo: what do you think  ?07:32
liushengzhenguo: it is like a phhsical Network Interface Card, and include all the attributes of the nic07:34
zhenguoliusheng: so you think a nic must be associate with a port?07:36
liushengzhenguo: yes07:36
zhenguoliusheng: for baremetal, as the physical interfaces is fixed, should we include all to the instance?07:37
zhenguoliusheng: like you may have a node which has 4 ports, but when request to creating a instance you may only set --nic with one net-id, so the instance should include one nic or 4 nics07:38
liushengzhenguo: but if want to use the physical interface, we must create a port with its mac firstly, right ?07:38
zhenguoliusheng: correct, if you want to make the interface work, you must create a neutron prot07:39
zhenguoliusheng: but the instance may have some nics that doesn't work now, and users can attach it to a neutron net work to make it work, make sense?07:40
zhenguoliusheng: or detach some nics07:40
liushengzhenguo: if we specified --nic with one net-id, I think the current code logic will create 4 ports in the net, and 4 nic objects will be also created internally07:40
zhenguoliusheng: I mean just specify --nic net-id=xx once07:41
liushengzhenguo: oh07:42
liushengzhenguo: users may need to create neutron port firstly if they want to attach a interface07:42
zhenguoliusheng: no07:42
zhenguoliusheng: we can provide a attach API07:43
liushengzhenguo: hmm I got you07:43
liushengzhenguo: if so, the only identity of a physical interface I think is the mac address07:44
liushengzhenguo: right  ?07:44
zhenguoliusheng: seems yes, but I'm not sure this make sense07:44
zhenguoliusheng: this means we should sync all ironic ports to our nics07:44
liushengzhenguo: just talked with zhenyu, he said seems Nova has a tendency that it is recommended to only support create instance by port specified07:46
liushengzhenguo: the creating instance with network specified way may will be deprecated07:46
liushengzhenguo: it is a legacy problem that Nova historical has nova-network07:47
zhenguoliusheng: oh, so users should create a port themselves?07:47
liushengzhenguo: yes07:47
zhenguoliusheng: hah, in fact, we don't support to specify port id now07:48
liushengzhenguo: lol07:48
liushengzhenguo: we should support that way :)07:49
zhenguoliusheng: sure, hah07:49
zhenguoliusheng: we can keep the current instance nics way, will test it and land it, thanks07:49
liushengzhenguo: thanks07:49
liushengzhenguo: btw, I need you to teach me about the console lol07:50
liushengzhenguo: do you have time ?07:50
zhenguoliusheng: hah, does it work now07:50
liushengzhenguo: for now, I only uploaded a experimental patch, throngh it I can start mogan-shellinaboxproxy and mogan-consoauth service07:51
liushengzhenguo: and a api get_vnc_console07:52
liushengzhenguo: I can get the a console url by the api paste it in browser and mogan-shellinaboxproxy can handle the request07:52
zhenguoliusheng: so it works now?07:53
liushengzhenguo: but I don't know how config ironic, the ironic node -- libvirt doman cannot open the connection, as you have suggested me, I can modify the xml to make it work07:54
liushengzhenguo: but I don't how to config ironic make it work07:54
liushengzhenguo: do you still remember ?07:55
liushengzhenguo:     <console type='pty'>07:55
liusheng      <target type='serial' port='0'/>07:55
liusheng    </console>07:55
zhenguoliusheng: you mean ironic shellinabox console also not work now?07:55
liushengzhenguo: the above xml you have suggested me to modify it to the xml07:55
zhenguoliusheng: you can enable ironic node console and get a console url07:55
zhenguoliusheng: so with 'virsh console' works not not?07:56
liushengzhenguo: cannot work, same problem07:56
zhenguoliusheng: I remember it works after that change07:57
liushengzhenguo: yes07:57
liushengzhenguo: definitely I cannot modify manually evey time :)07:58
zhenguoliusheng: hah07:58
zhenguoliusheng: so if you manually modifyed the xml file, it can work, right?08:03
liushengzhenguo: you can try in your bluecloud08:04
liushengzhenguo: error: internal error: character device <null> is not using a PTY08:05
*** litao has quit IRC08:05
*** litao has joined #openstack-mogan08:05
zhenguoliusheng: that's because your virsh xml still have problems08:06
zhenguoliusheng: you should make sure virsh console work first08:06
liushengzhenguo: yes, but I don't know how to config08:06
liushengzhenguo: may some related config options in ironic ?08:07
zhenguoliusheng: yes, I added that option,08:07
liushengzhenguo: which one ?08:07
zhenguoliusheng: but not sure whether it works or not08:07
zhenguoliusheng: wait for a sec, let me find08:07
liushengzhenguo: you have uploaded patch of mogan's devstack plugin ?08:08
zhenguoliusheng: no08:09
liushengzhenguo: you mean fixed in ironic ?08:10
zhenguoliusheng: Ironic seems already support this, which added by me after ssh cosnole support08:10
liushengzhenguo: ok, don't need any config option ?08:11
liushengzhenguo: I can try tup update ironic code  in my env and try again08:13
zhenguoliusheng: please hold on, let me figure it out08:13
liushengzhenguo: ok thanks08:14
zhenguoliusheng: as I understand if you set IRONIC_VM_LOG_CONSOLE to False, it will create a vm with pty console08:22
liushengzhenguo: you mean config it in local.conf ?08:23
zhenguoliusheng: yes, but seems it not work in my env08:23
liushengzhenguo: :(08:24
openstackgerritZhong Luyao proposed openstack/mogan master: Add fault object
zhenguoliusheng: you can read the code from here
liushengzhenguo: ok08:26
zhenguoluyao: sorry, seems the instance fault patch will conflict after this landed :(08:28
luyaozhenguo, never mind, i saw that, I will do rebase on it08:35
zhenguoluyao: thanks08:35
openstackgerritMerged openstack/mogan master: Split out network_info from Instance object
zhenguoluyao: and seems the patch only include fault object change right? you plan to add a following up patch for real instance faults added?08:36
luyaozhenguo, yes, I intend to do so08:37
zhenguoluyao: ok, thanks a lot08:37
liushengzhenguo:  seems it is relate with these lines of code08:38
luyaozhenguo, no thanks,:)08:38
zhenguoluyao: yes, and seems if you set the option to false, it will set console to pty08:39
zhenguooh, sorry send to wrong person08:40
openstackgerritZhenguo Niu proposed openstack/mogan master: [api-ef]Add locked to states API
zhenguowanghao: sorry, seems the quota patch needs to be rebased after the nics patch landed :(08:49
liushengzhenguo: yes, it seems only depend on IRONIC_VM_LOG_CONSOLE08:49
openstackgerritwanghao proposed openstack/mogan master: Quota support in Mogan(part 1)
liushengzhenguo: I will try again08:49
wanghaozhenguo: yes, it's coming08:49
zhenguowanghao: haha,08:49
wanghaozhenguo: :)08:50
zhenguowanghao: will land it soon after tested it08:50
zhenguoliusheng: yes, but on my env, I just checked the option is set,  but the xml file isn't as expected, you can try08:51
*** yuntongjin has quit IRC08:52
openstackgerritliusheng proposed openstack/mogan master: WIP: test support console -- ignore me
*** wanghao_ has joined #openstack-mogan09:00
*** wanghao has quit IRC09:04
*** yuntongjin has joined #openstack-mogan09:24
*** wanghao_ has quit IRC09:24
*** wanghao has joined #openstack-mogan09:25
*** wanghao has quit IRC09:32
*** litao has quit IRC09:34
*** zhangjl has quit IRC09:44
*** yuntongjin has quit IRC10:01
openstackgerritMerged openstack/mogan master: [api-ef]Add locked to states API
openstackgerritZhong Luyao proposed openstack/mogan master: Add fault object
*** kevinz has quit IRC10:12
*** Jeffrey4l has quit IRC10:30
*** Jeffrey4l has joined #openstack-mogan10:49
*** kevinz has joined #openstack-mogan12:18
*** kevinz has quit IRC12:18
*** yuntongjin has joined #openstack-mogan12:43
*** yuntongjin has quit IRC13:13
*** yuntongjin has joined #openstack-mogan13:14
*** yuntongjin has quit IRC13:46
*** yuntongjin has joined #openstack-mogan13:46
Kevin_Zhengwhat irc client you guys using14:01
*** yuntongjin has quit IRC14:09
openstackgerritMerged openstack/mogan master: Add fault object
*** Jeffrey4l has quit IRC17:59
*** openstackgerrit has quit IRC20:03
*** l4yerffeJ has quit IRC20:41
kongKevin_Zheng: irccloud20:49

Generated by 2.14.0 by Marius Gedminas - find it at!