Tuesday, 2017-01-10

*** wanghao has joined #openstack-mogan00:41
*** little has joined #openstack-mogan00:55
*** kevinz has joined #openstack-mogan01:25
*** yuntongjin has joined #openstack-mogan01:52
openstackgerritMerged openstack/nimble: Remove target_power_state  https://review.openstack.org/41776901:57
zhenguomorning mogan!02:19
*** zhangjl has joined #openstack-mogan02:20
yuntongjinmorning !02:28
*** yuntongjin has quit IRC02:29
*** yuntongjin has joined #openstack-mogan02:33
liushengzhenguo: some problems occured with tempest gate ?02:40
zhenguoliusheng: not sure if that related to us02:40
zhenguoliusheng: but seems not that stable02:40
liushengzhenguo: let me recheck https://review.openstack.org/#/c/417724/202:41
zhenguoliusheng: thanks02:41
liushengzhenguo: not sure it is because the tempest wait 3 mins :(02:42
zhenguoliusheng: a fixed time?02:42
liushengzhenguo: yes :(, it is because previously the tempest will fail in cleanup when tempest clean the securitygroup, because the sucuritygroup still in use after instance deleted02:43
liushengzhenguo: so I set an ugly 3min :(02:43
zhenguoliusheng: hah, so for now, how to handle that?02:44
liushengzhenguo: I have upload this02:44
zhenguoliusheng: let me take a look02:44
liushengzhenguo: because you told me, that the instance will be disassociated once instance deleted completelt02:44
zhenguoliusheng: hah02:45
zhenguoliusheng: seems that would have a race, as we use rpc cast for instance deleting, so in the wait_for method it will break out soon02:50
liushengzhenguo: you mean this patch ?02:51
zhenguoliusheng: yes02:51
zhenguoliusheng: the wait for method will only run once then just break out02:51
zhenguoliusheng: as deleting method will return immediately02:52
liushengzhenguo: yes, but it will wait the instance disappeared from mogan02:53
zhenguoliusheng: oh yes, sorry02:53
zhenguoliusheng: my mistake, it's in deleting status after cast the request, hah02:53
liushengzhenguo: as you told me, after the "deleting" status end, the instance will be dissasociated with node02:53
liushengzhenguo: yes02:54
zhenguoliusheng: yes, when it is set to deleted, the network should already been cleaned up02:54
liushengzhenguo: btw, I am not sure we need to support show deleted instance with admin users, currently it don't support, so I cache the NotFound02:54
zhenguoliusheng: maybe in the future we need to support a filter to list deleted instances, not sure02:55
zhenguoliusheng: I remember Kevin_zheng said nova will remove all 'deleted' filter02:56
liushengzhenguo: yeah, Nova support it, lol02:56
liushengzhenguo: really ?02:56
zhenguoliusheng: yes, in the last bug smash02:56
Kevin_Zhengguess what02:57
Kevin_Zhengnot removed02:57
Kevin_Zhengthey changed plans02:57
zhenguoKevin_Zheng: lol02:57
Kevin_Zhengthe fiedls I added in this schema will be allowed to use02:58
zhenguohah, deleted is still there02:58
Kevin_Zhengspent the whole cycle decided on the fields02:59
liushengKevin_Zheng: may searchlight will take over all these filters ?03:01
zhenguohah, thanks Kevin_Zheng for the information, so we can prepare to add deleted filter03:01
Kevin_Zhengyeah after support versioned notification in SL and nova sending more versioned notification03:02
liushengzhenguo: seems, this pass the gate, lol   https://review.openstack.org/#/c/417724/03:07
zhenguoliusheng: hah, will +A it to improve our gate, thanks03:08
liushengzhenguo: thanks:)03:08
* zhenguo brb03:15
openstackgerritMerged openstack/nimble: Move power target check before fetching instance from db  https://review.openstack.org/41774003:28
*** wanghao has quit IRC03:33
zhenguoliusheng, shaohe_feng: seems sync power states will be more complex than I expected, many race conditions03:33
openstackgerritMerged openstack/nimble: Don't need to wait fixed time after deletion in tempest  https://review.openstack.org/41772403:34
*** yuntongjin has quit IRC03:50
openstackgerritMerged openstack/nimble: Run sync_node_resources periodic task immediately  https://review.openstack.org/41780303:50
openstackgerritzhangjl proposed openstack/nimble: API-REF: Fix the typo in docs and ensure the consistency of instance_uuid  https://review.openstack.org/41819503:53
*** yuntongjin has joined #openstack-mogan03:53
*** yuntongjin has quit IRC04:06
*** zhangjl has quit IRC04:38
*** zhangjl has joined #openstack-mogan04:39
*** zhangjl has quit IRC04:42
*** zhangjl has joined #openstack-mogan04:44
*** wanghao has joined #openstack-mogan04:57
*** yuntongjin has joined #openstack-mogan05:28
*** zhangjl1 has joined #openstack-mogan05:37
*** zhangjl has quit IRC05:39
*** zhangjl1 has quit IRC05:42
*** zhangjl has joined #openstack-mogan06:10
*** yuntongjin has quit IRC06:20
openstackgerritliusheng proposed openstack/nimble: Add version tests for versioned notification class  https://review.openstack.org/41821206:34
*** yuntongjin has joined #openstack-mogan06:58
liushengzhenguo: should we still use set_power_state in API side to perform start/stop/reboot actions ?07:00
zhenguoliusheng: yes07:00
liushengzhenguo: ok, start/stop/reboot only in client side07:02
zhenguoliusheng: for api, PUT /instances/instance_uuid/states/power seems OK, but CLI should be more convinient for end users.07:02
liushengzhenguo: agree07:02
liushengzhenguo: currently, we don't have a cli command for get-state API, do you think we should add one ?07:09
zhenguoliusheng: yes, but not high priority things07:10
liushengzhenguo: hah07:10
liushengzhenguo: not sure what's the command name should be07:10
zhenguoliusheng: a couple things should be done in CLI : 1. replace --detailed with --long in listing instances 2. change to use openstack baremetal server boot instead of create, wdyt?07:10
liushengzhenguo: ok, the 1# has been done07:12
zhenguoliusheng: hah, sorry seems I missed that :P07:12
liushengzhenguo: the 2# is simple, so what's the most hight priority things for us now, lol07:13
zhenguoliusheng: seems CLI is ok with that, all high priority things are in mogan server. lol07:14
liushengzhenguo: yeah, if there is most hight priority thing and urgent one need me, please feel free to tell me, lol07:15
zhenguoliusheng: hah07:15
zhenguoliusheng: seems many tasks assigned to me https://etherpad.openstack.org/p/nimble-task , you can take over anyone you interested07:17
liushengzhenguo: ok07:17
zhenguoliusheng: I remember you are the owner of multi instances create?07:17
zhenguooh, seems little taken over it in last meeting07:17
liushengzhenguo: oh, yes, is it hight prior ?07:18
liushengzhenguo: ok07:18
liushengzhenguo: he can take it if he want07:18
zhenguoliusheng: hah, not sure little has been working on it07:18
zhenguoliusheng: I would like to align with nova in O release, so anything that we don't have is high priority thing, hah07:19
liushengzhenguo: yes, I think thare still are meny basic functions need to align07:20
zhenguoliusheng: yes, and I believe not all listed in the etherpad07:21
liushengzhenguo: just found a problem, when stop a server, the status is still active, is it reasonable ? and, the power_status don't show in 'server list'07:22
zhenguoliusheng: active is correct, I'm now refactor that part, will always return power state with instance07:23
liushengzhenguo: ok, so wait for your refactor job07:23
zhenguoliusheng: ok, will sent out a draft patch soon, need you all to revalue it as sync_power_state will introduce many race conditions :(07:24
liushengzhenguo: hah07:24
zhenguoliusheng: hah07:25
zhenguoliusheng: so the basic versioned notification has been done, right? I saw you show it in the rabbitmq, seems awesome07:27
liushengzhenguo: just basic frame and a instance.create.start notification, need to add many spcific notifications definitions, I just want to switch a job sometimes, lol07:28
zhenguoliusheng: hah, that's ok, a example notification is enough, other's can follow you, maybe for newcomers07:29
liushengzhenguo: yes07:29
zhenguoliusheng: just find you have a console task, are you still interesed in it?07:30
liushengzhenguo: really, I have forgot that!07:30
liushengzhenguo: may I can try07:31
zhenguoliusheng: hah, just find it in the etherpad07:31
liushengzhenguo: I have remembered you have done some attempts ?07:31
zhenguoliusheng: a console proxy?07:31
liushengzhenguo: yes07:31
zhenguoliusheng: I'm not sure if we can leverage nova's proxy07:31
zhenguoliusheng: but for a hybrid env with both vm and bm, not sure if there will be confilicts07:32
liushengzhenguo: it is better to get rid of Nova07:32
liushengzhenguo: not sure07:33
zhenguoliusheng: yes, now we can just copy serialconsole proxy from nova07:33
liushengzhenguo: ok07:33
liushengzhenguo: will try07:33
zhenguoliusheng: and ironic driver has already provide console related methods07:33
zhenguoliusheng: and for API, not sure how to construct07:34
liushengzhenguo: may need try to read the code of ironic driver of Nova07:34
zhenguoliusheng: yes, for our REST API, maybe just use GET /instances/instance_uuid/console ?07:35
liushengzhenguo: make sense to me07:36
zhenguoliusheng: ok, good luck!07:37
liushengzhenguo: lol07:37
liushengzhenguo: ... for nova boot command, it is 'openstack server create' in OSC07:42
liushengzhenguo: seems nova people don't like server boot07:42
zhenguoliusheng: really? oh if so, we should follow that07:42
liushengzhenguo: hah07:42
zhenguoliusheng: hah07:42
liushengzhenguo: yes07:42
zhenguoliusheng: oh yes, sorry for the misleading07:44
liushengzhenguo: np, I also thought that, lol07:44
zhenguoliusheng: hah07:45
Kevin_Zhengonly you two are talking07:49
zhenguoKevin_Zheng: hah, so I try to invite more people here07:50
*** yuntongjin has quit IRC07:51
*** wxy has joined #openstack-mogan07:55
*** yuntongjin has joined #openstack-mogan08:06
*** kevinz has quit IRC08:09
openstackgerritZhenguo Niu proposed openstack/nimble: [WIP] Save power_state to instance and sync with ironic  https://review.openstack.org/41824908:17
zhenguoliusheng: seems it's still --detailed https://github.com/openstack/python-nimbleclient/blob/master/moganclient/osc/v1/server.py#L18708:20
liushengzhenguo: you mean we need to use --long ?08:22
zhenguoliusheng: yes, seems all other projects suse --long08:22
liushengzhenguo: hmm, long seems not a specific word...08:24
zhenguoliusheng: seems yes, but in openstack seems long is a specific word, hah08:25
zhenguoliusheng: I mean in openstack CLI08:25
liushengzhenguo: yes, I confirmed it08:25
zhenguoliusheng: so we'd better to follow it08:25
liushengzhenguo: ok, so server list is GET instance/, with --long is GET/instance/detail, right ?08:26
zhenguoliusheng: seems like it is08:27
liushengzhenguo: any example except Nova is that usage ?08:27
liushengzhenguo: oh, saw it08:28
zhenguoliusheng: openstack baremetal node list08:28
liushengzhenguo: ok, will change it08:28
zhenguoliusheng: thanks08:29
liushengzhenguo: np, thanks for confirm that :)08:29
zhenguoliusheng: hah08:29
zhenguoliusheng: do we have a public ip in xi'an lib?08:35
liushengzhenguo: yes08:36
openstackgerritliusheng proposed openstack/python-nimbleclient: Replace --detailed with --long parameter in "server list" command  https://review.openstack.org/41825508:36
zhenguoliusheng: can we setup a http server there for our API docs?08:36
liushengzhenguo: you can call zhaobo, if you have needs08:37
liushengzhenguo: you mean mogan's api docs ?08:37
zhenguoliusheng: yes, ok will ping him later08:37
Kevin_ZhengI think we have problem08:38
liushengzhenguo: want to show to outside guys ? lol08:38
Kevin_Zhengour server cannot serve 80 port08:38
zhenguoany port is ok08:38
Kevin_Zhengthen it is ok08:38
zhenguohah, will talk with zhaobo later,08:39
Kevin_Zhengyou can just ask him to give you a mirror port08:39
Kevin_Zhengof your dev env08:39
liushengzhenguo: yes08:39
zhenguoseems OK08:39
liushengwe cannot publish our api docs in openstack.org now ?08:39
zhenguoliusheng: yes08:40
zhenguoonly official project can do that08:40
liushengzhenguo: just because we are not big-tent08:40
zhenguoliusheng: yes :(08:40
liushengzhenguo: any people want to read our docs now ? hah08:42
zhenguoliusheng: not sure, but we should prepare it first before  customers coming08:43
liushengzhenguo: hah08:46
zhenguoliusheng: hah, we shoud try to serve them well if they are interested08:46
liushengzhenguo: yes, indeed08:47
* zhenguo brb08:50
shaohe_fengzhenguo: I'm here.08:51
shaohe_fengzhenguo: have seen your message.  you should be careful for the race conditions08:51
zhenguoshaohe_feng: yes, will try to be careful09:01
littleliusheng: please assgin support-multiple-creation to me09:05
liushenglittle: ok09:06
littleThese two days is busy for VT09:06
liushenglittle: what's your lp id ?09:07
littleand solving question , so delayed09:07
liushenglittle: done09:07
liushenglittle: np :)09:08
littleliusheng: BTW, I create a bp, do you know how to delete it.09:09
liushenglittle: just change the Definition tag to "Obsolete"09:10
zhenguoshaohe_feng, yuntongjin: do you know why china mobile guys disappeared, not sure if they still want to take some tasks.09:11
openstackgerritMerged openstack/nimble: API-REF: Fix the typo in docs and ensure the consistency of instance_uuid  https://review.openstack.org/41819509:13
shaohe_fengyuntongjin: is our interface with china mobile09:15
shaohe_fengzhenguo: I find a problem09:15
zhenguoshaohe_feng: what problem09:15
shaohe_fengzhenguo: I change generate the configdrive before task flow09:15
shaohe_fengzhenguo: but, we need to get the node info to generate the configdrive09:16
shaohe_fengzhenguo: we did not attach a node to a instance, before task flow.09:16
zhenguoshaohe_feng: why need that? let me check09:17
shaohe_fengzhenguo: I also need to check it, why matafile need it.09:17
zhenguoshaohe_feng: seems no compelling reason for that, we can change a bit if no impact.09:19
shaohe_fengzhenguo: oh, check the code. seem only for the log. ")09:20
shaohe_fengzhenguo: then we do not need it.09:20
zhenguoshaohe_feng: hah, yes09:20
yuntongjinzhenguo: i'm not sure why09:23
zhenguoyuntongjin: hah09:23
yuntongjinchina mobile does have many baremetal requirement/issue that they hope would get help09:24
zhenguoyuntongjin: yes, we would like to help09:24
zhenguowe will have dedicated attention for their requirements09:25
yuntongjinwhich we could  discuss in our meeting09:25
zhenguoyuntongjin: yes but silenceli disappeared from irc09:25
*** wanghao has quit IRC09:26
yuntongjini can help  bring all the issue/requirement into our meeting09:27
zhenguoyuntongjin: ok, thanks, you can add a agenda to discuss the requirements09:27
yuntongjinonce china mobile folks realize that we are helping to address their requirement, i think they will get involved more09:28
zhenguoyuntongjin: hah, sure09:29
yuntongjinzhenguo: sure, will do09:30
zhenguoyuntongjin: thanks09:30
*** yuntongjin has quit IRC09:30
zhenguoshaohe_feng, liusheng: regarding https://review.openstack.org/#/c/417308/ , do you think the looping call is needed?09:33
*** openstack has joined #openstack-mogan14:30
openstackgerritZhenguo Niu proposed openstack/nimble: [WIP] Introduce instance task state  https://review.openstack.org/41846116:14
openstackgerritZhenguo Niu proposed openstack/nimble: [WIP] Introduce instance task state  https://review.openstack.org/41846116:37
openstackgerritMerged openstack/nimble: Add a looping call to wait for power action completed  https://review.openstack.org/41730817:10
*** Jeffrey4l_ has joined #openstack-mogan21:35
*** Jeffrey4l has quit IRC21:35
*** wxy has quit IRC23:21
*** wxy has joined #openstack-mogan23:38

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