| *** wanghao has joined #openstack-mogan | 00:58 | |
| zhenguo | morning mogan! | 00:59 |
|---|---|---|
| *** wanghao has quit IRC | 01:07 | |
| *** litao__ has joined #openstack-mogan | 01:11 | |
| *** wanghao has joined #openstack-mogan | 01:11 | |
| *** liujiong has joined #openstack-mogan | 01:22 | |
| wanghao | morning! | 01:33 |
| openstackgerrit | wanghao proposed openstack/mogan-specs master: Manage existing nodes in Mogan https://review.openstack.org/459967 | 01:35 |
| openstackgerrit | liusheng proposed openstack/mogan master: Clean the compute_node and compute_port objects and db interfaces https://review.openstack.org/478406 | 01:40 |
| openstackgerrit | liusheng proposed openstack/mogan master: Get rid of listing availability zone api and clean some legacy code https://review.openstack.org/478403 | 01:40 |
| openstackgerrit | liusheng proposed openstack/mogan master: Clean the methods about updating node resources to Mogan db https://review.openstack.org/478357 | 01:40 |
| openstackgerrit | liusheng proposed openstack/mogan master: Get rid of node listing api of Mogan https://review.openstack.org/478361 | 01:40 |
| openstackgerrit | liusheng proposed openstack/mogan master: Consume nodes resource in placement https://review.openstack.org/477826 | 01:40 |
| openstackgerrit | liusheng proposed openstack/mogan master: Refactor the scheduler to use placement service https://review.openstack.org/477426 | 01:40 |
| litao__ | morning patches | 01:44 |
| *** wanghao has quit IRC | 01:53 | |
| *** wanghao has joined #openstack-mogan | 02:14 | |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan master: Update ironic.driver.plug_vif implementation https://review.openstack.org/469766 | 02:18 |
| openstackgerrit | liusheng proposed openstack/mogan master: Clean the compute_node and compute_port objects and db interfaces https://review.openstack.org/478406 | 02:28 |
| openstackgerrit | liusheng proposed openstack/mogan master: Get rid of listing availability zone api and clean some legacy code https://review.openstack.org/478403 | 02:28 |
| openstackgerrit | liusheng proposed openstack/mogan master: Clean the methods about updating node resources to Mogan db https://review.openstack.org/478357 | 02:28 |
| openstackgerrit | liusheng proposed openstack/mogan master: Get rid of node listing api of Mogan https://review.openstack.org/478361 | 02:28 |
| openstackgerrit | liusheng proposed openstack/mogan master: Consume nodes resource in placement https://review.openstack.org/477826 | 02:28 |
| openstackgerrit | liusheng proposed openstack/mogan master: Refactor the scheduler to use placement service https://review.openstack.org/477426 | 02:28 |
| zhenguo | Xinran: I added you as a co-author to luyao's patch of refactoring vif attaching. | 02:31 |
| zhenguo | Xinran: and just filed a bug in ironic https://bugs.launchpad.net/ironic/+bug/1701991 | 02:32 |
| openstack | Launchpad bug 1701991 in Ironic "Unable to specify ironic port/portgroup id to use when attaching vif" [Undecided,New] | 02:32 |
| zhenguo | litao__: I found you also contribute to ironic community, not sure if you are interested in this bug :D | 02:33 |
| Xinran | zhenguo, okay thanks | 02:39 |
| zhenguo | Xinran: np, will test that soon when my env is ready | 02:40 |
| Xinran | zhenguo, do you mean add port_type in ironic like what I deleted in mogan | 02:40 |
| zhenguo | Xinran: not port type | 02:41 |
| zhenguo | Xinran: I would like to add port_id support when attaching vif | 02:41 |
| zhenguo | Xinran: ironic.vif_attach(neutron_port_id, ironic_port_id) | 02:41 |
| zhenguo | liusheng: https://review.openstack.org/#/c/476325/ , how about also moving the query.py to the following up patches which really uses it. | 02:48 |
| liusheng | zhenguo: no, it is used in this patch | 02:49 |
| liusheng | zhenguo: manager.py | 02:49 |
| zhenguo | liusheng: let me check | 02:50 |
| Xinran | zhenguo, so I think about adding another func like vif_attach_by_port_id(neutron_port_id, ironic_port_id), and we can call it in vif_attach. | 02:50 |
| zhenguo | liusheng: queryclient? | 02:51 |
| zhenguo | Xinran: not currently, we can just remove port type now | 02:51 |
| liusheng | zhenguo:oh, the queryclient can be moved to following patches | 02:52 |
| zhenguo | Xinran: let's wait for ironic guys to see whether they accept this proposal | 02:52 |
| zhenguo | liusheng: hah, yes, the queryclient | 02:52 |
| Xinran | zhenguo, hah okay. | 02:52 |
| liusheng | zhenguo: my mistake | 02:52 |
| zhenguo | liusheng: really a huge work, hah | 02:53 |
| liusheng | zhenguo: hah, just some copying and removing | 02:53 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Update ironic.driver.plug_vif implementation https://review.openstack.org/469766 | 02:53 |
| zhenguo | liusheng: seems most of the work have been done, | 02:54 |
| liusheng | zhenguo: yes | 02:54 |
| zhenguo | liusheng: ok, let's try to land them before the weekly meeting | 02:54 |
| liusheng | zhenguo: ok, sir :D | 02:55 |
| zhenguo | liusheng: lol | 02:55 |
| zhenguo | liusheng: this https://review.openstack.org/#/c/479120/ removes the cloudboot driver dir as it's empty for now | 02:56 |
| liusheng | zhenguo: ok | 02:57 |
| zhenguo | liusheng: a quick question, if power action failed, should we set the server status to fail? | 02:59 |
| liusheng | zhenguo: what's your meaning of the failure, the server's power status didn't changed ? | 03:01 |
| liusheng | zhenguo: if so, I don't think so | 03:01 |
| zhenguo | liusheng: https://review.openstack.org/#/c/476862/ | 03:01 |
| zhenguo | liusheng: I found if we soft power off the server it will be stuck in *ing | 03:02 |
| zhenguo | liusheng: if the ironic driver doesn't support htat | 03:02 |
| liusheng | zhenguo: I don't prefer to change the status of server to error | 03:02 |
| zhenguo | liusheng: that's also my concern | 03:03 |
| zhenguo | liusheng: as it's not really a server fail | 03:03 |
| liusheng | zhenguo: how about add a timeout, if power action failed, just abort the action | 03:03 |
| liusheng | zhenguo: and record a warning log | 03:03 |
| liusheng | zhenguo: yes | 03:03 |
| zhenguo | liusheng: as power action is a cast instead of call | 03:04 |
| zhenguo | liusheng: so the API will return with the server is powering now | 03:04 |
| liusheng | zhenguo: I mean in engine side | 03:04 |
| zhenguo | liusheng: you mean just change the *ing status to active? | 03:04 |
| liusheng | zhenguo: yes | 03:04 |
| zhenguo | liusheng: or off | 03:04 |
| liusheng | zhenguo: I think so | 03:04 |
| liusheng | zhenguo: and record some info in log | 03:05 |
| zhenguo | liusheng: seems ok | 03:05 |
| zhenguo | litao__: wdyt? | 03:05 |
| litao__ | zhenguo: ok, I will try | 03:07 |
| zhenguo | litao__: thanks | 03:08 |
| litao__ | zhenguo: np 😀 | 03:08 |
| *** Kevin_Zheng has joined #openstack-mogan | 03:14 | |
| openstackgerrit | Tao Li proposed openstack/mogan master: Fix reverting the network when plug vif failed https://review.openstack.org/477440 | 03:20 |
| openstackgerrit | liusheng proposed openstack/mogan master: Clean the compute_node and compute_port objects and db interfaces https://review.openstack.org/478406 | 03:22 |
| openstackgerrit | liusheng proposed openstack/mogan master: Get rid of listing availability zone api and clean some legacy code https://review.openstack.org/478403 | 03:22 |
| openstackgerrit | liusheng proposed openstack/mogan master: Reporting nodes resource to placement service https://review.openstack.org/476325 | 03:22 |
| openstackgerrit | liusheng proposed openstack/mogan master: Clean the methods about updating node resources to Mogan db https://review.openstack.org/478357 | 03:22 |
| openstackgerrit | liusheng proposed openstack/mogan master: Get rid of node listing api of Mogan https://review.openstack.org/478361 | 03:22 |
| openstackgerrit | liusheng proposed openstack/mogan master: Consume nodes resource in placement https://review.openstack.org/477826 | 03:22 |
| openstackgerrit | liusheng proposed openstack/mogan master: Refactor the scheduler to use placement service https://review.openstack.org/477426 | 03:22 |
| openstackgerrit | Merged openstack/mogan master: Remove cloudboot driver directory https://review.openstack.org/479120 | 03:36 |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Update ironic.driver.plug_vif implementation https://review.openstack.org/469766 | 03:51 |
| *** liujiong has quit IRC | 03:58 | |
| openstackgerrit | Xinran WANG proposed openstack/mogan master: Detach interface for server https://review.openstack.org/461767 | 05:49 |
| *** liujiong has joined #openstack-mogan | 06:04 | |
| openstackgerrit | liusheng proposed openstack/mogan master: Clean the compute_node and compute_port objects and db interfaces https://review.openstack.org/478406 | 06:19 |
| openstackgerrit | liusheng proposed openstack/mogan master: Get rid of listing availability zone api and clean some legacy code https://review.openstack.org/478403 | 06:19 |
| openstackgerrit | liusheng proposed openstack/mogan master: Reporting nodes resource to placement service https://review.openstack.org/476325 | 06:19 |
| openstackgerrit | liusheng proposed openstack/mogan master: Clean the methods about updating node resources to Mogan db https://review.openstack.org/478357 | 06:19 |
| openstackgerrit | liusheng proposed openstack/mogan master: Get rid of node listing api of Mogan https://review.openstack.org/478361 | 06:19 |
| openstackgerrit | liusheng proposed openstack/mogan master: Consume nodes resource in placement https://review.openstack.org/477826 | 06:19 |
| openstackgerrit | liusheng proposed openstack/mogan master: Refactor the scheduler to use placement service https://review.openstack.org/477426 | 06:19 |
| openstackgerrit | wanghao proposed openstack/mogan master: [WIP]Manage existing BMs: Part-1 https://review.openstack.org/479660 | 06:30 |
| wanghao | zhenguo: ping | 06:32 |
| zhenguo | wanghao: pong | 06:32 |
| wanghao | zhenguo: litao__ and I will start to code the manage existing patch this week, I will finish the part-1: list adoptable nodes and litao__ will finish the part-2: manage API. | 06:33 |
| zhenguo | wanghao: awesome! | 06:33 |
| wanghao | zhenguo: As I think we didn't have much issue in spec, so let it begin to land. :) | 06:34 |
| zhenguo | wanghao: yes, I also don't find big issues with the spec | 06:34 |
| zhenguo | wanghao: will try to let others review it as well | 06:34 |
| wanghao | zhenguo: yeah, that we can update spec even if we merged the code if it's necessary | 06:35 |
| zhenguo | wanghao: yes | 06:35 |
| wanghao | zhenguo: well, so I will add our patch into review list and we can review it too, thanks. | 06:35 |
| zhenguo | wanghao: sure, thanks for the efforts | 06:36 |
| wanghao | zhenguo: you're welcome | 06:36 |
| wanghao | zhenguo: BTW, I found a thing, that we pass a lot of context to driver, but not use it. Do you think will to remove them? | 06:36 |
| zhenguo | wanghao: that's also happen in nova drivers, I think we don't need it, but seems no rush to clean up it | 06:38 |
| zhenguo | wanghao: also in DB api | 06:39 |
| wanghao | zhenguo: sure, I will do it later before we release. | 06:39 |
| zhenguo | wanghao: ok, thanks | 06:39 |
| wanghao | zhenguo: np | 06:39 |
| *** wanghao_ has joined #openstack-mogan | 07:01 | |
| *** wanghao has quit IRC | 07:01 | |
| zhenguo | liusheng: have you ever seen this 'Unable to locate package libsystemd-dev' when installing devstack? | 07:01 |
| liusheng | zhenguo: didn't found that, can you install that package manually ? | 07:02 |
| zhenguo | liusheng: no, | 07:02 |
| zhenguo | liusheng: in ubuntu 14.04 | 07:03 |
| liusheng | zhenguo: oh, i always use 16.04 | 07:03 |
| zhenguo | liusheng: hah | 07:03 |
| zhenguo | liusheng: you have upgraded your vm in blue zone? | 07:03 |
| liusheng | zhenguo: openstack now recommend to use 16.04 than 14.04 | 07:04 |
| liusheng | zhenguo: yes | 07:04 |
| zhenguo | liusheng: yes, that's the problem | 07:04 |
| liusheng | zhenguo: may you need to pugrade your system :D | 07:04 |
| zhenguo | liusheng: ok, I will try to upgrade it | 07:04 |
| liusheng | zhenguo: ok | 07:05 |
| zhenguo | liusheng: what's the review hour etherpad url is? | 07:06 |
| liusheng | zhenguo: https://etherpad.openstack.org/p/MoganReviewHour | 07:08 |
| zhenguo | liusheng: ok, thanks | 07:08 |
| *** wanghao has joined #openstack-mogan | 07:16 | |
| *** wanghao_ has quit IRC | 07:19 | |
| zhenguo | liusheng, Xinran: wrt https://review.openstack.org/#/c/469766/ , placement patches need to update to solve conflicts if this is landed first | 07:26 |
| liusheng | yes, I found this patch have changed the compute_port object definition, but I will remove compute_port and compute_node objects | 07:28 |
| liusheng | we won't have dependency on node info in Mogan'db | 07:28 |
| zhenguo | liusheng: yes, but there's no dependences, so from the patch's view it's ok, hah | 07:28 |
| liusheng | zhenguo: ok,yes | 07:29 |
| Xinran | zhenguo, liusheng thanks | 07:35 |
| zhenguo | Xinran: np | 07:35 |
| Xinran | Btw, can you run python -m testtools.run mogan.tests.tempest.api.test_servers in your env? | 07:39 |
| zhenguo | liusheng: ^^ | 07:40 |
| zhenguo | liusheng: seems Xinran's problem is still the glance_api_servers config options | 07:41 |
| liusheng | Xinran, zhenguo I didn | 07:43 |
| Xinran | zhenguo, liusheng cant stop struggling with that... | 07:43 |
| liusheng | Xinran, zhenguo I didn't use testtools to run tempest | 07:43 |
| liusheng | tempest run -t --regex "^mogan\.tests\.tempest\.api\.test_servers\.BaremetalComputeAPIServersTest\.test_server_attach_interface" | 07:43 |
| liusheng | I use "tempest run" command | 07:44 |
| liusheng | Xinran: still, you need to confirm your glance running on uwsgi | 07:45 |
| liusheng | Xinran: if not, you need to config the port number in your glance_api_servers option | 07:45 |
| *** openstackgerrit has quit IRC | 07:48 | |
| Xinran | liusheng, glance is running on uwsgi | 07:48 |
| liusheng | Xinran: so you just need to config like this: glance_api_servers = http://10.229.40.107/image | 07:49 |
| *** openstackgerrit has joined #openstack-mogan | 07:49 | |
| openstackgerrit | wanghao proposed openstack/mogan master: Manage existing BMs: Part-1 https://review.openstack.org/479660 | 07:49 |
| Xinran | liusheng, ok I will try. Thanks a lot | 07:59 |
| liusheng | Xinran: np :D | 07:59 |
| zhenguo | liusheng: seems there will be collisions with placement, as before setting allocation, many requests may get a same node, right? | 08:11 |
| liusheng | zhenguo: but scheduler will set allocation | 08:12 |
| zhenguo | liusheng: when to set that | 08:13 |
| liusheng | zhenguo: after a node selected | 08:13 |
| zhenguo | liusheng: many parallel requests will get a same node with select destinations | 08:13 |
| zhenguo | liusheng: it's two REST API calls | 08:14 |
| zhenguo | liusheng: not atomic | 08:14 |
| liusheng | zhenguo: but I found there is a lock in scheduler | 08:14 |
| zhenguo | liusheng: hah, yes | 08:14 |
| zhenguo | liusheng: I mean, should we get rid of it or is there some good solutions | 08:15 |
| liusheng | zhenguo: seems nova also cannot support multiple-deployment of scheduler | 08:17 |
| *** wanghao has quit IRC | 08:17 | |
| zhenguo | liusheng: you mean nova also lock the whole process? | 08:17 |
| * zhenguo brb | 08:19 | |
| liusheng | zhenguo: not very sure, but there is a old problem that Nova cannot suuport multi-worker of scheduler service, since scheduling cannot handle parallel requests | 08:20 |
| *** wanghao has joined #openstack-mogan | 08:28 | |
| *** Xinran has quit IRC | 08:31 | |
| zhenguo | liusheng: ok | 08:31 |
| zhenguo | hi all, please help to review https://etherpad.openstack.org/p/MoganReviewHour | 08:34 |
| zhenguo | shaohe_feng: are you around? if so, please help to review the patches list, thanks! | 08:43 |
| wanghao | terrible network.... | 08:51 |
| shaohe_feng | zhenguo: yes. | 08:59 |
| shaohe_feng | zhenguo: OK. | 08:59 |
| zhenguo | shaohe_feng: thanks | 09:03 |
| zhenguo | shaohe_feng: especially the manage nodes spec and the placement patches list | 09:04 |
| shaohe_feng | zhenguo: OK. | 09:04 |
| *** openstackgerrit has quit IRC | 09:18 | |
| *** Xinran_ has joined #openstack-mogan | 09:24 | |
| *** openstackgerrit has joined #openstack-mogan | 09:30 | |
| openstackgerrit | Tao Li proposed openstack/mogan master: Fix reverting the network when plug vif failed https://review.openstack.org/477440 | 09:30 |
| openstackgerrit | Zhenguo Niu proposed openstack/mogan-specs master: Track resources using Placement https://review.openstack.org/475700 | 09:36 |
| *** wanghao has quit IRC | 09:38 | |
| Xinran_ | liusheng, hi, are you around? still doesn't work....I need your help... | 09:48 |
| liusheng | Xinran_: what's the error now ?> | 09:50 |
| Xinran_ | liusheng, the same | 09:50 |
| liusheng | Xinran_: cannot find the image ? | 09:50 |
| Xinran_ | liusheng, value required for option glance_api_servers in group [glance] | 09:50 |
| Xinran_ | liusheng, no , now I can create server. The error occurs when I run tempest test | 09:51 |
| liusheng | Xinran_: did you config glance_api_servers under [glance] in tempest.conf ? | 09:54 |
| Xinran_ | liusheng, I didn't found tempest.conf | 09:56 |
| liusheng | Xinran_: /etc/tempest/tempest.conf ? | 09:56 |
| liusheng | Xinran_: or you can edit in /opt/stack/tempest/etc/tempest.conf | 09:57 |
| Xinran_ | liusheng, no there is no folder named tempest | 09:57 |
| liusheng | Xinran_: did you install tempest with stack.sh ? | 09:58 |
| Xinran_ | liusheng, yes, in local.conf | 09:59 |
| Xinran_ | liusheng, i didn't install it manually | 09:59 |
| liusheng | Xinran_: does this exist ? /opt/stack/tempest/etc/tempest.conf | 09:59 |
| Xinran_ | liusheng, no ... | 10:00 |
| liusheng | Xinran_: no etc/tempest.conf in your local tempest repo ? | 10:00 |
| liusheng | Xinran_: that is strange | 10:00 |
| Xinran_ | liusheng, no... | 10:01 |
| liusheng | Xinran_: can you find tempest.conf in your system | 10:02 |
| Xinran_ | liusheng, if not , can I install tempest manually ? | 10:02 |
| Xinran_ | liusheng, no there's nothing about tempest | 10:02 |
| Xinran_ | liusheng, But before there were no tempest.conf either but it worked | 10:03 |
| liusheng | Xinran_ no, devstack will install tempest and config it | 10:04 |
| Xinran_ | liusheng, really strange | 10:11 |
| Xinran_ | liusheng, let me restack | 10:11 |
| *** dims has quit IRC | 10:14 | |
| *** dims has joined #openstack-mogan | 10:15 | |
| *** liujiong has quit IRC | 10:15 | |
| *** Xinran_ has quit IRC | 11:02 | |
| openstackgerrit | Merged openstack/mogan master: Update ironic.driver.plug_vif implementation https://review.openstack.org/469766 | 11:31 |
| openstackgerrit | Merged openstack/mogan master: Stop using deprecated 'message' attribute in Exception https://review.openstack.org/475627 | 11:54 |
| *** litao__ has quit IRC | 11:56 | |
| *** wanghao has joined #openstack-mogan | 12:45 | |
| *** wanghao has quit IRC | 13:35 | |
| *** wanghao has joined #openstack-mogan | 13:35 | |
| *** wanghao has quit IRC | 13:35 | |
| *** wanghao has joined #openstack-mogan | 13:36 | |
| *** wanghao has quit IRC | 13:36 | |
| *** wanghao has joined #openstack-mogan | 13:36 | |
| *** wanghao has quit IRC | 13:37 | |
| *** wanghao has joined #openstack-mogan | 13:37 | |
| *** wanghao has quit IRC | 13:38 | |
| *** wanghao has joined #openstack-mogan | 13:38 | |
| *** wanghao has quit IRC | 13:39 | |
| *** wanghao has joined #openstack-mogan | 13:39 | |
| *** wanghao has quit IRC | 13:39 | |
| *** wanghao has joined #openstack-mogan | 13:40 | |
| *** wanghao has quit IRC | 13:40 | |
| *** wanghao has joined #openstack-mogan | 13:41 | |
| -openstackstatus- NOTICE: We have switched now all docs publishing jobs to new documentation builds. For details see dhellmann's email http://lists.openstack.org/pipermail/openstack-dev/2017-July/119221.html . For problems, join us on #openstack-doc | 14:38 | |
| -openstackstatus- NOTICE: "We're experiencing a few problems with the reorg on docs.openstack.org and are looking into these..." | 15:18 | |
| *** ChanServ changes topic to ""We're experiencing a few problems with the reorg on docs.openstack.org and are looking into these..."" | 15:18 | |
| *** ChanServ changes topic to "Bugs: bugs.launchpad.net/mogan | Review: https://review.openstack.org/#/q/project:openstack/mogan,n,z" | 15:37 | |
| -openstackstatus- NOTICE: "docs.openstack.org is working fine again, due to move of new location, each repo needs to merge one change to appear on docs.o.o" | 15:37 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!