| *** wanghao has joined #openstack-mogan | 00:41 | |
| *** little has joined #openstack-mogan | 00:55 | |
| *** kevinz has joined #openstack-mogan | 01:25 | |
| *** yuntongjin has joined #openstack-mogan | 01:52 | |
| openstackgerrit | Merged openstack/nimble: Remove target_power_state https://review.openstack.org/417769 | 01:57 |
|---|---|---|
| zhenguo | morning mogan! | 02:19 |
| *** zhangjl has joined #openstack-mogan | 02:20 | |
| yuntongjin | morning ! | 02:28 |
| *** yuntongjin has quit IRC | 02:29 | |
| liusheng | morning | 02:31 |
| *** yuntongjin has joined #openstack-mogan | 02:33 | |
| liusheng | zhenguo: some problems occured with tempest gate ? | 02:40 |
| zhenguo | liusheng: not sure if that related to us | 02:40 |
| zhenguo | liusheng: but seems not that stable | 02:40 |
| liusheng | zhenguo: let me recheck https://review.openstack.org/#/c/417724/2 | 02:41 |
| zhenguo | liusheng: thanks | 02:41 |
| liusheng | zhenguo: not sure it is because the tempest wait 3 mins :( | 02:42 |
| zhenguo | liusheng: a fixed time? | 02:42 |
| liusheng | zhenguo: yes :(, it is because previously the tempest will fail in cleanup when tempest clean the securitygroup, because the sucuritygroup still in use after instance deleted | 02:43 |
| liusheng | zhenguo: so I set an ugly 3min :( | 02:43 |
| zhenguo | liusheng: hah, so for now, how to handle that? | 02:44 |
| liusheng | zhenguo: I have upload this | 02:44 |
| zhenguo | liusheng: let me take a look | 02:44 |
| liusheng | zhenguo: because you told me, that the instance will be disassociated once instance deleted completelt | 02:44 |
| zhenguo | liusheng: hah | 02:45 |
| zhenguo | liusheng: seems that would have a race, as we use rpc cast for instance deleting, so in the wait_for method it will break out soon | 02:50 |
| liusheng | zhenguo: you mean this patch ? | 02:51 |
| zhenguo | liusheng: yes | 02:51 |
| zhenguo | liusheng: the wait for method will only run once then just break out | 02:51 |
| zhenguo | liusheng: as deleting method will return immediately | 02:52 |
| liusheng | zhenguo: yes, but it will wait the instance disappeared from mogan | 02:53 |
| zhenguo | liusheng: oh yes, sorry | 02:53 |
| zhenguo | liusheng: my mistake, it's in deleting status after cast the request, hah | 02:53 |
| liusheng | zhenguo: as you told me, after the "deleting" status end, the instance will be dissasociated with node | 02:53 |
| liusheng | zhenguo: yes | 02:54 |
| zhenguo | liusheng: yes, when it is set to deleted, the network should already been cleaned up | 02:54 |
| liusheng | zhenguo: btw, I am not sure we need to support show deleted instance with admin users, currently it don't support, so I cache the NotFound | 02:54 |
| liusheng | s/cache/catch | 02:55 |
| zhenguo | liusheng: maybe in the future we need to support a filter to list deleted instances, not sure | 02:55 |
| zhenguo | liusheng: I remember Kevin_zheng said nova will remove all 'deleted' filter | 02:56 |
| liusheng | zhenguo: yeah, Nova support it, lol | 02:56 |
| liusheng | zhenguo: really ? | 02:56 |
| zhenguo | liusheng: yes, in the last bug smash | 02:56 |
| Kevin_Zheng | guess what | 02:57 |
| Kevin_Zheng | not removed | 02:57 |
| Kevin_Zheng | they changed plans | 02:57 |
| zhenguo | Kevin_Zheng: lol | 02:57 |
| Kevin_Zheng | https://review.openstack.org/#/c/408571/34/nova/api/openstack/compute/schemas/servers.py | 02:58 |
| Kevin_Zheng | the fiedls I added in this schema will be allowed to use | 02:58 |
| zhenguo | hah, deleted is still there | 02:58 |
| Kevin_Zheng | spent the whole cycle decided on the fields | 02:59 |
| Kevin_Zheng | LOL | 02:59 |
| liusheng | Kevin_Zheng: may searchlight will take over all these filters ? | 03:01 |
| zhenguo | hah, thanks Kevin_Zheng for the information, so we can prepare to add deleted filter | 03:01 |
| Kevin_Zheng | yeah after support versioned notification in SL and nova sending more versioned notification | 03:02 |
| liusheng | zhenguo: seems, this pass the gate, lol https://review.openstack.org/#/c/417724/ | 03:07 |
| zhenguo | liusheng: hah, will +A it to improve our gate, thanks | 03:08 |
| liusheng | zhenguo: thanks:) | 03:08 |
| * zhenguo brb | 03:15 | |
| openstackgerrit | Merged openstack/nimble: Move power target check before fetching instance from db https://review.openstack.org/417740 | 03:28 |
| *** wanghao has quit IRC | 03:33 | |
| zhenguo | liusheng, shaohe_feng: seems sync power states will be more complex than I expected, many race conditions | 03:33 |
| openstackgerrit | Merged openstack/nimble: Don't need to wait fixed time after deletion in tempest https://review.openstack.org/417724 | 03:34 |
| *** yuntongjin has quit IRC | 03:50 | |
| openstackgerrit | Merged openstack/nimble: Run sync_node_resources periodic task immediately https://review.openstack.org/417803 | 03:50 |
| openstackgerrit | zhangjl proposed openstack/nimble: API-REF: Fix the typo in docs and ensure the consistency of instance_uuid https://review.openstack.org/418195 | 03:53 |
| *** yuntongjin has joined #openstack-mogan | 03:53 | |
| *** yuntongjin has quit IRC | 04:06 | |
| *** zhangjl has quit IRC | 04:38 | |
| *** zhangjl has joined #openstack-mogan | 04:39 | |
| *** zhangjl has quit IRC | 04:42 | |
| *** zhangjl has joined #openstack-mogan | 04:44 | |
| *** wanghao has joined #openstack-mogan | 04:57 | |
| *** yuntongjin has joined #openstack-mogan | 05:28 | |
| *** zhangjl1 has joined #openstack-mogan | 05:37 | |
| *** zhangjl has quit IRC | 05:39 | |
| *** zhangjl1 has quit IRC | 05:42 | |
| *** zhangjl has joined #openstack-mogan | 06:10 | |
| *** yuntongjin has quit IRC | 06:20 | |
| openstackgerrit | liusheng proposed openstack/nimble: Add version tests for versioned notification class https://review.openstack.org/418212 | 06:34 |
| *** yuntongjin has joined #openstack-mogan | 06:58 | |
| liusheng | zhenguo: should we still use set_power_state in API side to perform start/stop/reboot actions ? | 07:00 |
| zhenguo | liusheng: yes | 07:00 |
| liusheng | zhenguo: ok, start/stop/reboot only in client side | 07:02 |
| zhenguo | liusheng: for api, PUT /instances/instance_uuid/states/power seems OK, but CLI should be more convinient for end users. | 07:02 |
| liusheng | zhenguo: agree | 07:02 |
| liusheng | zhenguo: currently, we don't have a cli command for get-state API, do you think we should add one ? | 07:09 |
| zhenguo | liusheng: yes, but not high priority things | 07:10 |
| liusheng | zhenguo: hah | 07:10 |
| liusheng | zhenguo: not sure what's the command name should be | 07:10 |
| zhenguo | liusheng: 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 |
| liusheng | zhenguo: ok, the 1# has been done | 07:12 |
| zhenguo | liusheng: hah, sorry seems I missed that :P | 07:12 |
| liusheng | zhenguo: the 2# is simple, so what's the most hight priority things for us now, lol | 07:13 |
| zhenguo | liusheng: seems CLI is ok with that, all high priority things are in mogan server. lol | 07:14 |
| liusheng | zhenguo: yeah, if there is most hight priority thing and urgent one need me, please feel free to tell me, lol | 07:15 |
| zhenguo | liusheng: hah | 07:15 |
| zhenguo | liusheng: seems many tasks assigned to me https://etherpad.openstack.org/p/nimble-task , you can take over anyone you interested | 07:17 |
| liusheng | zhenguo: ok | 07:17 |
| zhenguo | liusheng: I remember you are the owner of multi instances create? | 07:17 |
| zhenguo | oh, seems little taken over it in last meeting | 07:17 |
| liusheng | zhenguo: oh, yes, is it hight prior ? | 07:18 |
| liusheng | zhenguo: ok | 07:18 |
| liusheng | zhenguo: he can take it if he want | 07:18 |
| zhenguo | liusheng: hah, not sure little has been working on it | 07:18 |
| zhenguo | liusheng: I would like to align with nova in O release, so anything that we don't have is high priority thing, hah | 07:19 |
| liusheng | zhenguo: yes, I think thare still are meny basic functions need to align | 07:20 |
| zhenguo | liusheng: yes, and I believe not all listed in the etherpad | 07:21 |
| liusheng | zhenguo: 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 |
| zhenguo | liusheng: active is correct, I'm now refactor that part, will always return power state with instance | 07:23 |
| liusheng | zhenguo: ok, so wait for your refactor job | 07:23 |
| zhenguo | liusheng: 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 |
| liusheng | zhenguo: hah | 07:24 |
| zhenguo | liusheng: hah | 07:25 |
| zhenguo | liusheng: so the basic versioned notification has been done, right? I saw you show it in the rabbitmq, seems awesome | 07:27 |
| liusheng | zhenguo: just basic frame and a instance.create.start notification, need to add many spcific notifications definitions, I just want to switch a job sometimes, lol | 07:28 |
| zhenguo | liusheng: hah, that's ok, a example notification is enough, other's can follow you, maybe for newcomers | 07:29 |
| liusheng | zhenguo: yes | 07:29 |
| zhenguo | liusheng: just find you have a console task, are you still interesed in it? | 07:30 |
| liusheng | zhenguo: really, I have forgot that! | 07:30 |
| liusheng | zhenguo: may I can try | 07:31 |
| zhenguo | liusheng: hah, just find it in the etherpad | 07:31 |
| liusheng | zhenguo: I have remembered you have done some attempts ? | 07:31 |
| zhenguo | liusheng: a console proxy? | 07:31 |
| liusheng | zhenguo: yes | 07:31 |
| zhenguo | liusheng: I'm not sure if we can leverage nova's proxy | 07:31 |
| zhenguo | liusheng: but for a hybrid env with both vm and bm, not sure if there will be confilicts | 07:32 |
| liusheng | zhenguo: it is better to get rid of Nova | 07:32 |
| liusheng | zhenguo: not sure | 07:33 |
| zhenguo | liusheng: yes, now we can just copy serialconsole proxy from nova | 07:33 |
| liusheng | zhenguo: ok | 07:33 |
| liusheng | zhenguo: will try | 07:33 |
| zhenguo | liusheng: and ironic driver has already provide console related methods | 07:33 |
| zhenguo | liusheng: and for API, not sure how to construct | 07:34 |
| liusheng | zhenguo: may need try to read the code of ironic driver of Nova | 07:34 |
| zhenguo | liusheng: yes, for our REST API, maybe just use GET /instances/instance_uuid/console ? | 07:35 |
| liusheng | zhenguo: make sense to me | 07:36 |
| zhenguo | liusheng: ok, good luck! | 07:37 |
| liusheng | zhenguo: lol | 07:37 |
| liusheng | zhenguo: ... for nova boot command, it is 'openstack server create' in OSC | 07:42 |
| liusheng | zhenguo: seems nova people don't like server boot | 07:42 |
| zhenguo | liusheng: really? oh if so, we should follow that | 07:42 |
| liusheng | zhenguo: hah | 07:42 |
| zhenguo | liusheng: hah | 07:42 |
| liusheng | zhenguo: yes | 07:42 |
| zhenguo | liusheng: oh yes, sorry for the misleading | 07:44 |
| liusheng | zhenguo: np, I also thought that, lol | 07:44 |
| zhenguo | liusheng: hah | 07:45 |
| Kevin_Zheng | only you two are talking | 07:49 |
| zhenguo | Kevin_Zheng: hah, so I try to invite more people here | 07:50 |
| *** yuntongjin has quit IRC | 07:51 | |
| *** wxy has joined #openstack-mogan | 07:55 | |
| *** yuntongjin has joined #openstack-mogan | 08:06 | |
| *** kevinz has quit IRC | 08:09 | |
| openstackgerrit | Zhenguo Niu proposed openstack/nimble: [WIP] Save power_state to instance and sync with ironic https://review.openstack.org/418249 | 08:17 |
| zhenguo | liusheng: seems it's still --detailed https://github.com/openstack/python-nimbleclient/blob/master/moganclient/osc/v1/server.py#L187 | 08:20 |
| liusheng | zhenguo: you mean we need to use --long ? | 08:22 |
| zhenguo | liusheng: yes, seems all other projects suse --long | 08:22 |
| liusheng | zhenguo: hmm, long seems not a specific word... | 08:24 |
| zhenguo | liusheng: seems yes, but in openstack seems long is a specific word, hah | 08:25 |
| zhenguo | liusheng: I mean in openstack CLI | 08:25 |
| liusheng | zhenguo: yes, I confirmed it | 08:25 |
| zhenguo | liusheng: so we'd better to follow it | 08:25 |
| liusheng | zhenguo: ok, so server list is GET instance/, with --long is GET/instance/detail, right ? | 08:26 |
| zhenguo | liusheng: seems like it is | 08:27 |
| liusheng | zhenguo: any example except Nova is that usage ? | 08:27 |
| liusheng | zhenguo: oh, saw it | 08:28 |
| zhenguo | liusheng: openstack baremetal node list | 08:28 |
| liusheng | zhenguo: ok, will change it | 08:28 |
| zhenguo | liusheng: thanks | 08:29 |
| liusheng | zhenguo: np, thanks for confirm that :) | 08:29 |
| zhenguo | liusheng: hah | 08:29 |
| zhenguo | liusheng: do we have a public ip in xi'an lib? | 08:35 |
| liusheng | zhenguo: yes | 08:36 |
| openstackgerrit | liusheng proposed openstack/python-nimbleclient: Replace --detailed with --long parameter in "server list" command https://review.openstack.org/418255 | 08:36 |
| zhenguo | liusheng: can we setup a http server there for our API docs? | 08:36 |
| liusheng | zhenguo: you can call zhaobo, if you have needs | 08:37 |
| liusheng | zhenguo: you mean mogan's api docs ? | 08:37 |
| zhenguo | liusheng: yes, ok will ping him later | 08:37 |
| Kevin_Zheng | I think we have problem | 08:38 |
| liusheng | zhenguo: want to show to outside guys ? lol | 08:38 |
| Kevin_Zheng | our server cannot serve 80 port | 08:38 |
| zhenguo | any port is ok | 08:38 |
| Kevin_Zheng | then it is ok | 08:38 |
| zhenguo | hah, will talk with zhaobo later, | 08:39 |
| Kevin_Zheng | you can just ask him to give you a mirror port | 08:39 |
| Kevin_Zheng | of your dev env | 08:39 |
| liusheng | zhenguo: yes | 08:39 |
| zhenguo | seems OK | 08:39 |
| liusheng | we cannot publish our api docs in openstack.org now ? | 08:39 |
| zhenguo | liusheng: yes | 08:40 |
| zhenguo | only official project can do that | 08:40 |
| liusheng | zhenguo: just because we are not big-tent | 08:40 |
| zhenguo | liusheng: yes :( | 08:40 |
| little | I | 08:41 |
| liusheng | zhenguo: any people want to read our docs now ? hah | 08:42 |
| zhenguo | liusheng: not sure, but we should prepare it first before customers coming | 08:43 |
| liusheng | zhenguo: hah | 08:46 |
| zhenguo | liusheng: hah, we shoud try to serve them well if they are interested | 08:46 |
| liusheng | zhenguo: yes, indeed | 08:47 |
| * zhenguo brb | 08:50 | |
| shaohe_feng | zhenguo: I'm here. | 08:51 |
| shaohe_feng | zhenguo: have seen your message. you should be careful for the race conditions | 08:51 |
| zhenguo | shaohe_feng: yes, will try to be careful | 09:01 |
| little | liusheng: please assgin support-multiple-creation to me | 09:05 |
| liusheng | little: ok | 09:06 |
| little | These two days is busy for VT | 09:06 |
| liusheng | little: what's your lp id ? | 09:07 |
| little | and solving question , so delayed | 09:07 |
| little | eric-litao | 09:07 |
| liusheng | little: done | 09:07 |
| little | thanks | 09:07 |
| liusheng | little: np :) | 09:08 |
| little | liusheng: BTW, I create a bp, do you know how to delete it. | 09:09 |
| liusheng | little: just change the Definition tag to "Obsolete" | 09:10 |
| little | OK | 09:11 |
| zhenguo | shaohe_feng, yuntongjin: do you know why china mobile guys disappeared, not sure if they still want to take some tasks. | 09:11 |
| openstackgerrit | Merged openstack/nimble: API-REF: Fix the typo in docs and ensure the consistency of instance_uuid https://review.openstack.org/418195 | 09:13 |
| shaohe_feng | yuntongjin: is our interface with china mobile | 09:15 |
| shaohe_feng | zhenguo: I find a problem | 09:15 |
| zhenguo | shaohe_feng: what problem | 09:15 |
| shaohe_feng | zhenguo: I change generate the configdrive before task flow | 09:15 |
| shaohe_feng | zhenguo: but, we need to get the node info to generate the configdrive | 09:16 |
| shaohe_feng | zhenguo: we did not attach a node to a instance, before task flow. | 09:16 |
| zhenguo | shaohe_feng: why need that? let me check | 09:17 |
| shaohe_feng | zhenguo: I also need to check it, why matafile need it. | 09:17 |
| zhenguo | shaohe_feng: seems no compelling reason for that, we can change a bit if no impact. | 09:19 |
| shaohe_feng | zhenguo: oh, check the code. seem only for the log. ") | 09:20 |
| shaohe_feng | :) | 09:20 |
| shaohe_feng | zhenguo: then we do not need it. | 09:20 |
| zhenguo | shaohe_feng: hah, yes | 09:20 |
| yuntongjin | zhenguo: i'm not sure why | 09:23 |
| zhenguo | yuntongjin: hah | 09:23 |
| yuntongjin | china mobile does have many baremetal requirement/issue that they hope would get help | 09:24 |
| zhenguo | yuntongjin: yes, we would like to help | 09:24 |
| zhenguo | we will have dedicated attention for their requirements | 09:25 |
| yuntongjin | which we could discuss in our meeting | 09:25 |
| zhenguo | yuntongjin: yes but silenceli disappeared from irc | 09:25 |
| *** wanghao has quit IRC | 09:26 | |
| yuntongjin | i can help bring all the issue/requirement into our meeting | 09:27 |
| zhenguo | yuntongjin: ok, thanks, you can add a agenda to discuss the requirements | 09:27 |
| yuntongjin | once china mobile folks realize that we are helping to address their requirement, i think they will get involved more | 09:28 |
| zhenguo | yuntongjin: hah, sure | 09:29 |
| yuntongjin | zhenguo: sure, will do | 09:30 |
| zhenguo | yuntongjin: thanks | 09:30 |
| *** yuntongjin has quit IRC | 09:30 | |
| zhenguo | shaohe_feng, liusheng: regarding https://review.openstack.org/#/c/417308/ , do you think the looping call is needed? | 09:33 |
| *** openstack has joined #openstack-mogan | 14:30 | |
| openstackgerrit | Zhenguo Niu proposed openstack/nimble: [WIP] Introduce instance task state https://review.openstack.org/418461 | 16:14 |
| openstackgerrit | Zhenguo Niu proposed openstack/nimble: [WIP] Introduce instance task state https://review.openstack.org/418461 | 16:37 |
| openstackgerrit | Merged openstack/nimble: Add a looping call to wait for power action completed https://review.openstack.org/417308 | 17:10 |
| *** Jeffrey4l_ has joined #openstack-mogan | 21:35 | |
| *** Jeffrey4l has quit IRC | 21:35 | |
| *** wxy has quit IRC | 23:21 | |
| *** wxy has joined #openstack-mogan | 23:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!