Tuesday, 2017-04-18

*** wanghao has joined #openstack-mogan00:37
*** wanghao has quit IRC00:55
*** wanghao has joined #openstack-mogan00:56
*** wanghao_ has joined #openstack-mogan01:04
*** wanghao has quit IRC01:07
*** kevinz has joined #openstack-mogan01:13
*** wanghao has joined #openstack-mogan01:13
*** wanghao_ has quit IRC01:16
zhenguomorning mogan!01:31
*** wanghao has quit IRC01:36
*** wanghao has joined #openstack-mogan01:37
openstackgerritMerged openstack/mogan master: Fix doc generation for Python3  https://review.openstack.org/45720801:47
openstackgerritZhenguo Niu proposed openstack/mogan master: Change to use /flavors in URI  https://review.openstack.org/45722702:31
zhenguoliusheng: hi, I would like to replace /types with /flavors in URI, and then /instances to /servers maybe, wdyt?02:43
liushengzhenguo: totally agree02:43
zhenguoliusheng: hah02:43
liushengzhenguo: I have found your flavor patch02:43
zhenguoliusheng: but how about the db and objects, should we still keep instances and instance_types like nova?02:44
liushengzhenguo: maybe replace all of them ?02:45
zhenguoliusheng: ok, let me change the URI first02:45
liushengzhenguo: seems the instance concept nova also borrow from others02:45
liushengzhenguo: cool, thanks02:45
zhenguoliusheng: yes,02:45
liushengzhenguo: i will land the keypair feature asap02:47
zhenguoliusheng: thanks very much02:47
liushengzhenguo: lol02:47
zhenguoliusheng: hah, then I will add it to configdrive02:47
liushengzhenguo: ok02:48
zhenguoliusheng: ops, seems the moganclient should be changed first as our devstack plugin need to create a flavor :(02:52
liushengzhenguo: haha, that seems will lock the gate02:53
zhenguoliusheng: yes, do we have such test in client gate?02:53
liushengzhenguo: I don't think so02:54
zhenguoliusheng: so we can add a depends-on of client change :D02:54
liushengzhenguo: yes, please ping me if you  want to merge patch in client side02:55
zhenguoliusheng: ok, thanks02:55
openstackgerritZhenguo Niu proposed openstack/python-moganclient master: Replace /types with /flavors in URI  https://review.openstack.org/45743703:12
openstackgerritZhenguo Niu proposed openstack/mogan master: Change to use /flavors in URI  https://review.openstack.org/45722703:14
*** wanghao_ has joined #openstack-mogan03:26
*** wanghao_ has quit IRC03:27
*** wanghao has quit IRC03:29
*** lin_yang has quit IRC06:43
openstackgerritZhenguo Niu proposed openstack/mogan master: Change to use /flavors in URI  https://review.openstack.org/45722707:02
openstackgerritliusheng proposed openstack/mogan master: [PoC] keypair support  https://review.openstack.org/45720507:07
openstackgerritZhenguo Niu proposed openstack/mogan master: Change to use /flavors in URI  https://review.openstack.org/45722707:37
openstackgerritZhenguo Niu proposed openstack/mogan-ui master: Remove egg-info*  https://review.openstack.org/45749207:44
openstackgerritMerged openstack/mogan-ui master: Remove egg-info*  https://review.openstack.org/45749207:55
openstackgerritTao Li proposed openstack/mogan master: [WIP]Improve the multi-instance creation  https://review.openstack.org/45367508:16
*** litao has joined #openstack-mogan08:17
liushengzhenguo: FYI: the devstack env in physical machines is ready08:20
zhenguoliusheng: cool08:21
liushengzhenguo: :D08:21
zhenguoliusheng: so the next step, you can try to add a real node to mogan with the required driver information to test08:21
liushengzhenguo: ok, will try, I need to study more08:22
zhenguoliusheng: hah, at least you can test power on/off the node, and maybe console08:23
liushengzhenguo: a bit strange, I added a "disable_service nova" in local conf, but it still installed nova08:23
liushengzhenguo: cannot disable nova by this option ?08:23
zhenguoliusheng: you need to disalbe all nova services08:24
liushengzhenguo: but no effect, I have commented the "VIRT_DRIVER=ironic". lol08:24
liushengzhenguo: nova will use kvm08:24
zhenguoliusheng: lol08:25
litao: who knows the error "InternalServerError: No module named uts46data (HTTP 500)"08:27
zhenguolitao: what's that?08:28
litaoI created a server by mogan in devstack, the ironic-conductor raised the exception08:28
zhenguolitao: ironic need uts46data?08:29
litaoyes08:29
zhenguolitao: your ironic env is also in devstack or it's a real physical env?08:29
litaoThe ironic used the request module, and the request module add the feature08:29
litaodevstack env08:29
zhenguolitao: not sure what's happend :(08:30
* litao zhenguo: The version fo requests module is 2.12.508:30
liushenglitao: any error traceback ?08:31
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance Traceback (most recent call last):08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance   File "/opt/stack/mogan/mogan/engine/flows/create_instance.py", line 166, in _build_networks08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance     port_dict['id'])08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance   File "/opt/stack/mogan/mogan/engine/baremetal/ironic/driver.py", line 267, in plug_vif08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance     self.ironicclient.call("port.update", ironic_port_id, patch)08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance   File "/opt/stack/mogan/mogan/common/ironic.py", line 122, in call08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance     return self._multi_getattr(client, method)(*args, **kwargs)08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance   File "/usr/lib/python2.7/site-packages/ironicclient/v1/port.py", line 125, in update08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance     return self._update(resource_id=port_id, patch=patch)08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance   File "/usr/lib/python2.7/site-packages/ironicclient/common/base.py", line 191, in _update08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance     resp, body = self.api.json_request(method, url, body=patch)08:32
litao2017-04-17 05:09:27.255 TRACE mogan.engine.flows.create_instance   File "/usr/lib/python2.7/site-packages/ironicclient/common/http.py", line 560, in json_request08:32
*** openstackgerrit has quit IRC08:33
liushenglitao: just need the log messages near the "No module named uts46data"08:37
litao  File "/usr/lib/python2.7/site-packages/keystoneauth1/identity/generic/base.py", line 135, in _do_create_plugin08:40
litao    authenticated=False)08:40
litao  File "/usr/lib/python2.7/site-packages/positional/__init__.py", line 101, in inner08:40
litao    return wrapped(*args, **kwargs)08:40
litao  File "/usr/lib/python2.7/site-packages/keystoneauth1/identity/base.py", line 341, in get_discovery08:40
litao    authenticated=authenticated)08:40
litao  File "/usr/lib/python2.7/site-packages/positional/__init__.py", line 101, in inner08:40
*** litao has quit IRC08:40
*** litao has joined #openstack-mogan08:41
litaoliusheng:08:41
litaodo you receive my log message?08:41
liushenglitao: just some unrelated log :), may you can just paste it in paste.openstack.org08:42
litaoliusheng: just wait08:46
*** openstackgerrit has joined #openstack-mogan08:51
openstackgerritZhenguo Niu proposed openstack/mogan-ui master: Add servers list API  https://review.openstack.org/45752608:51
litaoliusheng:  pls see the link http://paste.openstack.org/show/606981/08:53
liushenglitao: you can try to update requests lib and idna lib08:54
liushengzhenguo: upgrade08:55
liushengliusheng: upgrade08:55
litaoliusheng: The requests and idna is the lastest version08:57
liushenglitao: clean the idna package and re-install it08:57
zhenguoliusheng: hi, this passed all tests https://review.openstack.org/#/c/457227/ , maybe we can land the client patch first09:12
liushengzhenguo: awesome!09:12
zhenguoliusheng: but the object and table are still named instance_type now :(09:13
zhenguoliusheng: maybe just leave that?09:13
liushengzhenguo: all others have been renamed to 'flavor' ?09:13
zhenguoliusheng: yes, the API related things have been changed09:14
liushengzhenguo: ok, we can just leave that now09:14
zhenguoliusheng: if someone want to do that, it's easy for new comers :D09:15
liushengzhenguo: hah, sure09:15
openstackgerritMerged openstack/python-moganclient master: Replace /types with /flavors in URI  https://review.openstack.org/45743709:18
zhenguoliusheng: client code is in, if there's no objections, I will land this https://review.openstack.org/#/c/457227/ , otherwise our gate will be blocked09:22
liushengzhenguo: ok09:23
zhenguoliusheng: thanks09:23
liushengzhenguo: I have to handle some conflicts. lol09:23
zhenguoliusheng: will that affect keypairs?09:23
liushengzhenguo: oh, maybe not conflict, only conflict with your PoC patch09:24
zhenguoliusheng: but there's no other choice, as if the patch not landed, your patch will not landed forever :D09:24
liushengzhenguo: it is ok09:24
liushengzhenguo: hah09:24
openstackgerritliusheng proposed openstack/mogan master: [PoC] keypair support  https://review.openstack.org/45720509:41
openstackgerritMerged openstack/mogan master: Change to use /flavors in URI  https://review.openstack.org/45722709:52
*** kevinz has quit IRC10:25
openstackgerritTao Li proposed openstack/mogan master: [WIP]Improve the multi-instance creation  https://review.openstack.org/45367510:43
zhenguoliusheng: hah, the flavor patch doesn't introduce any conflicts :D10:58
liushengzhenguo: yes, good to know that :P10:58
litaozhenguo:  I can't get the flavor after pulling the latest code10:59
zhenguolitao: really? how do you get the flavor11:00
zhenguolitao: moganclient or rest API?11:00
litaozhenguo: moganclient11:00
zhenguolitao: you need to upgrade the moganclient as well11:00
litaozhenguo:ok11:00
openstackgerritZhenguo Niu proposed openstack/mogan-ui master: Add servers list API  https://review.openstack.org/45752611:17
*** Xinran has quit IRC11:25
openstackgerritMerged openstack/mogan-ui master: Add servers list API  https://review.openstack.org/45752611:30
openstackgerritliusheng proposed openstack/mogan master: Add keypair support  https://review.openstack.org/45720511:47
litaoliusheng:  The issue remains after re-installing idna11:59
liushenglitao: it is strange, do you have confirmed the old files of idna under /usr/lib/python2.7/site-packages have been cleant12:02
openstackgerritZhenguo Niu proposed openstack/mogan-ui master: Add bare metal servers content  https://review.openstack.org/45759512:05
litaoliusheng: I downgraded the requests version, then it worked12:16
liushenglitao: ...12:16
openstackgerritliusheng proposed openstack/mogan master: Add keypair support  https://review.openstack.org/45720513:41
*** kevinz has joined #openstack-mogan14:01
*** kevinz has quit IRC14:13
*** kevinz has joined #openstack-mogan14:14
openstackgerritTao Li proposed openstack/mogan master: [WIP]Improve the multi-instance creation  https://review.openstack.org/45367514:23
*** kevinz has quit IRC15:00
*** kevinz has joined #openstack-mogan15:16
*** kevinz has quit IRC15:21
*** Kevin_Zheng has quit IRC15:57
*** litao has quit IRC16:49
*** harlowja_ has joined #openstack-mogan16:50
*** harlowja has quit IRC16:52
*** wanghao has joined #openstack-mogan17:09
*** wanghao has quit IRC17:41
*** wanghao has joined #openstack-mogan17:42
*** wanghao has quit IRC17:46
*** openstackgerrit has quit IRC20:33

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!