Wednesday, 2017-08-30

*** wanghao has joined #openstack-mogan00:39
wanghaomorning!00:53
zhenguomorning mogan and wanghao01:15
*** wanghao_ has joined #openstack-mogan01:17
*** wanghao has quit IRC01:21
liushengmorning01:21
openstackgerritliusheng proposed openstack/python-moganclient master: Format the output image uuid to name  https://review.openstack.org/49899801:21
zhenguoliusheng: maybe better to format the server/networks01:41
zhenguoliusheng: seems currently we it's ports instead of networks01:41
liushengzhenguo: you mean the network commands ?01:41
zhenguoliusheng: the Api01:42
zhenguoliusheng: like you GET servers/networks but actually get a list of ports01:42
zhenguoliusheng: currently it's like GET servers/networks/ports01:43
liushengzhenguo: maybe the server's networks is its netowrk info which is its ports info01:43
zhenguoliusheng: seems a bit weird01:44
liushengzhenguo: but servers/networks/ports seems a bit redundant01:45
zhenguoliusheng: yes01:46
liushengzhenguo: maybe servers/interfaces ?01:46
liushengzhenguo: in Nova's server body, it also call the interfaces "network"01:47
zhenguoliusheng: we need to make the notion clear about ports, interfaces, networks, and addresses in mogan01:48
liushengzhenguo: maybe networks of a server won't make users confused, hah, not sure01:48
zhenguoliusheng: maybe yes01:49
liushengzhenguo: it is easy to make users realize  that is network configurations of a server, including ip, mac, net, floatingip, blabla01:50
zhenguoliusheng: ok , let me check the API again01:50
openstackgerritZhenguo Niu proposed openstack/mogan master: Address bad indentation in docs  https://review.openstack.org/49900301:54
openstackgerritXinran WANG proposed openstack/mogan master: support specifying port_id when attaching interface  https://review.openstack.org/49412102:01
zhenguoliusheng: please have a look at this https://review.openstack.org/#/c/499003/02:10
liushengzhenguo: ok02:11
zhenguoliusheng: the 'aggregate list node' still conflict with 'aggregate list' right?02:16
liushengzhenguo: yes.. :(02:17
zhenguoliusheng: but seems it works02:18
liushengzhenguo: really ?02:19
liushengzhenguo: let me try02:19
liushengzhenguo: yes, don't know how it don't conflict any more, hah02:20
zhenguoliusheng: so we can keep it?02:20
liushengzhenguo: yes, sure02:22
zhenguoliusheng: ok02:22
zhenguoliusheng: maybe we need to add the CLI docs02:22
liushengzhenguo: yes, we need to improve the client docs02:23
*** litao__ has joined #openstack-mogan02:25
Xinranzhenguo,  do you know why I cant find created key in DB. It's always empty set02:25
zhenguoXinran:  new created?02:26
Xinranzhenguo, yes02:27
zhenguoXinran: I just tried, it presents in the key_pairs table02:27
Xinranzhenguo,  in my env, it's empty..02:28
Xinranmaybe some problem in my env02:28
zhenguoXinran: seems yes,02:28
Xinranzhenguo,  ok, let me check02:29
zhenguoXinran: for the key_name task, you can save the key with server when we create the server db record,02:29
zhenguo /me brb02:37
openstackgerritliusheng proposed openstack/python-moganclient master: Format the output image uuid to name  https://review.openstack.org/49899802:38
*** Xinran has quit IRC02:40
*** Xinran has joined #openstack-mogan02:41
litao__zhenguo: which is better? cast or call for managing server , what do you think?03:00
zhenguolitao__: I would like call03:00
zhenguolitao__: and I don't like the way to create db record first03:00
zhenguolitao__: we should prepare/check all things before that happen instead of leaving a dead server when error03:01
litao__zhenguo: Maybe call ironic API and neutron api wastes time03:01
zhenguolitao__:  a little time03:01
zhenguolitao__: which is acceptable03:01
litao__zhenguo:  maybe OK03:04
zhenguolitao__: which make things simple03:04
zhenguolitao__: you can do some basic check on api side, then to engine which constructs network info and set ironic node instance info, then create the db record03:05
zhenguolitao__: seems another API call can also manage the node before you set instance_uuid there, right?03:06
zhenguolitao__; seems collisions may happen03:06
litao__zhenguo: If the db record failed, should we  rollback the ironic node instance info?03:07
zhenguolitao__: yes03:07
zhenguolitao__: you just override the instance_uuid, right?03:09
litao__zhenguo: so i should lock the node when managing it03:09
zhenguolitao__: do you think collisions may happen03:09
litao__zhenguo: maybe03:09
zhenguolitao__: yes, seems you should lock it, and instead of just overridding, maybe better to check the instance_uuid existence, as admins may already remember the node uuid and manage it again and again03:10
litao__zhenguo: The get_manageable_servers method ensure to get the node without instance_uuid03:13
zhenguolitao__: seems yes :D03:13
zhenguolitao__: so you will still list all nodes and find the one you want?03:14
zhenguolitao__: can we provide a get single manageable node interface?03:14
litao__zhenguo: so I should also clear the instance _uuid if manage failed?03:15
zhenguolitao__: also ironic node list API but specify the node_uuid, not sure if we can03:15
zhenguolitao__: yes, seems only when creating db failed?03:15
litao__zhenguo: yes, get method is ok03:16
zhenguolitao__: ok03:16
litao__zhenguo: Implemented in API?03:16
litao__zhenguo: or just internal method03:16
zhenguolitao__: just driver interface03:17
litao__zhenguo: OK03:19
*** wanghao_ has quit IRC03:32
*** wanghao has joined #openstack-mogan03:33
* zhenguo brb03:40
zhenguoliusheng: hi, currently active node without instance_uuid will not got to placement, right?03:50
liushengzhenguo: yes03:52
zhenguoliusheng: so it's ok for manageable servers03:52
liushengzhenguo: may need to imporove as we disscussed03:52
zhenguoliusheng: yes03:53
zhenguoliusheng: I tested the server group patch, it works well03:54
zhenguoliusheng: just left some nits inline03:54
liushengzhenguo: ok, will update that according to your comments03:54
zhenguoliusheng: thanks03:54
* zhenguo brb03:54
openstackgerritliusheng proposed openstack/mogan master: Use server group in scheduler  https://review.openstack.org/49615104:07
XinranXinran,  ok, let me try thanks04:10
*** wanghao_ has joined #openstack-mogan04:57
*** wanghao has quit IRC04:58
*** shaohe_feng has quit IRC05:46
*** XueFengLiu has quit IRC05:47
*** XueFengLiu has joined #openstack-mogan05:47
*** shaohe_feng has joined #openstack-mogan05:51
*** wanghao has joined #openstack-mogan05:59
*** XueFengLiu has quit IRC06:02
*** XueFengLiu has joined #openstack-mogan06:02
*** wanghao_ has quit IRC06:02
zhenguoliusheng: hi06:19
zhenguoliusheng: about the log messages, why print the filtered nodes?06:20
liushengzhenguo: hi06:20
zhenguoliusheng: it's just because you filter other aggregates first06:20
liushengzhenguo: I think it is useful we printing more info if a InvalidNode error return06:20
zhenguoliusheng: it's not useful at all, and I got confuse with that info06:21
liushengzhenguo: why ?06:21
zhenguoliusheng: It's just because we filter other aggregates first in our internal process06:21
liushengzhenguo: no, it is not related with the order06:22
zhenguoliusheng: ?06:22
zhenguoliusheng: you print the filtered nodes before filter afffinity zone06:22
liushengzhenguo: ... it is  because you know the order of process06:23
zhenguoliusheng: why users need that info06:23
zhenguoliusheng: I know you have how many nodes left in the cloud?06:23
liushengzhenguo: it is help to find the reason about why InvalidNode06:23
liushengzhenguo: ok, will remove it06:24
zhenguoliusheng: we should not treat affinity zone  special as during filtering it's not different with other aggregates06:24
openstackgerritliusheng proposed openstack/mogan master: Use server group in scheduler  https://review.openstack.org/49615106:25
openstackgerritZhenguo Niu proposed openstack/mogan master: [WIP]Add root disk partitions support  https://review.openstack.org/49903906:27
*** wanghao has quit IRC06:35
*** wanghao has joined #openstack-mogan06:36
litao__zhenguo: There is no state transition in managing  node?06:45
litao__zhenguo: right?06:46
zhenguolitao__: yes06:46
litao__zhenguo: But If add some other process such as vif plug later,  is it ok?06:46
zhenguolitao__: seems ok06:48
litao__zhenguo: ok06:48
zhenguolitao__: but you should take care about the  server status, it can be active or stopped according to the power state06:48
litao__zhenguo: of course06:49
zhenguolitao__: ok06:49
openstackgerritXinran WANG proposed openstack/mogan master: support specifying port_id when attaching interface  https://review.openstack.org/49412107:05
*** wanghao_ has joined #openstack-mogan08:14
*** wanghao_ has quit IRC08:15
*** wanghao has quit IRC08:15
*** wanghao has joined #openstack-mogan08:16
openstackgerritMerged openstack/mogan master: Address bad indentation in docs  https://review.openstack.org/49900309:21
*** dims has quit IRC09:28
*** wanghao has quit IRC09:35
*** wanghao has joined #openstack-mogan09:35
*** wanghao has quit IRC09:36
*** wanghao has joined #openstack-mogan09:36
*** wanghao has quit IRC09:36
*** wanghao has joined #openstack-mogan09:37
*** wanghao has quit IRC09:37
*** wanghao has joined #openstack-mogan09:38
*** wanghao has quit IRC09:38
*** wanghao has joined #openstack-mogan09:38
*** wanghao has quit IRC09:39
*** wanghao has joined #openstack-mogan09:39
*** wanghao has quit IRC09:40
*** wanghao has joined #openstack-mogan09:40
*** wanghao has quit IRC09:40
*** wanghao has joined #openstack-mogan09:41
XinranHi, all . Does anyone know the command line to sync db ?09:57
openstackgerritMerged openstack/mogan master: Use server group in scheduler  https://review.openstack.org/49615109:57
openstackgerritMerged openstack/mogan master: Add socat console support  https://review.openstack.org/49383609:57
openstackgerritMerged openstack/mogan master: support specifying port_id when attaching interface  https://review.openstack.org/49412109:58
*** dims has joined #openstack-mogan09:58
openstackgerritMerged openstack/python-moganclient master: Format the output image uuid to name  https://review.openstack.org/49899810:01
zhenguoXinran: as we don't add db migration scripts, dbsync doens't work for us now :(10:17
openstackgerritZhenguo Niu proposed openstack/mogan master: [WIP]Add root disk partitions support  https://review.openstack.org/49903910:44
*** Jeffrey4l has quit IRC12:07
*** liusheng has quit IRC12:07
*** liusheng has joined #openstack-mogan12:07
*** Jeffrey4l has joined #openstack-mogan12:07
*** dims has quit IRC17:14
*** litao__ has quit IRC17:17
*** dims has joined #openstack-mogan17:18
*** dims has quit IRC19:28
*** dims has joined #openstack-mogan19:31

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